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

The goto-less goto!

Rate me:
Please Sign up or sign in to vote.
0.00/5 (No votes)
7 Feb 2011CPOL 4.2K   2  
As this piece of code is normally a function, should be more readable if all of it is put inside a function returning a bool, indicating success or failure.The cleanup function could be only resource deallocation, not a function at all. If it is intended to use with C++ or C#, a try..finally...
As this piece of code is normally a function, should be more readable if all of it is put inside a function returning a bool, indicating success or failure.

The cleanup function could be only resource deallocation, not a function at all. If it is intended to use with C++ or C#, a try..finally block should be more appropriate.

This piece of code use a try block, which is not super performatic at all, but ensure that all allocated resource are properly freed.

C++
bool DoOperations()
{
  try
  {
    if(condition1_fails)
    {
      return false;
    }
    ...
    if(condition2_fails)
    {
      return false;
    }
    ...
    ...
    if(conditionN_fails)
    {
      break false;
    }

    PerformActionOnAllSuccess();

    return true;
  }
  finally
  {
    if (condition1_resource != null)
    {
      // free condition1_resource
    }

    if (condition2_resource != null)
    {
      // free condition2_resource
    }

    ...

    if (conditionN_resource != null)
    {
      // free conditionN_resource
    }
  }
}

License

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


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

Comments and Discussions

 
-- There are no messages in this forum --