A couple of fixes and a couple of suggestions. First, the
mkdir thing has a precedence problem, as
tye suggests. Use parenthesis or
or to fix it:
mkdir("$dir", 0777) || die "Could not mkdir \"$dir\": $!\n";
Second, the
chop bothers me, 'cuz it's easy to grab something you don't want. (In this case, it's right, but I prefer
chomp).
Third, there are a couple of other ways for doing your menu stuff. Some people prefer this:
($menu eq 1) && addwant();
($menu eq 2) && addhave();
When I have to do something like this, I usually use a data structure of references:
my @actions = qw( \&addwant \&addhave ); # and so forth
# get input
if (defined (my $action = $actions[$menu])) {
$action->();
} else {
print "\n\n\nGoodbye!\n";
exit;
}
Finally, if you want to clear the screen, there's a bit in
perlfaq8 that might help. Not a big deal, though.
Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
Titles consisting of a single word are discouraged, and in most cases are disallowed outright.
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:
- a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, details, 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, summary, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
You may need to use entities for some characters, as follows. (Exception: Within code tags, you can put the characters literally.)
|
For: |
|
Use: |
| & | | & |
| < | | < |
| > | | > |
| [ | | [ |
| ] | | ] |
Link using PerlMonks shortcuts! What shortcuts can I use for linking?
See Writeup Formatting Tips and other pages linked from there for more info.