Geek』の書評でも書いたんだけど、コードレビューのときはHRTの精神を大事にしたい。HRT?HRTとは下記の3つの精神のことだ。阻害はHRT精神の欠如によって起こる。常に仲間に対して謙虚、尊敬、信頼の念を持とうな。2019以下、アンチパターンなコードレビューのシーンを先輩(👿)と後輩(👼)でお送りする。説明コードレビューじゃなくとも、テキスト・コミュニケーションであれば手段を問わず発生しうる問題。文字だけのコミュニケーションは、発言の背後にあるコンテキストや感情が汲み取りにくい。指摘して直してもらおう😃」なのかがわからないので、先輩・後輩という関係性を前提とした場合、後輩からは先輩が怒っていると思われていても仕方ない状況と言える。先輩の他の指摘に関しても同様に、「こう書くべき💢」なのか「個人的にこう書くべきだと思っているんだけどなぁ...🤔」なのかわからないし、「どうしてこんなひどいコード書いた?💢」なのか「このコードの意図が知りたいなぁ、どうしてこう書いたんだろう?🤔」なのかがわからない。👼「あー、そこは〇〇〇で、△△△だからそう書いたんですよね。説明コードレビューコメントに指示はあるが、理由がないケース。上記のケースだと「なぜ残念なコードと言えるのか?」というポイントが欠如している。コードに対する価値観が完全に合致していれば、「残念なコード」でレビュイーに伝わるが、大抵の場合はそうではないので理由がないと伝わらないケースが多い。レビュイーがジュニアレベルであればコード良し悪しの分別がつかないのでなおさら伝わらない。説明コードに対する誹謗中傷。これを乱用すると現場が殺伐とする。「クソコード」などのコードの誹謗中傷となるような言い回しは使わない「あまり強い言葉を遣うなよ。口頭でコミュニケーションするクソコードのような強い言葉を使いたくなるような場面は感情が昂ぶっているときであり、テキスト・コミュニケーションだとすれ違いが起きる可能性が高い。👿「これは今後メンテしていくのがとても辛いコードになりそうです。〇〇だし、△△ともなり、××だからです。一連のコード変更に対して大量のレビューコメントが付く事象のこと。シニアエンジニアがジュニアエンジニアのコードをレビューするときなど、レビュワー・レビュイー間のスキル・知識・経験に大きな差があるときに陥りやすい。大量のコメントを付ける/付けられることになるので、レビュワーにとってもレビュイーにとっても心的労力が高い。1つ1つのコメントがどれだけ丁寧だとしても、その数が多くなるとレビュイーに無力感を味あわせてしまうことになり心理的ストレスに繋がる可能性がある。可能な限り分割し1つ1つを小さい差分に保つコードレビューの代わりにペアプログラミングをするコードレビューが大変ならコードレビューをしない。ペアプロでその場で一緒に解決させるモブプログラミングでもOK研修の実施および内容見直しスキルに大きな差があることが原因なので、その差を埋めるための研修を実施もし新入社員向けに研修がないのであれば、新入社員を対象に技術研修を新たに計画すべきかもしれない。👼「なるほど、やってみます。👨「僕はそうは思いません。説明1つのコードレビューに端を発して長大な議論をスタートさせること。これが始まると一つのコメントに対してのスレッドがどんどん伸びていくことになる。設計などの大枠なトピックだと、長大な議論になりやすい。👨「僕はそうは思いません。答えはチームの心理的安全性を確保するためだ。全ては心理的安全性のため最近では共通認識になりつつあるので改めて言う必要はないかもしれないが、ハイパフォーマンスなチームにとって心理的安全性は極めて重要だ。それはGoogleの調査によって証明されている。つの力学のうち、圧倒的に重要なのが心理的安全性です。倍多い、という特徴がありました。「効果的なチームとは何か」を知るモヒカン族の方々は下記のように考えるかもしれない。「何を言ったってそれはコードに対する批判であって書いた人への批判ではない。「間違ったことは言っていない。気持ちはわからなくはないが、残念ながら我々は人間である。人間であるということは感情があるということ。そこに違和感があった場合、遠慮なく異を唱えることができるだろうか?「心理的安全性を気にしすぎて何も言えない」のではなく、お互いが年齢・ジェンダー・人種の区別なくフラットな関係性で何でも言い合えるようにするためのHRTであり、そのための心理的安全性である。そこは勘違いしてはいけない。チーム中にある個人の関係において、「様々な形で課題や問題についての提起がされる」ということに他なりません。(...中略...)つまるところ、心理的安全性が高いとは、「些細な問題であっても提起される」「多く問題に対して自己主張がなされる」という観測可能なチームの状態を意味しています。カナダ・バンクーバーでの修行時代を経て日本帰国。Railsと奮闘する日々。ブログでは技術ネタをよく書いています。