[PHP]POSTに失敗してしまいます

Question

PHP勉強中の者です。現在自動販売機作成ツールの管理画面を作成しているのですが、POSTの処理につまずいてしまっています。
ご教授頂ければ幸いです。
現在作成しているコードは以下の通りです。
DBへの接続情報は隠させていただいています。
※文字数制限の為続きはコメント欄にて書かせていただきます。

header("Content-type: text/html; charset=utf-8");
const HOST = '';
const DB_NAME = '';
const USER_NAME = '';
const PASSWORD = '';
$drink_name = '';
$drink_price = 0;
$drink_quantity = 0;
$drink_status = 0;
$drink_id = 0;
$tmp_file = '';
$file_name = '';
$upload_file_name = '';
$success_msg  = array();
$err_msg = array();
$drink_info_list = array();

//DB接続
if(($link = mysqli_connect(HOST,USER_NAME,PASSWORD,DB_NAME)) !== FALSE){
    
    mysqli_set_charset($link,'UTF8');
    
    if ($_SERVER['REQUEST_METHOD'] === 'POST') {
        
        $submit_type = $_POST['submit_type'];
        
        switch ($submit_type) {
            case 'add_item':
                // 新規商品追加の場合
                
                //POSTされた値を取得
                $drink_name = $_POST['drink_name'];
                $drink_price = $_POST['drink_price'];
                $drink_quantity = $_POST['drink_quantity'];
                $drink_status = (int)$_POST['public_status'];
                
                
                $tmp_file = $_FILES['drink_img']['tmp_name'];
                
                $file_name = 'img/' . $_FILES['drink_img']['name']; //hogehoge.php 
                
                
                // 登録値チェック
                /*
                
                17/08/15 
                issetは入力値判定には使えません。
                issetは、「宣言されているか」というチェックになります。
                
                入力されてるか、のチェックは言い換えると
                (文字入力の場合は)変数の中の文字数が0文字より多いか
                (数値の場合は)変数の中の数値が0より大きいか
                というチェックです。
                
                */
                
                // 1.名前が入力されているか
                if($drink_name === '' ){
                    $err_msg[] = '名前が入力されていません';
                }
                // 2.値段が入力されているか
                if($drink_price < 0 ){
                    $err_msg[] = '値段が入力されていません';
                }
                // 3.値段が数値か
                if(ctype_digit($drink_price) === FALSE){
                    $err_msg[] = '値段を入力してください';
                }
                // 4.在庫数が入力されているか
                if($drink_quantity < 0 ){
                    $err_msg[] = '個数が入力されていません';
                }
                // 5.在庫数が数値かどうか
                if(ctype_digit($drink_quantity) === FALSE){
                    $err_msg[] = '数値を入力してください';
                }
                // 6.ファイルの拡張子チェック
                // substr = sub string  -> 文字の抜き取り
                // strrpos = string reverse position -> 対象の文字が何文字目にあるか？
                // strpos = string position
                
                // strpos('あいうえお', 'あ') -> 0
                // strrpos('あいうえお', 'あ') -> 0
                
                 //strrpos('cola.jpg', '.') -> 4
                // substr('cola.jpg', 4) -> jpg
                
                // substr('apple.jpg', strrpos('apple.jpg', '.') + 1);
                // substr('apple.jpg', 5 + 1);
                // $ext = substr('apple.jpg', 6);//
                // 第一引数の文字を、第二引数の数値以降分抜き出す
                // 第三引数がある場合は、その数byte分抜き出す
                
                // strpos
                // strrpos
                
                $ext = substr($file_name, strrpos($file_name, '.') + 1);
                if(!($ext === 'jpeg' || $ext === 'jpg' || $ext === 'png')){
                    $error_msg[] = 'ファイル形式が異なります。画像ファイルはJPEG又はPNGのみ利用可能です。';
                }
                
                // エラーがなかったら(validation check に引っかからなかった場合に登録)
                if (count($error_msg) === 0) {
                    
                    mysqli_autocommit($link, false);
                    
                    // 1.INSERT to DB
                    
                    // create sql
                    $sql = 'INSERT INTO drink_info_table(drink_name,price,status) VALUES(' . '\'' .  $drink_name . '\',' . $drink_price . ',' . $drink_status . ')';
                    
                    // exec sql
                    if(mysqli_query($link,$sql) !== FALSE ){
                        
                        $drink_id = mysqli_insert_id($link);
                        
                        // create sql
                        $sql = 'INSERT INTO drink_stock_table(drink_id, stock_number) VALUES (' . $drink_id . ',' . $drink_quantity . ')';
                        
                        // exec sql
                        if(mysqli_query($link,$sql) !== FALSE ){
                            
                        } else{  
                            $err_msg[] = 'drink_stock_table:insertエラー:' . $sql;
                        }
                        
                    }else{  
                        $err_msg[] = 'drink_info_table:insertエラー:' . $sql;
                    }
                    
                    // 2.UPLOAD PIC
                    if (count($error_msg) === 0) {
                        if (is_uploaded_file($tmp_file) === TRUE){
                            $upload_file_name = 'img/' . $drink_id . '.' . $ext;
                            
                            if ( move_uploaded_file($tmp_file,$upload_file_name) === TRUE){//同名ファイルがアップロードされた場合の処理
                                
                            } else {
                                $err_msg[] = 'ファイルがアップロードできません';
                            }
                            
                        } else {
                            $err_msg[] = 'ファイルが選択されていません';
                        }
                    }
                    
                    // トランザクション成否判定
                    if(count($err_msg) === 0){
                        $success_msg[] = '商品追加成功';
                        mysqli_commit($link);
                    }else{
                        mysqli_rollback($link);
                    }
                }
                
                break;
                
            case 'change_value':
                // 在庫数変更の場合
                
                if(isset($_post['update_stock'])){
                    $update_stock = (int)$_post['update_stock'];
                    $sql = 'UPDATE drink_stock_table SET stock_number = ' . $update_stock . 'WHERE drink_id = ' . $drink_id;//間違え
                    if(mysqli_query($link,$sql) === TRUE){
                        $success_msg [] = '在庫変更成功'; 
                    }else{
                        $err_msg[] = 'UPDATE drink_stock_table:updateエラー:' . $sql;
                    }
                    //if($_POST['change_status']){
                    //    $success_msg [] = 'ステータス変更成功';
                    //}
                }$err_msg[] = '在庫変更失敗';
                
                break;
                
            case 'change_status':
                // ステータス変更の場合
                if(isset($_POST['change_status']) === '公開->非公開'){
                        $sql = 'UPDATE drink_info_table SET status = 0 WHERE drink_id = ' . $drink_id;
                        if(mysqli_query($link,$sql) === FALSE){
                            $err_msg[] = 'ステータス変更失敗';
                            
                        }
                }elseif(isset($_POST['change_status']) === '非公開->公開') {
                            
                        $sql = 'UPDATE drink_info_table SET status = 1 WHERE drink_id = ' . $drink_id;
                        if(mysqli_query($link,$sql) === FALSE){
                            $err_msg[] = 'ステータス変更失敗';
                        }
                }else{
                    $err_msg[] = 'ステータス変更失敗';
                }    
                break;
                
        }
        
        
    } // POST end
    
    //DBの値を取得
    $sql = 'SELECT drink_info_table.drink_id,drink_name,price,status,stock_number 
            FROM drink_info_table 
            JOIN drink_stock_table
            ON drink_info_table.drink_id = drink_stock_table.drink_id;';
    if($result = mysqli_query($link,$sql)){
        $i = 0;
        while($row = mysqli_fetch_assoc($result)){
            $drink_info_list[$i]['drink_id'] =  htmlspecialchars($row['drink_id'],ENT_QUOTES, 'UTF-8');
            $drink_info_list[$i]['drink_name'] =  htmlspecialchars($row['drink_name'],ENT_QUOTES, 'UTF-8');
            $drink_info_list[$i]['price'] =  htmlspecialchars($row['price'],ENT_QUOTES, 'UTF-8');
            $drink_info_list[$i]['status'] =  htmlspecialchars($row['status'],ENT_QUOTES, 'UTF-8');
            $drink_info_list[$i]['stock_number'] = htmlspecialchars($row['stock_number'],ENT_QUOTES, 'UTF-8');

            $i++;
            
            
        }
        
    }
}else{
    $err_msg[] = 'DB接続失敗';
}

