Учимся логике - страница 9

Ага :) это я немножко утрировал.

Вот еще один примерчег :) правда на MQL он вряд ли заработает как надо, на С++ заработает:

for (int i = OrdersTotal(); i--;)
{
   if (!OrderSelect(i - 1, SELECT_BY_POS)) continue;

   //...
}

Код вполне читаемый. Зря так.

Чтобы не расшифровывать, что имеется в виду (любой ордер на продажу), было бы разумнее расписать идентификаторы типов ордеров. Будет подлиньше, но намного понятнее.

С точки зрения лаконичности - да, это решение, но не слишком надежное. Компания-разработчик не гарантирует, что числовые значения этих констант никогда не изменятся.

В этом то вся и проблема. Можно сделать изящный код, который вдруг перестанет работать.

Не проверял, но сам так не стал бы писать.

Это уже не оптимизация, а извращ... пардон изощрение...

Вспомнилась милая задачка, как обменять значения между двумя численными переменными, не используя третьего. ))

В этом вся соль -- этот код не изящный. Вот такой будет изящней:

bool IsActiveType(int orderType)
{
   return (type == OP_BUY || type == OP_SELL);
}

bool IsAnySellType(int orderType)
{
   return (type == OP_SELL || type == OP_SELLLIMIT || type == OP_SELLSTOP);
}

{
   if (IsActiveType(OrderType()))
   {
      //...
   }

   if (IsAnySellType(OrderType()))
   {
      //...
   }
}

Ага :) вроде того.

Ага, вот-вот. Функции с телом в одну строку никто не отменял, они реально очень полезны.

Мне понравилось. Красиво смотрится

Только параметры функций убери, будет изящней. Если параметр не задан, то определять из OrderType() ордера, если задан, то по параметру.

Скушно мне тут с вами :)))