If at first you don't succeed... try, try again...

nospam@example.com (Simone) — Mon, 19 May 2008 16:57:02 -0700

Here's another interesting piece of code I've just digged up in a C# application I'm reviewing.

If you can come up with a bright idea about what those try/catch blocks are supposed to do, you've got more imagination than me...

class Foo
{
private int bar;

// ...snip...

public int GetNewBar()
{
int ret = 0;

lock (this)
{
try {
bar++;
ret = bar;
}
catch {
try {
bar++;
ret = bar;
}
catch {}
}
}

return ret;
}

// ...snip...
}

Don't code like this at home, kids

nospam@example.com (Simone) — Wed, 14 May 2008 10:25:13 -0700

Sometimes when you're reviewing someone else's code, you find interesting pieces of "art". Like the following loop construct in a C# application:

int i = anArray.Length - 1;

while (true)
{
if (i < 0)
break;
else
{
...
i--;
}
}

Doh.

You might be tempted to rewrite it as:

for( int i = anArray.Length - 1; i >= 0; i-- ) {
...
}

but it would be too simple, too boring, wouldn't it?

Simone 'Wiz' Tellini - Coding fun

If at first you don't succeed... try, try again...

Don't code like this at home, kids