Error 130 - invalid stop. Why? - Page 2

dwmcqueen · #11 (**permalink**) 12-11-2008, 04:16 AM

Here is my code - always give me a 130 error and sl and tp look a digit too short. Any thoughts??

if (RegularorCandleCloseStoploss == True)
{
if (StealthMode == False)
{
CalcStopLossPips = StopLossSell;
}
else
{
CalcStopLossPips = EmergencyStopLoss;
}
}
else
{
StopLossLevel =NormalizeDouble(Low[iLowest(NULL,0,MODE_LOW,NmbrBarsBackStopLoss,1)], 10);
CalcStopLossPips = (MathAbs(Bid + StopLossLevel)) / pp;
}

if (UseTakeProfit) TakeProfitLevel = NormalizeDouble(Bid - TakeProfitSell * Point, 10);
else TakeProfitLevel = 0.0;

Print("Sell Stoploss: ", DoubleToStr(StopLossLevel, Digits));
Print("TP: ", DoubleToStr(TakeProfitLevel, Digits));

Print("Bid: ", DoubleToStr(Bid, Digits));

Print("Digits: ", Digits);

Ticket = OrderSend(Symbol(), OP_SELL, lotsForOrder, Bid, Slippage, NormalizeDouble(Bid + (CalcStopLossPips * Point), Digits), TakeProfitLevel, "Sell(#" + InternalMagicNumber + ")", InternalMagicNumber, 0, DeepPink);

Roger09 · #12 (**permalink**) 12-11-2008, 06:28 AM

Why do you normalize 10 digits? Use Digits everywhere.

dwmcqueen · #13 (**permalink**) 12-11-2008, 01:26 PM

Thanks - still get the error, though. My code looks like this now:
if (RegularorCandleCloseStoploss == True)
{
if (StealthMode == False)
{
CalcStopLossPips = StopLossSell;
}
else
{
CalcStopLossPips = EmergencyStopLoss;
}
}
else
{
StopLossLevel =NormalizeDouble(Low[iLowest(NULL,0,MODE_LOW,NmbrBarsBackStopLoss,1)], Digits);
CalcStopLossPips = (MathAbs(Bid + StopLossLevel)) / pp;
}

if (UseTakeProfit) TakeProfitLevel = NormalizeDouble(Bid - TakeProfitSell * Point, Digits);
else TakeProfitLevel = 0.0;

Print("Sell Stoploss: ", DoubleToStr(StopLossLevel, Digits));
Print("TP: ", DoubleToStr(TakeProfitLevel, Digits));

Print("Bid: ", DoubleToStr(Bid, Digits));

Print("Digits: ", Digits);

OrderSend(Symbol(), OP_SELL, lotsForOrder, NormalizeDouble(Bid, Digits), Slippage, NormalizeDouble(Bid + (CalcStopLossPips * Point), Digits), NormalizeDouble(TakeProfitLevel, Digits), "Sell(#" + InternalMagicNumber + ")", InternalMagicNumber, 0, DeepPink);

Roger09 · #14 (**permalink**) 12-11-2008, 07:06 PM

Show me your experts logs, please.
What is pp?
I think you have a mistake when you calculate CalcStopLossPips.

dwmcqueen · #15 (**permalink**) 12-11-2008, 07:44 PM

pp is Point. The way the logic is working, though, this is getting executed:

CalcStopLossPips = StopLossSell;

Roger09 · #16 (**permalink**) 12-11-2008, 08:29 PM

OK, how much is StopLossSell?

dwmcqueen · #17 (**permalink**) 12-11-2008, 08:31 PM

It is set at 40.

Roger09 · #18 (**permalink**) 12-11-2008, 09:12 PM

Add
Print("SL: ", NormalizeDouble(Bid + (CalcStopLossPips * Point), Digits));
and show me your logs when you have an error.

dwmcqueen · #19 (**permalink**) 12-12-2008, 02:17 AM

2008.12.11 20:15:47 2008.09.17 00:00 FBMSoftwareEA USDJPY,H1: last error: 130
2008.12.11 20:15:47 2008.09.17 00:00 FBMSoftwareEA USDJPY,H1: Failed trade: OP_SELL USDJPY@105.63000000 tp@105.57000000 sl@105.66000000
2008.12.11 20:15:47 2008.09.17 00:00 FBMSoftwareEA USDJPY,H1: Failed to execute OP_SELL, after 0 retries
2008.12.11 20:15:47 2008.09.17 00:00 FBMSoftwareEA USDJPY,H1: Non-retryable error: 130
2008.12.11 20:15:47 2008.09.17 00:00 FBMSoftwareEA USDJPY,H1: OrderSend error 130
2008.12.11 20:15:47 2008.09.17 00:00 FBMSoftwareEA USDJPY,H1: About to attempt a OP_SELL 4.00000000 lots @105.63000000 sl:105.66000000 tp:105.57000000
2008.12.11 20:15:47 2008.09.17 00:00 FBMSoftwareEA USDJPY,H1: SL: 105.66
2008.12.11 20:15:47 2008.09.17 00:00 FBMSoftwareEA USDJPY,H1: Digits: 3
2008.12.11 20:15:47 2008.09.17 00:00 FBMSoftwareEA USDJPY,H1: Bid: 105.630
2008.12.11 20:15:47 2008.09.17 00:00 FBMSoftwareEA USDJPY,H1: TP: 105.570
2008.12.11 20:15:47 2008.09.17 00:00 FBMSoftwareEA USDJPY,H1: Sell Stoploss: 105.630
2008.12.11 20:15:47 2008.09.17 00:00 FBMSoftwareEA USDJPY,H1: Alert: SELL

Roger09 · #20 (**permalink**) 12-12-2008, 06:32 AM

I think you already understood your problem. You have Digits=3 and SL=40, it's too little. You have to set minimum SL(I don't no exactly your value) +Slippage.
You can check this value:
Print("Minimum SL - ",MarketInfo(Symbol(),MODE_STOPLEVEL));