Thread: Change lots/multiplier to lots=0.01 gives OrderSend error 130?

If it is returning an OrderSend error, then it appears to be attempting to trade. This suggests that your logic is OK, but it is impossible to tell from just those 2 snippets of code.

Error 131 is to do with the EA trying to send its order with an invalid trade size. Have you adjusted for minimum and maximum lots? Have you ensured that the lots use the correct lot step? Have you normalised the lot double before sending the order?

Execution Error Codes

Jeezz might have something...are you testing on a broker and/or account that will allow .01 Lots??? Many won't....FXDD etc...Posting the OrderSend lines (and broker) may help...

Hi jezzer and Bill, yes it is attempting to trade...the journal even shows it continuing even after the tester has apparently run its course...strange.

minimum and maximum lots.....no...i didn't do anything with that. But it had worked previously, until i changed that line in the first post.

normalized....just the 'extern double Lot=0.1;'

lot step....I'm not sure if it has it, or even needs it? I'm just multiplying times 2.

I'm trying this on both IBFX and Alpari UK demo. I switched to hardcoding Lots=0.1, not 0.01 thinking that would cover all brokers and atleast get this thing going. Funyoo posted a very basic MACD crossing the zero line with a martingale, that stepped down one after a win. That's where all this stems from. I just want it to go back to Lots=0.1 after a win.

below, i've posted his code for the ordersend,

Code:

// expert open position value
if((AddP()&&Add_Positions&&OP<=MaxOrders)||(OP==0&&!Add_Positions)){
if(OS==1){if(TP==0)TPI=0;else TPI=Bid-TP*Point;if(SL==0)SLI=0;else SLI=Bid+SL*Point;TK=OrderSend(Symbol(),OP_SELL,Lots,Bid,Slip,SLI,TPI,OrSt,Magic,0,Red);OS=0;return(0);}	
if(OB==1){if(TP==0)TPI=0;else TPI=Ask+TP*Point;if(SL==0)SLI=0;else SLI=Ask-SL*Point;TK=OrderSend(Symbol(),OP_BUY,Lots,Ask,Slip,SLI,TPI,OrSt,Magic,0,Lime);OB=0; return(0);}}
for(j=0;j<OrdersTotal();j++){if(OrderSelect(j,SELECT_BY_POS,MODE_TRADES)){if(OrderSymbol()==Symbol()&&((OrderMagicNumber()==Magic)||Magic==0)){TrP();}}}return(0);}

thanks for your input, I just don't want to bother you guys for such a silly thing. Always some newbie trying to reinvent the wheel

**** ok, I see now the original EA doesn't "OrderSend" on backtesting anyway! I was forward testing on demo last week, so I assumed it would backtest too! Apparently not. So maybe let's just forget it. Since my logical code made sense, maybe the thing will work in demo forward test anyway....sigh...****

As I said previously, error 131 is because the EA is trying to send an order of the incorrect lot size.

Once you have calculated your lot size, it is important to process the lot size further so that it is returned in a manner which your broker will accept. If you send an order for 1.5467 lots for instance, your broker's server will not understand.

Firstly, you need to ensure that it uses the correct lot step.

Code:

double LotStep=MarketInfo(Symbol(),MODE_LOTSTEP);
Lots=NormalizeDouble(Lots/LotStep,0)*LotStep;

Secondly, you need to check that it doesn't fall outside the minimum/maximum range that your broker will accept, so you need to run a line of code to do this.

Code:

double MinLotSize=MarketInfo(Symbol(),MODE_MINLOT);
if (Lots<MinLotSize) Lots=MinLotSize;

You would also run a similar line to ensure it didn't exceed the maximum acceptable lot size.

Once you've done that I suspect that your EA will start to send orders correctly.