The Weird and The Wonderful

The Weird and The Wonderful forum is a place to post Coding Horrors, Worst Practices, and the occasional flash of brilliance.

We all come across code that simply boggles the mind. Lazy kludges, embarrassing mistakes, horrid workarounds and developers just not quite getting it. And then somedays we come across - or write - the truly sublime.

Post your Best, your worst, and your most interesting. But please - no programming questions . This forum is purely for amusement and discussions on code snippets. All actual programming questions will be removed.

First Prev Next

Re: It's duplicated, too lazy to fix.

CDP180220-Jul-12 0:21

CDP1802

20-Jul-12 0:21

What do you have against PHP? It's soooo easy to learn and it has been used for soooo many big projects.

At least artificial intelligence already is superior to natural stupidity

Re: It's duplicated, too lazy to fix.

Andrei Straut20-Jul-12 2:57

Andrei Straut

20-Jul-12 2:57

Yes, and at the same time, it makes it awwwwwwfully easy to code badly.
You can code badly in absolutely any language, but bad code written in PHP just.has.that.special.flavor.

[rant start]
Now, assume we have some code (PHP) along the lines of:

$invoice_no = 1;

//some code

//Then someone comes who does not read docs, or has no idea that invoice_no should be an int, and does something with the following effect:
$invoice_no = $invoice_no . 'INV_123#';

//Some other code
return $invoice_no;

Now, try to do this in Java for instance:

int invoice_no = 1;
invoice_no = invoice_no + "INV_123#";
return invoice_no;

The compiler would be all over the place, screaming like a newborn baby. And that's just off the top of my head.

And speaking of compiler and autocomplete, how about contructs like:

$invoice_no = 'Whatever';

//Code

return $invoiceno; //first occurrence of $invoiceno, the IDE doesn't complain about it, is undefined, but you sucker will only know at runtime, from a !!!Notice!!!

Or that you can't really enforce a type parameter on a method (of course, there's get_class, but you'll only know about it at runtime).

Or autocomplete. I'a big fan of autocomplete. Now, I've done PHP with both Eclipse and Netbeans. Try to get autocomplete for something like:

$db = DatabaseService::getInstance();
$line = $db->getPoLineById($line_id);
$product = $db->searchProduct('id', $line->getProductId()); //This is where the magic ends. No autocomplete for 'line->'

Or debugging. var_dump, really? Now, I know there's XDebug, but try using that in a 4GB PHP project (I'm talking about a Magento website). Good luck with that.
[rant end]

Now, I do believe your remark was sarcastic. For being sarcastically funny, have a 5! Laugh | :laugh:

Full-fledged Java/.NET lover, full-fledged PHP hater.
Full-fledged Google/Microsoft lover, full-fledged Apple hater.
Full-fledged Skype lover, full-fledged YM hater.

Re: It's duplicated, too lazy to fix.

CDP180220-Jul-12 3:18

CDP1802

20-Jul-12 3:18

Andrei Straut wrote:
Now, I do believe your remark was sarcastic

Thanks. Indeed I don't jump off bridges just because it's especially easy on this particular bridge or many people have done so before. Not even the lemmings really do that. Smile | :)

I have started to ignore languages which are interpreted or are advertised as being especially noob-friendly on 8 bit machines. Especially the 'noob-unfriendly' things (like type safety) usually are supposed to help you and I have no interest in learning how to bypass them or constantly discussing why it's a bad idea to bypass them.

At least artificial intelligence already is superior to natural stupidity

Re: It's duplicated, too lazy to fix.

Andrei Straut20-Jul-12 3:31

Andrei Straut

20-Jul-12 3:31

CDP1802 wrote:
Especially the 'noob-unfriendly' things (like type safety) usually are supposed to help you and I have no interest in learning how to bypass them

So true. Now, please do go and convince everyone of this. Laugh | :laugh:

Full-fledged Java/.NET lover, full-fledged PHP hater.
Full-fledged Google/Microsoft lover, full-fledged Apple hater.
Full-fledged Skype lover, full-fledged YM hater.

Re: It's duplicated, too lazy to fix.

CDP180220-Jul-12 3:38

CDP1802

20-Jul-12 3:38

Hopeless. Those who go for 'easy' usually don't see the need for any pesky constraints and want to keep on going without them.

At least artificial intelligence already is superior to natural stupidity

Efficiency redux

John Hunley18-Jul-12 5:51

John Hunley

18-Jul-12 5:51

Just unearthed the following gem to display a progress bar:

C++

// display the battery gauge and percentage
switch(uiBattBarPosition)
{
   case(0):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"");
      break;
   case(1):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff");
      break;
   case(2):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff");
      break;
   case(3):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff");
      break;
   case(4):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff");
      break;
   case(5):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff");
      break;
   case(6):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff");
      break;
   case(7):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff\xff");
      break;
   case(8):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff\xff\xff");
      break;
   case(9):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff\xff\xff\xff");
      break;
   case(10):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff");
      break;
   case(11):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff");
      break;
   case(12):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff");
      break;
   case(13):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff");
      break;
   case(14):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff");
      break;
   case(15):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff");
      break;
   case(16):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff");
      break;
   case(17):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff");
      break;
   case(18):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff");
      break;
   case(19):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff");
      break;
   case(20):
      DisplayMessage(MAC_MSG_PTR(SS_BATTERY_CHARGE),s,"\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff\xff");
      break;
}   // end of switch

Makes you wonder if these people were getting paid by the line.