質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.34%

  • PHP

    21307questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

PHPで既存の画像と新しい画像を差し替えたい

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 591

study_m

score 16

前提・実現したいこと

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="OK">
        </form>

    </body>
</html>
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+1

おそらく、inputのformの記述が問題だと思われます。
期待されているのは$pro_image_name_oldには、"image.png"等の名前だと思いますが、実際には、<?print $pro_image_name_old; ?>という文字列が入ってきていそうです。
遷移元のフォームを見直してみては如何でしょうか?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/04/17 17:16

    ご回答ありがとうございます。
    遷移元の該当する部分を見てみましたが、どこが原因なのか自分では判断できませんでした。
    お手数ですが、追記のソースも確認していただけるでしょうか?
    よろしくお願いいたします!!

    キャンセル

  • 2017/04/17 17:25

    <?print~ ではなく<?php printなのでは?(php抜けてますよ)

    キャンセル

  • 2017/04/17 17:36

    なんと・・・php抜けていましたね!<?php printにしましたら、解決しました!!ありがとうございます!

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.34%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る

  • PHP

    21307questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。