Return to Revisions

The current version of the code is ...

try
{
    if (GetDataFromServer())
    {
        ProcessData();
    }
    else
    {
        return null;
    }
}
catch(Exception ex)
{
    return null;
}

... which makes sense.

The version which you originally posted said, DoSomethingElse(); instead of return null;.

The original version didn't make sense to me, because I didn't know:

• Might DoSomethingElse() throw an exception?
• Is it be harmful or expected to invoke DoSomethingElse() twice?

Because it didn't make sense I didn't answer it.

The reason why it didn't make sense is that you altered (i.e. obfuscated) it before you posted it on CR.