note Corion <p>Your generator cannot generate a password with the letters <c>a</c>, <c>b</c> or <c>c</c> due to the <c>+3</c> when selecting the random letter. See [doc://rand], which already returns a number in the range from <c>0..$len</c>.</p> <p>The variable <c>$len</c> is named wrong - it has the last index of <c>@lett</c>, not the number of elements in the array. This means that your password generator would never output the last element from <c>@lett</c> if it weren't for the <c>+ 3</c> from the previous step. I would use:</p> <c> $len = @lett; </c> <p>... which assigns to <c>$len</c> the number of elements in <c>@lett</c>.</p> <p>Other than that, using double quotes in a simple assignment is weird unless you have a special reason to do so - and if you are starting with Perl just now, those special reasons ("stringification") are still far away. So, no assignment should be of the form</p> <c> $foo = "$bar"; </c> <p>and always of the form</p> <c> $foo = $bar; </c> 1014922 1014922