勉強の為に簡単なECサイトを作ってます。
データは全てデータベースに格納されていて、データベースから持ってきたいと思っています。
写真のデータベースへの格納の仕方は、写真自体を格納する方法と、パスを格納する方法と２つあると思いますが、今回はパスを格納し、パスを持ってきて格納されてる場所にアクセスし、写真を表示させたいと思っています。

以下のソースで、きちんとDBにアクセスできて、パスも持ってきていているのですが、写真が表示されません。
どうしたらいいでしょうか。

php
1<?php
2//img_get.php//
3session_start();
4ini_set("display_errors","on");
5error_reporting(E_ALL | E_STRICT);
6ini_set("include_path","./includes");
7include("dbinfo.inc");
8// DB接続
9$dbLink = mysqli_connect($host, $user, $passwd, $dbname) or die ("Can't connect");
10	if (!$dbLink){
11		echo "connect failed";
12	}
13
14// Get img
15$sql = "SELECT catalog_id , path FROM product_img WHERE catalog_id = " . $_GET['catalog_id'];
16$result = mysqli_query($dbLink,$sql) or trigger_error($mysqli->error."[$sql]");
17$row = mysqli_fetch_assoc($result);
18if (!$result) {
19echo ("SQL error<BR>");
20exit;
21}
22
23$_SESSION['path'] = $row['path'];
24$_SESSION['catalog_id'] = $row['catalog_id'];
25
26
27//問題の箇所。。。
28if (file_exists($_SESSION['catalog_id'])) {
29    header('Content-Type: images/jpg');
30    readfile($_SESSION['catalog_id']);
31}
32
33// バイナリデータを直接表示
34echo $_SESSION['path']
35?>
36

表示させる画面です。

php
1
2<!DOCTYPE>
3<HTML>
4<HEAD>
5	<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
6  <title>Ex6 Sofa</title>
7  <link rel="stylesheet" href="custom.css">
8  <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.0/css/bootstrap.min.css">
9</HEAD>
10<BODY>
11  <div class="col-xs-6 col-xs-offset-3">
12          <h1>Online shop</h1><p>Hi, <?php echo $_SESSION['username']; ?>!!</p>
13<ul>
14  <li>Your customer ID is :<?php echo $_SESSION['cust_id']; ?></li>
15  <li>You logged in number of times:<?php echo $login_no; ?></li>
16  <li>Logtime：<?php echo $date; ?></li>
17</ul>
18  <p style='color: red; margin-bottom: 0; font-weight: bold'><?php echo $errorMessage; ?></p>
19
20  <h2>Sofa</h2>
21  <table border="1" width="500" cellspacing="0" cellpadding="5" bordercolor="#333333">
22  <tr>
23  <th>catalogID</th>
24  <th>Product name</th>
25  <th>price</th>
26  <th>Description</th>
27  <th>image</th>
28  </tr>
29  <tr>
30    <th><a href="/img_get.php?catalog_id= <?php echo $_SESSION['catalog_id'];?>"></a></th>
31    <th><?php echo $_SESSION['name']; ?></th>
32    <th><?php echo $_SESSION['price']; ?></th>
33    <th><?php echo $_SESSION['description']; ?></th>
34    <th><img src="img_get.php?catalog_id=1" /></th>
35  </tr>
36
37  </table>
38</div>
39</body>
40</html>
41

mpyw

2017/06/13 05:33

本題ではありませんが，SQLインジェクション脆弱性があったりしていろいろやばいので，参考程度にリンク貼っておきます http://qiita.com/mpyw/items/b00b72c5c95aac573b71

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

回答3件

ベストアンサー

おそらくfile_exists($_SESSION['catalog_id']))がうまくいっていないんですよね。
そしてpathには画像パスが入っているものと推察されますが、catalog_idには何が入っているのでしょうか？
あと、pathが画像フルパスなのであればfile_exists($_SESSION['path']))としreadfile($_SESSION['path']);です。
またechoは不要です。readfileのみで良いと思います。→PHPで画像出力
あと、ファイルが存在しない場合も考慮してnoimage画像も準備した方がいいですね。

