Beefy Boxes and Bandwidth Generously Provided by pair Networks
Don't ask to ask, just ask
 
PerlMonks  

Re: Re: Re: Hash references and illegal octal digits

by peppiv (Curate)
on Dec 24, 2002 at 20:02 UTC ( [id://222130]=note: print w/replies, xml ) Need Help??


in reply to Re: Re: Hash references and illegal octal digits
in thread Hash references and illegal octal digits

Oops! One more breakdown. It appears the code
my %hash = map { chomp; split /,/ } @array;
doesn't play well with this data.

__DATA__ 1001,choochoo 1002,candycane 1003,sockpuppet 1004,choochoo 1005,candycane 1006,sockpuppet6 1007, 1008,bar 0001,choochoo 0002,candycane 0003,sockpuppet 0109,choochoo

Where 1007 doesn't have any value. I have no idea how to treat this. Does anyone have any ideas?

peppiv

Replies are listed 'Best First'.
Re: Re: Re: Re: Hash references and illegal octal digits
by I0 (Priest) on Dec 24, 2002 at 23:18 UTC
    my %hash = map { chomp; split /,/,$_,2 } @array;
Re: Re: Re: Re: Hash references and illegal octal digits
by MarkM (Curate) on Dec 24, 2002 at 20:15 UTC

    It is usually considered bad form to modify $_ during a map{} operation. People expect map{} to create a new list from an existing list, without modifying the existing list.

    The expression can be written out clearer, more accurately, and using less intermediate memory by using the following code instead:

    my %hash; for (@array) { if (/^([^,]+)(?:,(.*))?$/) { $hash{$1} = $2; } }

    This also has the side effect of handling the case where the input takes the form "1009," or "1009". For "1009," the key will map to an empty string. For "1009" the key will map to the undefined value.

    The regular expression removes the need for chomp() as the $ at the end of the regular expression will not swallow the "\n".

    If efficiency is the true goal, the following code has the exact same effect:

    my %hash; /^([^,]+)(?:,(.*))?$/s && ($hash{$1} = $2) for @array;

    If a few CPU ticks can be spared, the former is far more preferable to the latter.

Re: Re: Re: Re: Hash references and illegal octal digits
by Mr. Muskrat (Canon) on Dec 24, 2002 at 20:12 UTC

    Just ensure that there is something after the comma... even if it is a space.
    If you have no control over the data then perhaps try something like this:
    my %hash = map { s/,\n/, \n/; chomp; split /,/ } @array;

Log In?
Username:
Password:

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://222130]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this?Last hourOther CB clients
Other Users?
Others having a coffee break in the Monastery: (3)
As of 2024-04-19 19:44 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found