Цикл индус

[Bugdrill]

Народ! Научите не писать индусский код.
Есть код:

		double max_n;
                int p = 0;
                for (int i = 0, size = n.size(); i < size; i++)
		{
			if (max_n == n[i])
			{
				if (i == 0)
				{
					p = 1;
					break;
				}
				if (i == 1)
				{
					p = 2;
					break;
				}
				if (i == 2)
				{
					p = 5;
					break;
				}
			}	
		}

Как такой участок кода должен выглядеть с точки зрения профи?
т.е. {0, 1, 2} соответствует {1, 2, 5}.
Может матем. формулой (какой)?

[Smitt&Wesson]

Примерно вот так:

double max_n;
int = p;
for (int i = 0, i = n.size(); i++)
{
	if (max_n == n[i])
	{
	      switch(i)
                {	
                     case 0:
                     {
                     		p = 1;
			break;
	          }
	          case 1:
	         {
			p = 2;
			break;
	         }
	         case 2: p = 5;
	}
       }	
}

[Bugdrill]

Smitt&Wesson
Имел ввиду switch?
По сути тоже самое.

[Smitt&Wesson]

Сообщение от Bugdrill

Smitt&Wesson
Имел ввиду switch?
По сути тоже самое.

Да, да. Блин, ещё не проснулся как следует .
Исправил.

[Serge_Bliznykov]

не знаю C, поэтому напишу только идею (в синтаксисе будут ошибки, но понять смысл вполне можно).

 int[] RecodeArray = new int[] {1,2,5};
 int p  = -1;
 for (int i = 0, size = n.size(); i < size; i++)
 {
  if (max_n == n[i]) {
     p =  RecodeArray[i];
     break
  }

p.s. в данном коде программист должен позаботится, чтобы таблица перекодирования покрывала полностью размер n.size, но, принципе, можно добавить проверку и выдачу сообщения, что не хватает значений для перекодирования (или использовать try .. except)


Хочу Вас так же предостеречь от сравнения вещественных чисел. Это не всегда будет работать так, как Вы ожидаете (подробности в темах о машинном представлении вещественных чисел).


К слову.
я поясните, что это за запись?!!!!!

int = p;

[Bugdrill]

Serge_Bliznykov
int = p; - я тоже не проснулся толком )) int p = 0;
Спасибо! Очень интересно!

[waleri]

С точки зрения профи не будет загадочных констант 0, 1, 5 и т.д. а все будет константных переменных или дефайнах - читабельно будет.

[Smitt&Wesson]

Сообщение от Bugdrill

Serge_Bliznykov
int = p; - я тоже не проснулся толком )) int p = 0;
Спасибо! Очень интересно!

Тоже это определение не заметил. А вообще, зачем переменной что-то присваивать? Ведь ей присваиваются значения в теле цикла и ренее нигде не используются в проверках.
Достаточно написать: int p;
Всё, въехал. Если max_n == n[i] нигде не совпадёт. Но можно записать и так:

if(max_n == n[i])
{
.........
}
else p = 0;

[Stilet]

		double max_n;
 int p = 0;
 int a[3]={1,2,5}
 for (int i = 0, size = n.size(); i < size; i++) if (max_n == n[i]) p=a[i];
		}

Подходит?

[Bugdrill]

Stilet
Как раз Serge_Bliznykov это и имел ввиду. Только break наверно нужен, так как в результате нужно получить первое совпадение.
На счет сравнения вещественного с целым. Я округляю вверх до целого: double max_n, забыл добавить.
На счет констант, незнаю, я сетку осциллоскопа рисую, что бы были красивые числа 10, 20, 30 ... Нужен десятичный логарифм именно от этих чисел.

В итоге хочу побороть в себе индуса ))