Q&A
###前提・実現したいこと
XAMPPとMySQLを使ってショッピングサイトを作っています。
登録した商品の画像を別の画像に差し替えたいです。
動作としては、下記のような期待結果になっていることを確認しました。
・別の画像に差し替えができている
・DBに登録される画像ファイル名がアップロードした別のファイル名に変更されている
ですが、下記のエラーメッセージが出ています。
具体的にどこを修正すればエラーが解決するのでしょうか？
###発生している問題・エラーメッセージ
unlink(./image/<?print $pro_image_name_old; ?>): No error in C:\xampp\htdocs\product\pro_edit_done.php on line 47
###該当のソースコード
<!DOCTYPE html> <html> <head> <meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/> <title>ろくまる農園</title> </head> <body> <?php try{ //■データベースが正常に動いているときに動く本来のプログラム $pro_code=$_POST['code']; $pro_name=$_POST['name']; $pro_price=$_POST['price']; $pro_image_name_old=$_POST['image_name_old']; $pro_image_name=$_POST['image_name']; $pro_code=htmlspecialchars($pro_code); $pro_name=htmlspecialchars($pro_name); $pro_price=htmlspecialchars($pro_price); //■DBにSQL文で命令する $dsn ='mysql:dbname=shop;host=localhost;charset=utf8'; $user ='root'; $password=''; $dbh =new PDO($dsn ,$user ,$password); $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); $sql ='UPDATE mst_product SET name=?,price=?,image=? WHERE code=?'; //■$stmtに$sqlを準備する $stmt =$dbh->prepare($sql); $data[]=$pro_name; $data[]=$pro_price; $data[]=$pro_image_name; $data[]=$pro_code; //■execure($data)でSQLで命令を出す $stmt->execute($data); //DBとのアクセスを切断する $dbh=null; //もし古い画像が空でないとき（古い画像があるとき）削除する //古い画像と新しい画像が同じだったら何もしない if($pro_image_name_old!=$pro_image_name){ unlink('./image/'.$pro_image_name_old); } print '修正しました<br/><br/>'; //データベースがダウンしているときに動くプログラム }catch(Exception $e){ print "ただいま障害によりサーバーに接続できません。大変ご迷惑をおかけしています。"; //強制終了 exit(); } ?> <a href="pro_list.php">戻る</a> </body> </html>
###遷移元のソース
エラーコードでprintで表示されているのは下記のvalueです。
<input type="hidden" name="image_name_old" value="<?print $pro_image_name_old; ?>">
<!DOCTYPE html> <html> <head> <meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/> <title>ろくまる農園</title> </head> <body> <?php try{ $pro_code =$_GET['procode']; //■DBにSQL文で命令する $dsn ='mysql:dbname=shop;host=localhost;charset=utf8'; $user ='root'; $password=''; $dbh =new PDO($dsn ,$user ,$password); $dbh->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION); //codeレコードのmst_proテーブルからnameカラムを選んで$sqlに代入する $sql ='SELECT name,price,image FROM mst_product WHERE code=?'; //■$stmtから全て取り出す(fetch~)ための準備をする $stmt =$dbh->prepare($sql); //取り出すデータを$pro_codeに絞る $data[]=$pro_code; //■execute($data)でSQLで命令を出す $stmt->execute($data); //■$stmtから全て取り出す $rec =$stmt->fetch(PDO::FETCH_ASSOC); $pro_name=$rec['name']; $pro_price=$rec['price']; //古くなった画像を定義する $pro_image_name_old=$rec['image']; if($pro_image_name_old==''){ $disp_image=''; } else{ $disp_image='<img src="./image/'.$pro_image_name_old.'">'; } //■DBとのアクセスを切断する $dbh=null; //データベースがダウンしているときに動くプログラム }catch(Exception $e){ print "ただいま障害によりサーバーに接続できません。大変ご迷惑をおかけしています。"; exit(); } ?> <!--■ここからHTML--> 商品修正<br /><br /> 商品コード<br /> <?php print $pro_code; ?> <br /><br /> <form method="post" action="pro_edit_check.php" enctype="multipart/form-data"> <!-- hiddenで受け取る--> <input type="hidden" name="code" value="<?php print $pro_code; ?>"> <!--古い画像（入れ替える前の画像）のデータをhiddenで送る--> <input type="hidden" name="image_name_old" value="<?print $pro_image_name_old; ?>"> 商品名<br /> <input type="text" name="name" style="width:200px" value="<?php print $pro_name; ?>"><br /> 価格<br /> <input type="text" name="price" style="width:50px" value="<?php print $pro_price; ?>">円<br /><br /> <?php print $disp_image; ?> <br /> 画像を選んでください<br /> <input type="file" name="image" style="width:400px"> <br /><br /> <input type="button" onclick="history.back()" value="戻る"> <input type="submit" value="ＯＫ"> </form> </body> </html>
回答1件
0
ベストアンサー
おそらく、inputのformの記述が問題だと思われます。
期待されているのは$pro_image_name_oldには、"image.png"等の名前だと思いますが、実際には、<?print $pro_image_name_old; ?>という文字列が入ってきていそうです。
遷移元のフォームを見直してみては如何でしょうか？
投稿2017/04/17 07:21
総合スコア3027
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
2017/04/17 08:16
2017/04/17 08:25
2017/04/17 08:36