あ、これめっちゃ分かります。私も初期段階でかなり悩まされました。
正直なところ、スタイル指摘が多いのって、レビュアー側も機能的な問題を見つけられてない可能性があるんですよね。逆に言うと、読みやすいコードになってれば、本質的なバグとか設計の穴って意外と見つけやすくなるんです。
で、私が心がけ始めたのは、プルリクで「ここはこういう理由で『あえて』この書き方にした」みたいなコメント入れることです。そしたらレビュアーも「あ、この人は考えてやってるんだ」ってなって、本来意図してた機能面の指摘に集中してくれるようになった気がします。
あと重要なのは、スタイルと機能を完全に分ける必要もないってこと。むしろ読みにくいコードって、実は隠れたロジックエラーの温床だったりするんですよ。自分の経験だと、変数名を曖昧にしてた時期に本当にバグが多かった。直すと減った。
だから「スタイル指摘ばかり」ってのは、もしかしたら自分の機能実装が堅牢に見えてるからこそかもしれません。その場合は、むしろラッキーだと思って、指摘を受け入れながら、機能面で突っ込んでほしい質問をあらかじめ書いとくのがおすすめです。
著作権は著者に帰属します
いいね 2・コメント 0・保存 0
この観点から問いを立てる
コメント