Experts: Diff_TF_MA_EA - page 5

New comment
 
fxsaber:

Apparently it's some kind of school of thought to write it like that

Why not like that?


ZY

Well, in general, fxsaber is right.

 
Artyom Trishkin:

aren't you aware of the requirement to match the MK style?

Of course not.

 
fxsaber:

Unknown, of course.

Come on )

 
Rashid Umarov:

Come on )

It is. There was one case when they didn't want to publish because the class name didn't have a "C" at the very beginning. But I don't think it falls under any rule. Rather, it was a personal preference of the reviewer.

 
fxsaber:

I replaced that.

Serious mistake.

What's in the link:

This is the wrong logic. After a failed and successful OrderSend, the current trading environment should be completely read again. This rule should always apply.

It was discussed yesterday - there is no such error there. The environment is updated when a position is opened. And when an unsuccessful opening, the list does not change - why should it be raced?

 
Artyom Trishkin:

What's on the link:

This is the wrong logic. After a failed and successful OrderSend, the current trading environment should be completely read again. This rule should always apply.

It was discussed yesterday - there is no such error. The environment is updated when a position is opened. And when opening a position is unsuccessful, the list does not change - why should it race?

I've already realised that I can't change your mind.

 
Rashid Umarov:
The codes have been updated.

There's still

double ll=trade.CheckVolume(symb,lot,symbol_info.Ask(),ORDER_TYPE_BUY);
 
fxsaber:

There's still

Won't re-read everything - meaning the cached value?

 
Rashid Umarov:

They won't reread everything - you mean the cached value?

Yes.

12345
New comment