Click here to Skip to main content
16,017,726 members
Articles / Programming Languages / C#
Alternative
Tip/Trick

The goto-less goto!

Rate me:
Please Sign up or sign in to vote.
4.95/5 (12 votes)
25 Jan 2011CPOL 41.8K   4   33
Alternative 2 can be enhanced by returning a booleen value if we want to code something similar to the original example. A (early) return value of false would indicate a failure which could then be handle by checking the function result.if (!DoAllActions()){ ...
Alternative 2 can be enhanced by returning a booleen value if we want to code something similar to the original example. A (early) return value of false would indicate a failure which could then be handle by checking the function result.

if (!DoAllActions())
{
   DoFailedCleanup();
}

bool DoAllActions()
{
   if (condition1_fails)
      return false;
   if (condition2_fails)
      return false;
   ...
   if(!conditionN_fails)
      retrun false;
   PerformActionOnAllSuccess();
   DoNormalCleanup();
   return true;
}


Alternatively, if each action are independant, a variation like this might be more appropriate:
if (DoAction1() && 
    DoAction2() && 
    ... && 
    DoActionN())
{
   PerformActionOnAllSuccess();
   DoNormalCleanup();
}
else 
{
   DoFailedCleanup();
}

bool DoAction1()
{
   bool condition1_fails = ...;
   return !condition1_fails;
}
...

License

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


Written By
Software Developer (Senior)
Canada Canada
Programmer at Maid LABS from 2003 (www.maidlabs.com)

Programmer-Analyst at Viasat Geo Technoligies from 1995 to 2002 (www.viasat-geo.com).

I have studied at École Polytechnique de Montréal in computer engineering.

Comments and Discussions

 
GeneralMy vote of 5 Pin
pasztorpisti26-Aug-13 9:34
pasztorpisti26-Aug-13 9:34 
GeneralRe: Surely anyone can program whatever he wants for himself and ... Pin
Wolfgang_Baron7-Apr-11 6:07
professionalWolfgang_Baron7-Apr-11 6:07 
GeneralRe: Clearly we have difference experiences - and debuggers. One ... Pin
EFEaglehouse7-Apr-11 4:05
EFEaglehouse7-Apr-11 4:05 
GeneralRe: The first person wanting to add a debug statement at the end... Pin
Wolfgang_Baron7-Apr-11 1:45
professionalWolfgang_Baron7-Apr-11 1:45 
GeneralSorry, but in C++ at least, not being able to add a debug st... Pin
jefito12-Apr-11 3:02
jefito12-Apr-11 3:02 
GeneralMultiple returns look clear only to those, who have written ... Pin
Wolfgang_Baron9-Feb-11 5:32
professionalWolfgang_Baron9-Feb-11 5:32 
GeneralRe: Never say never. Sometimes the clearest code is the shortest... Pin
EFEaglehouse5-Apr-11 4:32
EFEaglehouse5-Apr-11 4:32 
GeneralIn fact, with this pattern where the complex function is spl... Pin
Philippe Mori9-Feb-11 5:00
Philippe Mori9-Feb-11 5:00 
GeneralReason for my vote of 1 People who put more than one return ... Pin
Wolfgang_Baron9-Feb-11 4:35
professionalWolfgang_Baron9-Feb-11 4:35 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
pasztorpisti26-Aug-13 9:27
pasztorpisti26-Aug-13 9:27 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
Wolfgang_Baron7-Oct-13 3:08
professionalWolfgang_Baron7-Oct-13 3:08 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
pasztorpisti7-Oct-13 3:24
pasztorpisti7-Oct-13 3:24 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
Wolfgang_Baron7-Oct-13 5:10
professionalWolfgang_Baron7-Oct-13 5:10 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
pasztorpisti7-Oct-13 5:39
pasztorpisti7-Oct-13 5:39 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
Wolfgang_Baron7-Oct-13 13:08
professionalWolfgang_Baron7-Oct-13 13:08 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
pasztorpisti7-Oct-13 22:37
pasztorpisti7-Oct-13 22:37 
Wolfgang_Baron wrote:
lets just agree on not agreeing before we start throwing (pun intended) things at each

... and 1-2 sentences later:
Wolfgang_Baron wrote:
You seem to prefer the kind of spaghetti like sequences of code with lots of return statements, because they are easily written without having to think too much before hand.

Spaghetti and multiple returns are two different things. A function can be spaghetti with both multiple returns and with single point of return but with so many years of experience you probably know this. Please point at my sentence where I state that I like/prefer spaghetti, I think you could easily find posts where I state the opposite. Just a single or two return statements at the beginning of a function (handling for example two simple sanity checks) saves you from embedding the rest of the function into nested blocks and from writing terribly long if conditions to analyze. Could you please make my spaghetti function below more readable without multiple returns because I'm lazy to think too much:
C++
float Func()
{
    float res1 = Func1();
    if (res1 < 0)
        return 0;
    float res2 = Func2();
    if (res2 < 0)
        return 0;
    float res3 = Func3();
    if (res3 < 0)
        return 0;
    return res1 + res2 + res3;
}

Wolfgang_Baron wrote:
I have long years of experience with both kinds of code

I know there tere are several respectful members on codeproject with extensive experience in a lot of areas but I have never seen them reasoning in a topic with having X years of experience.
Wolfgang_Baron wrote:
Whether or not the rest of your code is RAII is not always under your control. If you have to use library code, that isn't, your simply are out of luck.

Not true. This is something that makes multiple returns safe even if you return from the middle of a function. Multiple returns often happen at the beginning of a function (sanity check, simple cases) where you usually have nothing to cleanup anyway.
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
Wolfgang_Baron9-Oct-13 14:59
professionalWolfgang_Baron9-Oct-13 14:59 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
Philippe Mori9-Oct-13 17:35
Philippe Mori9-Oct-13 17:35 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
pasztorpisti9-Oct-13 22:07
pasztorpisti9-Oct-13 22:07 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
Philippe Mori10-Oct-13 2:24
Philippe Mori10-Oct-13 2:24 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
pasztorpisti10-Oct-13 2:46
pasztorpisti10-Oct-13 2:46 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
Wolfgang_Baron9-Oct-13 22:29
professionalWolfgang_Baron9-Oct-13 22:29 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
Philippe Mori10-Oct-13 2:36
Philippe Mori10-Oct-13 2:36 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
pasztorpisti9-Oct-13 22:18
pasztorpisti9-Oct-13 22:18 
GeneralRe: Reason for my vote of 1People who put more than one return ... Pin
Wolfgang_Baron9-Oct-13 22:59
professionalWolfgang_Baron9-Oct-13 22:59 

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.