Q&A
button要素単体では単なるボタンでしかないので、jsと組み合わせるかinput type="submit"でいいかも。
（ミニマルに説明したかった意図はわかるんですが、真似して動かないって言われそうな気がして。）
前提・実現したいこと
phpとMysqlを利用して簡易的な掲示板を作っています。
もともと二つに分けていた メッセージを表示するphp と メッセージを送信するphp を一つにしようとしていますがうまくいきません
二つのボタンの処理を別々にしようとしているんですがどちらのボタンを押しても同じ動作が行われてしまいます。
$a1_d=$_POST["a1"];
$a2_d=$_POST["a2"];
この部分が定義されてないみたいなんですが解決の仕方がわかりません。
どのようにすれば実現できるか教えてください。
よろしくお願いします。
発生している問題・エラーメッセージ
Notice: Undefined index: a1 in C:\xampp\htdocs\keiziban.php on line 5 Notice: Undefined index: a2 in C:\xampp\htdocs\keiziban.php on line 6
該当のソースコード
html
1<head> 2<meta charset="UTF-8"> 3</head> 4<form method="post" action="keiziban.php"> 5メッセージを表示します<BR> 6 <input type="hidden" name="form_id"> 7 <input type="submit" value="メッセージ標示"name="sel"> 8</form> 9 10<form action="keiziban.php" method="post"> 11 名前を入力して <input type="text" name="a1"> <br> 12 メッセージを入力して <input type="text" name="a2" size="150"> <br> 13 <input type="hidden" name="form_id"> 14 <input type="submit" value="メッセージ送信" name="ins"> 15</form> 16
PHP
1<?php 2$s=mysqli_connect("localhost","ユーザー名","パスワード")or die("失敗しました"); 3 4mysqli_select_db(null ?? $s,"db1"); 5$a1_d=$_POST["a1"]; 6$a2_d=$_POST["a2"]; 7$sel=mysqli_query(null ?? $s,"select*from tb order by bang"); 8$ins=mysqli_query(null ?? $s,"INSERT INTO tb(nama,mess) 9VALUES('$a1_d','$a2_d')"); 10switch ($_POST["form_id"]) { 11 case 'sel': 12while($kekka=mysqli_fetch_array($sel)) { 13 print $kekka[0]; 14 print ":"; 15 print $kekka[1]; 16 print "<BR>"; 17 } 18 break; 19 case 'ins': 20 while($kekka=mysqli_fetch_array($ins)) { 21 print $kekka[0]; 22 print ":"; 23 print $kekka[1]; 24 print "<br>"; 25 } 26 break; 27} 28mysqli_close($s); 29print "<br><A href='keiziban.html'>トップメニューに戻ります</A>"; 30 ?>
回答2件
0
ベストアンサー
複数のフォーム内容をまとめて送ることはできません。
上の form の submit をすると、送られるのは上の form の内容のみです (a1・a2 は送信されない)。
<form> (略) <input type="hidden" name="mode" value="list"> (略) </form> <form> (略) <input type="hidden" name="mode" value="addmessage"> (略) </form>
などとして、受け側では
if ( $_POST['mode'] === 'list' ){ 表示の際にやりたいこと } if ( $_POST['mode'] === 'addmessage' ){ メッセージ投稿の際にやりたいこと }
などとしてはどうでしょうか。
<form> をひとつにまとめてしまうのも手ですが、どちらの submit を押したかの判別が必要になります。投稿2021/01/24 09:24
総合スコア2022
0
片方ずつしか送れないので、同name別valueの要素を設けて、
value値で分岐かければ良いです。
それに、出ているのはエラーではなくNotice
片方にしかない項目を参照しようとしているため出ています。
「分岐用以外の項目」の参照は分岐かけてからで良いでしょう。
html
1<form method="post"> 2<input type="hidden" name="mode" value="view"> 3<button>表示</button> 4</form> 5<form method="post"> 6<input type="hidden" name="mode" value="send"> 7<button>送信</button> 8</form>
php
1$mode = filter_input(INPUT_POST,'mode'); 2 3if($mode != null){ 4 switch($mode){ 5 case 'view': 6 //1つ目のフォームから来た時の処理 7 break; 8 case 'send': 9 //2つ目のフォームから来た時の処理 10 break; 11 default: 12 die('不正なパラメータ'); 13 } 14} 15
SQLインジェクションだったり、XSSだったりの対策がされてないのが気になりますが、
そこは「どういう脅威か」を調べたうえで適切に対応してください。
投稿2021/01/24 09:25
編集2021/01/24 09:26
総合スコア80875
submitのnameでボタンを見分ける方法も回答つくだろうなと思って別の方法も提案してみました。
本来なら要件別ならリクエストを受け取る先も別にするか、もしくはリクエストは全てバックグラウンドで非同期で受け付けるほうが望ましいとは思ってます。
ビュー側にロジックごちゃごちゃ書くべきではないですし、既にかなりごちゃっとしてましたし。
回答ありがとうございました。ロジックのほうはごちゃごちゃ書かずにシンプルにまとめた方がいいんですね。覚えておきます。
>ロジックのほうはごちゃごちゃ書かずにシンプルにまとめた方がいいんですね。
自分が書いたコードでも１週間、いや３日見なかったら他人のコードです。
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
退会済みユーザー
2021/01/25 00:11
2021/01/25 13:46 編集
2021/01/25 13:48
2021/01/25 14:33 編集
2021/01/25 14:38
2021/01/25 14:59
2021/01/25 16:09