編集履歴

回答編集履歴

短縮

2019/02/20 02:18

投稿

スコア80850

test CHANGED Viewed

@@ -506,6 +506,36 @@
+ちなみに[短く書く](http://php.net/manual/ja/language.operators.comparison.php#language.operators.comparison.ternary)ならこう。
+```php
+function get(int $num):array{
+    $a = [
+        ["id"=>1,
+        "name"=>"test"
+        ],
+        ["id"=>2,
+        "name"=>"test2"
+        ],
+    ];
+    return (array_key_exists($num,$a))?$a[$num]:[];
+}
+```
 # デバッグについて

修正

2019/02/20 02:18

投稿

m.ts10806

スコア80850

test CHANGED Viewed

@@ -398,7 +398,9 @@
 実際のプログラムでは入力値が入ってきたりクエリストリングから受け取ったパラメータが入ってきたりして、本当に何が来るか分からないわけです。
-もし新しいPHP環境であれば、[引数の型を制限](http://php.net/manual/ja/functions.arguments.php#functions.arguments.type-declaration)できます。
+下記のようにして[引数の型を制限](http://php.net/manual/ja/functions.arguments.php#functions.arguments.type-declaration)します。
@@ -442,7 +444,7 @@
-`PHP Notice:  Undefined offset: 2 in XXX.php on line 11```
+`PHP Notice:  Undefined offset: 2 in XXX.php on line 11`

修正

2019/02/20 01:33

投稿

m.ts10806

スコア80850

test CHANGED Viewed

@@ -398,7 +398,7 @@
 実際のプログラムでは入力値が入ってきたりクエリストリングから受け取ったパラメータが入ってきたりして、本当に何が来るか分からないわけです。
-もし新しいPHP環境であれば、引数の型を制限できます。
+もし新しいPHP環境であれば、[引数の型を制限](http://php.net/manual/ja/functions.arguments.php#functions.arguments.type-declaration)できます。
@@ -456,7 +456,7 @@
-阻止するために、「存在するキーが与えられていたら取得」としましょう。
+阻止するために、[「存在するキーが与えられていたら取得」](http://php.net/manual/ja/function.array-key-exists.php)としましょう。
@@ -492,7 +492,7 @@
 ```
-ついでに、返り値の型も宣言しておき、存在した場合、そのまま返却、そうでない場合は空の配列を返すようにしました。
+ついでに、[返り値の型も宣言](http://php.net/manual/ja/functions.returning-values.php#functions.returning-values.type-declaration)しておき、存在した場合、そのまま返却、そうでない場合は空の配列を返すようにしました。
 これでひとまずいい感じにはできましたね。

修正

2019/02/20 01:10

投稿

m.ts10806

スコア80850

test CHANGED Viewed

@@ -556,7 +556,7 @@
   var_dump($mouths);
-  //攻略
+  //後略
 }

修正

2019/02/20 01:04

投稿

m.ts10806

スコア80850

test CHANGED Viewed

@@ -18,7 +18,7 @@
 その考えは今すぐ捨てましょう。
-この内容で「なぜNULLになるのか」という質問が出る時点で質問者さんは思ったとおりコードを動かせる息には到底達していません。
+この内容で「なぜNULLになるのか」という質問が出る時点で質問者さんは思ったとおりコードを動かせる域には到底達していません。
@@ -572,9 +572,11 @@
-**質問者さんは思ったとおりコードを動かせる息には到底達していません。**
+**質問者さんは思ったとおりコードを動かせる域には到底達していません。**
+----------
@@ -585,3 +587,17 @@
 mouthって「口」ですけど。
 「口コミ」なら「reviews」ですね。レビュー。
+他人が見ても意味が分かる内容にしておかないと、数ヵ月後自分が見たときも意味が分かりませんよ。
+言語に限らず、下記のような記事はきちんと読んでおいて取り入れるようにしましょう。
+- [良いコードを書く技術（まとめ）](https://qiita.com/NoriakiOshita/items/e60ab5bb01b90d927ae5)
+- [プログラミング勉強を加速させる7つの習慣](https://qiita.com/YudaiTsukamoto/items/42a8df22ca4c6b327dfd)
+- [初心者を戒めるPHP](https://qiita.com/tadsan/items/fb496e450fc27c8c4494)
+- [初心者プログラマが犯しがちな過ち25選](https://qiita.com/rana_kualu/items/379eefb3a40c6b44cb92)

大幅修正

2019/02/20 01:01

投稿

m.ts10806

スコア80850

test CHANGED Viewed

@@ -1 +1,587 @@
+質問者さんの過去質問を見ましたが、毎度されている**デバッグ依頼**ですね。
+本来**非推奨な質問の域**ですが、今回だけ答えてみます。
+今後同様の質問がしなくてもいいようになれば幸いですね（ﾆｯｺﾘ
+# 結論から
+[変数のスコープ](http://php.net/manual/ja/language.variables.scope.php)をきちんと理解されていないことから「なぜNULLになるのでしょうか」という質問になっています。
+プログラムは指示したとおりにしか動きません。自分が思ったとおりに動くと思っていませんか？
+その考えは今すぐ捨てましょう。
+この内容で「なぜNULLになるのか」という質問が出る時点で質問者さんは思ったとおりコードを動かせる息には到底達していません。
+# ミニマムコード
+質問者さんのやろうとしていることは下記と同等です。
+```php
+$a = [
+    ["id"=>1,
+    "name"=>"test"
+    ],
+    ["id"=>2,
+    "name"=>"test2"
+    ],
+];
+function get($num){
+    $data = $a[$num];
+    return $data;
+}
+var_dump(get(1));
+```
+[require_once()](http://php.net/manual/ja/function.require-once.php)しているから状況が違う、
+と思われたのであればマニュアルをきちんと読み直してください。
+同等であるため[inlucde_once()](http://php.net/manual/ja/function.include-once.php)の方に説明がありますが、
+`include_once 命令は、スクリプトの実行時に指定 したファイルを読み込み評価します`
+これはPHPの場合、実行時にそこを通ったらそのプログラムを実行することを意味します。
+つまり、コードが書いてあるのと同義です。
+さて、本題のミニマムコードに戻りますが。
+「デバッグ依頼である」と私が結論づけるのも無理がなく、上記、質問者さんのコードと同等のコードを実行するとエラーになります。
+```
+PHP Notice:  Undefined variable: a in XXX.php on line 13
+```
+質問者さんのほうでも同じコードを実行してみてください。
+もし上記Noticeが出ていないのでしたら、エラー表示がOnになっていません。
+学習段階であったり本番リリース前であったり、不完全なコードの状態では何が起きるか分かりません。
+必ずエラー表示をOnにしてください。
+- [PHPのエラー表示設定について](https://qiita.com/shotets/items/3c95aef631b2c5eadae5)
+※もしかしたら既に過去に指摘を受けているかもしれませんね。でもそれだけ大事です。
+# PHPマニュアル確認
+で、ここで私が「結論」にリンクを貼った[変数のスコープ](http://php.net/manual/ja/language.variables.scope.php)の項を見てみてください。
+序盤に下記のようなコードがありますね。
+```php
+$a = 1; /* グローバルスコープ */
+function test()
+{
+    echo $a; /* ローカルスコープ変数の参照 */
+}
+test();
+```
+このコード、私のミニマムコードとよく似ていると思いませんか？
+それはつまり、質問者さんのコードと同等であることを示します。
+ちなみにこのコードも同じNoticeを出力します。
+`Undefined variable: a`
+「未定義の変数：a」
+エラーの調べかたとか読み方「質問前にすること」は下記が参考になりますよ。
+- [エラーメッセージの読み方と対処, 検索や質問の原則](https://qiita.com/cannorin/items/eb062aae88bfe2ad6fe5)
+**「読んで理解して自身で実践できるようになるまで質問しない」くらいの気持ちで読んでください**
+つまり、変数が未定義であることを指します。
+なぜかって、それはコメントに書いてある通りで、
+**PHPにおいて、関数は自身の中で定義された変数しか参照できないから**　です。
+それがコメントで書いてある「ローカルスコープ変数の参照」の意味です。
+JavaScriptと同じ感覚ではいけません。PHPの仕様です。覚えましょう。
+> PHPマニュアルより抜粋
+このスクリプトは、出力を全く行いません。これは、echo 命令がローカ ル版の $a 変数を参照しているにもかかわらず、このスコープでは値が代入されていないからです。～中略～PHP では、グローバル変数は、関数の内部で使用する場合、関数の内部でグローバルとして宣言する必要があります。
+# まず一歩目
+[変数のスコープ](http://php.net/manual/ja/language.variables.scope.php)にあるようにglobalキーワードを使うとひとまずエラーが解消できます。
+```php
+$a = [
+    ["id"=>1,
+    "name"=>"test"
+    ],
+    ["id"=>2,
+    "name"=>"test2"
+    ],
+];
+function get($num){
+    global $a;
+    $data = $a[$num];
+    return $data;
+}
+var_dump(get(1));
+```
+出力結果
+```
+array(2) {
+  ["id"]=>
+  int(2)
+  ["name"]=>
+  string(5) "test2"
+}
+```
+# 2歩目
+でもこれって意味がありません。
+globalキーワードを使う是非はひとまず置いといて、
+変数$aってどこも使ってませんよね。このget()関数のためだけに使うのは意味がないです。
+引数に渡すのがその次。
+```php
+$a = [
+    ["id"=>1,
+    "name"=>"test"
+    ],
+    ["id"=>2,
+    "name"=>"test2"
+    ],
+];
+function get($num,$a){
+    $data = $a[$num];
+    return $data;
+}
+var_dump(get(1,$a));
+```
+でも、わざわざ定義した変数を引数で渡すって変な感じですよね。
+それならこの方がいい。
+```php
+function get($num){
+    $a = [
+        ["id"=>1,
+        "name"=>"test"
+        ],
+        ["id"=>2,
+        "name"=>"test2"
+        ],
+    ];
+    $data = $a[$num];
+    return $data;
+}
+var_dump(get(1));
+```
+関数の中に定義してしまおうってことですね。
+結局、変数$aはget()以外で直接どこからも参照されることがないのでしたら、これで充分です。
+「データとしてしっかり持ちたいんだ」ということでしたら、
+その変数$aをコード内に直接変数で持つのではなくデータベースで取得するか、クラスにしましょう。
+```php
+class a{
+    private $a = [
+        ["id"=>1,
+        "name"=>"test"
+        ],
+        ["id"=>2,
+        "name"=>"test2"
+        ],
+    ];
+    function getByNum($num){
+        return $this->a[$num];
+    }
+}
+$obj_a = new a();
+var_dump($obj_a->getByNum(1));
+```
+privateなので$obj_a->a;のようには参照できません。
-[変数のスコープ](http://php.net/manual/ja/language.variables.scope.php)
+- [プロパティのアクセス権](http://php.net/manual/ja/language.oop5.visibility.php)
+# もう一歩
+クラスのメンバーでも関数でもなんでも同じですが、メソッドって**引数に何が入ってくるか知りません**。
+「プログラム組んでるんだから制御できるだろ」って思ったら大間違いです。
+関数で準備した以上は、誰から実行されるかなんてわからないわけです。
+複数人で同じプロジェクトで開発していたら、同じように使えるわけですし、引数って言っても変数です。変数は何が入っているか分からないもので、後から書き換え可能です。
+実際のプログラムでは入力値が入ってきたりクエリストリングから受け取ったパラメータが入ってきたりして、本当に何が来るか分からないわけです。
+もし新しいPHP環境であれば、引数の型を制限できます。
+※とりあえず「それならこれでいい」のコードで
+```php
+function get(int $num){
+    $a = [
+        ["id"=>1,
+        "name"=>"test"
+        ],
+        ["id"=>2,
+        "name"=>"test2"
+        ],
+    ];
+    $data = $a[$num];
+    return $data;
+}
+```
+これでget("a")とか、int（整数型）以外を与えるとエラーが出るようになりました。
+でも、これでも不十分です。
+確実に存在する値が引数として与えられるか保証はありませんよね。
+この場合、2とか与えられたら下記のようなエラーが出ます。
+`PHP Notice:  Undefined offset: 2 in XXX.php on line 11```
+未定義の[オフセット](https://wa3.i-3-i.info/word11923.html):2
+つまり、配列に存在しないポイント、という意味ですね。
+※配列にキーが存在しない場合と同等です
+阻止するために、「存在するキーが与えられていたら取得」としましょう。
+```php
+function get(int $num):array{
+    $a = [
+        ["id"=>1,
+        "name"=>"test"
+        ],
+        ["id"=>2,
+        "name"=>"test2"
+        ],
+    ];
+    if(array_key_exists($num,$a)){
+        return $a[$num];
+    }
+    return [];
+}
+```
+ついでに、返り値の型も宣言しておき、存在した場合、そのまま返却、そうでない場合は空の配列を返すようにしました。
+これでひとまずいい感じにはできましたね。
+あとは自身のコードに考え方を反映するだけ！
+頑張ってください。
+# デバッグについて
+そういえば、各所でvar_dump()を入れられてデバッグをされていますが、本来確認しなければならない項目を確認していません。
+ひとつめ。
+「処理がここまで通ってるよ」という確認は必要ですが、sampleという文言だけではどこかは分かりません。
+せめて一意になる数字にしましょう。
+ちなみに下記のように書くとそのプログラムの行数が出ます。
+```php
+echo __LINE__."<br />\n";//重なってはいけないので改行を入れておく
+```
+ふたつめ。
+確認しなければならない変数。
+$mouthsという変数をグローバルスコープで下記のように確認したことで
+```php
+//表示される
+//var_dump($mouths[0]);
+```
+getMouths()内で使えるぜ！と思ったかもしれませんが、根拠はどこですか？
+いずれにしても、本当に使えるかどうか確認しなければなりませんよね。
+```php
+function getMouths($itemperpage, $currentpage)
+{
+  var_dump($mouths);
+  //攻略
+}
+```
+こうしておけば、最初のNotice`PHP Notice:  Undefined variable:`が出て気づけたでしょうね。
+（エラー表示OFFだとnullかな）
+デバッグの仕方、本来確認すべき項目はきちんとおさえるようにしないと、デバッグがデバッグになりません。気をつけましょう。
+**質問者さんは思ったとおりコードを動かせる息には到底達していません。**
+蛇足：
+きちんと意味のある変数名つけてますか？
+mouthって「口」ですけど。
+「口コミ」なら「reviews」ですね。レビュー。

修正

2019/02/20 00:55

投稿

m.ts10806

スコア80850

test CHANGED Viewed

	@@ -1 +1 @@
1	- 変数のスコープ
1	+ [変数のスコープ](http://php.net/manual/ja/language.variables.scope.php)