回答編集履歴
2
ad
test
CHANGED
@@ -1,3 +1,5 @@
|
|
1
|
+
・`.html`でPHPを動くようにサーバー側の設定をしていなければ、`.php`でしかPHPコードとして認識しません。
|
2
|
+
|
1
3
|
・PHPの文字列連結は`.`です。([PHPマニュアル](https://www.php.net/manual/ja/language.operators.string.php))
|
2
4
|
|
3
5
|
・変数と定数が正しく使い分けできていません。現状だと変数で定義したのに定数として使おうとしています。([参考記事](https://qiita.com/mpyw/items/95e205056e25b7a59dfa) ※少し古い記事ですが基本は変わってません)
|
1
ed
test
CHANGED
@@ -1,8 +1,13 @@
|
|
1
1
|
・PHPの文字列連結は`.`です。([PHPマニュアル](https://www.php.net/manual/ja/language.operators.string.php))
|
2
|
+
|
2
3
|
・変数と定数が正しく使い分けできていません。現状だと変数で定義したのに定数として使おうとしています。([参考記事](https://qiita.com/mpyw/items/95e205056e25b7a59dfa) ※少し古い記事ですが基本は変わってません)
|
3
|
-
|
4
|
+
|
5
|
+
・PDO::query()の返却値が0ではない・・・?確かに`PDOStatement|false`なので([PHPマニュアル](https://www.php.net/manual/ja/pdo.query.php))falseの場合のチェックをしたいのかもしれませんが厳密比較してしまうと想定の結果が得られないのでは?([簡易動作確認](https://paiza.io/projects/1UmrtICQCEcD-vhu4fkDCA)) つまり実行結果falseでもtrue、そうでなくてもtrueなのでif文で分岐させている意味がないです。
|
6
|
+
|
4
7
|
・SELECT結果ならPDOStatementそのままではなくfetchAllしましょう([PHPマニュアル](https://www.php.net/manual/ja/pdostatement.fetchall.php))
|
8
|
+
|
5
9
|
・エラーハンドリングしましょう。([PHPマニュアル](https://www.php.net/manual/ja/pdo.error-handling.php))
|
10
|
+
|
6
11
|
・mysql_closeはじめ、mysql_*の機能はPHP7から削除されています。([PHPマニュアル](https://www.php.net/manual/ja/function.mysql-close.php))
|
7
12
|
コネクションのクローズはnullを代入です。([PHPマニュアル](https://www.php.net/manual/ja/pdo.connections.php]))
|
8
13
|
|