そういえば、これだけ見た感じだとセッションで画像情報を他の機能に持ちまわる等はなさそうなのでセッション利用しなくても良いと思いますよ。

サンプルコード

PHP
1//冒頭省略
2
3$image_path = "noimage.jpg"; //noimage画像
4$sql = "SELECT catalog_id , path FROM product_img WHERE catalog_id = " . mysqli_real_escape_string($dbLink,$_GET['catalog_id']);
5$result = mysqli_query($dbLink,$sql) or trigger_error($mysqli->error."[$sql]");
6if (!$result) {
7 $image_path = "noimage.jpg";
8}else{
9 $row = mysqli_fetch_assoc($result);
10 if (file_exists($row['path'])) {
11  $image_path = $row['path'];
12 }
13}
14
15header('Content-Type: images/jpeg');
16readfile($image_path);
17
18//以上

もし画像出力の確認をしたければ、ブラウザでimg_get.php?catalog_id=1を直接実行すると良いです。正常確認ができてからHTMLに埋め込んだ方がトラブルが少ないです。

追記

PHPで画像を表示させる場合、実行しているPHPからの相対パス、または絶対パスでなければなりません。
DBに保存しているパスがWebRootからのパスであれば調整する必要があります。ご注意ください。

例：
画像がhttp://hogehoge/img/1.jpgにありDBでは/img/1.jpgとありhttp://hogehoge/img_get.phpを実行する場合、

PHP
1$filepath = ".".$row["path"];
2//または
3$filepath = __DIR__.$row["path"]; //こちらの方が推奨

投稿2017/06/08 21:26

編集2017/06/13 02:57

m.ts10806

総合スコア80850

YukaSaku

2017/06/09 03:07

matsuさん、またヘルプしていただきありがとうございます！！ catalog_idにはauto_increment で入れた商品IDが入ってます。これはprimaryで、もう一つのDB、product_descriptionのcatalog_idと共通しており、このcatalog_idが共通したレコード情報を同じsofa.phpにあるテーブルの同じ行に表示したいと思っています。今は、写真が表示できないでいるので、まだそこまで至ってないです。 pathはそれぞれ、フルパスが入ってます。 readfile($_SESSION['path']);に変えてみたのですが、結果は同じでパスがきちんとechoされますが、sofa.phpで、写真が表示されません。

m.ts10806

2017/06/09 03:59 編集

> <a href="/img_get.php?catalog_id= <?php echo $_SESSION['catalog_id'];?>"></a> これは何がしたいのでしょうか。同じPHPを利用しているようですが。画像表示と文字出力は同時にはできません。回答をきちんと読んで理解してください。 echoは不要。$_SESSIONも不要です。 sofa.phpというのも質問のどこにもないのに今出てきているので何のことが分からないですし。後付が多すぎると正しい回答が得られず、何度もやりとりしなければならなくなるので出せる情報は正確に全て出すように心がけてください。

m.ts10806

2017/06/09 03:54

後半の処理をざっとサンプルとして書きました。勝手にsqlのエスケープも入れてますが、外部入力をGETで受け付けるので必須のため入れています。

m.ts10806

2017/06/09 04:01

回答の最後に書きましたが、画像出力の確認がしたければブラウザでimg_get.php?catalog_id=1を直接実行すると良いです。そこで正しい処理が書かれていれば画像だけが表示されるはず。余計にechoは不要ですね。

m.ts10806

2017/06/09 04:08

もしかしたらパスの問題もあるかもしれないので追記しました。

YukaSaku

2017/06/09 04:45

