php - このPHP関数の何が問題になっていますか?

Question

この関数の何が問題なのか誰か教えてもらえますか?

$reg_errors 配列からエラーを取得し、それらのエラーを同じスクリプトの別の場所で使用するために、この関数を作成しました。

function testimonialErrors ($reg_errors) {
    if ( !empty($reg_errors) ) {
            foreach ( $reg_errors AS $error) {
                echo "<li>$error</li>";
            }                                       
    }
    return $error; 
}

それから私はこのようにその関数を呼び出しました..

if ( !empty($reg_errors) ) {
    echo '<div class="error">
                <img src="images/error.png" />
                <h3>Errors,</h3>
                <ul>';
                echo testimonialErrors($reg_errors);
        echo '</ul>
            </div>';
}

しかし、このコードは機能していません。

更新:これは私の新しいコードです

function tesimonialErrors ($reg_errors) {
    if ( !empty($reg_errors) ) {
            foreach ( $reg_errors AS $error) {
                echo "<li>$error</li>";
            }                                       
    }
    return $error; 
}

そしてそれをこのように呼んだ

if ( !empty($reg_errors) ) {
    echo '<div class="error">
                <img src="images/error.png" />
                <h3>Errors,</h3>
                <ul>';
                tesimonialErrors($reg_errors);
        echo '</ul>
            </div>';
}

score 4 · Accepted Answer

variable scopeを調べる必要があります。現時点で$reg_errorsは、関数のスコープで定義されていません。関数に渡すことができます：

function tesimonialErrors ($reg_errors) {
  ...

そしてそれを次のように呼び出します：

tesimonialErrors($reg_errors);

score 0 · Accepted Answer

なぜこのようにしないのですか？

if ( !empty($reg_errors) ) {
    echo '<div class="error">
            <img src="images/error.png" />
            <h3>Errors,</h3>
            <ul>
               <li>';
    echo implode('</li><li>', $reg_errors);
    echo '    </li>
            </ul>
        </div>';
}

score -1 · Accepted Answer

function testimonialErrors ($reg_errors) {
    if ( !empty($reg_errors) ) {
            foreach ( $reg_errors AS $error) {
                echo "<li>$error</li>";
            }                                       
    }
    return $error;  // ****remove return statement**** 
}

score -1 · Accepted Answer

@Akam が指摘したように、 $reg_errors はグローバルスコープにあるため、またはのいずれglobal $reg_errors;かを使用する必要があります$GLOBALS['reg_errors']。

tesimonialErrorsまた、関数名を適切な英語に変更することをお勧めします (それはtestimonialErrors. そうしないと、The Next Guy が TDWTF に投稿してしまう可能性があります。

score -2 · Accepted Answer

関数に追加global $reg_errors;します。

function testimonialErrors () {
    global $reg_errors;
    if (!empty($reg_errors)) {
        foreach ($reg_errors as $error) {
            echo "<li>$error</li>";
        }
    }
}

php - このPHP関数の何が問題になっていますか?

5 に答える 5

Related

Reference