Trade manager button ea problem

[Fabien S]

Hey guys i wanted to create this trade manager ea that puts trades when pressing buy ( SL on 200 ema) however once i load the project, the ea doesnt work and reattaches itself all the time. Does anybody know where i have made an error? I appreciate every help. project https://fxdreema.com/shared/Yfq4NQgub

[trader.philipps]

@Fabien-S If you try to compile your project in Meta Editor you'll see the message:
'Sell' - undeclared identifier Trade Manager.mq4 3585 22

If I take a look at the line 3585 it's like this:
SkipObjects = (int)Sell;

So you use Sell that is not declared somewhere. I will try to find where it comes from.

[trader.philipps]

@Fabien-S Here we go ..

Block 702

You put Sell ther which is interprted as an integer variable. However it's not defined. Probably that field should be empty.

Aditionally it looks like you missed one connection between Blocks 51 and 54, but that was not the issue why the EA unloaded.

[Fabien S]

Thanks a lot!! @trader-philipps

[Fabien S]

https://fxdreema.com/shared/Yfq4NQgub
in the last version the button functionality of buy and sell doesnt seem to work, i think it has something to do with the 200 ema SL variable @trader-philipps could you help me out there aswell, i would really appreciate it!

[trader.philipps]

@fabien-s First of all check the labels as they seem to be inconsistent. On init you use sell and buy for instance, but later on Sell and Buy. Don't know if that matters, but I think so. Further I'd avoid blankspace in the labels. Mayne it doesn't matter, but better use underscore or camelCase instead.

Once the buttons work again in general, you maybe need to check if your TP variable is valid. For instance if price is above 200 ema and you make a buy order, the TP value would be below open proce and that's not valid.

Let me know when you have a newer version to check further.

[kelianmorr]

Nice. post

[Fabien S]

@trader-philipps you saved me a lot of time and helped me in this Project. I appreciate that Big thumbs up

[trader.philipps]

@fabien-s 4 eyes see more than 2. Sometimes I also seek for ages in my own projects before I find the issue.

[onafets]

@fabien-s hi Thank you for your contribution, is great!

Could I have the ex4, I don't seem to be able to export to either .ex4 or .mq4

Thanks in advance
s

[jjtrader]

@fabien-s said in Trade manager button ea problem:

@trader-philipps you saved me a lot of time and helped me in this Project. I appreciate that Big thumbs up
Hi, did you ever get a working trade manager with buttons? I am trying to code one right now and having difficulty. Thanks!