入力フォームから送られてきたidレコードがテーブルに存在したらupdate、存在していなかったら何が間違っているかメッセージで出力する方法。

Question

実現したいこと

入力フォームから送られてきたidとheightの値をtableのheightカラムにupdateしたいです。
該当するidレコードがなかったらレコードがない旨をメッセージとして出力したいです。

発生している問題・エラーメッセージ

tableにはid 1のレコードしか存在していませんが、
入力フォームにid 2や3などを入力しても
データを更新しましたと出力されてしまいます。
Countのwhereの部分が間違っているのでしょうか？
ちゃんと条件分岐するためにはどうすればよいでしょうか。

try {
$id = $_POST["id"];
$height = $_POST["height"];

if ($id === "" || $height === "" ) {    
    echo "入力に不備があります。";
} elseif (strlen($height)<2 || strlen($height)>3 ||! ctype_digit($height) || 
substr($height,0,1) === "0") {
    echo "2〜3桁の正しい半角数字で入力してください。";
} else {
    $sql = "SELECT COUNT(id) FROM users WHERE id = 1";
    $stmt = $dbh->prepare($sql);
    $stmt->execute();
    $result = $stmt->fetchAll(PDO::FETCH_ASSOC);

    if  ($result > 0) {
        $sql = "UPDATE users SET height = :height WHERE id = :id";
        $stmt = $dbh->prepare($sql);
        $params = array(":id" => $id, ":height" => $height);
        $stmt->execute($params);
        echo "データを更新しました。";
    }else {
        echo "該当するレコードがありません。";
    }   
}

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[YAmaGNZ]

SELECT COUNT(id) FROM users WHERE id = 1
と１固定にしているのは正しいのですか？

[m.ts10806]

質問タグにPHP追加してください。

Answer 1

コメントで指摘されているようにPOSTのIDをとっているのに id = 1と固定にしているのと、
せっかくSQLでcountとってるのに$resultを直参照しているのは意味がないのでは？
countだとSQLに構文エラーや参照ミス（テーブルやカラム名の間違い等）がなければ必ず1件取得できますし（SQLをDBに対して直実行してみてください）、PDOStatement::fetchAllの返却値は結果セットの配列なので$result > 0は間違いなくtrueになります。
結果セット配列からcountを参照しましょう（キーはvar_dump()などで確認のこと）

Comments

[Anpontangaya]

回答ありがとうございます！
まずsqlをdbに直実行したいのですが、こちらのid=1の1の部分をどのように書けばうまく取得できるのかわからずにいます。
教えていただけたらありがたいです。

[m.ts10806]

直に実行する場合はGUIツールやコマンドラインで対象DBにログインして行いますし、
あくまでモックなのでそこは固定値になります。POSTされるであろう値を何種類か試せばよいです。
PHPから実行＝アプリケーション通しての実行　で、
値などがバインドされるので、直実行はバインドされた前提で、「想定の結果が得られるか」をアプリケーション実行前に確認するためのものです。
開発時には構文エラー程度は取り除いた状態でアプリケーションから実行されるべきなので、直実行により想定の結果が得られるかを確認するのが先です。

[Anpontangaya]

申し訳ございません。
勉強不足なのは承知なのですが、初心者で内容が少し難しい部分があります、、、
実際にコードを入力するとどのようなものになるか教えていただけると幸いです。

[m.ts10806]

DBが設置されているサーバに対してコマンドライン（Windowsならコマンドプロンプト）で接続するケースと
既存のツール（MySQLならphpMyAdminやWorkBenchなどが代表。他にも色々）で接続するケース

後者の方が画面で見られるので楽だとは思いますが、そこは色々と試されると良いかと。

理解してもらいたい点としては、
DBはアプリケーション（今回の場合はPHPプログラム）とは別であり、必ずアプリケーションから実行しなければ結果を得られないわけではなく、DBだけで単体で結果を得ることができるということです。
PHPがやっているのはSQL文の形をした文字列とパラメータを引き渡すのみ。あくまで処理・実行結果の返却はDB側で行われているので、それぞれ単体で実行できるわけです。
※アプリケーションから渡された情報をDB側で確認するのは簡単ではないので、あくまで問題切り分けのために直実行を必ず先にして想定の結果を得られるか確認しましょうということです

[Anpontangaya]

ありがとうございます！
確認してみます！

Answer 2

PDOであればupdateを発行したあとrowCountして、0なら更新されなかった、1なら更新されたです。
ただidが存在しない場合と、idが存在するがsetするheightが保持しているheightと同じ場合ともにupdateはされなかったと判断されますので、もしその場合分けが必要であれば、一つは更新が0のときはselectですでにデータが存在するか確認する、もしくはテーブルに更新日時欄(たとえばmodifiedカラム)を設定して、なにかデータ投入するたびにmodifiedをnowで上書きするとよいでしょう

Comments

[Anpontangaya]

回答ありがとうございます！
場合分けが必要なのですが、まずidレコードが存在すればupdateし、なければ該当するレコードがないというふうにしたいのですが
Selectでうまく取得できずにいます、、、
Serect countのwhereの部分のidをどのように書けばうまく取得できるのか教えていただけたら幸いです。

[yambejp]

ですので・・・すでにidが存在してheightの値が同じで更新されなかった場合はどうしたいのですか？

[Anpontangaya]

すみません。
そちらは特に場合分けが必要ないみたいでした。

[yambejp]

であれば、回答した通り「PDOであればupdateを発行したあとrowCountして、0なら更新されなかった、1なら更新された」が必要十分です

[Anpontangaya]

かしこまりました！
答えていただきありがとうございます！