Question

I’m maintaining a number of console applications at work and one thing I’ve been noticing in a number of them is that they call Environment.Exit(0).

A sample program would look like this:

public class Program {     public static void Main(string[] args)     {         DoStuff();         Environment.Exit(0);     } } 

I don’t understand what the intent of the original programmer was in doing this? In my mind even without the Environment.Exit statement the program should exit just fine. That said, for one of these programs, it’s console window has been remaining even after it was supposed to have closed so I’m really not sure what’s going on there….

Am I missing something here? Or is there a reason why Environment.Exit should be called in this instance?

Answer 1

The only reason for calling Exit() as the last line of the Main method is if there might be other foreground threads running. They would stay running if execution just fell off the end of Main. Even in this case, it would usually be a better idea either to put in some explicit graceful termination into the other threads – or make them background threads to start with.

If you ever want to return a different exit code from Main, the simpler way to achieve that is to declare it to return int.

In short, I don’t think you need Environment.Exit() here, and it’s worth asking your colleagues exactly why they’re using it – chances are they won’t be able to give you a good reason, and it’s another bit of fluff you can cut out.