This seems to work fine, albeit I wasn't able to reproduce the error you were getting of uninitialised value. If you still get the error, maybe try dumping both @nup and @genpass to separate text files and look for special control characters. Also, make sure the number of elements in each array tally with what your expecting.
#!/usr/bin/perl -w use strict; use warnings; my @genpass; my $pass = "new.pass"; open (PASS, ">", $pass) or die "Cannot open $pass: $!"; my $p = 1; while ( $p < 53 ){ @genpass = generatePassword(10); print PASS "@genpass \n"; $p++; } close PASS; open (PASS, $pass) or die "Cannot open $pass: $!"; @genpass = (<PASS>); chomp @genpass; print scalar (@genpass) , " <- genpass\n"; my $users = "new.users"; #open (USERS, "<", $users) or die "Cannot open $users: $!"; my @nup = (<DATA>); print scalar (@nup) , " <- nup\n"; #close USERS; open (USERS, ">", $users) or die "Cannot open $users: $!"; foreach my $line (@nup){ $line =~ s/xyz456/$genpass[0]/; print USERS $line; shift(@genpass); } close USERS; sub generatePassword { my $length = shift; my $password = ""; my $possible = 'abcdefghijkmnpqrstuvwxyz23456789ABCDEFGHJKLMNPQRST +UVWXYZ'; while (length($password) < $length) { $password .= substr($possible, (int(rand(length($possible)))), + 1); } return $password; } __DATA__ dn: uid=1234567,ou=people,dc=abc,o=oursite.com objectClass: posixAccount objectClass: shadowAccount objectClass: account objectClass: top uid: 1234567 cn: 1234567 uidNumber: 31472 gidNumber: 7546 gecos: First Last,ABCDEFG297,####### homeDirectory: /home/1234567 loginShell: /bin/csh userPassword: xyz456 dn: uid=1234567,ou=people,dc=abc,o=oursite.com objectClass: posixAccount objectClass: shadowAccount objectClass: account objectClass: top uid: 1234567 cn: 1234567 uidNumber: 31472 gidNumber: 7546 gecos: First Last,ABCDEFG297,####### homeDirectory: /home/1234567 loginShell: /bin/csh userPassword: xyz456 dn: uid=1234567,ou=people,dc=abc,o=oursite.com objectClass: posixAccount objectClass: shadowAccount objectClass: account objectClass: top uid: 1234567 cn: 1234567 uidNumber: 31472 gidNumber: 7546 gecos: First Last,ABCDEFG297,####### homeDirectory: /home/1234567 loginShell: /bin/csh userPassword: xyz456

In reply to Re: Why do I get uninitialized value in substitution (s///) in my array? by desemondo
in thread Why do I get uninitialized value in substitution (s///) in my array? by xjlittle

Title:
Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post, it's "PerlMonks-approved HTML":



  • 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:
    & &amp;
    < &lt;
    > &gt;
    [ &#91;
    ] &#93;
  • Link using PerlMonks shortcuts! What shortcuts can I use for linking?
  • See Writeup Formatting Tips and other pages linked from there for more info.