過去ログ - ここだけ全員焼肉店(焼肉の夏、日本の夏) 793店舗目
↓
1-
覧
板
20
45
:
エグゼリカ
[sage]
2020/10/17(土) 01:41:48.72 ID:1cwd6BvF0
あるところにとってもDDD意識した設計されてるシステムがありました
エンジニア歴3年の人が上げてきたコードの話
・そのエンティティにまるで関連性のない、そこまで責任を及ばせるべきでないドメインのメソッド実装しようとしてるんだけどこれどうしましょう
・思いっきりレイヤー構造無視されてるんだけど、リクエスト受け取った直後のAPIのトップレベルのコントローラ層から直でリポジトリ呼ぼうとしてるんですけどこれどうしましょう
・ていうかなんでトップレベル層に業務ロジックずらずら書いてんすか、他のレイヤー無視ですか、既存のAPIと見比べて違和感覚えないんですか、どうしましょうこれ
・お願い、大した理由もなくむやみにforを3重にも4重にもすんのやめて!リストから特定条件抽出するだけならstream使って!あぁ…ループの最奥で繰り返し同じSQL発行してることにも気づいてないな、どうしましょう
・Javaでは変数の先頭は小文字です ← 一緒にレビューに入ってくれたVPoEの原文ママ
<<前のレス[*]
|
次のレス[#]>>
1002Res/1939.58 KB
↑[8]
前[4]
次[6]
板[3]
1-[1]
l20
このスレッドは過去ログ倉庫に格納されています。
もう書き込みできません。
過去ログ - ここだけ全員焼肉店(焼肉の夏、日本の夏) 793店舗目 -パー速VIP http://ex14.vip2ch.com/test/read.cgi/part4vip/kako/1596710681/
VIPサービス増築中!
携帯うpろだ
|
隙間うpろだ
Powered By
VIPservice