コードの健全性: 礼儀正しいレビュー == 役立つレビュー

.app 1 .dev 1 #11WeeksOfAndroid 13 #11WeeksOfAndroid Android TV 1 #Android11 3 #DevFest16 1 #DevFest17 1 #DevFest18 1 #DevFest19 1 #DevFest20 1 #DevFest21 1 #DevFest22 1 #hack4jp 3 11 weeks of Android 2 A MESSAGE FROM OUR CEO 1 A/B Testing 1 A4A 4 Accelerator 6 Accessibility 1 accuracy 1 Actions on Google 16 Activation Atlas 1 address validation API 1 Addy Osmani 1 ADK 2 AdMob 32 Ads 71 Ads API 10

[newnakashima]

コードの質が低すぎるというか、どう見てもテキトーに書いただろコレというコードに出会うとキツい表現になってしまうんだよな。。。レビューのときは菩薩にならないと。

[gagegogi]

大事

[hatecafe]

画面がはみ出て隠れてて縮小もできなくて、読めない。。iPhoneSE。。/直ってた。読む、

[nemusg]

そもそもの人としての最低限レベルの話なんだけどコードレビューになると上手くできない人もいるにはいる

[siguma_sig]

重要度をprefixにつけるのはたしかにありだなー

[Kwappa]

礼節の話だ

[takepon0928]

大事。。。

[modal_soul]

感情的でネガティブな表現は、相手に対するネガティブな印象を自分に刷り込んでしまうから、慎重に推敲するよう心がけてる。敬意を払い、謙虚に振る舞え。

[ytRino]

日本語版来てた

[mohno]

（“コード”レビューはほとんどしないが）「きつい表現だったりすると、好ましくない結果が生まれる場合」←耳が痛い。でも、“規格と違う”と言ってるのに直さなかったり、具体的に修正文を寄越せとか言うんだぜ。

[manhole]

「べからず」がありがたい

[mux03]

言い方重要．どういうケースだろうが，受け取る人の気持ちになって，有益なコメントかどうかの判断が必要

[nishiotsutomu]

コードレビューというかコミュニケーションの基本的な立ち位置の話だった

[tossy_yukky]

人間としてみたいな話だった

[yuki_2021]

コードレビューもコミュニケーションが大事よね。感情的になったらいけないわ。

[RabbitBit]

コードレビューは、職人の視点で行うと感情的になるので、コンパイラーのような中立的で客観的な視点を意識する必要があるのかなと思っている。

[mztnex]

当然といえば当然かもだけど、googleでもこういう点を気遣ってるんだな（あるいは気遣わなければいけない、か）

[dahiyu]

参考にする

[Soraneko]

👀

[nigohiroki]

具体的じゃないレビューはやめろ！というキツいレビューをしたことあるので反省。

[ryuichi1208]

「ここ妥協しちゃうの？w」っていうレビューコメント貰ったことあるなー🤣新人たる所以かな。

[hirata_yasuyuki]

正しさしかない。

[tettekete37564]

今日。レビュイーの書いたクラス〇〇のインスタンス△△があり定義の見つからないxxというメソッドのコールが書かれている。このxxというのは何をコールしてますか？の質問に〇〇をコールしてます。という答え・・・

[clash_m45]

良い記事

[hiro14aki]

シンプルで良い。

[yoiIT]

これはコードレビューに限らず、提案資料やリスクレビューなど、あらゆるレビューに言えること。

[ono_matope]

簡潔でいいな。私もコードレビューでは主語 you を避けて we を好んで使う。 > （例:「あなた」や「あなたの」）が含まれているだけでも、コードを改善するという目的の妨げになることがあります。

[maecchi]

“具体的で実現可能なフィードバックを提供する”って意外と難しいんですよね。ただ礼儀正しいレビューだと実装者の心理的安全性も増すのでできる限り心掛けたい

[peketamin]

説明力が鍛えられる。レビューは会話なので「なんで？」みたいな突き放したコメントはダメ絶対。

[Sei]

コードに限らずだわな、よくさせたいと思うなら伝わりやすく考慮するというそれだけの話だと思う。

[twol]

やさしいせかい

[hylom]

先日Twitterでも話題になっていたコードレビューの話。

[csouls]

"Unless you make me? :)" はどう見ても挑発なので、良い例えだ

[katsyoshi]

なるほどな

[ShinSemiya]

「礼儀正しいレビュワー（なぜか最新のロシア製装備を持っている）」