テーブルの内容を<a>タグでくるんだ時のアンカー

質問

下記のPHPスクリプトにおいて、テーブルの内容を<a＞タグで
挟み、データベースの内容を表示しているのですが、
先頭レコードのアンカーだけが表示されず、ただのテキストになってしまいます。
二番目以降のレコードはテーブルの内容がアンカー付きで表示されるのですが、
先頭レコードにアンカーをつけるには、どこがまちがっているのか
ご指導お願いいたします。

mysql_set_charset("utf-8");
//$sql = "SELECT * FROM hospytal;";
//$result = mysql_query($sql, $con) or die(mysql_error());

if($_POST['radio_btn']==="1"){
$sql = "SELECT * FROM hospytal order by manzokudo desc;"; // 満足度の降順
$result = mysql_query($sql, $con) or die(mysql_error());
}else{
$sql = "SELECT * FROM hospytal order by kuchikomi desc;"; // 口コミ数の降順
$result = mysql_query($sql, $con) or die(mysql_error());
}

$rowco = mysql_num_rows($result); // テーブルの行数
for($i=0; $i<$rowco; $i++){
$row = mysql_fetch_row($result); // 取得された行に対応する配列を返し、内部のデータポインタを前に進める
echo '<a href="'.$i.'.php?sid='.$row[0].'">';
echo '<table width="500" height="120" border="1">';
echo '<tr>';
echo '<td width="500" height="30">'.$row[1].'</td>';
echo '</tr><tr>';
echo '<td width="500" height="30">'.$row[4].'&nbsp;&nbsp;&nbsp;'.$row[5].'件</td>';
echo '</tr><tr>';
echo '<td width="500" height="30">'.$row[7].'</td>';
echo '</tr><tr>';
echo '<td width="500" height="30">'.$row[9].'</td>';
echo '</tr>';
echo '</table>';
echo '</a>';
}
mysql_close($con); // データベースとの接続を解除する

回答No.1

agunuz · ベストアンサー

まず

>先頭レコードのアンカーだけが表示されず、ただのテキストになってしまいます。

どういうhtmlソースが返っているかを確認してください。anchor要素とtable要素をまとめて出力しているので抜け落ちる可能性はありません。最初の <a の前に閉じていない二重引用符があったりしてhtmlソースとして解釈出来ない状態になっていたりはしませんか？

以下はソースを眺めていて気になった点を書きます。

まずネイティブなMySQL関数はすでに非推奨です。今から書くならPDOをお勧めします。

http://php.net/manual/ja/mysqlinfo.api.choosing.php

>mysql_set_charset("utf-8");

MySQLには、utf-8という文字セットはありません。utf8です。

>$rowco = mysql_num_rows($result);

行数を得るのはなんのためでしょうか？見たところforで回している中でもレコード番号を使っているのは

>echo '<a href="'.$i.'.php?sid='.$row[0].'">';

の部分だけですが、0.phpや1.phpは存在するのですか？普通は同じphpスクリプト名（たとえばview.phpとか）でQueryStringで引き渡すIDだけを変えると思いますが・・・。

もし表示上で「行番号」が必要なら

$cnt = 0;
while ($row = mysql_fetch_row($result)) {
$cnt++;
// $rowを使った処理
}

で済みます。行数を得てからforで回す意図がわかりません。

なお、そもそもanchorはインライン要素なのでtable（ブロック要素）を内包するのはマズイです。各々のtd要素内の文字列に対してanchor要素を書いてください。

（PDOの例）

<?php
// table要素の書式
$table = <<<EOD
<table width="500" height="120" border="1">
<tr>
<td width="500" height="30"><a href="view.php?sid=%s">%s</a></td>';
</tr>
<tr>
<td width="500" height="30"><a href="view.php?sid=%s">$s&nbsp;&nbsp;&nbsp;%d件</a></td>';
</tr>
<tr>
<td width="500" height="30"><a href="view.php?sid=%s">%s</a></td>';
</tr>
<tr>
<td width="500" height="30"><a href="view.php?sid=%s">%s</a></td>';
</tr>
</table>
EOD;

try {
$pdo = new PDO("mysql:host=localhost;dbname=dbname;charset=utf8", 'user', 'pass');
$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

if($_POST['radio_btn']==="1"){
$stmt = mysql_query('SELECT * FROM hospytal order by manzokudo desc');
} else {
$stmt = mysql_query('SELECT * FROM hospytal order by kuchikomi desc');
}

while ($row = $stmt->fetch(PDO::FETCH_OBJ)) {
printf($table,
$row->id, $row->name,
$row->id, $row->rank, $row->cnt,
$row->id, $row->address,
$row->id, $row->address,
$row->id, $row->open);
}
} catch (PDOException $e) { // tryブロックの終了（例外を捕捉）
// 例外処理（適切に書き直してください）
die('database error<br />' . $e->getMessage());
}

お礼

長文のご説明、本当にありがとうございました。
感謝しております。