Q&A
ご回答ありがとうございました。
なるほど！そのような書き方もあるのですね、勉強になります。
2つの変数があり、3つのパターンがあるとします。
変数 $a $b
パターン 1.$aのみが存在する 2.$bのみが存在する 3.$aと$b両方存在する
※両方存在しないことは考えられないとします。
二次元配列にしたいのですが、以下のようにではなくもう少し簡潔に書けないでしょうか。
if(isset($a)) { $arr = [ 'a' => $a ]; } elseif (isset($b)) { $arr = [ 'b' => $b ]; } elseif (isset($a) && isset($b)) { $arr = [ 'a' => $a, 'b' => $b ]; } return $arr;
よろしくお願いします。
追記
皆様、多くのご回答ありがとうございました。
期待する動作をいただいたご回答で確認が取れました。
ベストアンサーを選ぶのが本当に難しかったのですが一番回答が早かった方にしました。
ご協力ありがとうございました。
回答5件
0
その配列をその後にどう使うか次第なのでは…。
自分なら最後だけ返してnullチェックは使う側で行う。
$arr = [ 'a' => $a, 'b' => $b, ];
if(!empty($arr['a']))
どのパターンでもチェックは必要。$arr['a']が存在しないも、存在するけどnullも使う側から見れば同じこと。
存在しなくても常にデフォルトを設定するとか。PHP7の書き方。
$arr = [ 'a' => $a ?? 'default', 'b' => $b ?? 'default', ];
投稿2018/04/20 08:48
総合スコア10377
0
$aもしくは$bがnullだったときにどうしたいかですね
とりあえずこれで
PHP
1$arr=[]; 2$a=1; 3$b=2; 4foreach(["a","b"] as $val){ 5 if(isset($$val)) $arr[$val]=$$val; 6} 7var_dump($arr);
投稿2018/04/20 08:45
総合スコア114812
0
こうかなぁ。
PHP
1if(isset($a)) $arr['a'] = $a; 2if(isset($b)) $arr['b'] = $b;
ちなみにこれ二次元配列じゃないですよ。
一次元の連想配列です。
投稿2018/04/20 08:30
総合スコア16998
最初に配列を作成せずにいきなり[]=で作成することも動かなくはないですが、おすすめできない、とのことです。
http://php.net/manual/ja/language.types.array.php （上20%ぐらいの位置に、そういう記述がありました）
そこは敢えて元のコードの動きを再現しました。
あ、ちょっと意味合いが違うか。
「両方存在しないことは考えられない」パターンであれば、$arrが生成されない心配はないかと思います。
その一文忘れてました。
そうですね、その前提条件があるなら初期化しておいたほうがいいですね。
> ちなみにこれ二次元配列じゃないですよ。
> 一次元の連想配列です。
失礼しました。頭がこんがらがっちゃって、もう少し勉強します。
ご回答ありがとうございました。
0
配列に追加していく形で書くのも1つの方法かなと思います。
php
1$arr = []; 2if(isset($a)) $arr['a'] = $a; 3if(isset($b)) $arr['b'] = $b;
投稿2018/04/20 08:29
総合スコア145183
まるっきりかぶりましたｗ
最初に初期化すべきなんですね。
自分でももっと勉強します。
ありがとうございました。
0
ベストアンサー
PHP
1$arr = (isset($a) && isset($b)) 2 ? [ 'a' => $a, 'b' => $b ] 3 : ( (isset($a))? [ 'a' => $a ] : [ 'b' => $b ] );
投稿2018/04/20 08:29
退会済みユーザー
総合スコア0
もうけっ！
ありがとうございました！三項演算子の書き方勉強になりました。
PHPは三項演算子の仕様がおかしいので、: 以降については必ず ( ) で括ってくださいマシマシ。
そうだったんですか！知りませんでした。
括ります！
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
退会済みユーザー
2018/04/20 09:16