掲示板の個々のスレッドとレスポンスを紐づけたい

Question

実現したい事

掲示板のスレッドごとのレスポンスを表示できるようにしたいです。
例えば「スレッドA」には「スレッドA」に書き込まれたレスポンスを、「スレッドB」には「スレッドB」に書き込まれたレスポンスを表示するようにしたいです。

困っている事

個々のスレッドごとにレスポンスを表示したいのですが、どのスレッドでも同じレスポンスが表示されます。
スレッドごとにレスポンスを表示する方法が分からないのですが、どのようにすれば実装できるでしょうか？

DBに登録されている情報

＜スレッドのテーブル＞
テーブル名：thread_data
・id（連番）
・thread_id(スレッド作成者を区別するためのID)
・thread_title（スレッドのタイトル）
・thread_author（スレッドを作った人）
・thread_contents（スレッド詳細）
・created_at（スレッド作成日時）

＜レスポンスのテーブル＞
テーブル名：res_data
・id（連番）
・res_id（投稿者を区別するためのID）
・res_author（投稿者の名前）
・res_contents（書き込む内容）
・created_at（投稿日時）

<!-----------index.phpのファイル（スレッド一覧を表示する） ------------->
<?php
require_once('config.php');
require_once('common.php');

$pdo = new PDO(DSN, DB_USER, DB_PASS);
$thread_list = $pdo->prepare("SELECT * FROM thread_data ORDER BY id DESC");
$thread_list->execute();

?>
<!------　中略 --------->

	<section class="test_wrapper">
		<h1>掲示板</h1>
		<div class="btn_block">
			<a class="btn" href="./thread_add.php">スレッドを作成する</a>
		</div><!--/.btn_block -->
		<h2>スレッド一覧</h2>
		<div class="thread_hd_block">
			<p>スレッド名</p>
			<p>投稿日時</p>
			<p>スレッド作成者</p>
		</div><!--/.thread_hd_block -->
		<?php foreach($thread_list as $tl) : ?>
		<div class="thread_list_block">
			<?php $id = $tl['id'] ?>
			<a href="thread.php?id=<?php echo $id; ?>">
				<h3><?php echo h($tl['thread_title']) ?></h3>
				<p><?php echo h($tl['created_at']) ?></p>
				<p>作成者：<?php echo h($tl['thread_author']) ?></p>
			</a>
		</div><!--/.thread_list_block -->
	<?php endforeach; ?>
	</section>

<!--------- thread.phpのファイル（スレッドと書き込みを表示する） ------>
<?php
require_once('config.php');
require_once('common.php');
$id = $_GET['id'];

$pdo = new PDO(DSN, DB_USER, DB_PASS);
$result = $pdo->prepare("SELECT * FROM thread_data WHERE id = :id");
$result->bindParam(":id", $id);
$result->execute();

$res_data = $pdo->prepare("SELECT * FROM res_data ORDER BY id DESC");
$res_data->execute();
?>

<!------　中略 --------->

