PHPでMySQLに保存されている画像データが表示されない

Question

ラージオブジェクト (LOB) を参考に、
MySQLでmediumblob型の画像データが保存されているのですが、
うまく取得し表示することができません。

ログにはfpassthru()の第一引数にresouceではなくてstringが指定されていますと言われているので、
型変換みたいなのが必要なのでしょうか？

参考元のコメント欄を見ても同じエラーになった人がいるようです。

MySQL Workbench から登録したバイナリデータをダウンロードすると
正しく画像が表示されているので、登録時の処理には問題ないと思っています。

どこに問題があるのか指摘していただけませんでしょうか。

###ログ

Warning: fpassthru() expects parameter 1 to be resource, string given in /app/php/show_image.php

###ソース

show_image.php

  $pdo = new PDO('mysql:*****');
  $stmt = $pdo->query("SELECT type, imgdat FROM test_table LIMIT 1");
  $stmt->bindColumn(1, $typ, PDO::PARAM_STR, 256);
  $stmt->bindColumn(2, $dat, PDO::PARAM_LOB);
  $stmt->fetch(PDO::FETCH_BOUND);

  header("Content-Type: $typ");
  fpassthru($dat);

regist_image.php

  $db = new PDO('mysql:*****');
  switch($_SERVER['REQUEST_METHOD']){
    case 'POST' :
      if($_FILES['upImage']){
        $fp = fopen($_FILES["upImage"]["tmp_name"], "rb");

        $extension = pathinfo($_FILES["upImage"]["name"], PATHINFO_EXTENSION);

        $stt1 = $db->prepare("INSERT INTO test_table ( imgdat, extension, error, type, size) select ?, ?, ?, ?, ?");
        $stt1->bindValue(1, $fp, PDO::PARAM_LOB);
        $stt1->bindValue(2, $extension);
        $stt1->bindValue(3, $_FILES["upImage"]["error"] );
        $stt1->bindValue(4, $_FILES["upImage"]["type"] );
        $stt1->bindValue(5, $_FILES["upImage"]["size"] );

        $db->beginTransaction();
        $stt1->execute();
        $db->commit();
        fclose($fp);
      }
      break;

    default :
      break;
  $db = null;
  }
} catch (PDOException $e){
	die($e->getMessage());
}

TABLE

CREATE TABLE test_table (
  imgdat mediumblob ,
  extension varchar(5),
  error int ,
  type varchar(50),
  size varchar(50)
);

Answer 1

ベストアンサー後ですが、無事に画像を表示することができました。

文字コードを「UTF-8(BOM)」→「UTF-8」に変更したら出力することができました。
ブラウザの文字コードも「UTF-8」です。
コードにもherokuにも問題はありませんでした。

ベストアンサーはすごく迷いましたが、crhgさんにさせていただきます。
hirohiroさんも何回も回答してくださったり、サイトを検索していただいたりしてありがとうございました。m(__)m

---

自分用にまとめると、画像をherokuでmysqlに保存→出力するまでに気を付けることとして、

• .user.ini に upload_max_filesize の値をファイルサイズ以上に設定したファイルをアップロードすること
• max_allowed_packet もファイルサイズ以上に設定すること
• DBのカラムの型をmediumblobにすること
• mysql で使う場合は PDO::PARAM_LOB という型を使うこと
• PDO::PARAM_LOB はストリームで返されないこと
• 文字コードを統一すること

かなぁと。。

show_image.php

