select join 、論理削除ボタンについて

2つのテーブルを結合して、各項目に論理削除ボタンを付けたいです。
xamppでの作成中です

このソース自体は記述途中です。

<!DOCTYPE html>
<html lang="ja">

	<head>
	<meta http-equiv="Content-Type" content="text/html"; charset="UTF-8" />
		<title>status</title>
	</head>
	
	<body>
	
		<h1>使用状況画面</h1>
		
		<p><input type="button" name="regist" onclick="location.href= 'registration.php'" value="新規登録"></p>
		
		<form method = "post" action ="change.php">
		
	<table border = "1" width = "auto">
	
		<tr>
		<th>設備No</th>
		<th>設備名</th>
		<th>使用者</th>
		<th>開始時間</th>
		<th>終了時間</th>
		<th>使用目的</th>
		<th>削除</th>
		</tr>
		
	<?php
		
		//PDO生成
		$pdo = new PDO ('mysql:host=localhost;dbname=simplefacilityusesystem;charset=utf8', 'maint', '1qaz!QAZ');		//データベース接続
		foreach ( $pdo->query ( 'select facility from status left join using facility_number' ) as $row ) {
			
			echo '<tr>';
			echo '<td>', $row ['facility_number'], '</td>';
			echo '<td>', $row ['facility_name'], '</td>';
			echo '<td>', $row ['name'], '</td>'
			echo '<td>', $row ['start'], '</td>;
			echo '<td>', $row ['end'], '</td>;
			echo '<td>', $row ['purpose'], '</td>;
			echo '<td><input type="submit value="削除" name="flag">', $row [''], '</td>';
			echo '<td><input type="hidden" name="id" value="<?=$row['id']?>"';
			echo '</tr>';
			echo "\n";
		}
		
		if(isset($_POST["flag"])){
		$sql="delete from status where ";
		$sql->execute;
		}
		
		
	?>
		
	</table>
		
		</form>
		
		
		<br><p><input type="submit" onclick="location.href= 'menu.php'" value="前画面に戻る"></p>
		
	</body>
	
</html>

yambejp

2018/05/18 03:10

クォーテーションの閉じ忘れなど大量にあります。ロジックを質問する以前にケアレスミスを全部なおしてください。（今のままだと何をおしえてもタイプミスで絶対に動きません）

ryusan_012

2018/05/18 03:40

申し訳ございません

yambejp

2018/05/18 03:56

「select facility」としてカラムを１つしか参照していませんが、その後のループでは指定していないカラムを参照しているように見えます。どういう根拠でしょうか？

m.ts10806

2018/05/18 04:00

前回の私の回答がまるで活かされていませんが・・・。サンプルコードも添付しましたが、全く参考にしなかったのでしょうか？

m.ts10806

2018/05/18 04:01

回答前に指摘した接続情報のマスクかける部分とかも含めて諸々、一通り見直してください。

m.ts10806

2018/05/18 04:07

この記事も読んだ上なんですよね？https://qiita.com/rana_kualu/items/95f0c8be51e8665015d5#ide%E3%82%92%E4%BD%BF%E3%81%88

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

回答1件

ベストアンサー

一応こんなかんじですか？
hiddenでidを埋めて削除させるなら行ごとにformを設定するのが妥当。

PHP
1<?php
2try{
3  $pdo = new PDO($dsn, $user,$password);
4  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 
5  $pdo->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_ASSOC);
6  $pdo->setAttribute(PDO::ATTR_EMULATE_PREPARES,false);
7  $id=filter_input(INPUT_POST,"id");
8  $del=filter_input(INPUT_POST,"del");
9  if(!is_null($del)){
10    $sql="delete from status where id=?";
11    $stmt = $pdo->prepare($sql);
12    $stmt->execute([$id]);
13  }
14  $sql='select facility from status ';
15  $stmt=$pdo->query($sql);
16  $rows=$stmt->fetchAll(PDO::FETCH_ASSOC);
17  }
18}catch(PDOException $e){
19  die($e->getMessage());
20}
21
22?>
23
24<p><input type="button" name="regist" onclick="location.href= 'registration.php'" value="新規登録"></p>
25<table border = "1" width = "auto">
26<tr>
27<th>設備No</th>
28<th>設備名</th>
29<th>使用者</th>
30<th>開始時間</th>
31<th>終了時間</th>
32<th>使用目的</th>
33<th>削除</th>
34</tr>
35<?php
36  foreach($rows as $row){
37    echo <<<eof
38<tr>
39<td>{$row['facility_number']}</td>
40<td>{$row['facility_name']}</td>
41<td>{$row['name']}</td>
42<td>{$row['start']}</td>
43<td>{$row['end']}</td>
44<td>{$row['purpose']}</td>
45<td>
46<form method="post">
47<input type="submit" value="削除" name="del">
48<input type="hidden" name="id" value="{$row['id']}">
49</form>
50</td>
51</tr>
52eof;
53 }
54</table>
55<p><input type="submit" onclick="location.href= 'menu.php'" value="前画面に戻る"></p>
56

投稿2018/05/18 04:20

yambejp

総合スコア114839

m.ts10806

2018/05/18 05:18

前回質問（ https://teratail.com/questions/126731 ）からの流れだとするとテーブルがfacilityでその中にstatusカラムがありそうなんですよね。（論理削除　らしいので）この質問者さん　基本的なところの理解がだいぶ疎かになっているまま突き進んでいるのでコードが滅茶苦茶です・・・。