Помогите решить Error 1 с OrderModify - страница 4

Новый комментарий
 
Roman Meskhidze:
исправился 
Я применил стилизатор кода перед вставкой - и Вам советую.
 
Roman Meskhidze:

Оксана, вы гений

Так будет верно?

А почему вы в первом цикле проверяли на Магик и Символ, а во втором цикле только на Магик? Это грубая ошибка.

Я вообще бы по-другому сделала, в функцию отправляла бы Тип, с которым работать: TrailTP(int type) - чтобы на будущее не было ошибок.

И там, где вы проверяете на Магик и Символ, проверяла бы еще на Тип!

И в коде уместнее не printf(), а Print()

 
Karputov Vladimir:
Я применил стилизатор кода перед вставкой - и Вам советую.
На стиль и цвет товарищей нет! ;)
 
Boris:
На стиль и цвет товарищей нет! ;)

Поддерживаю. После стилизатора все скобки:

void Func()
  {
   if(true)
     {
      if(true)
        {
        }
     }
  }

А мне еще с флеша удобнее так:

void Func(){  
  if(true){    
    if(true){
    }  
  }
}
 
Oksana Berenko:

в функцию отправляла бы Тип, с которым работать: TrailTP(int type) - чтобы на будущее ...

Будущее v.1.0: ... пропустим для краткости

Будущее v.2.0: ... пропустим для

Будущее v.3.0: ... пропустим

Будущее v.4.0: ...

Будущее v.5.0: 

Move_SL(
        double d_Trail_Trigger,
        double d_BreakEvevn_Trigger = -1,
        double d_Trail_Distance = -1,
        double d_Trail_Step = -1,
        double d_BreakEvevn_Profit = 0,
        int i_Magic = -1,
        ENUM_ORDER_GROUP e_Op_Type = ORD_ALL,
        datetime t_Time_From = 0,
        ENUM_BALANCE e_Profit = SUM_ANY
) {
        // 63 строки
}

где

enum ENUM_ORDER_GROUP {
        ORD_BUY_IN_MARKET,      // all buy market orders
        ORD_SELL_IN_MARKET,     // all sell market orders
        ORD_MARKET,             // all market orders
        ORD_PENDINGS,           // all pending orders
        ORD_BUY_PENDINGS,       // all buy pending orders
        ORD_BUY_ALL,            // all buy orders
        ORD_SELL_PENDINGS,      // all sell pending orders
        ORD_SELL_ALL,           // all sell orders
        ORD_LIMITS,             // all limit-orders
        ORD_STOPS,              // all stop-orders
        ORD_ALL                 // all orders
};
enum ENUM_BALANCE {
        SUM_ANY,        // all
        SUM_PROFIT,     // profit only
        SUM_LOSS        // loss only
};

Да фьюча стилл коминг ...

 

Oksana Berenko:

А мне еще с флеша удобнее так:

Аналогично. Стилизатор в стиле Метаквотов - с намертво вшитыми личными привычками разрабов и без пользовательских настроек
 
Oksana Berenko:

А почему вы в первом цикле проверяли на Магик и Символ, а во втором цикле только на Магик? Это грубая ошибка.

Я вообще бы по-другому сделала, в функцию отправляла бы Тип, с которым работать: TrailTP(int type) - чтобы на будущее не было ошибок.

И там, где вы проверяете на Магик и Символ, проверяла бы еще на Тип!

И в коде уместнее не printf(), а Print()

вы не поверите, но так и сделал - передаю тип в функцию )))

Еще раз огромное спасибо 

 
Oksana Berenko:

Поддерживаю. После стилизатора все скобки:

А мне еще с флеша удобнее так:

Надо же, какое совпадение! Получается компактнее по ширине и высоте! Правда, последние закрывающие скобки выстраиваю в линию! Таким образом легче и быстрее работать, и код занимает в полтора раза меньше линий. Мой вариант Вашего примера:

void Func(){  
  if(true){    
    if(true){
} } }
 
Karputov Vladimir:
Я применил стилизатор кода перед вставкой - и Вам советую.
С форматированием через SRC все ясно - оно необходимо. А вот с использованием стилизатора буду согласен тогда, когда MQ сделают его настройку и возможность иметь несколько наборов стилей. Я использую codestyle от MS и от стиля MQ мне становится грустно.
 
Oksana Berenko:

А почему вы в первом цикле проверяли на Магик и Символ, а во втором цикле только на Магик? Это грубая ошибка.

Я вообще бы по-другому сделала, в функцию отправляла бы Тип, с которым работать: TrailTP(int type) - чтобы на будущее не было ошибок.

И там, где вы проверяете на Магик и Символ, проверяла бы еще на Тип!

И в коде уместнее не printf(), а Print()

Спасибо за вашу помощь. Результат не заставил себя долго ждать <ССЫЛКА НА ПЛАТНЫЙ СИГНАЛ УДАЛЕНА>

 А в чем разница принципиальная между printf и Print() ? 

Я учился по учебнику mql4.com, а выходит что там все устарело?

Как следить за обновлениями, ведь иначе в будущих билдах есть вероятность неработоспособности советников? 

 
Roman Meskhidze:

А в чем разница принципиальная между printf и Print() ? 

Я учился по учебнику mql4.com, а выходит что там все устарело?

Как следить за обновлениями, ведь иначе в будущих билдах есть вероятность неработоспособности советников? 

printf, он же PrintFormat, более удобен и универсален, чем Print. Учебник описывает старую версию MQL4, так что устарел. Общего списка обновлений по версиям нет, некоторые сведения печатаются на этом форуме. 
123456
Новый комментарий
Причина обращения: