エムスリーテックブログエムスリーテックブログ読者になるエムスリーテックブログエムスリー(m3)のエンジニア・開発メンバーによる技術ブログです2018-12-13コードレビューを支える『褒め文化』AdventCalendarコードレビュー、好きですか?エンジニアリンググループの山口です。クラウド電子カルテ「エムスリーデジカル」を開発しています。今回は、チームに根ざしている『褒め文化』についてお話しします。『褒め文化』とは簡単に言えば、コードレビューで褒める文化です。コメントに対してコメントしている様子とても簡単です。簡単なのですが、前職(SIer)ではこういった経験が全く無かったため*1、join直後は(良い意味で)驚いたのが印象に残っています。褒めるけっこう安易に安直に褒められますし、褒めます。思ったことを素直にコメントにしてしまいます。褒め文化の効用ここからは「※個人の感想です」になってしまいますが、こうした褒め文化は、レビュア・レビュイどちらの立場でも非常にありがたい存在です。指摘されることを恐れないレビュイの立場では、レビュー指摘の捉え方が大きく変わりました。これまでは、指摘はある種の減点評価であり、指摘を受けることは即ち品質に問題のあるコードを書いたのだ、という認識でした。指摘ゼロとなるよう念入りに自己レビューし、指摘ゼロを達成して自尊心が満たされる、そんな状況でした。現在は、指摘と褒めを同時に受け取るため、そこそこ書けたけど、より良くする余地があると考えるようになりました。指摘を過度に恐れることはなく、「より良いコードを書き、サービスも自分も成長するためのプロセス」としてレビューを受け止めています。指摘することを恐れないレビュアの立場でも、やはり捉え方が大きく変わりました。レビュー指摘はコードに対する指摘であって、人格否定ではありません。それを理解していても、相手に誤解なく伝えるのは難しいと感じていました。軽微な内容であれば、心の中で(まあいっか...)と指摘を諦めることさえありました。安易で安直な"褒め"が入ると、レビューの雰囲気は変わります。良くも悪くもコードに対して率直な意見を述べる場。これまでレビュー指摘の文言を考えるのに苦労していましたが、不要になりました。褒めも指摘も、シンプルな表現で済ませています。議論、相談、無知を恐れないもっと言うと、レビュアとレビュイは「上下関係」ではなく「対等な関係」という考え方があるように思います。*2例えば、「もっと良い書き方があれば知りたい」と思えば、そこから議論をはじめてより良いコードが生まれることもあります。レビュイのコードを見て、レビュアが新たな知識を得ることもあります。praising!このように『褒め文化』は、心理的に良い影響を与えるとともに、レビューの質も向上させることが出来ます。みなさんもぜひ試してみてください。エムスリーのエンジニアリンググループは、ソフトウェアエンジニアが楽しく働けるような文化が根付いています。カジュアル面談で、気軽にお話ししてみませんか?jobs.m3.com参考デキるプログラマだけが知っているコードレビュー7つの秘訣*1:品質担保のみを目的としていたという背景もありますが*2:チームでレビュアを固定しておらず、誰もがレビュアでありレビュイである、という状況も影響しているかもしれません。読者になる関連記事2023-12-252023年は3段階でシフトチェンジ!2024年はさらに加速してやっていきます!皆さんこんにちは、こんばんは。こんにちは、VPoEの河合(@vaaaaanquish)です。最近遊...2022-03-29トイルを自動化したら、チームにひろまった話こんにちは。Scalaとまどマギのマミさんが大好きな安江です。皆さん、こんにちは、こんばんは。エムスリーで執行役員VPoE兼P...2020-11-20刺激語カードを使ってソフトウェアレビューこんにちは。9注目記事はてなブログをはじめよう!m3techさんは、はてなブログを使っています。ブログを報告する引用をストックしましたストック一覧を見る閉じる引用するにはまずログインしてくださいログイン閉じる引用をストックできませんでした。再度お試しください閉じる限定公開記事のため引用できません。