1
私は大量のコードを扱っていますが、簡潔にするためにif文の中に変数を設定しています。例：if ($subscriber = $this->subscriber->addSubscriber($request->all()))。下の例を展開してください。PHP if文の中で変数と式を組み合わせる
これはうまく動作し、おそらく3値ステートメントを使用する良い指標です。しかし、私が自分自身で問題を作り出す前に、それは深夜のリファクタリングにつながります。
私はこの問題があるか悪いのか尋ねる必要があると感じていますか？使用前、つまり伝統的なアプローチを宣言する必要がありますか？申し訳ありませんが、これは基本的な質問ですが、ここやPHPフォーラムで何も見つかりません。
public function postSubscriber(Request $request)
{
if ($subscriber = $this->subscriber->addSubscriber($request->all())) {
return response()->json($subscriber, 200);
}
return response()->json("Oops! something went wrong with your subscription.", 500);
}
あなたのコードはちょうどいいです。 – PeeHaa
三元はおそらくコードを読みにくくするでしょう。このような短いメソッドの場合でも、if文には何も問題はありません –
より簡潔なコードを読みやすくするかどうかは非常に主観的です。私にとって、コードをより多くのステップに分割する主な利点は、ブレークポイントをより正確に設定できるようにすることで、デバッグを少し素早く行うことができることです。 if条件での代入によって、一部のIDE（Netbeansがiircを実行する）で警告が表示されますが、これをオフにすることはできますが、誤って後で行うと警告しません。 –