別フレームにjsで変数を送信することができません。

表題の件ですが、データベースの中にテーブルリストがtable1~10まであるのですが、どのテーブル名でリンクを飛ばしてもtable2.phpにはtable10としか表記されません。
どなたかご教授宜しくお願い致します。

//frame.php//
<html>
<head>
<title></title>
</head>

<frameset cols="200,*">

<frame src="table.php" name="left" title="左フレーム">
<frame src="table2.php" name="right" title="右フレーム">

</html>

//table.php//
<html>
<head>
</head>

<body>

<?php
error_reporting(E_ALL ^ E_NOTICE ^ E_DEPRECATED);

// データベースサーバへの接続
$db = mysql_connect('xxx','xxx','xxx');

// テーブルの一覧表示の関数呼び出し（ユーザ定義関数）
show_tables('xxx',$db);

// データベースサーバの切断
mysql_close($db);

// ----------------------------------------------
// テーブルの一覧表示の関数の定義
function show_tables($db_name,$db)
{
// 指定されたデータベース内のテーブルリストの取得
$rs = mysql_list_tables($db_name,$db);

// 結果セット内のレコード数の取得
$num_rows = mysql_num_rows($rs);

print "<table>\n";
print "<tr>\n";
print "<td align=center>Tables in {$db_name}</td>\n";
print "</tr>\n";

// テーブルがある場合
if($num_rows > 0)
{
// 結果セット内のレコードを順次参照
for($i = 0; $i < $num_rows; $i++)
{
// テーブル名の取得
$table_name = mysql_table_name($rs,$i);
?>

<script type="text/javascript">
<!--
function jump(){
	parent.right.location.href = "table2.php?table_name=<?php print $table_name;?>";
}
// -->
</script>

<?php
// テーブル名の表示
print "<tr>\n";
print '<td><a href="#" onClick="jump(); return false;">';
print $table_name;
print '</a></td>';

print "</tr>\n";
}
}

// テーブルが無い場合
else
{
print "<tr>\n";
print "<td>テーブルはありません</td>\n";
print "</tr>\n";
}
print "</table>\n";

// 結果セットの解放
mysql_free_result($rs);
}
?>
</div>

</body>
</html>

//table2.php//
<?php 
$table_name=$_REQUEST['table_name'];
print $table_name;
?>
```

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

回答2件

jump()をループ内で関数宣言しているから10個のjump()が定義されてますね。
そして、それぞれが同じ名前であるためループ内で上書きしてます。
最終的に10個目のjump()のみが存在する形です。
なので、どこで呼び出しても10個目のjump()が実行されます。

対策としてはjump()をループの外で定義して、渡す引数でどこへ移動させるかを決めるような方法にするのが良いと思います。

php
1<script>
2function jump(table_name){
3//処理
4}
5</script>
6
7<?php
8//テーブルを出力するループ処理
9?>

投稿2017/06/08 15:54

oskbt

総合スコア1895

alkeran

2017/06/09 13:40

ループの外で定義すると以下の部分が「ページのソースを表示」で確認すると何も表示されなくなるのですが...。 <?php print $table_name; ?>

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

ベストアンサー

下記のように2回同じ名前の関数を定義すると後者に上書きされます。

HTML
1<script type="text/javascript">
2<!--
3function jump(){
4    parent.right.location.href = "table2.php?table_name=1";
5}
6// -->
7</script>
8<script type="text/javascript">
9<!--
10function jump(){
11    parent.right.location.href = "table2.php?table_name=2";
12}
13// -->
14</script>

また、table要素の直下にscript要素を入れるのは文法違反です。

【<table> - HTML | MDN】
https://developer.mozilla.org/ja/docs/Web/HTML/Element/table

JavaScriptの関数jumpを引数を取るようにしてみてはいかがでしょうか。

HTML
1<script type="text/javascript">
2<!--
3function jump( num ) {
4    parent.right.location.href = "table2.php?table_name=" + num;
5}
6// -->
7</script>
8<table>
9<!--  -->
10</table>

PHP
1print "<tr>\n";
2print '<td><a href="#" onClick="jump(';
3print $table_name;
4print '); return false;">';
5print $table_name;
6print '</a></td>';
7print "</tr>\n";