Renko Trend Trading EA NOT WORKING - PLEASE HELP

[czjlj]

I don't know why its not working.
Can anybody fix this EA?
Your kind help in this regard is much appreciated.

Thanks!

[phil_trade]

I don't know why its not working.
Can anybody fix this EA?
Your kind help in this regard is much appreciated.

Thanks!

May be you can first explain :

1 - How is it suppose to work

2 - what is the problem

Philippe

[czjlj]

When I used default sets,the EA get error 131.
Then I set lot=0.1,but it trade only once and no more.
I'm using Ironfx broker terminal (build 509)
I need some one to fix this bug, 'cause I'm not a coder.

[czjlj]

If I set Rsi=false,the EA get error 4108.

222.png

333.png

[phil_trade]

When I used default sets,the EA get error 131.
Then I set lot=0.1,but it trade only once and no more.
I'm using Ironfx broker terminal (build 509)
I need some one to fix this bug, 'cause I'm not a coder.

It's just a PARAM problem cause your broker accept only 0.1 lot for this pairs.

But, before saying there is a bug in an EA, please learn more about forex ( Min lot etc )

[phil_trade]

If I set Rsi=false,the EA get error 4108.

222.png

333.png

Same advice as previously ! Learn more about forex and about this EA

This message is about ECN Broker Yes/No. Look deeper in parameter.

and please...learn more

[czjlj]

If I set Rsi=false,the EA get error 4108.

222.png

333.png

Same advice as previously ! Learn more about forex and about this EA

This message is about ECN Broker Yes/No. Look deeper in parameter.

and please...learn more

Thanks for your Replies!

I'm not a newbie.Just describe the errors for you.
I think I can solve those problems,like error 131,error 4108 etc.
But the ea trade only once with default parameter(lot>=0.1).I have no solution.

[SteveHopwood]

If I set Rsi=false,the EA get error 4108.

222.png

333.png

Same advice as previously ! Learn more about forex and about this EA

This message is about ECN Broker Yes/No. Look deeper in parameter.

and please...learn more

Thanks for your Replies!

I'm not a newbie.Just describe the errors for you.
I think I can solve those problems,like error 131,error 4108 etc.
But the ea trade only once with default parameter(lot>=0.1).I have no solution.

Listen dimwit. I can get deeply offended when some cretin accuses some of my code to not work because the user is too thick to understand the basics.

Others generously provided the answers here yet you appear to be too stupid to understand them.

Your membership here hangs by a thread and one more insulting post will result in its termination. Learn the basics before you post here again.

Capiche?

[czjlj]

SteveHopwood
I will try to learn.
Thank you for your teachings so much.
Please accept my apologies for any trouble my mistake has caused you.

[czjlj]

Look at the fourth and the fifth TradeDirection = none please.
Aren't they Effective？

Code: Select all

void GetBrickDirection()
{
   //Sets TradeDirection to up, down or none depending on their direction for the previous LookBackBricks bricks

   if (OldBars == Bars) return;//Only check at the start of each new candle
   //OldBars is reset in start() so that the ea can check the trailing stop loss at the start of each new brick
   
   TradeDirection = none;
   
   //Previous brick direction
   if (Close[1] > Open[1] ) TradeDirection = up;
   if (Close[1] < Open[1] ) TradeDirection = down;
   
   //Confirm that previous LookBackBricks were in the same direction
   for (BrickCount = 2; BrickCount <= LookBackBricks; BrickCount++)
   {
      //Check rising bricks
      if (TradeDirection == up)
      {
         if (Close[BrickCount] < Open[BrickCount] ) 
         {
            TradeDirection = none;
            return;
         }//if (Close[BrickCount] < Open[BrickCount] )  

      }//if (TradeDirection == up)

    
      //Check rising bricks
      if (TradeDirection == down)
      {
         if (Close[BrickCount] > Open[BrickCount] ) 
         {
            TradeDirection = none;
            return;
         }//if (Close[BrickCount] > Open[BrickCount] )  
      
      }//if (TradeDirection == down)
      
   }//for (BrickCount = 0; BrickCount <= Bars; BrickCount++)
   
   //Got this far, so all the bricks have been going in the same direction.
   //Check that the prior to BrickCount was in the opposite direction.
   //This is intended to avoid getting into a trade late in a move.
   if (TradeDirection == up)
   {
      if (Close[BrickCount] > Open[BrickCount] )//Was another riser
      {
         TradeDirection = none;
         return;
      }//if (Close[BrickCount] > Open[BrickCount] )
   }//if (TradeDirection == up)

   if (TradeDirection == down)
   {
      if (Close[BrickCount] < Open[BrickCount] )//Was another faller
      {
         TradeDirection = none;
         return;
      }//if (Close[BrickCount] > Open[BrickCount] )
   }//if (TradeDirection == down)

   
   
}//void GetBrickDirection()

Let's look at this code again.
Is modify if (OrdersTotal() == 0 || !TradeExists) better than if (!TradeExists)?

Code: Select all

   ///////////////////////////////////////////////////////////////////////////////////////////////         
   //Trading
   if (!TradeExists)
   {
      LookForTradingOpportunities();
   }//if (!TradeExists)
   /////////////////////////////////////////////////////////////////////////////////////////////// 

Maybe I'm wrong.
Please correct me.
Thx!