sofa.phpの<a href="/img_get.php?catalog_id= <?php echo $_SESSION['catalog_id'];?>"></a>は、直接catalog_id=1と書いて写真を表示させるのではなく、このように$_SESSION['catalog_id']を使えば、汎用性があるのかなと思って書いたのですが、img_get.phpでループも何もさせてないので、意味ないですね。指摘ありがとうございます。サンプルもありがとうございます。 pathは、 Ex6/img_get.php Ex6/Sofa.php Ex6/images/Square_Table_main.jpg （←写真のパス）で、DBのpathの１レコードにはimages/Square_Table_main.jpg と入っています。なので、img_get.phpをimg_get.php?catalog_id=1で実行すると、今まだ最後のラインのechoが入っているので、images/Square_Table_main.jpgが表示されます。パスの書き方の説明もありがとうございます。今、真夜中で、頭が回らなくなってきたので、こちらのパスの書き方を元に８時間後くらいにまた返信させていただきます。いつもありがとうございます。

m.ts10806

2017/06/09 04:55

真夜中？ああ、日本国外にお住まいですね。了解です。 img_get.phpでループさせるというのが何をしたいのか分かりませんが、そうなるとやはり ↓この部分に使われている$_SESSIONの意図が不明です。どこで取得されてどこで設定されているのか。 <th><a href="/img_get.php?catalog_id= <?php echo $_SESSION['catalog_id'];?>"></a></th> <th><?php echo $_SESSION['name']; ?></th> <th><?php echo $_SESSION['price']; ?></th> <th><?php echo $_SESSION['description']; ?></th> <th><img src="img_get.php?catalog_id=1" /></th> 当画面に表示させるだけのためなら要らないと思うのですが。。今回とは別問題ですね。セッションは便利ですが何でもかんでもセッション利用は良くないということで。ひとまず。

YukaSaku

2017/06/09 14:06

おはようございます。今、このサンプルコードを元に、img_get.php?catalog_id=1で実行したら、エラーメッセージ表示にブラウザで表示されず、新しくphpファイルが作られて勝手にダウンロードされてその中にエラーメッセージが表示されていました。以前もコードの中のif文、→　//if (file_exists($_SESSION['catalog_id'])) { これを削除すると、同じ現象でエラーメッセージは新しく作られたphpファイルに書き出されました。なぜでしょうか？またこの中身は以下のように書かれています。 <br /> <b>Warning</b>: mysqli_real_escape_string() expects exactly 2 parameters, 1 given in <b>/home/vol14_5/ihostfull.com/uoolo_19138640/htdocs/PHP/Ex6/img_get.php</b> on line <b>15</b><br /> <iframe src='http://ifastnet.com/pop.html' name='frame1' scrolling='no' frameborder='no' align='center' MARGINWIDTH='0' MARGINHEIGHT='0' height = '10px' width = '10px'> </iframe><br /> <b>Notice</b>: Undefined variable: mysqli in <b>/home/vol14_5/ihostfull.com/uoolo_19138640/htdocs/PHP/Ex6/img_get.php</b> on line <b>16</b><br /> <iframe src='http://ifastnet.com/pop.html' name='frame1' scrolling='no' frameborder='no' align='center' MARGINWIDTH='0' MARGINHEIGHT='0' height = '10px' width = '10px'> </iframe><br /> <b>Notice</b>: Trying to get property of non-object in <b>/home/vol14_5/ihostfull.com/uoolo_19138640/htdocs/PHP/Ex6/img_get.php</b> on line <b>16</b><br /> <iframe src='http://ifastnet.com/pop.html' name='frame1' scrolling='no' frameborder='no' align='center' MARGINWIDTH='0' MARGINHEIGHT='0' height = '10px' width = '10px'> </iframe><br /> <b>Notice</b>: [SELECT catalog_id , path FROM product_img WHERE catalog_id = ] in <b>/home/vol14_5/ihostfull.com/uoolo_19138640/htdocs/PHP/Ex6/img_get.php</b> on line <b>16</b><br /> <iframe src='http://ifastnet.com/pop.html' name='frame1' scrolling='no' frameborder='no' align='center' MARGINWIDTH='0' MARGINHEIGHT='0' height = '10px' width = '10px'> </iframe><br /> <b>Warning</b>: readfile(noimage.jpg): failed to open stream: No such file or directory in <b>/home/vol14_5/ihostfull.com/uoolo_19138640/htdocs/PHP/Ex6/img_get.php</b> on line <b>34</b><br /> <iframe src='http://ifastnet.com/pop.html' name='frame1' scrolling='no' frameborder='no' align='center' MARGINWIDTH='0' MARGINHEIGHT='0' height = '10px' width = '10px'> </iframe>

