Looks like
jdporter covered the major points,
except for the fact that your use of "join()" (with the
embedded "map" and "rand") won't work, either.
Slow things down a bit, give yourself some space, and use
a loop to generate the candidate password -- put it in a
subroutine, maybe, to keep it clear:
...
my $pw;
do {
$pw = make_new_pw();
} while ( grep { $pw eq $_ } @stored );
...
# update: moved the declaration of @chars into the subroutine,
# where it belongs:
sub make_new_pw
{
my @chars = ( "A" .. "Z", "a" .. "z", 0 .. 9, qw( ! @ $ % ^ & *) )
+;
my $newpw = "";
for ( 1 .. 17 ) {
$newpw .= $chars[ rand @chars ];
}
return $newpw;
}
One closing tip: I've seen people get really bolixed because
they are sent a password like "lXIg1v0WO" -- given the
chance to control which letters are available to go into a random password,
I would tend to leave "I1l" and "0O" out of the set, because
they are too easily confused and misread.
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.