コードレビュー時のコメントの意図を明確にする

2023/02/15 29 users 意図 疑念 レビュイー プレ コメント

コードレビューあるあるなんですけど、レビューする時に「これってなんでこうしてるんですか？」みたいなピュアなコメントを書いてしまうと、その意図がレビュイーに伝わらなくて、「詰められてる」「コードに疑念を持たれている」「修正依頼をされている」「純粋に質問されている」という解釈のブレを生んでしまって物... 続きを読む

レビュー文化の会社で、楽に生きられる思考法 - ビープラウド社長のブログ

2018/03/19 43 users ビープラウド社長 レビュイー レビュアー 思考法 人格攻撃

2018 - 03 - 19 レビュー文化の会社で、楽に生きられる思考法 仕事論 「コードを憎んで人を憎まず」 コードレビュー時のレビュアー（レビューする側）の考え方（レビュー時に人格攻撃にならない、感じられないように気をつけるなど）として、良く使われる言葉です。 一方で、レビュイー（レビューを受ける側）はどのように考えればよいでしょうか。 マシンやプログラムからエラーメッセージが出ることを 「怒... 続きを読む

悪いコードを憎んで人を憎まず！ プルリク送付前に心がけたいコードレビューのコミュニケーション術 - エンジニアHub｜若手Webエンジニアのキャリアを考える！

2018/01/24 826 users エンジニアHub GitHub jun 若手Webエンジニア

2018 - 01 - 24 悪いコードを憎んで人を憎まず！ プルリク送付前に心がけたいコードレビューのコミュニケーション術 コードレビューを円滑に進め、より学びを促進するために重要な「コードレビュー時のコミュニケーション」について、現役エンジニア・池田 惇さんの経験とともに考えてみます。 GitHub 池田惇 コードレビュー Tweet アプリエンジニアの池田 惇 （いけだ・じゅん／ @jun_... 続きを読む

ソシャゲエンジニアの自分がコードレビュー時に重視する箇所32選 - Qiita

2016/04/07 672 users Qiita ソシャゲエンジニア 自分

コードレビューで土日に安寧を ソーシャルゲームは、ユーザアクセス集中と、それに伴うユーザデータ増加によって劇的に負荷が上がり、（主に土日に）サービスに影響を与えがちです。 問題があるコードは、たとえ負荷テストを行っても、作成したシナリオによっては見つけられない可能性もあります。 そういった見えない不安を払拭するという意味でも、コードレビューは重要だと思っています。 【ステキポイント】 ・ ソースを... 続きを読む