Dice Tries to be "Techy"

Here is an ad that Dice published recently:


if (myJob == boring) {
     // Go to Dice for great Java jobs
     sendRedirect("http://www.dice.com");
} else {
     suck_it_up();
}


I thought this ad was cute when I first saw it. However, the second time it popped up on my web browser I realized something wasn't quite right.

Dice is a website for job postings. They ran this ad on SourceForge looking for Java programmers who are on the market. The thing is, if you actually read the code, the else clause doesn't make sense. OK sure, if myjob is boring then go look for a new job, but the else clause says suck it up. That effectively states, if your job is not boring then suck it up, which seems odd.

The proper code might be something like this:


if (myJob == boring) {
     // Go to Dice for great Java jobs
     sendRedirect("http://www.dice.com");
} else {
     // You job isn't boring
     celebrate();
}


Alternatively, I'd accept this:


if (myJob == boring && CanQuit) {
     // Go to Dice for great Java jobs
     sendRedirect("http://www.dice.com");
} else {
     // Can't quit or job not boring
     suck_it_up();
}


It still don't make much sense for the job is not boring case but it's better.