5

最初に、以下のコードをできるだけ小さくしようとしたと言いましょう。

        <?php
        // required variables (make them explciit no need for foreach loop)

        $getyear       = (isset($_POST['year'])) ? $_POST['year'] : '';
        $getpass       = (isset($_POST['studentpass'])) ? $_POST['studentpass'] : '';
        $getretypepass = (isset($_POST['retypepass'])) ? $_POST['retypepass'] : '';
        $errormsg      = (isset($errormsg)) ? $errormsg : '';

        $validSubmission = isset($_POST['registerbtn']) && $_POST['year'] && $_POST['studentpass'] && $_POST['retypepass'];

        $min_year = 1;
        $max_year = 10;
        $years    = range($min_year, $max_year); // returns array with numeric values of 1900 - 2012
        $yearHTML = '';
        $yearHTML .= '<select name="year" id="yearDrop">' . PHP_EOL;
        $yearHTML .= '<option value="">Please Select</option>' . PHP_EOL;
        foreach ($years as $year) {
            if (!$validSubmission && isset($_POST['year']) && $year == $_POST['year']) {
                $yearHTML .= "<option value='" . $year . "' selected='selected'>$year</option>" . PHP_EOL;
            } else {
                $yearHTML .= "<option value='" . $year . "'>$year</option>" . PHP_EOL;
            }
        }

        $yearHTML .= '</select>';


        if ((isset($_POST['registerbtn']))) {
            if (in_array($_POST['year'], $years) === true) {
                $getyear = (int) $_POST['year'];
            }
            $getpass       = $_POST['studentpass'];
            $getretypepass = $_POST['retypepass'];


            if ($getyear) {
                if ($getpass) {
                    if (strlen($getpass) <= 5) {
                        $errormsg = "The Password must be a minimum of 6 characters or more";
                    } else {
                        if ($getretypepass) {
                            if ($getpass === $getretypepass) {
                                //perform 2 queries, one query contains $aliasnumrows and other contains $numrows

                                if ($aliasnumrows == 0) {
                                    if ($numrows == 0) {
                                        //perform query which contains $numrows


                                        if ($numrows == 1) {
                                            $errormsg = "<span style='color: green'>Student has been Registered</span>";

                                            $getyear = "";


                                        } else {
                                            $errormsg = "An error has occured, Student has not been Registered";

                                        }

                                    }

                                    else {
                                        $errormsg = "There is already a Student with that Username";
                                    }
                                }

                                else {
                                    $errormsg = "There is already a Student with that Alias";
                                }
                            }

                            else {
                                $errormsg = "Your Passwords did not match";
                            }
                        }

                        else {
                            $errormsg = "You must retype your Password to Register";
                        }
                    }
                } else {
                    $errormsg = "You must enter in a Password to Register";
                }

            }

        else{
    $errormsg = "You must enter in Student's current Academic Year to Register";
    }

    }

$form = "
<form action='" . htmlentities($_SERVER["PHP_SELF"]) . "' method='post'>
  <table>
  <tr>
  <td></td>
  <td id='errormsg'>$errormsg</td>
</tr>
  <tr>
  <td>Year:</td>
  <td>{$yearHTML}</td>
  </tr>
  <tr>
  <td>Password:</td>
  <td><input type='password' name='studentpass' value='' /></td>  
  </tr>
  <tr>
  <td>Retype Password:</td>
  <td><input type='password' name='retypepass' value='' /></td>  
  </tr>
  <tr>
  <tr>
  <td></td>
  <td><input type='submit' value='Register' name='registerbtn' /></td>
  </tr>
  </table>
  </form>";

  echo $form;

わかりました、これは私が抱えている問題です。私は何年もの間ドロップダウンメニューを持っており、パスワードを入力してパスワードを再入力するための2つのテキスト入力があります。これで、ユーザーがドロップダウンメニューから年を選択してフォームを送信すると、パスワードを入力する必要があることを示すエラーメッセージが表示され、年ドロップダウンメニューにはユーザーが年ドロップダウンメニューから選択したオプションが表示されます。

私が抱えている問題は、以下のメッセージのいずれかが表示された場合、ドロップダウンオプションが「選択してください」の表示に戻ることです。私の質問は、次のエラーメッセージが表示されたときに、ドロップダウンメニューが[選択してください]オプションに戻るのを停止するにはどうすればよいですか?

$errormsg = "There is already a Student with that Username";
$errormsg = "There is already a Student with that Alias";
$errormsg = "Your Passwords did not match";
$errormsg = "You must retype your Password to Register";

変更が必要だと思うコード行は次のとおりです。

    $validSubmission = isset($_POST['registerbtn']) && $_POST['year'] && $_POST['studentpass'] && $_POST['retypepass'];
4

3 に答える 3

1

あなたが探しているのはこれだと思います:

// This is just a suggestion, but I would set these to null instead of an empty string:
    $getyear       = (isset($_POST['year'])) ? $_POST['year'] : null;
    $getpass       = (isset($_POST['studentpass'])) ? $_POST['studentpass'] : null;
    $getretypepass = (isset($_POST['retypepass'])) ? $_POST['retypepass'] : null;
    $errormsg      = (isset($errormsg)) ? $errormsg : null;
// You have already defined these, so no need to use $_POST here
// Also, I like to compare it to an actual value rather than just doing if ($variable)
// to avoid unforseen circumstances. Note, isset() returns false if $var == null.
$validSubmission = (isset($_POST['registerbtn']) && isset($getyear) && isset($getpass) && isset($getretypepass));

次に、後でロジックが逆になったと思います。

    foreach ($years as $year) {
        if ($validSubmission && $year == $getyear) {
            $yearHTML .= "<option value='" . $year . "' selected='selected'>$year</option>" . PHP_EOL;
        } else {
            $yearHTML .= "<option value='" . $year . "'>$year</option>" . PHP_EOL;
        }
    }
于 2012-12-08T00:19:26.283 に答える
1

Year ドロップダウンを再入力するとき、実際には、提出の一部が有効かどうかは気にせず、関係なく再入力したいだけです。そのためのロジックを削除するだけです。

foreach ($years as $year) {
    if (isset($_POST['year']) && $year == $_POST['year']) {
        $yearHTML .= "<option value='" . $year . "' selected='selected'>$year</option>" . PHP_EOL;
    } else {
        $yearHTML .= "<option value='" . $year . "'>$year</option>" . PHP_EOL;
    }
}
于 2012-12-08T00:42:55.790 に答える
0

問題は、サーバーに投稿し直し、サーバーがページを再レンダリングしていることです。これを適切に、かつすべてのブラウザーで機能するようにするには、$_POST の結果を (適切にエスケープした後に) 入力要素にエコーする必要があります。

同様に、選択ボックスに $_POST 値を使用して、属性を設定するoptionタグを決定する必要があります。selected="selected"

于 2012-12-08T00:19:02.930 に答える