YukaSaku

2017/06/13 02:45

おっしゃる通り、pathに問題がありました。 imageそれぞれのpathになぜか　"\"　が入ってました。消したところ、きちんと表示されました。ありがとうございました。

m.ts10806

2017/06/13 02:53

登録処理と画像の配置処理等を見ていないので確実なところは言えないですが、windows環境であればパス区切りは\だからですね。

m.ts10806

2017/06/13 02:58 編集

エラーの件ですが、サンプルコードに誤りがありましたので修正します。お返事遅れてすみません。 mysqli_real_escape_string($_GET['catalog_id']) ↓ mysqli_real_escape_string($dbLink,$_GET['catalog_id'])

YukaSaku

2017/06/13 03:28

いつも詳しい説明ありがとうございます。自分なりに噛み砕いて、自分用メモに保存してます。

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

まずはimg_get.php?catalog_id=xxxでアクセスしてエラーを確認することです。
$sql = "SELECT catalog_id ,・・・
については画像の表示は1つしかできないので保険でLIMIT 1をつけておいてください

readfile($_SESSION['catalog_id']);
についてはidじゃなくてpathを読むのでは？

header('Content-Type: images/jpg');
は、jpegの方がよいかもしれません

あとせっかくDBで管理するのですから、product_imgテーブルには
mimeや縦、横サイズ、altテキストなどを保持して、画像を取り出す際に
imgタグに渡してやると拡張性が高くなると思います

投稿2017/06/09 00:21

yambejp

総合スコア114829

YukaSaku

2017/06/09 03:35

img_get.php?catalog_id=xxxでエラーを修正し、上記コード現在はエラーはありません。 readfile($_SESSION['catalog_id']);は、pathに直して実行してみたのですが、写真は表示されませんでした。 LIMITつけてみました。ありがとうございます。

yambejp

2017/06/09 03:38 編集

> 上記コード現在はエラーはありませんそれでも画像が表示されないでしたら、なにが表示されるのでしょうか？場合によってはimg_get.phpにBOMがついていたらBOMなしで保存しなおしてください

YukaSaku

2017/06/09 03:57

img_get.php?catalog_id=xxxでは、エラーがなければ何も表示されないので、一番最後の行の echo $_SESSION['path'] で、入ってるパスを表示させています。すみません、BOMがよくわかりません。

yambejp

2017/06/09 04:04

では、パスが間違っているんですよ img_get.phpからの相対位置なのかdocument_rootからの絶対位置なのかはっきりさせて、きちんと参照できる設定をしてください BOMはファイルの基本なので調べて覚えて下さい UTF8で保存するときにはかならず設定が必要なないようです

YukaSaku

2017/06/09 04:22

ありがとうございます。調べてみます。

YukaSaku

2017/06/13 02:46

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

画像のパスはpathとcatalog_idのどちらに入っているのでしょうか？
名前からしてpathと思われるので、下記のようになるのではと思います。
また、他の回答にもありますが、$_SESSIONに入れる必要はないと思われます。

PHP
1//$_SESSION['path'] = $row['path'];     ←これは不要
2//$_SESSION['catalog_id'] = $row['catalog_id'];     ←これは不要
3
4if (file_exists($row['path'])) {
5    header('Content-Type: images/jpg');
6    readfile($row['path']);
7}
8
9// バイナリデータを直接表示
10//echo $_SESSION['path']     ←これは不要