コードレビューの「純粋に質問ですが」は「勝手に指摘だと受けとってよくわからん修正するんだろ?質問なんだよ!修正すんなよ!」という意思表示な説
- whappitele
- 161398
- 231
- 259
- 37
コードレビューの「純粋に質問ですが」は表現を柔らかさ目的でなく、「お前は質問だって言わないと勝手に指摘だと受けとってよくわからん修正するんだろ?質問なんだよ!!修正すんなよ!」という意思表示で、むしろ元より殺伐となってる可能性すらあります。
2023-02-13 16:43:45「なんで○○なんですか?」 「修正しました」 twitter.com/irof/status/15…
2022-10-06 08:58:56レビューでコメントしたら「必ず何か変更しなければならない」となってるのはあんまよくなくて、(レビューの位置付けにもよるけど)そう言うもんじゃ無いよーとかは重ねて言わなきゃなんない。
2022-10-06 08:53:36焼肉エンジニア / そけっとちゃん(#ちょうぜつエンジニアめもりーちゃん)エヴァンジェリスト
@a_suenami 「なんで○○なんですか?」という質問を 「○○であることが問題なので、合理的な回答がないなら変更すべき」 と勝手に脳内解読して 「○○使わないようにしました!(ドヤァ」 ってやるの、やりがち 本当に、やりがち……(自戒
2023-02-14 12:51:24「お前のコードの意図がわからん」 「修正しました」 「何でや。さっぱり分からんわ!!!」 twitter.com/a_suenami/stat…
2023-02-14 10:20:59「(これは指摘ではなくて)質問です。」……という言い出し、するなあ。こっちは単にポリシーや背景を聞きたいだけなのに、「修正しました」って返されちゃうもんなあ。 twitter.com/a_suenami/stat…
2023-02-14 08:12:51これ、よく言っちゃうなあ 殺伐となっちゃってるのか。。。 twitter.com/a_suenami/stat…
2023-02-14 12:57:25これはわかります… 「動作として支障はないが、どうしてその実装に至ったのか」を確認者は聞きたいやつ でも回答者はそこで考える→視点が整理される→改良案思いつく→我慢がきかずうっかり直しちゃうのパターンになるのもわかる twitter.com/a_suenami/stat…
2023-02-14 07:58:18怯えられてる人にどんな言葉をかけても... というのはあるでしょうね。 (そうならないようにたち振る舞わないといけないわけですが... twitter.com/a_suenami/stat…
2023-02-14 12:29:15意図を知りたいときには素直に「質問」って言葉を使えばいいと思うし、そうじゃないなら「直した方がいいと思う」って言えばいいし、「純粋に質問」って言ったら純粋じゃない質問もあるのかって話にもなるかもしれないからあんまり飾らないで伝えたらいいのでは〜とは思う。 twitter.com/a_suenami/stat…
2023-02-14 12:51:28これは使う時があるのです。 逆に質問された側ではその指摘を元に修正(改良)案思いついちゃうことはよくある。 twitter.com/a_suenami/stat…
2023-02-13 21:15:48最近は修正して欲しいときは「修正オネシャス」って明言するようにしてる。(質問で修正されないとは言ってない) twitter.com/a_suenami/stat…
2023-02-13 20:29:21request changesを投げてないのに、会話を放棄してアドホックな変更入れたコミットを指摘事項反映とか糞コメントとともに積んで仕事した気になってんじゃねえぞ? ってことなら、なるほど確かに。 twitter.com/a_suenami/stat…
2023-02-13 23:31:47なぜ1を選んだの? じゃあ2番 それなら2とおもった理由は? 3番で ──みたいな。理由を聞かれたら「間違い」とおもう義務教育の敗北を感じる twitter.com/a_suenami/stat…
2023-02-13 21:20:26理由などない、ネットで検索したら落ちてたコードに書かれていた 聞かれたからには求められたものと違うのだろう、ええと、こうかな? ──という読みかたは失礼だろうかなあ……
2023-02-13 21:20:27線形探索を選んだ理由は? インターフェースを使わなかったのはなぜ? いろいろ設計・実装の選択の理由があるはずで、それを聞きたい……のだけれど。ただなんとなくだとか知っているのがそれだけだったとか、理由などない、ないのだ、たぶん……
2023-02-13 21:42:11