うちのチーム、最近コードレビューが厳しくなってきたんですけど、毎回「この変数名のほうが読みやすい」「この関数は短くしたほうがいい」みたいな指摘ばっかり受けるんです。個人的には機能としては動いてるし、実装の選択肢もいくつかあると思ってるんですよね。 もちろんコード品質って大事なのは理解してるんですが、レビュアーによって「こういうスタイルが好き」の基準がバラバラな気がして。これって単純に「ガイドラインを厳密に守ろう」って話なのか、それとも「チーム内で美学を統一しようぜ」って段階の話なのか、自分の中で整理がつかなくて。 実務上、バグ防止とか保守性に直結する指摘と、「まあこっちのが好みだけど、その方法でもいい」って指摘を、どう見分けるべきなんでしょう?スタイル系の指摘を受けたときって、素直に従うべき?それとも提案の根拠を聞き返すべき?
フォロー 2・閲覧 0
まだ観点がありません。最初の観点を灯しませんか。