Предупреждение в цикле foreach?

Почему я получаю следующее предупреждение:

[Пятница, 25 марта 19:19:21 2011] [ошибка] [клиент 127.0.0.1] Предупреждение PHP: для foreach() указан неверный аргумент в /var/www/register.php в строке 60, реферер: http://localhost/register.php

при запуске следующего скрипта:

<?php
    error_reporting( E_ALL );
?>

<html>
    <head>
        <title></title>
        <link rel="icon" type="image/png" href="favicon.ico">

        <?php
            if( $_SERVER['REQUEST_METHOD']=='POST' ) {
                $err = array();

                if( empty( $_POST['display_name'] ) ) $err[] = "display name field is required";
                if( empty( $_POST['email'] ) ) $err[] = "email field is required";
                if( empty( $_POST['password'] ) ) $err[] = "password field is required";

                if( !$err ) {
                    try {
                        $DBH = new PDO( "mysql:host=localhost;dbname=database1", "user", "pass" );
                        $DBH -> setAttribute( PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION );

                        $STH = $DBH -> prepare( "insert into table1 (display_name, email, password) values ( :display_name, :email, :password )" );

                        $STH -> bindParam( ':display_name', $_POST['display_name'], PDO::PARAM_STR, 100 );
                        $STH -> bindParam( ':email', $_POST['email'], PDO::PARAM_STR, 100 );
                        $STH -> bindParam( ':password', $_POST['password'], PDO::PARAM_STR, 100 );

                        $STH -> execute();

                        $STH = $DBH -> prepare( "insert into table2 ( username, status, users_id ) values ( :username, :status, :users_id )" );

                        $strStatus = 1;

                        $STH -> bindParam( ':username', $_POST['display_name'], PDO::PARAM_STR, 100 );
                        $STH -> bindParam( ':status', $strStatus, PDO::PARAM_INT, 1 );
                        $STH -> bindParam( ':users_id', $_POST['referer'], PDO::PARAM_INT, 1 );

                        $STH -> execute();

                        $DBH = null;
                    } catch( PDOException $e ) {
                        echo $e -> getMessage();
                    }

                    header( "Location: ".$_SERVER['PHP_SELF'] );
                    exit;
                } else {
                    foreach( $_POST as $key => $val ) {
                        $form[$key] = htmlspecialchars($val);
                    }
                }
            } else {
                $form['display_name'] = $form['email'] = $form['password'] = '';
            }
        ?>
    </head>

    <body>
        <?php foreach( $err as $line ) { ?>
        <div style="error"><?php echo $line; ?></div>
        <?php } ?>

        <h1>register</h1>

        <form method="post">
            referers id:<br />
            <input type="text" name="referer" /><br /><br />

            name:<br />
            <input type="text" name="display_name" value="<?php echo $form['display_name']; ?>" /><br /><br />

            email:<br />
            <input type="text" name="email" value="<?php echo $form['email']; ?>" /><br /><br />

            password:<br />
            <input type="text" name="password" value="<?php echo $form['password']; ?>" /><br /><br />

            <input type="submit" value="register" />
        </form>
    </body>
</html>

php
person oshirowanen    schedule 25.03.2011    source источник

Ответы (4)


arrow_upward
2
arrow_downward

Вы получаете сообщение об ошибке, потому что пытаетесь получить доступ к переменной, которая при определенных условиях не существует. Если бы форма вашей страницы не была отправлена, переменная $err не существовала бы.

Вы можете исправить это, поместив $err = array(); вне вашего оператора if. Изменять:

if( $_SERVER['REQUEST_METHOD']=='POST' ) {
   $err = array();

To:

$err = array();
if( $_SERVER['REQUEST_METHOD']=='POST' ) {
person Tim Cooper    schedule 25.03.2011

arrow_upward
0
arrow_downward

Эта ошибка возникает, когда аргумент не является массивом. Вы должны поместить if(is_array($_POST)) перед/вокруг вашего foreach, чтобы убедиться, что он не содержит нулевое значение.

person tvkanters    schedule 25.03.2011

arrow_upward
0
arrow_downward

Я думаю, что ваш сеанс не всегда инициализируется, поэтому вы получаете недопустимый аргумент, потому что он не существует

person yogsma    schedule 25.03.2011

arrow_upward
0
arrow_downward

Такое предупреждение вы получаете, когда пытаетесь выполнить итерацию над чем-то, что не является массивом.

Перед следующей строкой:

foreach( $err as $line ) {

Убедитесь, что $err действительно является массивом.


Два решения:

  • инициализировать $err раньше, за пределами условия, которое проверяет, является ли $_SERVER['REQUEST_METHOD']=='POST', поэтому он по крайней мере содержит пустой массив
  • Или не пытайтесь перебирать $err, если он не установлен, используя isset() проверить на это
person Pascal MARTIN    schedule 25.03.2011