Comments

[退会済みユーザー]

何を実現したいのか、しかしどういうエラーが発生しどういう現象が発生するのか、を書いてほしい。

[kei344]

「失敗」とはどのような状況でしょうか。出ているエラーなどと併せて、具体的に記述されたほうが回答を得られやすいと思います。

[newyee]

ただいまコメント欄につい追記させて貰いました。すみません。コードが長くなってしまい。切り抜いての説明が難しいと思ったもので...

Answer 1

$_POSTです。$_postではありません。

isset($_post['update_stock'])

こちらを

isset($_POST['update_stock'])

と変更してみてください。

また、SQL文に誤りがあります。

$sql = 'UPDATE drink_stock_table SET stock_number = ' . $update_stock . 'WHERE drink_id = ' . $drink_id;//間違え

となっていますが、

$sql = 'UPDATE drink_stock_table SET stock_number = ' . $update_stock . ' WHERE drink_id = ' . $drink_id;//間違え

のように、WHEREの前に空白ができるようにしてください。

また、UPDATEの際、$drink_idが常に0になってしまっていますので

<input type="hidden" name="drink_id" value="<?php echo $drink_id ?>">

をHTMLに含めて、PHP側で

if (isset($_POST['drink_id'])){
  $drink_id = $_POST['drink_id'];
}

も追加してください。

Comments

[newyee]

早速の回答ありがとうございます。
訂正してみた所、isset($_POST['update_stock'])ここの部分に関しましてはTUREとなり解決することができました。
ただUPDATEエラーがでてしまったんですけど、これはPOSTと関係ない所ですもんね。。。

[tsuemura]

