Помогите найти ошибку в коде

А как же это?

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

Проверь все-таки последнюю функцию на правильность работы

Все функции правильные, ошибка где-то здесь if ((TimeCurrent()- timeopensell) < 60*Period())break; То есть если убрать timeopenbuy и timeopensell все работает, только открывает кучу ордеров на одном баре. Я хочу как-то это убрать. Не получается. Помогите, пожалуйста

Как должна выглядеть функция, так (как сейчас)

int CheckOrders(int type) {
   bool check = false;                                 
   for(int i=1; i<=OrdersTotal(); i++) {
      if (OrderSelect(i-1,SELECT_BY_POS))  {                                       
         if (OrderSymbol()!=Symbol())continue;
         if (OrderMagicNumber()==magic && OrderType()== type)
            check = true;
         break;
      }
   }
   return (check);                
}     

Или так?

int CheckOrders(int type) {
   bool check = false;                                 
   for(int i=1; i<=OrdersTotal(); i++) {
      if (OrderSelect(i-1,SELECT_BY_POS))  {                                       
         if (OrderSymbol()!=Symbol())continue;
         if (OrderMagicNumber()==magic && OrderType()== type) {
            check = true;
            break;
         }
      }
   }
   return (check);                
}     

И чем различаются оба варианта?

Это не принципиально. Искать нужно во времени

Если есть логическая в одной функции, то она может быть и в других.

Верхний вариант не будет работать правильно. Если условие if (OrderMagicNumber()==magic && OrderType()== type) ложно, то дальше идет брейк и мы можем не попасть на желаемый ордер

Я же говорю, без времен все работает, все функции корректны. Попробуйте, пожалуйста, отвлечься от функций и подумать над временем

А на каком таймфрейме вы проверяете эксперт?