in reply to New to Programming and I have chose PERL

Move your print statement to inside the braces for the if statement.

I'd also suggest looking at reformating a bit so you can more easily see what goes together:

#!/usr/local/bin/perl -w $input = ''; #temporary input $hrate = 0; #pay rate $yrate = 0; #yearly salary while () { print 'Do you wish to calculate yearly salary (y) or hourly rate?( +h)'; chomp ($input = <STDIN>); if ($input eq '') { last; } if ($input =~ /h/) { print "Enter your hourly rate:"; chomp ($hrate = <STDIN>); $yrate = ($hrate * 40) *52; print "Your yearly salary is $yrate\n"; } }
(I'm not going to get into the One True Brace Style religious flamefest - so we'll just use the brace positioning you're already using.)

Hope that helps.

Replies are listed 'Best First'.
Re^2: New to Programming and I have chose PERL
by Lunchy (Sexton) on Jan 12, 2005 at 18:25 UTC
    I'm sure this is overkill ( I saw others using regexs so I thought I'd go all out. ;)), but this is how I like to do command line driven programs like this. Basically I'm creating a lookup table with the %actions hash. When someone enters their choice, it checks to see if that choice exists in the lookup table, if it does, it'll execute the sub reference that the lookup table points to. If the choice does not exist in the lookup table, it goes on to throw the error and starts the loop over again. While I understand you're new to perl, this may give you a little something to chew on while you learn. :-D
    use strict; my %actions = ( 'y' => \&calcHourly, 'h' => \&calcYearly, 'q' => sub { exit(); }, ); while ( 1 ) { print "\nEnter y (or) h (or) q to calculate Yearly salary(y) or Ho +urly rate(h) q to Quit: "; chomp( my $myAction = <STDIN> ); if ( exists $actions{ $myAction } ) { $actions{ $myAction }->(); next; } print "\nError: \"$myAction\" is not a valid choice, please try ag +ain\n\n"; } sub calcHourly { print "Enter your hourly rate: "; chomp ( my $hourrate = <STDIN> ); my $yearrate = ( $hourrate * 40 ) * 52; print("\nYour Annual salary is: \$$yearrate\n\n"); } sub calcYearly { print "Enter your yearly salary: "; chomp ( my $yearrate = <STDIN> ); my $hourrate = ( $yearrate / 52 ) / 40; print("\nYour hourly rate is: \$$hourrate\n\n"); }
    (oops, replied to the wrong one, doh!)