Please Help - How to reset Variables in indicator

if(((mStoredTimeBearish + mCheckingTime) > iTime(NULL, 1, 0)) && mPriceAction == 3)
{
            if(mLowVolumePrice < iClose(NULL, 1, 1))
            {
               if((iVolume(NULL, 1, 1) < (iVolume(NULL, 1, iLowest(NULL, 1, MODE_VOLUME, mNoBars, 2)))) && (iClose(NULL, 1, 1) < iOpen(NULL, 1, 1)))
               {  
               mStoredMinVolumeTime = iTime(NULL, 1, 1);
               mPriceAction = 4;
                    ObjectCreate("mDivLine5"+Time[1], OBJ_VLINE, 0, 0, 0, 0, 0);
                    ObjectSet("mDivLine5"+Time[1], OBJPROP_COLOR, Lime);
                    ObjectSet("mDivLine5"+Time[1], OBJPROP_STYLE, STYLE_DOT);
                    ObjectSet("mDivLine5"+Time[1], OBJPROP_WIDTH, 1); 
                    ObjectSet("mDivLine5"+Time[1], OBJPROP_TIME1, iTime(NULL, 1, 1)); 
                    ObjectSet("mDivLine5"+Time[1], OBJPROP_BACK, true);
               }
            }
           if(mLowVolumePrice > iClose(NULL, 1, 1))
           {
                           DrawLine1(White, Time[1]);
                           mStoredTimeBearish = 0;
            }
}

Quoting crusader1

Disliked

Hi guys can anybody tell me how can I reset all variables that I have make in my coding? Because I have problem with this part of code... In theory when I get White line which is that last "if" I give variable mStoredTimeBearish zero value... this mean that after that, first "if" in this part of code will not be true anymore because 0+mCheckingTime is not bigger than iTime(NULL, 1, 0). But problem is that I am still receiving Lime line after I received White line which is wrong and stupid. Can someone tell me what I am doing wrong. I have attached...

Ignored

if(((mStoredTimeBearish + mCheckingTime) < iTime(NULL, 0, 0)))
{
               if((iVolume(NULL, 0, 2) > (0.95*(iVolume(NULL, 0, iHighest(NULL, 0, MODE_VOLUME, mNoBackCheck, 3))))) && (iClose(NULL, 0, 2) < iOpen(NULL, 0, 2)) && (iClose(NULL, 0, 1) > iOpen(NULL, 0, 1)))
                  {
                  if(iTime(NULL, 0, 0) > mStoredTimeBearish)//What is the point? Must be true
                    { 

                     datetime opencandle = iTime(NULL, 0, 2);
                     datetime closecandle = opencandle + (Period()*60)-1;
                     int shiftM5first = iBarShift(NULL,1,opencandle,false);
                     int shiftM5last = iBarShift(NULL,1,closecandle,false);
                     int M5LowestPriceShift = iLowest(NULL,1,MODE_LOW,shiftM5first-shiftM5last,shiftM5last + 1);
                     int M5HighestVolumeShift = iHighest(NULL,1,MODE_VOLUME,shiftM5first-shiftM5last,shiftM5last);
                     int mVolumeMaxLastBar = (iVolume(NULL, 1, M5HighestVolumeShift));
                     double mPriceMinLastBar = (iLow(NULL, 1, M5LowestPriceShift));
                     int mM1LowestPriceCandle = iLowest(NULL, 1, MODE_LOW, (M5LowestPriceShift - 1), 1);
                        if((M5LowestPriceShift == M5HighestVolumeShift) && (mVolumeMaxLastBar > (iVolume(NULL, 1, iHighest(NULL, 1, MODE_VOLUME, (M5HighestVolumeShift - 1), 1)))))
                        if(mPriceMinLastBar < (iLow(NULL, 1, iLowest(NULL, 1, MODE_LOW, (M5LowestPriceShift - 1), 1))))
                        if(iTime(NULL, 1, 0) > mAlertTime)
                          {                     
                     //if(M5LowestPriceShift == M5HighestVolumeShift)
                     {
                    ObjectCreate("mDivLine0"+Time[1], OBJ_VLINE, 0, 0, 0, 0, 0);
                    ObjectSet("mDivLine0"+Time[1], OBJPROP_COLOR, Lime);
                    ObjectSet("mDivLine0"+Time[1], OBJPROP_STYLE, STYLE_SOLID);
                    ObjectSet("mDivLine0"+Time[1], OBJPROP_WIDTH, 1); 
                    ObjectSet("mDivLine0"+Time[1], OBJPROP_TIME1, iTime(NULL, 1, M5HighestVolumeShift)); 
                    ObjectSet("mDivLine0"+Time[1], OBJPROP_BACK, true);
                    Alert(Symbol() + Period() + "M" + " Candle Number with Highest Volume V=" + mVolumeMaxLastBar + " Volume candle v=" + M5HighestVolumeShift + "HighestPriceCandle ="+ M5LowestPriceShift + " LowestPrice ="+ mPriceMinLastBar + " LowestPriceCandleAFTER =" + mM1LowestPriceCandle);
                    mStoredTimeBearish = iTime(NULL, 0, 0);
                    mStoredSellVolume = iVolume(NULL, 0, 2);
                    mHighVolumePrice = iHigh(NULL, 0, 2);
                    mPriceAction = 2;
                    mAlertTime = iTime(NULL, 1, 0);
                    //DrawHline(Red, mHighVolumePrice);
                    // Comment("HighVolume");
                     }
                           }
                   }                           
                  }
}

