Click here to Skip to main content
15,880,796 members
Articles / Programming Languages / C#
Alternative
Tip/Trick

The goto-less goto!

Rate me:
Please Sign up or sign in to vote.
3.25/5 (4 votes)
24 Jan 2011CPOL 9.5K   2   5
I'm a C# programmer. I try to avoid goto at all costs, and generally I try to avoid throwing exceptions for the purpose of controlling the flow of execution. I've always been of the opinion that exceptions are for exceptional situations and indicate errors rather than expected conditions.I...
I'm a C# programmer. I try to avoid goto at all costs, and generally I try to avoid throwing exceptions for the purpose of controlling the flow of execution. I've always been of the opinion that exceptions are for exceptional situations and indicate errors rather than expected conditions.

I suggest encapsulating this sort of code in its own method. You can just return from the method early if one of the goto conditions is met.

public void SomeMethod()
{
    MethodThatReturnsEarlyIfCertainConditionsAreMet();
}

private void MethodThatReturnsEarlyIfCertainConditionsAreMet()
{
    // Using a random number is for illustrative purposes only;
    // in a real situation you might be testing some other kind
    // of value set elsewhere in the code or whatever.
    Random random = new Random();
    int i = random.Next(10);

    if (i < 10) return; // early return; our condition was met.

    DoOtherWork();

    i = random.Next(10);

    if (i < 5) return; // early return; a different condition was met.

    DoMoreOtherWork();

    // etc.
}


While I like the idea of all methods having only one return, in practice I don't feel that it's worth it, especially for situations like this.

I don't know how this sort of thing would behave in C++, but I know in C#, the CLR would handle cleanup of your locals when the method returns, which ultimately includes calling destructors if they are used, though Microsoft's recommendation in .NET languages is to implement IDisposable when you have unmanaged resources to clean up.

License

This article, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)


Written By
United States United States
This member has not yet provided a Biography. Assume it's interesting and varied, and probably something to do with programming.

Comments and Discussions

 
GeneralFine if you have no cleanup. Pin
EFEaglehouse5-Apr-11 4:21
EFEaglehouse5-Apr-11 4:21 
GeneralReason for my vote of 3 This is fine if you have no cleanup ... Pin
EFEaglehouse5-Apr-11 4:20
EFEaglehouse5-Apr-11 4:20 
Reason for my vote of 3
This is fine if you have no cleanup to do. Otherwise, this applies to a different situation than the original article.
GeneralI prefer multiple returns in most cases - the meat of the me... Pin
SimmoTech3-Mar-11 21:21
SimmoTech3-Mar-11 21:21 
GeneralAlso, only one return is easier for debugging. Only one spot... Pin
P2Labs14-Feb-11 2:04
P2Labs14-Feb-11 2:04 
GeneralReason for my vote of 1 Having one only one return statement... Pin
Wolfgang_Baron9-Feb-11 4:40
professionalWolfgang_Baron9-Feb-11 4:40 

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Praise Praise    Rant Rant    Admin Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.