[過去ログ] スレ立てるまでもない質問はここで 162匹目 (1002レス)
1-

このスレッドは過去ログ倉庫に格納されています。
次スレ検索 歴削→次スレ 栞削→次スレ 過去ログメニュー
655
(1): (オッペケ Src1-Xv9b) 2022/11/13(日)21:03 ID:wphs1sqGr(5/9) AAS
>>652
だから前提で業務ロジックが複雑って言ってんじゃん
複雑な業務ロジックはコメントないと他人が理解できないわけ

てかお前のゴミコードはコメントゼロなの?
656
(1): (ワッチョイ b507-z+Fy) 2022/11/13(日)21:25 ID:pp5TDwJ00(4/8) AAS
>>655
無駄に複雑になってるのはお前の責任も多いぞって話だ
そういうのをスパゲティって言うんだよ
知らなきゃググれ
657
(2): (オッペケ Src1-Xv9b) 2022/11/13(日)21:47 ID:wphs1sqGr(6/9) AAS
>>656
業務が複雑なのは俺のせいか
お前マジで楽な仕事しかしたことなさそうだな
参考にならんからもうレスしなくていいぞ
658
(1): (ワッチョイ b507-z+Fy) 2022/11/13(日)21:57 ID:pp5TDwJ00(5/8) AAS
>>657
業務が複雑な話をしてるんじゃなくお前のコードが複雑になってる話をしてるんだがこれがわからないとなるとさすがにプログラミング向いてないから若いうちにやめた方がいいぞ
659: (ワッチョイ e535-3ACM) 2022/11/13(日)22:02 ID:NKPiR9HJ0(1/2) AAS
比較メソッドの実装はどこにすべき?→DBに任せるべき
コメントの場所はどこにすればいい?→コメントが要らないようにすべき

こういうの多いね、ここ。
660: (オッペケ Src1-Xv9b) 2022/11/13(日)22:04 ID:wphs1sqGr(7/9) AAS
>>658
コメントの位置の質問くらいでイライラすんなよw
カスみてえなメンタルだなww
661
(1): (ワッチョイ a301-xL83) 2022/11/13(日)22:05 ID:0j2qr49h0(2/2) AAS
>>657
ifとかelse-ifとかにコメント入れないと理解できなくなるというのは100%プログラマーの責任だぞ
業務ロジックが複雑なのが理由なんじゃなくてそのロジックを整理してプログラムで表現できてないのが理由なんだから

ただ理由を書く場合とかコメントを書いておいたほうがいいケースがあるのは確か
そういう場合のコメント位置はifやelse-ifブロックの中に書くのが主流
次点でifやelse-ifと同じ行
ifやelseの上の行にあるとプログラムの構造が見にくくなるから好まれない
662
(1): (オッペケ Src1-Xv9b) 2022/11/13(日)22:15 ID:wphs1sqGr(8/9) AAS
>>661
いやマジで業務が次々と変わるんだわ
新たな条件が鬼のようにくるんだよ
知らない言葉が新しく作られ、意味のわからない造語が条件を意味していと言ってくる

さらに先方から来るデータがゴミクソ過ぎて日付の位置に名前が記載されたり酷いんだよ
直してくれっていっても相手が大企業で絶対に譲らねえんだ
それをプログラマーのせいってことか?

つまり前半の部分は質問してないんだから答えるな
わかったか?

後半は好む好まないではなくコーディング規約とかでどの言語、どのフレームワークでも同じように規約されてるレベルのものなのか?
663: (ワッチョイ b507-z+Fy) 2022/11/13(日)22:33 ID:pp5TDwJ00(6/8) AAS
知らない言葉w
664: (ワッチョイ b507-z+Fy) 2022/11/13(日)23:12 ID:pp5TDwJ00(7/8) AAS
業務の質問を5chでしてイライラするボンクラよw
だがほんとは業務じゃないだろ
なぜなら「プログラマーはめんどくせーのばっか」の一言でお前がプログラマーじゃないのがわかる
初心者なら無駄に見栄張らず初心者らしく謙虚にするのがいいと思うぞw
665: (オッペケ Src1-Xv9b) 2022/11/13(日)23:26 ID:wphs1sqGr(9/9) AAS
業務の質問はしてない
コメントの質問を業務の質問と勘違いするアホがいるようだが
666: (ワッチョイ b507-z+Fy) 2022/11/13(日)23:30 ID:pp5TDwJ00(8/8) AAS
業務気取ってるが業務じゃないと言ってるのが読み取れないお粗末な言語能力よw
667: (ワッチョイ e535-3ACM) 2022/11/13(日)23:38 ID:NKPiR9HJ0(2/2) AAS
もういいだろ…
668: (ワッチョイ 8553-XNY3) 2022/11/13(日)23:46 ID:G9mkcghf0(1) AAS
ヒント:ワッチョイ
669: (ワッチョイ 05ac-sptE) 2022/11/14(月)00:09 ID:OqYLF+gF0(1/2) AAS
まあそのコメントなら最初のifの上で説明するやろな
elseifの上にまで下げなきゃいけないほど、最初のifの中を大きくしない

