http://www.perlmonks.org?node_id=1039445


in reply to Help Extract these lines

Maybe something like this (quick proposal, untested): 

my %my_hash;
while <DATA> {
     chomp;
     my $id = (split /:/, $_)[1] if /id/;
     if (/name/) {
          my $name =~ /: "(\d+)"/;
          $my_hash{$name} = $id;
     }
}

[download]

Replies are listed 'Best First'.
Re^2: Help Extract these lines
by ww (Archbishop) on Jun 17, 2013 at 21:32 UTC

    "untested."

    Yup, Re: Help Extract these lines doesn't even compile.
    Syntax error at Ln 2 ( which, in turn, produces a spurious error re closing curly braces ).


    If you didn't program your executable by toggling in binary, it wasn't really programming!

[reply]

      Alright, I just typed an approach for a solution directly on the web page, not having access to a Perl compiler when I did it (and I said that I could not test). What I typed does not compile, but it takes less than 5 seconds to find out why and to correct it. One really easy to find compile error (adding parens after the while keyword) and one other small stupid mistake just as easy to find and to correct, and it basically works: 

      use Data::Dumper;
my %my_hash;
my $id;
while (<DATA>) {
     chomp; s/\r//g;
     $id = (split /:/, $_)[1] if /id/;
     if (/name/) {
          my $name = $1 if /\: "(\w+)"/;
          $my_hash{$name} = $id;
     }
}
print Dumper \%my_hash;
__DATA__
"id": "xx-ada-qwebasd",
"name": "telphone",
"id": "fasda-asd-123123-fkja123a",
"name": "car",
"id": "97f921-a312-fas2",
"name": "ball",

      [download]
      which prints: 
      $ perl   2_lines.pl
$VAR1 = {
          'ball' => ' "97f921-a312-fas2",',
          'car' => ' "fasda-asd-123123-fkja123a",',
          'telphone' => ' "xx-ada-qwebasd",'
        };

      [download]
      I admit the data still needs a bit more cleaning, but that's really boilerplate code, and I think that this untested code basically gave a very good track towards the solution.
[reply]
[d/l]
[select]

In Section Seekers of Perl Wisdom