【PHP】ボタンに応じて、挨拶を変えたい。

プログラミング初学者です。
MVCの練習で、簡単なプログラムを PHP5.6.40 / MVCモデルにて
実装を図っています。

やりたいこと

ボタン毎に違うメッセージを表示させたい。
朝のボタンをクリック時には　おはようございます、(名前)さん。
昼のボタンをクリック時には　こんにちは、(名前)さん。

該当のソースコード

PHP
1<!-- Controller -->
2<?php
3    // 機能部分(Model)の読み込み
4    require_once "./greeting.php";
5
6    if(!isset($_POST["flag"])){ // 初めて画面を開いたとき
7        init();
8
9    } else if($_POST["flag"] = "1" || $_POST["flag"] = "2") {   // ボタンが押されたとき        
10        edit();
11    }
12    // View の読み込み
13    require_once "./nameform.php";
14?>

php
1<!-- Model -->
2<?php
3    function init() {   //初期値(空白)を設定
4        $_POST["namae"] = "";
5    }
6
7    function edit(){    //空白のとき メッセージを準備
8        if(empty($_POST["namae"])){
9            $_POST["msg"] = "入力されていません。";
10        }
11    }
12?>

php
1<!-- View -->
2<html>
3<head>
4    <meta charset=utf-8></meta>
5    <script>
6        function sample() {
7            document.getElementById("flag").value = "1";
8        }
9        function sample2() {
10            document.getElementById("flag").value = "2";
11        }
12    </script>
13    <title>名前入力</title>
14</head>
15<body>
16    <?php
17        if(isset( $_POST["msg"])){
18            echo $_POST["msg"];
19        } else if (isset($_POST["flag"]) && $_POST["flag"] = "1") {
20            echo "おはようございます、{$_POST["namae"]}さん". $_POST["flag"];
21        } else if (isset($_POST["flag"]) && $_POST["flag"] = "2") {
22            echo "こんにちは、{$_POST["namae"]}さん";
23        }
24    ?>
25    <form action="" method="post">
26        名前:<input type="text" name="namae" value="<?php echo $_POST["namae"]; ?>"><br>
27        <input type="hidden" name="flag" id="flag" value="">
28
29        <input type="submit" onclick="sample();" value="朝">
30        <input type="submit" onclick="sample2();" value="昼"><br>
31    </form>
32</body>
33</html>

試したこと

var_dump()を行い、値の受け渡しができているかの確認をしました。
flagの値の受け渡しができているのですが、
どちらを押しても　"1"が送られており、おはようのメッセージが表示されます。

拙い文章、思い付きだらけのコードではございますが、
ご教授いただければ助かります。

m.ts10806

2020/03/19 07:50

根本的な話ですが、過去案件のサポートなどの理由がない限り、既にサポートが終わったPHPのバージョンで学習をするのは意味がありません。 7系最新を使いましょう。

nnn_kkk

2020/03/19 07:59

ありがとうございます。すぐにでも最新のものに変更したいと思います。

行動規範の内容に同意します

回答2件

ベストアンサー

他にもいろいろありそうな気はしますが、とりあえずは比較演算子が間違っています

PHP
1    } else if (isset($_POST["flag"]) && $_POST["flag"] = "1") {

↓

PHP
1    } else if (isset($_POST["flag"]) && $_POST["flag"] === "1") {

他のif内の比較演算子も同様に確認してください。
https://www.php.net/manual/ja/language.operators.comparison.php

投稿2020/03/19 07:50

編集2020/03/19 07:51

tanat

総合スコア18713

nnn_kkk

2020/03/19 07:55

素早いご対応ありがとうございました。ほかの間違えてそうなところに関しては、これから学習して改善していけるよう頑張ります。

tanat

2020/03/19 08:08

全体を通して確認するのはなかなか大変なので、最小限の範囲を切り出して挙動を確認出来るようにするとデバッグが捗る＆質問もしやすくなるかと思います。例えば今回の場合だと ifの直前で変数に入ってる値は正しいことはわかっている →ifの条件文がおかしい可能性が高い ↓ こんな感じで確認用のphpファイルを作ってアクセスしてみて、ifの挙動を確認してみる。 ↓ 他の箇所では無く、必ずこの中に問題の箇所があることが確認できる ↓ １行ずつしらみつぶしに確認してみたりリファレンスを確認してみたりする <?php $flag = "1"; $msg = "test"; $name = "太郎" if(isset( $msg)){ echo $msg; } else if (isset($flag) && $flag = "1") { echo "おはようございます、{$name}さん". $flag; } else if (isset($flag) && $flag = "2") { echo "こんにちは、{$name}さん"; }

行動規範の内容に同意します

~~```html
<input type="submit" onclick="sample();" value="朝">
<input type="submit" onclick="sample2();" value="昼"><br>

inputのtypeがsubmitの時は、onclickより先にsubmitが行われてしまいます。
なので、例えばtypeをbuttonにして、onclick時に発火されるJavaScriptの側で処理の最後にformをsubmitする処理を書いてあげるなどの方法がいいと思います。~~

すみません、全然頓珍漢なことを書いてしまいました。

投稿2020/03/19 07:55

編集2020/03/19 07:58