php クリックした画像ごとに表示するページを変える

Question

やりたいこと

dbにあるパスから画像を一覧表示して画像をクリックした時にクリックした画像のページに表示したいです。

現状

どの画像をクリックしても最後のidの画像ページが表示されてしまいます。

コード

db接続は省いてますが接続自体は問題ありません。
画像はディレクト内、db内には画像パス、idなどがあります。

//画像一覧ページ
            session_start();
            $userId = $_SESSION["userId"];

            $stmt = $mysqli->prepare("SELECT * FROM images where userId = ?");
            if($stmt->execute(array($userId))){
              while($row = $stmt->fetch()){

                $url = "../../contents/illust/" . $row["url"];
                $id = $row["id"];
                $_SESSION["id"] = $id;
                echo '<a href="../post/php/loadillust.php"><img src="'.$url.'" alt="'.$row['title'].'" /></a>';
              }
            }

//画像表示ページ
  session_start();

  $id = $_SESSION["id"];

  $sql = "SELECT * FROM images WHERE id='$id'";

  $stmt = $mysqli->prepare($sql);
  $stmt->execute();
  $row = $stmt->fetch(PDO::FETCH_ASSOC);
  $stmt->execute();

  while($row = $stmt->fetch(PDO::FETCH_ASSOC)){
    $url = $row['url'];
    $type = $row["type"];
  }
  $stmt = null;

  $img = file_get_contents('../../../contents/illust/' . $url);

  if($type == "jpeg"){
    header("Content-type: image/jpeg");
    echo $img;
    exit;
  }elseif($type == "png"){
    header("Content-type: image/png");
    echo $img;
    exit;
  }else{
    echo "システムエラー";
  }

Comments

[m.ts10806]

画像表示ページ　で　 $_SESSION["id"]　は何が渡ったのか確認しましたか？

[m.ts10806]

あと、画像一覧ページではprepareでパラメータセットしているのに画像表示ページでは直に値を突っ込んでいます。この実装のぶれはなぜでしょう。

[comgi]

idをecho表示させたところ最後の画像のidのみ出力されました。

すみません、制作日の違いです。ページによってバラバラなんですが、正直問題なく処理してくれれば大丈夫と思いそのままにしていましたが、統一した方がいいでしょうか？

[m.ts10806]

>idをecho表示させたところ最後の画像のidのみ出力されました。

これは「最後の画像をクリックしたとき」ですか？それとも「どの画像をクリックしても」ですか？
まぁおおよそ分かるんですが・・

実装は統一すべきです。実装のブレはそのままバグに繋がります。

Answer 1

ループでずっと$_SESSION["id"]が上書きされ続けているので最後のIDのみが残っている状態です。
リンクでGETパラメータを付与するなどしてください。

<a href="../post/php/loadillust.php?id='.htmlspecialchars($row["id"], ENT_QUOTES, 'UTF-8').'">

$id = filter_input(INPUT_GET,"id");

Comments

[comgi]

$stmt = $mysqli->prepare("SELECT * FROM images where userId = ?");
if($stmt->execute(array($userId))){
while($row = $stmt->fetch()){

$url = "../../contents/illust/" . $row["url"];
echo '<a href="../post/php/loadillust.php"><img src="'.$url.'" alt="'.$row['title'].'" /></a>';
echo '<a href="../post/php/loadillust.php?id='.htmlspecialchars($row["id"], ENT_QUOTES, 'UTF-8').'">';
}
}
/////
$id = filter_input(INPUT_GET,"id");

$stmt = $mysqli->prepare("SELECT * FROM images WHERE id = ?");
if($stmt->execute(array($id))){
while($row = $stmt->fetch()){
$url = $row['url'];
$type = $row["type"];

$img = file_get_contents('../../../contents/illust/' . $url);
}
}

この記述で合ってますでしょうか？表示ページに何も表示されず真っ白なままです。idのみの表示も同じ状態です。
もしよろしければ、原因を教えて頂けませんでしょうか。

[m.ts10806]

ロジックを1つずつ確認してください。
executeの結果、file_get_contentsの結果。
最後の
if($type == "jpeg"){　　までそのままなら何かしら出るはずですが、
エラー表示はONにして確認しましたか？
https://qiita.com/shotets/items/3c95aef631b2c5eadae5

[comgi]

すみません
コードをいろいろといじってる内に成功しました。
とても助かりました、ありがとうございます！

[m.ts10806]

最終的にどうなったか残してください。
書いた通りにしか動かないので「なぜか動いた」という状態は次のバグを生むことになります。

[comgi]

$url = "../../contents/illust/" . $row["url"];
echo '<a href="../post/php/loadillust.php?id='.htmlspecialchars($row["id"], ENT_QUOTES, 'UTF-8').'">';
echo '<img src="'.$url.'" alt="'.$row['title'].'" />';

imgの出力の場所と方法を変更しました。
大丈夫でしょうか？

[m.ts10806]

大丈夫かどうかはこちらは知らないです。
なぜなら要件次第だから。仕様は作る人が決めるものだから。

[comgi]

そうですよね、、、
何から何まですみませんありがとうございます。

[m.ts10806]

いえ、こちらは単に「やりたいこと」に対して「やったこと」「起きている現象（できてないこと）」を見て、「どうしたら起きている現象がやりたいことに近づけるか」をアドバイスしているだけなので。
内容が具体的であればあるほど答えやすく、抽象的であればあるほど答えにくい、それだけです。