Помогите пожалуйста. Какие-то ошибки в советнике - страница 2

 
Vinin:

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

Там у него в ините тикету присвоено значение ноль.

И в старте сразу же выбор по тикету. Тикет разве нулевым бывает? Даже потом, когда (если) появится существующий тикет, при выборе по его номеру нужно ещё проверить из какого списка выбран ордер. При отсылке торгового запроса нет никаких проверок на корректность передаваемых параметров...

Короче - всё фтопку сналёту, даже без тщательной проверки кода, а так - с беглого взгляду... Всё криво...

 
artmedia70:

Там у него в ините тикету присвоено значение ноль.

И в старте сразу же выбор по тикету. Тикет разве нулевым бывает? Даже потом, когда появится существующий тикет, при выборе по его номеру нужно ещё проверить из какого списка выбран ордер. При отсылке торгового запроса нет никаких проверок на корректность передаваемых параметров...

Короче - всё фтопку с налёту, даже без тщательной проверки кода, а так - с беглого взгляду... Всё криво...



С небольшими поправками все будет работать. Доказывать надо или как?
 
Vinin:

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

Каким образом считать? Параметров хватает, т.к. первы ордер в функции init он открывает, и закрывает, а дальше выдает ошибки..
 
Vinin:

С небольшими поправками все будет работать. Доказывать надо или как?
Давай в скайпе посоревнуемся... :)))
 
Нулево значение переменной тикету присваивается для избегания нахождения мусора в переменной и объявляется она первоначально не в ините, а в инете она сразу получает значение открытого ордера и дальше с ней все нормально
 
qaliti:
Нулево значение переменной тикету присваивается для избегания нахождения мусора в переменной и объявляется она первоначально не в ините, а в инете она сразу получает значение открытого ордера и дальше с ней все нормально

То-то там сплошные ошибки - оч нормально...

ЗЫ. Только сейчас обратил внимание - кто ж в ините отсылает торговые приказы? Нонсенс.

 

artmedia70:


ЗЫ. Только сейчас обратил внимание - кто ж в ините отсылает торговые приказы? Нонсенс.



Можно, но не всегда. При набрасывании на график работать будет, при загрузке из шаблона может и нет.
 
alsu:

Можно, но не всегда. При набрасывании на график работать будет, при загрузке из шаблона может и нет.

Нужен надёжный советник или так... ?

Тут и для тестера-то в этом коде много чего дописывать нужно. Местами менять закрытие с открытием, делать необходимые проверки, лот рассчитать по-человечески. Логику даже не смотрел.

ЗЫ. Любые вероятности неадекватного поведения кода нужно сводить к минимуму, а тут логика построена на этом. Я ж говорю - нонсенс.

 
Так как мне исправить именно эти две ошибки?
 
qaliti:
Так как мне исправить именно эти две ошибки?

Прочитать что они означают, постараться понять почему они появляются у вас и переписать код заново.

Причина обращения: