Click here to Skip to main content
65,938 articles
CodeProject is changing. Read more.
Articles / Languages / C#

The goto-less goto!

0.00/5 (No votes)
7 Feb 2011CPOL 4.4K  
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)