<?php
  $pdo = new PDO('mysql:*****);

  $stmt = $pdo->query("SELECT extension, imgdat FROM test_table LIMIT 1");
  $stmt->bindColumn(1, $ext, PDO::PARAM_STR, 256);
  $stmt->bindColumn(2, $dat, PDO::PARAM_LOB);
  
  $result = $stmt->fetch(PDO::FETCH_BOUND);

  $contents_type = array(
    'jpg'  => 'image/jpeg',
    'jpeg' => 'image/jpeg',
    'png'  => 'image/png',
    'gif'  => 'image/gif',
    'bmp'  => 'image/bmp',
  );

  $fp = fopen('php://memory', 'r+');
  
  $result = fwrite($fp, $dat,strlen($dat));
  
  if($result == strlen($dat)){
    header("Content-Type: $contents_type[$ext]");
    rewind($fp);
    fpassthru($fp);
  }else{
    echo("miss");
  }
  fclose($fp);
  
  $pdo = null;
?>

Answer 2

本来はbindColumnで指定した変数にstreamが返ってくる仕様のはずなのですがデータそのものが返却されてしまうというPDO_MYSQLのバグで、2007年に報告されているのですがいまだに直されていません。

Bug #40913 PDO_MYSQL: PDO::PARAM_LOB does not bind to a stream for fetching a BLOB

2012-11-07 12:1の[e-mail]さんのコメントによれば簡単に直せるものではなさそうですから、当面は今の動作に合わせて記述するしかないと思われます。

Comments

[退会済みユーザー]

回答ありがとうございます。バグなんですね。。
ということは画像データを取得して表示したいときは、PDOは使わず、mysqliを使用した方がいいということでしょうか？
それとも画像はmysqlに保存しない方がいいということでしょうか？

[crhg]

mysqliは使ったことがないので詳しくないのですがざっと見たところstreamで受け取るような機能は見つからないので、それだったらPDOで通常のやり方で受け取るのと一緒ですから、あえて使うメリットはないような気がします。

streamで受け取ってfpassthruで送るやり方が使えない場合、一度画像データを変数に格納することになるのでメモリ使用量は増えます。画像をDBに保存するデメリットはDBが肥大化するのと処理能力的に不利というのが一般的ですが、メモリ使用量が増えることで処理能力面のデメリットがさらに大きくなったと言えるでしょう。DBに置くことは扱いが楽であるとか整合性をとりやすいメリットもあるのでどちらを重視するかになってくると思います。

[退会済みユーザー]

herokuというSaaSを使っているのですが、そこは画像を保存できないみたいなのでDBに保存するしかなさそうですね。。

そして、↓のソースのようにメモリーをfopenして文字列を書き込んでみたのですが、うまくいきませんでした。

もしかしたらherokuに問題があるかもしれません。
DBの容量が5MBしかないなど(画像は4MBと少し)...。

画像は別のところに保存した方がいいのかもと考えています。

show_image.php
```php
$pdo = new PDO('mysql:*****');
$stmt = $pdo->query("SELECT type, imgdat FROM test_table LIMIT 1");
$stmt->bindColumn(1, $typ, PDO::PARAM_STR, 256);
$stmt->bindColumn(2, $dat, PDO::PARAM_LOB);
$stmt->fetch(PDO::FETCH_BOUND);

$fp = fopen('php://memory', 'r+');
$result = fwrite($fp, $dat);

if($result == strlen($dat)){
header("Content-Type: $typ");
rewind($fp);
fpassthru($fp);
}
fclose($fp);
$pdo = null;
```

Answer 3

コメントを間違えて回答にしてしまったので削除

Answer 4

ひょっとしてfpassthru();にバイナリを直接流し込んだからではないでしょうか？
phpマニュアルのページ

fpassthru — ファイルポインタ上に残っているすべてのデータを出力する

$datはファイルポインタではなくて、恐らくデータそのもの？

後、エラーには直接関係ないかもしれませんが、Content-Typeの指定は、「"Content-Type: $typ"」では$typが文字のまま出てしまいそうです。

Comments

[hirohiro]

当初書式のミスでデータにアクセスできていないのでは無いかと思いそのように指摘する内容を書きましたが、失礼しました。質問欄の書式で問題ないみたいでした。（公式のマニュアルに同じような書式がありました）
後はこれもあまり自信が無いのですが、fpassthru();にバイナリを直接流し込んだからではないでしょうか？
>>fpassthru — ファイルポインタ上に残っているすべてのデータを出力する
http://php.net/manual/ja/function.fpassthru.php
$datはファイルポインタではなくて、データそのものですよね？多分

というわけで、こちらの内容で書き直しました。

[退会済みユーザー]

回答ありがとうございます。
$datはバイナリを取得しているのでfpassthruにデータそのものを流し込んでいます。
マニュアルのページにはラージオブジェクトという型を指定をしてあげれば、勝手にポインタになってくれているとあったので、サンプルみたいにfpassthruを使えるのではと思っていました。
が、crhgさんが回答してくださっているようにバグみたいです。

echo($dat)とやってみてもうまくいかなかったのですが、データそのものをそのまま表示する方法はないのでしょうか？

[hirohiro]

>>echo($dat)とやってみてもうまくいかなかった
なるほど、このような書式でだめだったということなのですね？
header("Content-Type: $typ");
echo $dat;
これはどのようにうまくいかなかったのでしょう？バイナリデータが文字で書き出されてしまったのでしょうか？
もしかするとstrlen()か何かで大きさを確認してみると$datは空かも知れませんね。
（crhgさんの書き込みからはバイナリデータは受け渡しされるようですが）

もしbindColumnに拘りが無ければ使わない方法を試してみてはどうでしょう？
$stmt = $pdo->query("SELECT type, imgdat FROM test_table LIMIT 1");
$img = $stmt->fetch();
header("Content-Type: ".$img->type);
echo $img->imgdat;
mysqlからの画像出力で多く紹介されているのはこちらのパターンかと
参考：http://www.kantenna.com/pg/2010/04/phpmysql.php

[退会済みユーザー]

このサイトを参考にしたりフェッチの仕方を変えたりもしてみましたがなかなかうまくいきませんでした。
strlen()も調べてみましたが画像サイズ数ありましたね。。
なかなか詰んでます(笑)
今は文字列をストリームで読み込む方法を探してます

サイトなど調べてくださってありがとうございます。