Can't exit at 5am using ToTime

[ShruggedAtlas]

Trying to exit positions at 5am after entering after 1am. Entries are working fine but the exits occur just one bar later instead of staying in the position until 5am. What am I doing wrong?

Exit code is as follows:

                switch(Position.MarketPosition)
                {
                    case MarketPosition.Short:
                        if (ToTime(Time[0]) >= 50000)
                            
                            Print("Time is " + Time.ToString());
                            Print("Exiting SHORT Position");
                            ExitShort(shares,"ES", "E1");    // ES is name of exit and E1 is the entry name                                      
                            // ExitShort(int quantity, string signalName, string fromEntrySignal)
                            break;
                        
                    case MarketPosition.Long:
                        if (ToTime(Time[0]) >= 50000)
                            
                            Print("Time is " + Time.ToString());
                            Print("Exiting LONG Position");
                            ExitLong(shares,"ES", "E1");    // ES is name of exit and E1 is the entry name
                                                              // ExitShort(int quantity, string signalName, string fromEntrySignal)
                        
                            break;
                }

Also, curiously, the only print statement in the exit code that prints to output is the phrase 'Exiting short/long position'. The "Time is xxxx" statement does not print. I don't know why.

[NinjaTrader_PaulH]

Hello ShruggedAtlas,

Thanks for your post.

From the code you posted it appears you are missing "{}" after the if statement to contain all the statements. As written, the if statement only affects the next statement which is a print statement, all others would be immediately executed.

              switch(Position.MarketPosition)
                {
                    case MarketPosition.Short:
                        if (ToTime(Time[0]) >= 50000)
                           [COLOR="Red"] {[/COLOR]
                            Print("Time is " + Time.ToString());
                            Print("Exiting SHORT Position");
                            ExitShort(shares,"ES", "E1");    // ES is name of exit and E1 is the entry name              
[COLOR="red"]                           } [/COLOR]                       
                            // ExitShort(int quantity, string signalName, string fromEntrySignal)
                            break;
                        
                    case MarketPosition.Long:
                        if (ToTime(Time[0]) >= 50000)
                           [COLOR="red"] {[/COLOR]
                            Print("Time is " + Time.ToString());
                            Print("Exiting LONG Position");
                            ExitLong(shares,"ES", "E1");    // ES is name of exit and E1 is the entry name
[COLOR="red"]                            }   [/COLOR]                                                          // ExitShort(int quantity, string signalName, string fromEntrySignal)
                        
                            break;
                }

[ShruggedAtlas]

grrrr...I should have known. Thank you that was the solution precisely.