$_POSTに文字列が代入されない

皆様いつもお世話になっております。

実行環境は以下の通りです。
PHP:7.2.1
phpMyAdmin:4.7.7
MySQL:5.6.38

こちらのデータベースのmountの数値を

PHPを使って増減させたいのですが、$_POST['code']に値がうまく入らず、エラーが出てしまいます。

PHP
1<?php
2
3function accessSQL(){
4    $user = "root";
5    $pass = "root";
6    $dbh = new PDO('mysql:host=localhost;dbname=hozon;charset=utf8', $user, $pass);
7    return $dbh;
8}
9
10try {
11    $dbh = accessSQL();
12    $stmt = $dbh->prepare("SELECT id, code, prod_name, mount, category, end_date from hozon where id = 56");
13    $stmt->execute();
14    $value_array = $stmt->fetchALL();
15        foreach ($value_array as $value) {
16            $value['code'];
17            $value['prod_name'];
18            $value['mount'];
19        }
20    }catch (PDOException $e){
21        echo $e->getMessage();
22    }
23
24
25if (isset($_POST['change'])){
26    echo 'new_mount is : '.$_POST['new_mount'];//テスト
27    echo 'code is : '.$_POST['code'];//テスト：ここの$_POST変数にcodeが代入されない
28    try{$dbh = accessSQL();
29        $stmt = $dbh->prepare("update hozon set mount = :mount where code = :code");
30        $stmt -> bindParam(':mount', $_POST['new_mount'],PDO::PARAM_STR);
31        $stmt -> bindParam(':code',$_POST['code'], PDO::PARAM_STR); //ここの$_POST変数にコードが代入されない
32        $stmt -> execute();
33    }catch (PDOException $e){
34        echo $e->getMessage();
35    }
36}
37
38if (isset($_POST['cancel'])){
39    [header('Location:./index.php')];
40    exit();
41}
42?>
43<!doctype html>
44<html lang="ja">
45<head>
46    <meta charset="UTF-8">
47    <meta http-equiv="X-UA-Compatible" content="ie=edge">
48    <!--Import materialize.css-->
49    <link type="text/css" rel="stylesheet" href="css/materialize.min.css"  media="screen,projection"/>
50    <link type="text/css" rel="stylesheet" href="css/base.css"  media="screen,projection"/>
51    <title>保存期限管理システム</title>
52</head>
53<form action="update.php" method="post">
54    <div class="row input-field">
55        <div class="col s2">
56            <label>&nbsp;</label>
57            <input type="text" name="code" placeholder="商品番号" value="<?php echo $value['code'];?>" disabled>
58        </div>
59        <div class="col s5">
60            <label>&nbsp;</label>
61            <input type="text" name="prod_name" placeholder="商品名" value="<?php echo $value['prod_name']; ?>" disabled>
62        </div>
63        <div class="col s5">
64            <label>*個数（半角数字で入力してください）</label>
65            <input type="number" name="new_mount" placeholder="個数" min="1" value="<?php echo $value['mount']; ?>" required>
66        </div>
67
68        <div class="row">
69            <input type="submit" name="change" value="登録" class="regist btn btn-large">
70            <input type="submit" name="cancel" value="キャンセル" class="btn-large cancel">
71        </div>
72    </div>
73</form>
74

どうしたら$_POST['code']に値が入るようになるでしょうか？アドバイスとご教授のほどよろしくお願いします。

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

回答4件

ベストアンサー

DBから受け取ったcodeをinputにうけてpostしなおしたときの処理ですね？

<input type="text" name="code" placeholder="商品番号" value="<?php echo $value['code'];?>" disabled>

disabled属性のタグはサブミットしたときにデータ渡しされません
disabled→readonlyとするか
disabledをとってtype="hiddenなどに変更するとよいでしょう

投稿2018/05/11 03:02

yambejp

総合スコア114829

kalon

2018/05/11 04:02

解決しました！「disabled属性のタグはサブミットしたときにデータ渡しされません」全く知りませんでした。的確なアドバイスをありがとうございます。

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

php
1echo 'code is : '.$_POST['code'];
2→
3var_dump($_POST);

と変更してみて、中身を確認してください。

このスクリプト、大量に問題点があるので、全体見直したほうが良いかと。

ざっくっと
・入力値確認なしに DB に突っ込もうとしている
・foreach ($value_array as $value) の $value を <input value=…> まで引きずっている & foreach 内で何もやっていない。
・$dbh 作成時、ATTR_ERRMODE の設定がされていない。
・$dbh 作成時、なぜ try - catch しない？
・（出力時にエスケープはしてほしいなぁ）

投稿2018/05/11 03:09

退会済みユーザー

総合スコア0

kalon

2018/05/11 04:49

ご丁寧なご指摘に感謝です。 >・入力値確認なしに DB に突っ込もうとしている実装します。 >foreach ($value_array as $value) の $value を <input value=…> まで引きずっている & foreach 内で >何もやっていない。 foreach ($value_array as $value)をしないで、データベースからとってきた値を$value['code']のような連想配列にすることは出来るのでしょうか？　＆ <input value=…>まで引きずらない方法は他にあるのでしょうか？ >・$dbh 作成時、ATTR_ERRMODE の設定がされていない。 $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);の一文を入れれば良いのでしょうか？ >・$dbh 作成時、なぜ try - catch しない？ function accessSQL(){ try { $user = "root"; $pass = "root"; $dbh = new PDO('mysql:host=localhost;dbname=hozon;charset=utf8', $user, $pass); return $dbh; }catch (PDOException $e){ echo $e->getMessage(); } } こういうことでしょうか？ >・（出力時にエスケープはしてほしいなぁ htmlspecialcharですね？実装します。

退会済みユーザー

2018/05/11 09:35

https://qiita.com/mpyw/items/b2a2b0e8e60ed4680f06 私は、mpyw さんの書いている以下の記事で学習したのでオススメです。・PHPでデータベースに接続するときのまとめ・PDOフェッチパターン大全・「何故htmlspecialcharsを通すのか？」を一言でどうぞ「PHPでデータベースに接続するときのまとめ」はボリュームもあり、中身も濃いので、初心者には難しいと思いますが、DB 接続する度に読み返すと、理解が進みます。頑張ってください。

kalon

2018/05/11 09:55

教えてくださってありがとうございます。わかりました、読んでみます。

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

$_POST['code']が空ということは、更新処理をするフォームに対して送ってきていないということになると思います。
別のphpファイルなどで表示しているフォームでしょうから、そちらのソースを見せてもらわないとなんとも言えません。
codeをcodaとかタイプミスしているんじゃないでしょうか。hidden属性で持たせるべきcodeが欠落していないでしょうか。

画面遷移も適切に過不足なく質問内に説明がほしいです。
入力→確認→更新処理と画面遷移してそれぞれphpファイルが別なのかどうかとか。
別だとした場合、
確認画面のformの中に<input type="hidden" name="code" value="<?php echo $_POST['code']?>" />とか必要ですが、いかがでしょうか。

投稿2018/05/11 02:58

編集2018/05/11 03:07

退会済みユーザー

総合スコア0