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


in reply to finding unique items in an array, from a text file

hiya - looks a bit overcomplicated, should be really simple. IMHO you don't need @lines, @uniq and %seen... only need one hash to do this.

I'd prefer to do something like this (off the top of my head, code not tested):

# open the file into FH my %uniq; $uniq{$_} = 1 while (<FH>); print join "\n", keys %uniq; # close the file

Hope that helps. All the best.

Replies are listed 'Best First'.
Re^2: finding unique items in an array, from a text file
by johngg (Canon) on Jan 13, 2009 at 23:27 UTC

    I think there might be a couple of problems with your code.

    • You join the output with \ns but you did not chomp the input so you will get extra blank lines.
    • Hash keys are unordered so the line order of the original file will be scrambled. This may or may not be an issue.

    The grep solutions suggested might be better if line order is to be preserved. Something like (again, not tested).

    ... my %seen = (); print grep ! $seen{ $_ } ++, <$fh>; ...

    Cheers,

    JohnGG

      All good points. ;)