Code: Select all

// Open a BUY when the price is above ALL 3 SMAs, and the two most recent completed bars have an open below their close, and the current bar has 2 pip movement from open in the LONG direction. 

// My attempt to show BUY

   if(Open[2] < Close[2] && Open[1] < Close[1] && Open[0] < (Open[0] + 2) && SMA1_Period < Open[2] && SMA2_Period < Open[2]   &&  SMA3_Period < Open[2])
   {
      ticket=OrderSend(Symbol(),OP_BUY,Lots,Ask,Slippage,Ask-StopLossPips,Ask+TakeProfitPips,"Viks_Three_Little_Ducks",MagicNumber,0,Green);
     if(ticket<0)
       {

Code: Select all

//Here is an easy way to troubleshoot these nested Conditions...

if(Open[2] < Close[2])
{
	Print("3rd Candle Open < Close");
	if( Open[1] < Close[1])
	{
		Print("Second Candle Open < Close");
		if(Open[0] < (Open[0] + 2 * Point ) //You need to get the pip value. This will depend on pair.
		{
			Print("Open was " + 2* Point+ " pips away from open");
			if(SMA1_Period < Open[2]) //SMA1 Value must be calculated somewhere else
			{
				Print("Next condition met");
				if(adnauseaum...............)
				{
					Print("All conditions met sending order here.");
					ticket= OrderSendFunction();
				}
			}
		}
	}
}

//Alternate
if(Open[2] < Close[2]) Print("3rd Candle Open < Close");
if( Open[1] < Close[1]) Print("Second Candle Open < Close");
//Troubleshooting is stoneage on this platform
//But you need to make sure there is no logic error in your conditions

Code: Select all

//+------------------------------------------------------------------+
bool IsCriticalError( int error)
   {
	switch( error)
	{
		case 4:	Print("Server is busy, will try again");
					return(false);  
		case 135:Print("Price has changed, will try again");
					return(false);  
		case 136:Print("No prices, will try again");
					return(false);  
		case 137:Print("Broker is busy, will try again");
					return(false);  
		case 146:Print("Terminal is busy, will try again");
					return(false);  
      case  2: Print("General error. No trades will be made");
					return(true);  
		case  5: Print("Old version. No trades will be made");
					return(true);  
		case 64: Print("Account is blocked. No trades will be made");
					return(true);  
		case 133:Print("Trade is prohibited. No trades will be made");
					return(true);  
		case 134:Print("Insufficient funds. No trades will be made");
					return(true);  
		default: 
			break;
   }
   
   Print("Error ", ErrorDescription( error), ". No trades will be made"); 
	return(true);  
}

Code: Select all

//Code Snippet from Steve Hopwoods shell EA code - Thanks Steve
void DisplayUserFeedback()
{
   if (IsTesting() && !IsVisualMode()) return;