Правильность написания и отработки кода экспертом

[Oleg Konovalov]

if(

// условие на вход ПЕРВОЕ, состоит из нескольких управляющих блоков (сигнальный, регулирующий, ...подтверждающий)
(
...
)
) opSellFirst= True;

if(
// условие на вход ВТОРОЕ, состоит из нескольких управляющих блоков (сигнальный, подтверждающий, регулирующий, ...)
(
...
)
) opSellSecond= True;

if (opSellFirst == true || opSellSecond == true) opSell= true;

Здравствуйте.

Прошу знающих людей подсказать, насколько данная конструкция кода правильна для использования в торговом советнике и не возникнет ли с ней сложностей?

[KONDOR]

konovalov: насколько данная конструкция кода правильна для использования в торговом советнике и не возникнет ли с ней сложностей?

bool opSellFirst= false; //  перед сравнением условиий нужно "обнулить" переменную, если этого не сделать, то всегда будет true, независимо от условий
 if(... ) opSellFirst= true;

bool opSellSecond= false; // также и здесь
 if(... ) opSellSecond= true;

 if (opSellFirst == true || opSellSecond == true) opSell= true;


также для ускорения выполнения кода можно применять вложенные условия
вместо if(x>0 && y>0 && z>0) .....;

if(x>0) { 
  if(y>0)  {
    if(z>0) ....;
           }
         }

[Vladyslav Goshkov]

KONDOR:

if (opSellFirst == true || opSellSecond == true) opSell= true;

Можно так:

bool opSell = (opSellFirst||opSellSecond);

[Oleg Konovalov]

Спасибо за ответы.