Re^2: problem with deleting a row

Replies are listed 'Best First'.
Re^3: problem with deleting a row by Laurent_R (Canon) on Nov 21, 2013 at 18:37 UTC
This: `while (<FILE>) { @array = <FILE>;` [download] is just wrong. Either you read the file line by line with a while loop, or you "slurp" it into an array, but don't do both. The you are constructing a @a array and print $a, a scalar variable. The following pragmas: `use strict; use warnings;` [download] would have picked up these errors.	[reply] [d/l] [select]
Re^3: problem with deleting a row by Ea (Chaplain) on Nov 22, 2013 at 11:04 UTC
I've been writing Perl for 17 years and I'm still learning new and better ways to write code. Here's another way based on kcott's excellent answer below. `#!/usr/bin/perl -w use strict; use autodie; my ($user_file, $changed_file) = qw{test.txt test.txt.mod}; open my $in_fh, '<', $user_file; open my $out_fh, '>', $changed_file; print "Which user? "; my $user = <STDIN>; chomp $user; print $out_fh grep !/^$user,/, <$in_fh>;` [download] This one slurps the whole file with the angle brackets, passes it through grep to filter the output which gets printed to the output file. Note the exclamation mark in front of the regex. It's the "not" operator which says you don't want the matching lines in your output. Keep working at it. From the long list of replies, you've certainly piqued the Monks' interest. Sometimes I can think of 6 impossible LDAP attributes before breakfast.	[reply] [d/l]
Re^4: problem with deleting a row by Laurent_R (Canon) on Nov 23, 2013 at 16:56 UTC
`print $out_fh grep !/^$user,/, <$in_fh>;` [download] As far as I can say, this does not work properly. Using the example under the Perl debugger I gave above in this post Re^5: problem with deleting a row, I obtain the following with your syntax: `DB<3> $user = "foo"; DB<10> print join " ", grep !/^$user,/, qw /foo foobar bar barfoo fo +bar foobaz /; foo foobar bar barfoo fobar foobaz DB<11> print join " ", grep /^$user,/, qw /foo foobar bar barfoo fob +ar foobaz /; DB<12>` [download] This apparently does not work because $user is interpreted literally as a pattern. As I said in the post mentioned above, you need to force evaluation of $user into the content of the variable (I gave in that post two possible ways of doing that) to get this to work properly.	[reply] [d/l] [select]
Re^3: problem with deleting a row by brianMonk (Initiate) on Nov 21, 2013 at 18:42 UTC
Well I managed to do this. The code returns every line except the one that the user wants to delete entering first username. However, if in my test.txt file two usernames start with the same letter (for example, la123 and lv444), the program will remove both lines, regardless user input that is for example la123. `$file = "test.txt"; print "Enter the username you want to remove from the information.txt +file\n"; $user = <STDIN>; open( FILE, $file) or die "cannot open > test.txt: $!"; while (<FILE>) { chomp; @array = <FILE>; @a = grep (!/^[$user]/, @array); print @a; }` [download]	[reply] [d/l]
Re^4: problem with deleting a row by Laurent_R (Canon) on Nov 21, 2013 at 19:03 UTC
Why are you using square brackets around the $user variable? Don't forget to chomp $user.	[reply]
Re^4: problem with deleting a row by brianMonk (Initiate) on Nov 21, 2013 at 19:56 UTC
Lauren, if I remove square brackets around $user, the program will just output every line refusing to perform grep function.	[reply]
Re^5: problem with deleting a row by Laurent_R (Canon) on Nov 23, 2013 at 11:57 UTC
if I remove square brackets around $user, the program will just output every line refusing to perform grep function. This is because you need to force the evaluation of the $user variable. This is an example under the Perl debugger of one possible way to do it: `DB<1> @array = qw /foo foobar bar barfoo fobar foobaz /; DB<2> x @array; 0 'foo' 1 'foobar' 2 'bar' 3 'barfoo' 4 'fobar' 5 'foobaz' DB<3> $user = "foo"; DB<4> @a = grep /^\Q$user\E/, @array; DB<5> print "@a"; foo foobar foobaz DB<7> print join " ", grep !/^\Q$user\E/, @array; bar barfoo fobar` [download] But using square brackets does not work as expected because it is defining a character class: `DB<8> print join " ", grep !/^[$user]/, @array; bar barfoo DB<9> print join " ", grep /^[$user]/, @array; foo foobar fobar foobaz` [download] On the other hand, using `${user}` instead of `$user` will be sufficient to get it to work properly: `DB<27> print join " ", grep /^${user}/, @array; foo foobar foobaz DB<28> print join " ", grep !/^${user}/, @array; bar barfoo fobar` [download]	[reply] [d/l] [select]