deeej Posted August 13, 2007 Share Posted August 13, 2007 Hi,I am stuck with this random password generator I want it to create a random password for every entry in my database but when I submit the code it does the same random password for all the entries. code: // random password gen function createRandomPassword() { $chars = "abcdefghijkmnopqrstuvwxyz023456789"; srand((double)microtime()*1000000); $i = 0; $pass = '' ; while ($i <= 7) { $num = rand() % 33; $tmp = substr($chars, $num, 1); $pass = $pass . $tmp; $i++; } return $pass; } // set new password for all $password = createRandomPassword(); $q1 = "update members set password = '$password'"; Probably really easy, hope some one can help thanks.!! Link to comment https://forums.phpfreaks.com/topic/64695-loop-probably-really-easy/ Share on other sites More sharing options...
trq Posted August 13, 2007 Share Posted August 13, 2007 $pass = $pass . $tmp; should be... $pass = $tmp; Link to comment https://forums.phpfreaks.com/topic/64695-loop-probably-really-easy/#findComment-322595 Share on other sites More sharing options...
Orio Posted August 13, 2007 Share Posted August 13, 2007 @thropre- why ??? He/She wants a 7 chars long string I guess, not 1 char. Seems ok to me... Try removing the srand() line, using srand() is not really needed since php seeds the rand() function itself. Orio. Link to comment https://forums.phpfreaks.com/topic/64695-loop-probably-really-easy/#findComment-322601 Share on other sites More sharing options...
deeej Posted August 13, 2007 Author Share Posted August 13, 2007 the code works but it set's all the passwords to only 1 random generated password I want every entry to have its own random password, do not have a clue how to do it though. Thanks Link to comment https://forums.phpfreaks.com/topic/64695-loop-probably-really-easy/#findComment-322605 Share on other sites More sharing options...
deeej Posted August 13, 2007 Author Share Posted August 13, 2007 sorry pressed quote instead of modify Link to comment https://forums.phpfreaks.com/topic/64695-loop-probably-really-easy/#findComment-322607 Share on other sites More sharing options...
Recommended Posts
Archived
This topic is now archived and is closed to further replies.