Weekly MAx and Low - Previous and Current

To add comments, please log in or register
Diego Castilho
1064
Diego Castilho  

Hi, 

I´m trygin to add in  my indicator the information in Text about the status of the week. I mean, current week with minimum, compared with previous week, max or neutral (wihtout max or min) . 

Values of max, min of previous week and current week are ok. I have printed it and confirmed throught the chart.

Problem, there is something else to do with if .. because, it doe show the correct value.  Not sure why it is not working - any help is appreaciated.

maxWAtual = Max value of current week 

maxWAnterior = Max value of Previous week

minWAtual = Min value of current week

minWAnterior = Min value of Previous week.



  // Weekly Max and Min 

  

    if (maxWAtual > maxWAnterior && minWAtual < minWAnterior){

    string Weekmax = " W - Max";

    ObjectCreate (0, textWmax, OBJ_LABEL , 0 , 0 , 0 ); 

    ObjectSetInteger (0, textWmax, OBJPROP_CORNER , CORNER_LEFT_LOWER); 

    ObjectSetInteger (0, textWmax, OBJPROP_ANCHOR , ANCHOR_LEFT_LOWER ); 

    ObjectSetInteger (0, textWmax, OBJPROP_XDISTANCE , 130); 

    ObjectSetInteger (0, textWmax, OBJPROP_YDISTANCE , 03); 

    ObjectSetString (0, textWmax, OBJPROP_TEXT , Weekmax); 

    ObjectSetInteger (0, textWmax, OBJPROP_COLOR , clrRed); 

    ObjectSetString (0, textWmax, OBJPROP_FONT , "Trebuchet MS" ); 

    ObjectSetInteger (0, textWmax, OBJPROP_FONTSIZE , 10); 

    ObjectSetInteger (0, textWmax, OBJPROP_BACK , true ); // object in the backgroun

    }

    

    if (minWAtual > minWAnterior && maxWAtual < maxWAnterior){

    string Weekmin = "W - Min";

    ObjectCreate (0, textWmax, OBJ_LABEL , 0 , 0 , 0 ); 

    ObjectSetInteger (0, textWmax, OBJPROP_CORNER , CORNER_LEFT_LOWER); 

    ObjectSetInteger (0, textWmax, OBJPROP_ANCHOR , ANCHOR_LEFT_LOWER ); 

    ObjectSetInteger (0, textWmax, OBJPROP_XDISTANCE , 130); 

    ObjectSetInteger (0, textWmax, OBJPROP_YDISTANCE , 03); 

    ObjectSetString (0, textWmax, OBJPROP_TEXT , Weekmin); 

    ObjectSetInteger (0, textWmax, OBJPROP_COLOR , clrRed); 

    ObjectSetString (0, textWmax, OBJPROP_FONT , "Trebuchet MS" ); 

    ObjectSetInteger (0, textWmax, OBJPROP_FONTSIZE , 10); 

    ObjectSetInteger (0, textWmax, OBJPROP_BACK , true ); // object in the backgroun

    

    }

    

    if (minWAtual < minWAnterior && maxWAtual < maxWAnterior) {

    string Weekneutral = "W - Neutral";

    ObjectCreate (0, textWmax, OBJ_LABEL , 0 , 0 , 0 ); 

    ObjectSetInteger (0, textWmax, OBJPROP_CORNER , CORNER_LEFT_LOWER); 

    ObjectSetInteger (0, textWmax, OBJPROP_ANCHOR , ANCHOR_LEFT_LOWER ); 

    ObjectSetInteger (0, textWmax, OBJPROP_XDISTANCE , 130); 

    ObjectSetInteger (0, textWmax, OBJPROP_YDISTANCE , 03); 

    ObjectSetString (0, textWmax, OBJPROP_TEXT , Weekneutral); 

    ObjectSetInteger (0, textWmax, OBJPROP_COLOR , clrRed); 

    ObjectSetString (0, textWmax, OBJPROP_FONT , "Trebuchet MS" ); 

    ObjectSetInteger (0, textWmax, OBJPROP_FONTSIZE , 10); 

    ObjectSetInteger (0, textWmax, OBJPROP_BACK , true ); // object in the backgroun

    }

    

    if (minWAtual > minWAnterior && maxWAtual > maxWAnterior){

    string Weekmaxmin = "W - Max,Min";

    ObjectCreate (0, textWmax, OBJ_LABEL , 0 , 0 , 0 ); 

    ObjectSetInteger (0, textWmax, OBJPROP_CORNER , CORNER_LEFT_LOWER); 

    ObjectSetInteger (0, textWmax, OBJPROP_ANCHOR , ANCHOR_LEFT_LOWER ); 

    ObjectSetInteger (0, textWmax, OBJPROP_XDISTANCE , 130); 

    ObjectSetInteger (0, textWmax, OBJPROP_YDISTANCE , 03); 

    ObjectSetString (0, textWmax, OBJPROP_TEXT , Weekmaxmin); 

    ObjectSetInteger (0, textWmax, OBJPROP_COLOR , clrRed); 

    ObjectSetString (0, textWmax, OBJPROP_FONT , "Trebuchet MS" ); 

    ObjectSetInteger (0, textWmax, OBJPROP_FONTSIZE , 10); 

    ObjectSetInteger (0, textWmax, OBJPROP_BACK , true ); // object in the backgroun

    

    }



    Thanks in advance!

Sergey Golubev
Moderator
108344
Sergey Golubev  

Forum on trading, automated trading systems and testing trading strategies

When you post code please use the CODE button (Alt-S)!

Use the CODE button

Magruder Trading Group, Inc.
167
Chad Magruder  

