Welcome to the Lounge

Forogar wrote:

This caused horribly messy code

Forogar wrote:

introduces an execution statement (return) on the same line as the "if" which is bad coding practice

MSBassSinger wrote:

my tolerance for lazy programmers that just want to hack up some code and get the minimum done with the least time - is very low.

void SomeFunction(SomeThing s)
{
	if (s == null || s.Thing == null) 	return;
	//... more code
}

void doSomething(var a)
{
    if (a == null)
    {
        throw new Exception("a == null");
        // or return with no meaningful "why"
    }
    else
    {
        //...
    }
}

void doSomething(var a)
{
    if (a != null)
    {
        //...
        // Many lines so that the error handling is separated from the test
        //...
    }
    else
    {
        throw new Exception("a == null");
        // or return with no meaningful "why"
    }
}

  while (true) {
  // Comment Describing Primary Condition 1
  if (condition1) {set %; set sMsg; break}

  // Comment Describing Primary Condition 2
  if (condition2) {set %; set sMsg; break}

  // Calculate complex logic...

  break; // prevent a loop
  }
  logIt("Rate Calculation",sMsg, % as string);
  setup return variables
  return;
}

dandy72 wrote:

That was intentional in my response.

dandy72 wrote:

Political correctness gone mad. But then, I don't know of any other type of political correctness