in reply to Re^2: The trap of reference numification
in thread The trap of reference numification

Sounds like you needed Hungarian notation. Not the evil Systems notation, but the good Apps notation. Refer to http://en.wikipedia.org/wiki/Hungarian_notation for more info on the difference. FWIW, the inventor invented the Apps version, but it was mutated into the Systems version due to a poor choice of words in the original whitepaper.
My criteria for good software:
  1. Does it work?
  2. Can someone else come in, make a change, and be reasonably certain no bugs were introduced?
  • Comment on Re^3: The trap of reference numification

Replies are listed 'Best First'.
Re^4: The trap of reference numification
by perrin (Chancellor) on Nov 12, 2005 at 05:45 UTC
    That sounds like a cure that's worse than the disease to me. I'd prefer a less pathologically magical Perl over code full of prefixed variable names.
[reply]
      Instead of: 
      my $COMMIT_CHUNK_SIZE;
my $line_num;
my $row = [ ... ];

      [download]
      Apps Hungarian would say to use something like: 
      my $cnt_COMMIT_CHUNK_SIZE;
my $cnt_line_num;
my $db_row = [ ... ];

      [download]
      (I completely made up the prefixes. They're there just to point out how one could do it.)

      Now, when we rewrite the snippet samtregar posted, it's immediately obvious that it's wrong. 

      if ($db_row and ($db_row % $cnt_COMMIT_CHUNK_SIZE == 0)) {
    $dbh->commit;
}

      [download]
      You're having a $db_ and a $cnt_ variable interact without some sort of mediation. That's an obvious and inspectable red flag. In fact, when using Apps Hungarian, people have reported an inability to make these kinds of errors. It's kinda like Perl makes it really really hard to have dangling pointers. You can still do it, but you have to intentionaly disregard your best judgement to do it.
      My criteria for good software:
      1. Does it work?
      2. Can someone else come in, make a change, and be reasonably certain no bugs were introduced?
[reply]
[d/l]
[select]

In Section Meditations