[過去ﾛｸﾞ] PHP質問・雑談スレ4【初心者お断り(ROM歓迎)】 [無断転載禁止]©2ch.net (1002ﾚｽ)
上下前次1-新
このｽﾚｯﾄﾞは過去ﾛｸﾞ倉庫に格納されています｡
次ｽﾚ検索 歴削→次ｽﾚ 栞削→次ｽﾚ 過去ﾛｸﾞﾒﾆｭｰ
596: 580 2018/07/22(日)22:39 ID:??? AAS
ん？俺がワンライナーとか言ったのがまずかったのか？
普通三項演算子はワンライナーで書くもんだからそう言っただけなのだが
別にそこはメインじゃないし主題ではない

?:という2つの演算子しか使っておらず、間違い探しをするようなものでもなく
何がわかりづらいのか理解できないって話をしただけ

ちなみに会話してんの俺じゃないぞ・・・
597: 2018/07/22(日)22:40 ID:??? AAS
少ない行がいいなら最期に¥rと¥nを置き換えて1行にすればいいやんｍ
598: 2018/07/22(日)22:44 ID:??? AAS
>普通三項演算子はワンライナーで書くもんだから

俺ルール広めようとしないでくれない？
599: 580 2018/07/22(日)22:49 ID:??? AAS
主観ではあるけど
googleで画像検索したりしても、実際ワンライナーが多数派じゃない？
そもそもネストするようなケースじゃほとんど使われないし
これも主観だけど感覚的に正しいと思う
600: 2018/07/22(日)22:53 ID:??? AAS
改行とインデントで整えないなら1層ですら可読性低いんだけど

しかし、なんで一行・1行などと書かないでワンライナーって書くのかね
文字数多いじゃん検索用ってか
601(1): 2018/07/22(日)23:10 ID:??? AAS
if (age < 18) {
　　if (gender == 'male') {
　　　　title = 'Master';
　　} else {
　　　　title = 'Miss';
　　}
} else {
　　if (gender == 'male') {
　　　　title = 'Mr';
} else {
省8
602(1): 580 2018/07/22(日)23:14 ID:??? AAS
可読性の定義は人によってさまざまだけど
長ったらしいのはかえって可読性が下がると俺は思う
>>601では真ん中が一番読みやすい
603(1): 2018/07/22(日)23:16 ID:??? AAS
ビットにしてswitchしよう
604: 580 2018/07/22(日)23:19 ID:??? AAS
一番上がおかしかった
if (age < 18) {
　　if (gender == 'male') {
　　　　title = 'Master';
　　} else {
　　　　title = 'Miss';
　　}
} else {
　　if (gender == 'male') {
　　　　title = 'Mr';
省5
605(1): 2018/07/22(日)23:20 ID:??? AAS
一番下もおかしいんだが
php書けないのか
606: 580 2018/07/22(日)23:25 ID:??? AAS
>>605
これPHPじゃないし
三項演算子の可読性について、stackoverflowで引き合いに出されてたコード
PHPは評価の仕方が他の言語と違うので
そもそも一番下のようなコードは書いたことがない
607: 2018/07/22(日)23:33 ID:??? AAS
>>602
同意

ネストしないなら3項演算子がシンプルでよいがネストさせるヤツは死ねと思ってしまう
608: 2018/07/22(日)23:37 ID:??? AAS
俺Pythonistaだけど、{}が読みづらい原因だから、こうするといいよ。

if ($age < 18)
　　if ($gender == 'male')
　　　　$title = 'Master';
　　else
　　　　$title = 'Miss';
else
　　if ($gender == 'male')
　　　　$title = 'Mr';
　　else
省1
609: 2018/07/22(日)23:43 ID:??? AAS
PSR-2>呼んだ？
610(2): 2018/07/22(日)23:59 ID:??? AAS
$titles = [
　'male'　 => [ 'u18' => 'Master', 'other' => 'Mr' ],
　'female' => [ 'u18' => 'Miss',　 'other' => 'Ms' ],
];

$age_category = ( $age < 18 ) ? 'u18' : 'other' ;
// もしくは
if( $age < 18 )
　$age_category = 'u18';
else
　$age_category = 'other';
省6
611: 2018/07/23(月)00:00 ID:??? AAS
実務的にはそれぞれ属性を自分のobjectに聞いて、って感じのが多いかな
612: 2018/07/23(月)00:14 ID:??? AAS
stackoverflow探してみたけどそれっぽいのが見つからないわ
613: 2018/07/23(月)00:48 ID:??? AAS
外部ﾘﾝｸ:www.quora.com
stackoverflowではないな
614: 2018/07/23(月)01:16 ID:??? AAS
echo 'こんにちは、' . ($name && strlen($name) > 0) ? trim($name) : "anonymous" . "さん";

$name = ($name && strlen($name) > 0) ? trim($name) : "anonymous";
echo "こんにちは、{$name}さん";

var = a ? b : c;を徹底して、
それ以外を組み込んだ、上のような使い方をしなければ問題ないと思う。
Viewには処理済み変数のみを埋め込むのを徹底すればいい。
615: 2018/07/23(月)01:22 ID:??? AAS
表示名の場合はだいたい
$nameを先に'anonymous'で初期化しておいて
セッションがあってDBから取ってくる等の時に$nameを上書きする
ってやるよね
上下前次1-新書関写板覧索設栞歴
あと 387 ﾚｽあります
ｽﾚ情報 赤ﾚｽ抽出 画像ﾚｽ抽出 歴の未読ｽﾚ AAｻﾑﾈｲﾙ
ぬこの手 ぬこTOP 0.009s