note davido <p>Perl's identifiers are case sensitive. <c>%userHash</c> is not the same as <c>%userhash</c>.</p> <p>One of the first things I do in such situations is a global case-insensitive search for the identifier using my editor. As I iterate through the search results it's much easier for my eyes to spot when something changes than by just skimming through the code.</p> <p>One thing that might have made it easier to spot would be following Schwern's [http://www.google.com/url?sa=t&rct=j&q=&esrc=s&source=web&cd=1&cad=rja&ved=0CDMQFjAA&url=http%3A%2F%2Fschwern.dreamhosters.com%2Ftalks%2FSkimmable%2520Code%2520-%2520YAPC-NA-2008.pdf&ei=c_ejUIyeCIn2iwKV2ICIBg&usg=AFQjCNFyDXKGARY4BFQzyNZGIhaOOP6TXA&sig2=vpjnFkItkYyg8QUEqI6kOQ|Skimmable Code] talk's advice. Moving some of the complexity in your while/if constructs into subroutines would move the usage of <c>%userhash</c> visually closer to its declaration, which might be a good thing.</p> <div class="pmsig"><div class="pmsig-281137"> <br /><p>Dave</p> </div></div> 1003880 1003880