This is a nicely-written program, well done! If the C++ version is up to this standard, you should get high marks on the assignment.
I did notice one logical error:
print "Division by zero not allowed, starting over..." and redo
if ( $frac01 =~ /^[\s\S]?\/?0$/ );
If the user enters 1/0, this correctly flags division by zero. But,
- If the user enters 12/0, the regex fails and division by zero is not caught.
- If the user enters 10, the regex matches and gives a false positive.
How to fix this? One way would be to extract the denominator (if any) and test whether that is zero. But this is Perl, so TMTOWTDI.
Now to style: In Perl, it’s better to declare a variable as late as possible in the code. So, looking at this line:
my ( $f1, $f2, $frac01, $frac02, $command, $total ) = (0) x 6;
actually only $command needs to be declared before the outer while loop. The rest can safely be declared at the point of first use. (And yes, I would declare $total separately within each of the if / elsif blocks.)
Hope that helps,
Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
Read Where should I post X? if you're not absolutely sure you're posting in the right place.
Please read these before you post! —
Posts may use any of the Perl Monks Approved HTML tags:
You may need to use entities for some characters, as follows. (Exception: Within code tags, you can put the characters literally.)
- a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, div, dl, dt, em, font, h1, h2, h3, h4, h5, h6, hr, i, ins, li, ol, p, pre, readmore, small, span, spoiler, strike, strong, sub, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
Link using PerlMonks shortcuts! What shortcuts can I use for linking?
See Writeup Formatting Tips and other pages linked from there for more info.
| & || & |
| < || < |
| > || > |
| [ || [ |
| ] || ] ||