Beefy Boxes and Bandwidth Generously Provided by pair Networks
We don't bite newbies here... much
 
PerlMonks  

Re: Fix-it fix-it fix-it!

by davido (Archbishop)
on Nov 14, 2012 at 19:41 UTC ( #1003881=note: print w/replies, xml ) Need Help??


in reply to Fix-it fix-it fix-it!

Perl's identifiers are case sensitive. %userHash is not the same as %userhash.

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.

One thing that might have made it easier to spot would be following Schwern's Skimmable Code talk's advice. Moving some of the complexity in your while/if constructs into subroutines would move the usage of %userhash visually closer to its declaration, which might be a good thing.


Dave

Replies are listed 'Best First'.
Re^2: Fix-it fix-it fix-it!
by squimby (Acolyte) on Nov 14, 2012 at 19:43 UTC
    Pardon my french, but dammit! I feel like an idiot. Removing this thread.
        Noted - maybe somebody else can use my script in the future if they correct that mistake :P

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://1003881]
help
Chatterbox?
[choroba]: Robocoptic!
[erix]: I like Coptics :)
[LanX]: Copts
[erix]: sorry - just pulling your leg
[robby_dobby]: erix: It's a little late for that :-)
[robby_dobby]: I was falling asleep at my chair and threw out that one bit about participating in a YAPC. Things went from there :-)
[erix]: never too late to pull legs
[LanX]: Official YAPC Africa 2013 Announcement
erix always game to pull legs from sleeping cells
[erix]: hm nice might do a spot of diving - Red Sea is said to be beautiful

How do I use this? | Other CB clients
Other Users?
Others perusing the Monastery: (14)
As of 2017-04-24 16:15 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    I'm a fool:











    Results (442 votes). Check out past polls.