今どきの言語や設計ならそうならないように大抵できると思うわ
670
(1): (ワッチョイ 035a-BELy) 2022/11/14(月)01:26 ID:il96DJqz0(1) AAS
サーバー側から処理が終わったことをフロントエンドに通知したいんだけど何かいい方法ある?
定期的にサーバーにリクエスト送って確認するしかない?
671: (ワッチョイ e535-3ACM) 2022/11/14(月)01:45 ID:qP7RtejA0(1) AAS
それがポーリングでもうひとつはWebSocketがあるけどそこまでしたいかどうか
672: (ワッチョイ a301-xL83) 2022/11/14(月)02:20 ID:kBkzjc2d0(1) AAS
>>670
push notification
673
(1): (スップ Sd03-+47b) 2022/11/14(月)09:19 ID:9Mh86GGod(1) AAS
もう終わってるみたいだが、そんなに複雑なif文ならメソッド化かな
メソッドコメントにいくらでも説明書けるし、if文も入れ子にして一つ一つは簡単になるはずだよね

この人には聞き入れては貰えないだろうが…
674
(2): (スップ Sd43-+47b) 2022/11/14(月)09:28 ID:xMKESfkXd(1) AAS
>>673
自分で書いといてあれだがbooleandeで判断できないならenamがいいわな
終わった話に無駄なこと書いてたわ
上とこれ共々無かったのとに…
675
(1): (ワッチョイ b507-z+Fy) 2022/11/14(月)10:17 ID:HDY6oj/W0(1/2) AAS
>>674
しかもそれ俺がとっくに書いてるやつ
676: (オッペケ Src1-Xv9b) 2022/11/14(月)10:30 ID:3xPuxRkkr(1) AAS
結局どいつも答えられなくてワロタw
自称プログラマーのくせに課題に対して明確な解決すらできずに別の案を提示w
しかもこんな低レベルな質問内容に休日からブチ切れww
677: (ワッチョイ b507-z+Fy) 2022/11/14(月)10:48 ID:HDY6oj/W0(2/2) AAS
答えてんじゃん
糞コードだって
理由を二つ書いたのが読めないのか?

二つ目の理由読めばわかるがelseの前に空行入れるとかセンス無さすぎて笑うわw
678
(2): (ブーイモ MM39-5kBA) 2022/11/14(月)11:43 ID:WYNEj24HM(1/2) AAS
>>674
booleanで判断できないってどういう状況?
679: (ブーイモ MM39-5kBA) 2022/11/14(月)11:46 ID:WYNEj24HM(2/2) AAS
>>662
スパゲッティコーダーの典型的な言い訳じゃん
現在進行形で負債を量産してることに早く気付け
680
(2): (アウアウウー Saa9-z+Fy) 2022/11/14(月)11:52 ID:Ukj+JKk5a(1) AAS
>>678
平日ダイヤ、休日ダイヤ、臨時ダイヤの三種類で処理を変えたい時とかあるわな
その時はbooleanじゃなくダイヤの種類をenumで返せばいい
この質問者は「臨時ダイヤになる条件が次々変わるのにどうすりゃいいの!?」って発狂してるがそんなのどのダイヤになるのかという判断だけカプセル化して処理から分離すべきなんだよな
681: (オッペケ Src1-sptE) 2022/11/14(月)11:56 ID:+TCudK18r(1) AAS
もしかしたら、こいつの会社は毎日のように会社の業種とかが変わるのかもしれんよw
682: (スップ Sd03-+47b) 2022/11/14(月)12:38 ID:EH4Yfs9Id(1/2) AAS
>>675
なので無かったことにして欲しい…
683: (スップ Sd03-+47b) 2022/11/14(月)12:43 ID:EH4Yfs9Id(2/2) AAS
>>678
>>680 さんが説明してくれてたわ
684
(2): (ブーイモ MM4b-5kBA) 2022/11/14(月)23:32 ID:V3+D5mP/M(1) AAS
>>680
分岐が三種類あるからそれYes/Noの二値しかないbooleanで表せないという話?
だとするとenumにする判断基準としてはちょっと微妙

それにenumを使ったとしてもenumを返す関数に条件記述が移動するだけなら質問者のスパゲッティコードは解消しないので
まずはif (is臨時ダイヤ()){…}のように条件記述を簡潔化することから始めたほうがいいと思う
1-
あと 318 レスあります
スレ情報 赤レス抽出 画像レス抽出 歴の未読スレ

ぬこの手 ぬこTOP 0.019s