質問編集履歴
2
コードの冒頭に$ids = getIds();を追加しました。$idsの出どころがわからないと思ったため。
title
CHANGED
|
File without changes
|
body
CHANGED
|
@@ -5,6 +5,7 @@
|
|
|
5
5
|
|
|
6
6
|
## 元のコード（サンプルのため変数名などは簡略化してます）
|
|
7
7
|
```PHP
|
|
8
|
+
$ids = getIds();
|
|
8
9
|
$datas = getDatas();
|
|
9
10
|
|
|
10
11
|
// $datasが空でforeachに入らないケースがあります
|
|
@@ -18,6 +19,7 @@
|
|
|
18
19
|
|
|
19
20
|
## 指摘して以下のように直すべきと考えているコード
|
|
20
21
|
```PHP
|
|
22
|
+
$ids = getIds();
|
|
21
23
|
$datas = getDatas();
|
|
22
24
|
|
|
23
25
|
// ※指摘点 foreachの前に配列の初期化をしておきたい
|
1
はじめにの文章を掲題に沿うように修正。
title
CHANGED
|
File without changes
|
body
CHANGED
|
@@ -1,5 +1,5 @@
|
|
|
1
1
|
## はじめに
|
|
2
|
-
PHPのコードレビューをしていて、配列の変数の初期化の指摘
|
|
2
|
+
PHPのコードレビューをしていて、配列の変数の初期化のタイミングを指摘したいのですが、
|
|
3
3
|
言語化ができていないので、指摘できない状態にいます。
|
|
4
4
|
言語化の手助けをお願いします。
|
|
5
5
|
|