[traderdoc]

Ne, nicht so richtig.

Beser ist auch beim Schreiben des Codes den Button mit dem # zu benutzen, um den Code schön zu strukturieren. Das liest sich besser.
CurrentTime ist nicht nötig und ich habe mal ein paar Klammern gestrichen.

extern string TradeTime_1_Begin = "09:30";
extern string TradeTime_1_End = "11:30";
extern string TradeTime_2_Begin = "15:50";
extern string TradeTime_2_End = "20:30";
//+------------------------------------------
datetime Time_1_Begin, Time_1_End;
datetime Time_2_Begin, Time_2_End;
//+------------------------------------------
void init() {
   Time_1_Begin = StrToTime(TimeToStr(TimeCurrent(), TIME_DATE)+" "+TradeTime_1_Begin);
   Time_1_End   = StrToTime(TimeToStr(TimeCurrent(), TIME_DATE)+" "+TradeTime_1_End);
   Time_2_Begin = StrToTime(TimeToStr(TimeCurrent(), TIME_DATE)+" "+TradeTime_2_Begin);
   Time_2_End   = StrToTime(TimeToStr(TimeCurrent(), TIME_DATE)+" "+TradeTime_2_End);
}
//+------------------------------------------
bool TradingTime() {
   if ((Time_1_Begin < Time_1_End && (TimeCurrent() < Time_1_Begin || TimeCurrent >= Time_1_End)) ||
      (Time_2_Begin < Time_2_End && (TimeCurrent() < Time_2_Begin || TimeCurrent() >= Time_2_End)))
   return(false);

   return(true);
}

traderdoc