コードレビュー時のコメントの意図を明確にする
コードレビューあるあるなんですけど、レビューする時に「これってなんでこうしてるんですか？」みたいなピュアなコメントを書いてしまうと、その意図がレビュイーに伝わらなくて、「詰められてる」「コードに疑念を持たれている」「修正依頼をされている」「純粋に質問されている」という解釈のブレを生んでしまって物事が円滑にいかないみたいなことは発生します。 コメントする側からするとただ単に質問しているだけなのにどうして・・・と思いがちなのですが、コードレビューに慣れていない人はこういう重要度や意図がはっきりしないコメントをたくさん書いてレビュイーを困惑させがちというのはあるんじゃないでしょうか。 重要度や意図がはっきりしないコメントを書くのが常になってしまうと、GitHubでPRを出してレビューをしてもらってマージするという開発者が何度も何度も回していくワークフローに滞りが発生してしまう、もしくはそれどころ
コードレビューの「純粋に質問ですが」は「勝手に指摘だと受けとってよくわからん修正するんだろ？質問なんだよ！修正すんなよ！」という意思表示な説
すえなみ @a_suenami コードレビューの「純粋に質問ですが」は表現を柔らかさ目的でなく、「お前は質問だって言わないと勝手に指摘だと受けとってよくわからん修正するんだろ？質問なんだよ！！修正すんなよ！」という意思表示で、むしろ元より殺伐となってる可能性すらあります。
コードレビューコメントは純粋な質問でも詰問してるように聞こえたり嫌味を言ってるように聞こえたりするのでテキストコミュニケーションは難しい
Junichi Ito (伊藤淳一) @jnchito 「なぜここで1を加算してるんでしょうか？」 とか、 「シャドーイングってご存じですか？」 みたいなコードレビューコメントって、純粋な質問であっても受け取り方によっては詰問してるように聞こえたり、嫌味を言ってるように聞こえたりするので、テキストコミュニケーションって難しい。 Junichi Ito (伊藤淳一) @jnchito コードレビューにおいて疑問文が詰問に聞こえちゃうパターンなので「純粋に質問」を冒頭に付けてみた。 「なんでtrueにしたんですか？」が「普通はやらないでしょ！💢」に聞こえてしまう日本語の難しさよ……。 pic.twitter.com/3hDi6sV6FP twitter.com/jnchito/status…
開発の現場に「WinWinコミュニケーション」が増えると、仕事が楽しくなる
伝え方を変えるだけで、これだけ変わる 先日、以下のツイートをしました。 上記のやりとりは、実際の開発の現場であったコミュニケーションを元に書いています。 Badコミュニケーション 現実の私たちは、下記のようなBadコミュニケーションを取ってしまうこともあるのではないでしょうか。 ×「Hogeがないから、このコードはダメだ。却下」 → ワイのモチベダウン わたしは、このようなコードレビューが苦手で、怖いです。 レビューのコメントってどうしても冷たくみえてしまうし、自分が傷つけられてる感じがするからです。 その結果、どうしても前向きに作業に取り組むことができなくなってしまいます。 気分が落ちたことで、さらにうっかりミスをしたりして、再度レビューで詰められたこともあります。 WinWinコミュニケーション 一方で、以下のやりとりが、WinWinコミュニケーションです。 〇「作業ありがとうございま
気が狂った設計 - hitode909の日記
大きめのこととか，自信のないところを触るときは，コード書く前に，こういう作戦考えてみたけどどうですかって聞いてみたり，こういうことやりたいんだけど一緒に考えませんかって，いっしょに話して設計考えたりするとよいと思う． 一緒に考えたすぐあとに気が狂った設計とか言い出したらおかしいので，未然に変な設計のままコード書いてしまうのを防げる． 特に辛い気持ちになるのが、「気が狂った設計」「クソコード」「（こんな実装は）有り得ない」といった言葉だ。 Pull Requestのレビューが辛くて会社をやめたい 単に言葉が強いのはよくないと思う．我が社にはそんな強い言葉でレビュー書く人はいない． 我が社には，普段から強い言葉を発する人もいなくて，みんな物腰柔らかな変な言葉を話している． 言葉使いや文体は，ずっと過ごしてると同僚から移ったりするので，普段からそういう言葉を話していると，全体の雰囲気も悪くなりそ
1
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
処理を実行中です
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
