回答編集履歴
5
見直し
answer
CHANGED
@@ -22,4 +22,7 @@
|
|
22
22
|
|
23
23
|
> 返り値
|
24
24
|
|
25
|
-
> この関数が成功した場合の返り値は、取得形式によって異なります。 失敗した場合は常に FALSE を返します。
|
25
|
+
> この関数が成功した場合の返り値は、取得形式によって異なります。 失敗した場合は常に FALSE を返します。
|
26
|
+
|
27
|
+
それと、fetch()するまえに、
|
28
|
+
fetch()結果が入る`$rec`を参照しているのもおかしい。
|
4
見直し
answer
CHANGED
@@ -12,8 +12,11 @@
|
|
12
12
|
`while(true)`なんて、
|
13
13
|
ブロック内でbreak;を書いているとはいえ
|
14
14
|
まず書いちゃダメ。
|
15
|
+
break;の条件に至らないと無限ループとして処理が終わらず、
|
16
|
+
実行時間の時間切れで強制終了になるだけだし。
|
17
|
+
せめて
|
15
18
|
`while ($rec = $stmt->fetch(PDO::FETCH_ASSOC)) {`
|
16
|
-
|
19
|
+
とかにしませんか。
|
17
20
|
|
18
21
|
[PHP: PDOStatement::fetch - Manual](https://www.php.net/manual/ja/pdostatement.fetch.php#refsect1-pdostatement.fetch-examples)
|
19
22
|
|
3
見直し
answer
CHANGED
@@ -13,4 +13,10 @@
|
|
13
13
|
ブロック内でbreak;を書いているとはいえ
|
14
14
|
まず書いちゃダメ。
|
15
15
|
`while ($rec = $stmt->fetch(PDO::FETCH_ASSOC)) {`
|
16
|
-
などとするならわかるけど。
|
16
|
+
などとするならわかるけど。
|
17
|
+
|
18
|
+
[PHP: PDOStatement::fetch - Manual](https://www.php.net/manual/ja/pdostatement.fetch.php#refsect1-pdostatement.fetch-examples)
|
19
|
+
|
20
|
+
> 返り値
|
21
|
+
|
22
|
+
> この関数が成功した場合の返り値は、取得形式によって異なります。 失敗した場合は常に FALSE を返します。
|
2
見直し
answer
CHANGED
@@ -7,4 +7,10 @@
|
|
7
7
|
```
|
8
8
|
|
9
9
|
データの取得や表示が終わっていないのに
|
10
|
-
`$dbh=null;`をやる意味もわからない。
|
10
|
+
`$dbh=null;`をやる意味もわからない。
|
11
|
+
|
12
|
+
`while(true)`なんて、
|
13
|
+
ブロック内でbreak;を書いているとはいえ
|
14
|
+
まず書いちゃダメ。
|
15
|
+
`while ($rec = $stmt->fetch(PDO::FETCH_ASSOC)) {`
|
16
|
+
などとするならわかるけど。
|
1
見直し
answer
CHANGED
@@ -4,4 +4,7 @@
|
|
4
4
|
↓
|
5
5
|
```php
|
6
6
|
$sql='SELECT code,name FROM mst_staff;';
|
7
|
-
```
|
7
|
+
```
|
8
|
+
|
9
|
+
データの取得や表示が終わっていないのに
|
10
|
+
`$dbh=null;`をやる意味もわからない。
|