<section class="thread_wrapper">
		<div class="thread_btn_block">
			<a class="btn" href="index.php">スレッド一覧に戻る</a>
		</div><!--/.thread_btn_block -->
		<div class="thread_first">
			<?php foreach($result as $r) : ?>
				<h1><?php echo h($r['thread_title']) ?></h1>
				<div class="thread_info_block">
					<p>スレッド作成者：<?php echo h($r['thread_author']) ?></p>
					<p><span>スレッド作成者ID：</span><?php echo h($r['thread_id']) ?></p>
				</div><!--/.thread_info_block -->
				<div class="contents_block">
					<?php echo h($r['thread_contents']) ?>
				</div><!--/.contents_block -->
		<?php endforeach; ?>
		</div><!--/.thread_first -->
		<div class="res_wrapper">
			<div class="res_make_block">
				<h2>コメントを投稿する</h2>
				<form method="post" action="res_add_done.php">
					<p>名前</p>
					<input type="text" name="res_author">
					<p>投稿内容</p>
					<textarea name="res_contents"></textarea>
					<input type="submit" value="投稿する">
					<?php
					//ipアドレスを元にidを生成。ipアドレスと日付が一致するとユニークなIDが生成される
					$ip = $_SERVER["REMOTE_ADDR"];
					$timestamp = date("Y/m/d");
					$secret = "fqeifxhfnfianqwafipoz";
					$id_hash = hash_hmac("sha1" , $timestamp.$ip, $secret);
					$id_base64 = base64_encode($id_hash);
					$res_id = substr($id_base64, 0, 8);
					?>
					<input type="hidden" name="res_id" value="<?php echo $res_id; ?>">
					<!--スレッドのidを保存する-->
					<input type="hidden" name="id" value="<?php echo $id ?>">
				</form>
			</div><!--/.res_make_block -->
			<div class="res_area">
				<?php
				//初期のレスナンバー
				$res_number = 1;
				?>
				<?php foreach($res_data as $res) : ?>
				<div class="res_block">
					<div class="res_author_info_block">
						<p><?php echo $res_number; ?>：</p>
						<p style="padding-right: 20px;"><?php echo h($res['res_author']) ?></p>
						<p style="padding-right: 20px;">ID：<?php echo h($res['res_id']) ?></p>
						<p>投稿日時：<?php echo h($res['created_at']) ?></p>
					</div><!--/.res_author_info_block -->
					<div class="res_write"><?php echo h($res['res_contents']) ?></div>
				</div><!--/.res_block -->
					<?php
					$res_number++;
				endforeach;
				?>
			</div><!--/.res_area -->
		</div><!--/.res_wrapper -->
	</section><!--/.thread_wrapper -->

コード

ご教示いただけますと幸いです。
ご回答お願い致します。

Answer 1

レスポンスからどのスレッドを指すのかを示すように
レスポンステーブルにスレッドIDを持たせないと実現できないかと。

---

テーブル名：thread_data
・id（連番）
・thread_id(スレッド作成者を区別するためのID)

テーブル名：res_data
・id（連番）
・res_id（投稿者を区別するためのID）

thread_data.id と thread_data.thread_id の違い、
res_data.id と res_data.res_id の違いをはっきりさせといたほうがいいよ。
これ、混乱のもとになるから。

idってカラムを設けることで
自テーブル内のユニークさを担保するけど、
それと違う自テーブル内の別idって
なにか計算式で算出するのかユーザー記入なのかわからないけど、
中身の違うものに同じようにidってつけると混乱するよ。

autherのテーブルを別途設けるなら、
res_autherを持たずにres_data.auther_idなどとしたら
確実に投稿者を区別できると思うけど。
じゃないと、同じ投稿者名を使いまわしされれば、
なりすましとか起こりうる。

Comments

[Keichi_Negishi]

ご回答ありがとうございます。やってみます。

[Keichi_Negishi]

アドバイスありがとうございます。

＞中身の違うものに同じようにidってつけると混乱する
→分かり辛いなと思っていましたが、やはりマズイのですね。

投稿者のIDに関しては投稿者のIPアドレスを元にIDを作成しているのですが、
これは問題でしょうか。

[退会済みユーザー]

ユーザー側に見せる必要があるのであれば、あえて冗長にidentifierとするとか。
見せる必要がなければ、そもそもシステム内でidで識別できているので省略するのも一考。

[Keichi_Negishi]

お返事遅くなりましたができました！
ありがとうございます。

Answer 2

「所属」を特定する情報を保持していないからです。
レスポンステーブルに「親スレッドid」をもたせ、レスポンステーブル検索時に対象のスレッドidで絞り込みます。

「レスに対するレス」があるなら更にレスポンステーブルに「親レスid」をもたせる事でできます。nullだったら最も親で、再帰的に子レスを取得します。

Comments

[Keichi_Negishi]

ご回答ありがとうございます。やってみます。

[Keichi_Negishi]

お返事遅くなりましたができました！
ありがとうございます。