エラーの内容を教えてもらえますか？
mysqli_error($link)
で取得できると思います。

[newyee]

返信ありがとうございます。
下記がエラーの内容となります
「UPDATE drink_stock_table:updateエラー:UPDATE drink_stock_table SET stock_number = 0WHERE drink_id = 0」
在庫数を入力しても、stock_number = 0となっていることから反映されていないのではないかと思っています...

[tsuemura]

回答に追記しましたが、
SET stock_number = 0WHERE drink_id = 0
となっていますので、
SET stock_number = 0 WHERE drink_id = 0
のように、前の文と後の文の間に空白を空けましょう。

[newyee]

ありがとうございます。エラーもでず在庫数変更と表示もでました。
ただ、一点だけ気になることがあるのですが、エラーなくUPDATEが成功しているのですが、入力した在庫数がデフォルトの表示から変わらず、例えば「3→4」に変更した場合でも「3」のままの表示なっているんですよね...データベースの中身も変更がないままです。
<input type="text"name="update_stock" value="<?php echo $values['stock_number']?>">
ここの$values['stock_number']に問題があるとみているのですが、何故なのか分からないです...

[tsuemura]

いや、$drink_idが常に0になっているからではないでしょうか。
HTMLのformに、データベースから取得したdrink_idを載せて、POSTリクエストにdrink_idが含まれるようにしてください。回答にサンプルを追記しておきます。

[newyee]

ありがとうございます。
参考にさせて頂きます。

Answer 2

<!DOCTYPE html>
<html lang="ja">
<head>
    <meta charset="UTF-8">
    <title>自動販売機</title>
    <style>
        table,th,tr,td{
            border:solid 1px;
        }
    </style>
</head>
<body>
    <?php foreach($success_msg as $value){?>
        <p><?php echo $value;?></p>
    <?php } ?>
    <?php foreach($err_msg as $value){?>
        <p><?php echo $value;?></p>
    <?php }  ?>
    <h1>自動販売機管理ツール</h1>
    <section>
        <h2>新規商品追加</h2>
        <form method="post" enctype="multipart/form-data">
            <div>
                <label>名前<input type="text" name="drink_name"></label>
            </div>
            <div>
                <label>値段<input type="text" name="drink_price"></label>
            </div>
            <div>
                <label>個数<input type="text" name="drink_quantity"></label>
            </div>
            <div><input type="file"name="drink_img"></div>
            <div>
                <select name="public_status">
                    <option value="0">非公開</option>
                    <option value="1">公開</option>
                </select>    
            </div>
            <div><input type="submit" value="送信"></div>
            <input type="hidden" name="submit_type" value="add_item" />
        </form>
    </section>
    <section>
        <h2>商品情報変更</h2>
        <table>
            <caption>商品一覧</caption>
            
            <tbody>
                <tr>
                    <th>商品画像</th>
                    <th>商品名</th>
                    <th>価格</th>
                    <th>在庫数</th>
                    <th>ステータス</th>
                </tr>
                    <!-- foreachで回す -->
                    <!-- 在庫数変更のform -->
                    
                    <?php foreach ($drink_info_list as $values) {
                    //var_dump($values);
                    //exit();
                    ?>
                            
                            <tr>
                        
                                <form method ="post">  
                                    <td><img src="<?php $upload_file_name ;?>"></td>
                                    <td><?php echo $values['drink_name']; ?></td>
                                    <td><?php echo $values['price']?></td>
                                    <td>
                                        <input type="text"name="update_stock" value="<?php echo $values['stock_number']?>">
                                        <input type="submit"value="変更">
                                    </td>
                                    <input type="hidden" name="submit_type" value="change_value" />
                                </form>
                                <!-- ステータス変更のform -->
                                <form method ="post">
                                    <td>
                                        
                                        <?php if((int)$values['status'] === 1){  ?>
                                            <input type="submit" name="change_status" value="公開->非公開">
                                        <?php } else {  ?>
                                            <input type="submit" name="change_status" value="非公開->公開">
                                        <?php }  ?>
                                        
                                        
                                    </td>
                                    <input type="hidden" name="submit_type" value="change_status">
                                </form> 
                                
                                <?php }  ?>
                            </tr>
                    
            </tbody>
        </table>
    </section>
</body>
</html>

ここまでが現在作成しているコードとなります。

Comments

[newyee]

長くなってしまいすみません。
切り抜いてご説明するのは難しいと思ったもので...
問題となっている箇所なのですが、HTML内の、
<input type="text"name="update_stock" value="<?php echo $values['stock_number']?>"←ここの部分でPOSTされた値が if(isset($_post['update_stock']))←ここのif文でFALSEとなってしまい在庫数を変更することができません。
ステータス変更の部分についても同様に、
<?php if((int)$values['status'] === 1){ ?>
<input type="submit" name="change_status" value="公開->非公開">
<?php } else { ?>
<input type="submit" name="change_status" value="非公開->公開">
<?php } ?>
ここでPOSTした値が、FALSEとなってしまいステータス変更ができずにいます...