if(((mStoredTimeBearish + mCheckingTime) < iTime(NULL, 0, 0)))
{
               if((iVolume(NULL, 0, 2) > (0.95*(iVolume(NULL, 0, iHighest(NULL, 0, MODE_VOLUME, mNoBackCheck, 3))))) && (iClose(NULL, 0, 2) < iOpen(NULL, 0, 2)) && (iClose(NULL, 0, 1) > iOpen(NULL, 0, 1)))
                  {
                  if(iTime(NULL, 0, 0) > mStoredTimeBearish)//What is the point? Must be true

Quoting crusader1

Disliked

Yep you are right Gurmai that if is not important there... But it is not also wrong or? My point is why on that code section that I have posted up "if" can be skipped Can you help me with that part? I can explain you whole code each line what must do or better what could be done. Thanks for help Gurmai!

Ignored

Quoting crusader1

Disliked

Yep you are right Gurmai that if is not important there... But it is not also wrong or? - this was answer to your Q about if(iTime(NULL, 0, 0) > mStoredTimeBearish)//What is the point? Must be true My point is why on that code section that I have posted up "if" can be skipped Can you help me with that part?

Ignored

Quoting Kilian19

Disliked

I don't really understand what you want as well sorry. But one thing that helps troubleshooting is putting print statements at the important sections and to look if the variables really contain the values they are supposed to hold.

Ignored

//+------------------------------------------------------------------+
void log(string s1,    string s2="",  string s3="",  string s4="",  string s5="",  string s6="",  string s7="",  string s8="",
         string s9="", string s10="", string s11="", string s12="", string s13="", string s14="", string s15="")   {
//+------------------------------------------------------------------+
// Outputs up to 15 values to Microsoft DebugView app (debugging alternative for MQL4)
// see http://www.forexfactory.com/showthread.php?t=245303 thread for more info
   string out = StringTrimRight(StringConcatenate(WindowExpertName(), ".mq4 ", Symbol(), " ", s1, " ", s2, " ", s3, " ", s4, " ", s5,
                    " ", s6, " ", s7, " ", s8, " ", s9, " ", s10, " ", s11, " ", s12, " ", s13, " ", s14, " ", s15));
   OutputDebugStringA(out);
}

Quoting hanover

Disliked

{quote}

Ignored

Quoting Gumrai

Disliked

I have looked through your code and although I cannot work out what you are trying to achieve, I may or may not have an explanation You posted a 1 minute chart in your first post. In this case it is normal to expect that the indicator has been running on the 1 minute chart. My question to you is did you run the indicator in a different time-frame and then change to the M1 chart? If you run this indicator on M15 or M30 charts, then your posted chart may make sense as the dotted lime line will be created before the White line, so before mStoredTimeBearish...

Ignored

if(((mStoredTimeBearish + mCheckingTime) < iTime(NULL, 5, 0)))
{
               if((iVolume(NULL, 5, 2) > (0.95*(iVolume(NULL, 5, iHighest(NULL, 5, MODE_VOLUME, mNoBackCheck, 3)))))
               && (iLow(NULL, 5, 2) < iLow(NULL, 5, (iLowest(NULL, 5, MODE_LOW, 5, 3)))) 
               && ((iClose(NULL, 5, 2) < iOpen(NULL, 5, 2)) || ((iClose(NULL, 5, 2) > iOpen(NULL, 5, 2)) && ((iHigh(NULL, 5, 2) - iOpen(NULL, 5, 2)) < (iOpen(NULL, 5, 2) - iLow(NULL, 5, 2)))))
               && (iClose(NULL, 5, 1) < iOpen(NULL, 5, 1)))
                  {
                  if(iTime(NULL, 5, 0) > mStoredTimeBearish)
                    { 
                    mStoredTimeBearish = iTime(NULL, 5, 0);
                    mLowVolumePrice = iLow(NULL, 5, 2);
                     datetime opencandle = iTime(NULL, 0, 2);
                     datetime closecandle = opencandle + (Period()*60)-1;
                     int shiftM5first = iBarShift(NULL,1,opencandle,false);
                     int shiftM5last = iBarShift(NULL,1,closecandle,false);
                     int M5LowestPriceShift = iLowest(NULL,1,MODE_LOW,shiftM5first-shiftM5last,shiftM5last + 1);
                     int M5HighestVolumeShift = iHighest(NULL,1,MODE_VOLUME,shiftM5first-shiftM5last,shiftM5last);
                     int mVolumeMaxLastBar = (iVolume(NULL, 1, M5HighestVolumeShift));
                     double mPriceMinLastBar = (iLow(NULL, 1, M5LowestPriceShift));
                     int mM1LowestPriceCandle = iLowest(NULL, 1, MODE_LOW, (M5LowestPriceShift - 1), 1);
                    ObjectCreate("mDivLine21"+Time[1], OBJ_VLINE, 0, 0, 0, 0, 0);
                    ObjectSet("mDivLine21"+Time[1], OBJPROP_COLOR, Lime);
                    ObjectSet("mDivLine21"+Time[1], OBJPROP_STYLE, STYLE_SOLID);
                    ObjectSet("mDivLine21"+Time[1], OBJPROP_WIDTH, 1); 
                    ObjectSet("mDivLine21"+Time[1], OBJPROP_TIME1, iTime(NULL, 1, M5HighestVolumeShift)); 
                    ObjectSet("mDivLine21"+Time[1], OBJPROP_BACK, true);
                    mPriceAction = 3;
                    }
                  }
}               
if(((mStoredTimeBearish + mCheckingTime) > iTime(NULL, 1, 0)) && mPriceAction == 3)

Quoting hanover

Disliked

{quote} Somewhat off topic, sorry, in that it doesn't address the OP's question, but I've found this a much better way of debugging than either MQL4's Print() or Comment() statements. Three main reasons: 1. In some cases, output from Print() is buffered/delayed until the indicator being debugged is either re-initialized, or removed from the chart (which forces MT4 to output all of the buffered data to the log file). 2. With Print(), you need to use DoubleToStr() to output any double...

Ignored

Quoting crusader1

Disliked

{quote} Hi Gumrai. This indi works for two TFs. First part of code (red colored code) is for conditions that must happen on Upper TF for example on 5M and if this conditions are there then indi swicth to Lower TF on this example it was 1M and then continue to search condition there.

Ignored

Quote

Disliked

My question to you is did you run the indicator in a different time-frame and then change to the M1 chart?

Quoting Gumrai

Disliked

{quote} You didn't answer my question {quote}

Ignored

Quoting crusader1

Disliked

{quote} Ok sorry. Yes this attached pic is from 5M TF and I have only switched to 1M to see better each line. But answer is YES

Ignored