Beefy Boxes and Bandwidth Generously Provided by pair Networks
Problems? Is your data what you think it is?
 
PerlMonks  

Re^7: chop vs chomp

by eric256 (Parson)
on May 11, 2007 at 16:43 UTC ( #614961=note: print w/replies, xml ) Need Help??


in reply to Re^6: chop vs chomp
in thread chop vs chomp

You've proved quite a few times that chomp is not a good choice, in your case. Your case however does not represent everyone else's common uses. By your own definition here, a new line could exist and there could still be an error. In that case shouldn't your code be looking at the structure of the data it is attempting to load and relying on some other input other than the newline to mean your data is valid?

You've also mentioned chop several times but unless you (chop() eq $/ ) or die chop doesn't do you any more good than chomp did. I also believe it has been pointed out that your chop construct will actualy fail if $/ is more than one line, and in addition chomp isn't mean to remove record seperators, it is meant to portable remove a new line. As a feature chomp doesn't fail if there is no new line, but few of perls functions will die if they fail.

You attack everyone here as following groupthink and yet you can't seem to accept a new point of view yourself. I havn't seen anyone claim that chomp is ALWAYS the right choice, yet you claim it is ALWAYS the wrong choice. That flies in the face of the facts before you, chomp is quite often exactly what people want it to be, and when it isn't, then don't use it. Don't through it (and the people who use it) out simply because you have had a bad experience with it. Can it be misused? Of course it can, and is, but that realy doesn't mean it is always a bad choice.

If you want to use it as a measure of skill then fine, but be sure to make your requirements such that chomp alone is the wrong choice. Maybe something like "Write a script to parse a data file removing line endings and verify that each line ends with a terminator. Assume the files are provided on the command line and output them to STDOUT". That is my first attempt writing such a problem so please don't be too critical ;)

#!/usr/bin/perl use strict; use warnings; while (<>) { chomp or die "Record is missing terminating new line."; print; }


___________
Eric Hodges

Replies are listed 'Best First'.
Re^8: chop vs chomp
by Moron (Curate) on May 11, 2007 at 17:09 UTC
    Your example looks fine. But you see if chomp had never come along, people wouldn't be having to focus so lopsidedly on this issue at all - they'd be taking a balanced view of data quality and would not dream of doing this - this is another of my objections - it's the tail of the input line wagging the dog!!
    __________________________________________________________________________________

    ^M Free your mind!

      So....the fault is mine? None of my data files have a trailing new line, and I wouldn't dream of assuming that a newline meant a complete record even if they did. You on the other hand always have a trailing newline and assume it is significant? If my data never (and I do mean never) has a trailing newline then I should be fixing all those horrible tools that are providing me incomplete data. (/me heads off to slap newlines on all his csv files and apologize.). Sorry I didn't realize that soo many of my tools are broken and you are absolutely correct, ALL FILES MUST END IN \n.

      You talk of lopsided focus and yet you don't seem to be able to see that not everyone deals (or wants to deal with, or even should have to deal with) files that have a trailing newline. I've conceded many times now that in some cases you might in fact be right, it is only your blanket (lopsided) claim that chomp is ALWAYS bad that I take issue with. Ah well, it is done, you are of your opinion, and you showed me a use for chomps return value. Thank you for that, I can only hope that maybe you will see a little bit of the other side of the coin here and not right off programmers for using a specific tool when it fits their task (not your task).


      ___________
      Eric Hodges

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://614961]
help
Chatterbox?
and all is quiet...

How do I use this? | Other CB clients
Other Users?
Others about the Monastery: (1)
As of 2018-07-22 01:26 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    It has been suggested to rename Perl 6 in order to boost its marketing potential. Which name would you prefer?















    Results (451 votes). Check out past polls.

    Notices?