ここだけ全員焼肉店（焼肉の夏、日本の夏） 793店舗目
↓ 1- 覧 板 20
39:参代目 ﾄｩﾙｰﾃﾞ、ｻｰﾆｬ、ﾍﾙﾏ SW勢192人 ﾗﾝｽ勢6人 SU84　店員299+108[sage]
2020/10/16(金) 21:01:08.22 ID:L6CZK5ZU0
ええんやで

40:エグゼリカ[sage]
2020/10/16(金) 23:12:52.78 ID:YrbNWS7PO
人に教えるって…大変だね…(コードレビュー…)

41:さとりん[sage]
2020/10/17(土) 00:14:01.17 ID:i6AXUcPqO
>>40
人に教えるって大変だね…
(製造経験ほぼゼロの奴に教えるとかいう)

42:さとりん[sage]
2020/10/17(土) 00:15:30.91 ID:i6AXUcPqO
コードレビューは基本的にコーディング規約と違うものは指摘、他は好みじゃないって感じに伝えてるなぁ。
あとは不要な広いスコープの抑制、長すぎるメソッドの分割、重複する処理をまとめるよう伝える、とかかなぁ

43:さとりん[sage]
2020/10/17(土) 00:19:58.79 ID:i6AXUcPqO
ヘボン式の徹底、スペルミスの対策
後はそのシステムのコード特有の癖とあってるかの確認とかかなぁ
主に略称。例えば加入日がjoinDateなのかjoinDayなのかkanyuHiなのかkanyuBiなのか、とか

44:さとりん[sage]
2020/10/17(土) 00:21:22.58 ID:i6AXUcPqO
自分の場合は、これが美しいよりはこれはマジで許せない改修する時にそのコードみたら書いた奴ぶっ飛ばすってなるのは何かな、で見るなぁ

(自分の場合は趣味のコードと仕事のコードで評価軸変えてるし)

45:エグゼリカ[sage]
2020/10/17(土) 01:41:48.72 ID:1cwd6BvF0
あるところにとってもDDD意識した設計されてるシステムがありました
エンジニア歴3年の人が上げてきたコードの話

・そのエンティティにまるで関連性のない、そこまで責任を及ばせるべきでないドメインのメソッド実装しようとしてるんだけどこれどうしましょう
・思いっきりレイヤー構造無視されてるんだけど、リクエスト受け取った直後のAPIのトップレベルのコントローラ層から直でリポジトリ呼ぼうとしてるんですけどこれどうしましょう
以下略 AAS


46:エグゼリカ[sage]
2020/10/17(土) 01:44:43.10 ID:1cwd6BvF0
で、俺がそれ見て何から突っ込んだらいいの…て困惑してるとこにその一緒にレビューに入ってくれた人が
ここ違う、ここダメ、ここにこのロジック書かないでとものすごい勢いでコメントしまくってて、もう俺いらないじゃんってね
まったくなんでこの人このコードをこの短時間で理解して次々的確にコメントできんのかと

つくづく頭の出来が違いすぎると久々に無力感を感じたお話

47:さとりん[sage]
2020/10/17(土) 01:54:58.18 ID:i6AXUcPqO
あるある

48:さとりん[sage]
2020/10/17(土) 01:55:27.07 ID:i6AXUcPqO
マジでウチのくそソース読ませたいわ

常識に捕らわれると死ぬ

49:エグゼリカ[sage]
2020/10/17(土) 02:06:58.85 ID:1cwd6BvF0
最近C#あんま触ってないからなー
読めるかなー

1002Res/1939.58 KB
↑[8] 前[4] 次[6] 書[5] 板[3] 1-[1] l20

VIPサービス増築中！
携帯うpろだ|隙間うpろだ
Powered By VIPservice