Only thing I can see with this code not working the you expect is if you have the names wrong for the types of weeks you are trying to display in text. As it is now the logic is this...

W-Max = Higher High & Lower Low - Engulfing bar
W-Min = HL & LH - Inside bar
W-Neutral = LL & LH - Down
W-Max,Min = HL & HH - Up

I think you need to swap the names around for the Text variable, try this code...


maxWAtual = Max value of current week 

maxWAnterior = Max value of Previous week

minWAtual = Min value of current week

minWAnterior = Min value of Previous week.



  // Weekly Max and Min 

  

    if (maxWAtual > maxWAnterior && minWAtual < minWAnterior){

    string Week = " W - Max,Min";

    ObjectCreate (0, textWmax, OBJ_LABEL , 0 , 0 , 0 ); 

    ObjectSetInteger (0, textWmax, OBJPROP_CORNER , CORNER_LEFT_LOWER); 

    ObjectSetInteger (0, textWmax, OBJPROP_ANCHOR , ANCHOR_LEFT_LOWER ); 

    ObjectSetInteger (0, textWmax, OBJPROP_XDISTANCE , 130); 

    ObjectSetInteger (0, textWmax, OBJPROP_YDISTANCE , 03); 

    ObjectSetString (0, textWmax, OBJPROP_TEXT , Week); 

    ObjectSetInteger (0, textWmax, OBJPROP_COLOR , clrRed); 

    ObjectSetString (0, textWmax, OBJPROP_FONT , "Trebuchet MS" ); 

    ObjectSetInteger (0, textWmax, OBJPROP_FONTSIZE , 10); 

    ObjectSetInteger (0, textWmax, OBJPROP_BACK , true ); // object in the backgroun

    }

    

    if (minWAtual > minWAnterior && maxWAtual < maxWAnterior){

    string Week = "W - Neutral";

    ObjectCreate (0, textWmax, OBJ_LABEL , 0 , 0 , 0 ); 

    ObjectSetInteger (0, textWmax, OBJPROP_CORNER , CORNER_LEFT_LOWER); 

    ObjectSetInteger (0, textWmax, OBJPROP_ANCHOR , ANCHOR_LEFT_LOWER ); 

    ObjectSetInteger (0, textWmax, OBJPROP_XDISTANCE , 130); 

    ObjectSetInteger (0, textWmax, OBJPROP_YDISTANCE , 03); 

    ObjectSetString (0, textWmax, OBJPROP_TEXT , Week); 

    ObjectSetInteger (0, textWmax, OBJPROP_COLOR , clrRed); 

    ObjectSetString (0, textWmax, OBJPROP_FONT , "Trebuchet MS" ); 

    ObjectSetInteger (0, textWmax, OBJPROP_FONTSIZE , 10); 

    ObjectSetInteger (0, textWmax, OBJPROP_BACK , true ); // object in the backgroun

    

    }

    

    if (minWAtual < minWAnterior && maxWAtual < maxWAnterior) {

    string Week = "W - Min";

    ObjectCreate (0, textWmax, OBJ_LABEL , 0 , 0 , 0 ); 

    ObjectSetInteger (0, textWmax, OBJPROP_CORNER , CORNER_LEFT_LOWER); 

    ObjectSetInteger (0, textWmax, OBJPROP_ANCHOR , ANCHOR_LEFT_LOWER ); 

    ObjectSetInteger (0, textWmax, OBJPROP_XDISTANCE , 130); 

    ObjectSetInteger (0, textWmax, OBJPROP_YDISTANCE , 03); 

    ObjectSetString (0, textWmax, OBJPROP_TEXT , Week); 

    ObjectSetInteger (0, textWmax, OBJPROP_COLOR , clrRed); 

    ObjectSetString (0, textWmax, OBJPROP_FONT , "Trebuchet MS" ); 

    ObjectSetInteger (0, textWmax, OBJPROP_FONTSIZE , 10); 

    ObjectSetInteger (0, textWmax, OBJPROP_BACK , true ); // object in the backgroun

    }

    

    if (minWAtual > minWAnterior && maxWAtual > maxWAnterior){

    string Week = "W - Max";

    ObjectCreate (0, textWmax, OBJ_LABEL , 0 , 0 , 0 ); 

    ObjectSetInteger (0, textWmax, OBJPROP_CORNER , CORNER_LEFT_LOWER); 

    ObjectSetInteger (0, textWmax, OBJPROP_ANCHOR , ANCHOR_LEFT_LOWER ); 

    ObjectSetInteger (0, textWmax, OBJPROP_XDISTANCE , 130); 

    ObjectSetInteger (0, textWmax, OBJPROP_YDISTANCE , 03); 

    ObjectSetString (0, textWmax, OBJPROP_TEXT , Week); 

    ObjectSetInteger (0, textWmax, OBJPROP_COLOR , clrRed); 

    ObjectSetString (0, textWmax, OBJPROP_FONT , "Trebuchet MS" ); 

    ObjectSetInteger (0, textWmax, OBJPROP_FONTSIZE , 10); 

    ObjectSetInteger (0, textWmax, OBJPROP_BACK , true ); // object in the backgroun

    

    }
Diego Castilho
1064
Diego Castilho  
Chad Magruder:

Only thing I can see with this code not working the you expect is if you have the names wrong for the types of weeks you are trying to display in text. As it is now the logic is this...

W-Max = Higher High & Lower Low - Engulfing bar
W-Min = HL & LH - Inside bar
W-Neutral = LL & LH - Down
W-Max,Min = HL & HH - Up

I think you need to swap the names around for the Text variable, try this code...


Now, i is working properly . 


Thanks for your help ! really appreacited.

To add comments, please log in or register