Измените оператор while на foreach - PullRequest
0 голосов
/ 13 октября 2011

У меня есть оператор while, который выполняется правильно, но он должен быть оператором foreach, поскольку выполняется несколько вставок, и он не генерирует случайный пароль или не отправляет электронное письмо для каждого из них. Вот код:

        while (($data = fgetcsv($handle, 1000, ",")) !== FALSE)
     {

        $password = createRandomPassword(); 

        // Add the user to the database:
        $sql = "INSERT into users(account_id,first,last,email,pass) values('$id', '$data[0]','$data[1]','$data[2]','$password')";
        $r = mysqli_query ($dbc, $sql) or trigger_error("Query: $sql\n<br />MySQL Error: " . mysqli_error($dbc));


            if (mysqli_affected_rows($dbc) == 1) { // If it ran OK.

                    $emailBody = "<html><body>";

                    $emailBody .= "<br><br><b>Hey $first $last!</b>";

                    $emailBody .= "<br><br>You've been added to our site and your password is: $password";

                    $emailBody .= "</body></html>";

                    mail("$ln_first $ln_last <$ln_email>", "Your Account" , $emailBody, "From: YourCompany <noreply@yourcompany.com>\nContent-Type: text/html");

            }

     fclose($handle);
    echo '<p><div class="alert green">You have successfully imported users.</div></p>';
 }

Ответы [ 2 ]

3 голосов
/ 13 октября 2011

Не думаю, что с вашей петлей while что-то не так - вы закрываете ручку слишком рано! fclose и последние echo строки должны быть после этой последней фигурной скобки.

0 голосов
/ 13 октября 2011

Проблемы, как я вижу, в том, что вы не определили переменные, которые вы используете для отправки электронных писем:

$ln_first $ln_last <$ln_email>

Я не могу видеть, откуда пришли эти переменные, или эти

$emailBody .= "<br><br><b>Hey $first $last!</b>";
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...