AIに書かせたコードで、PRを出す。
しばらくすると、レビューのコメントが付く。
「ここ、なんでこう書いたの？」
その瞬間、頭が真っ白になる。
で、とりあえず返す。
「一旦、調べさせてください」
でも、調べても答えが出てこない。
エラーの直し方なら、AIに聞けばすぐに返ってくるのに。
自分が何も決めずに書かせたコードだと、AIに聞き直しても
返ってくるのはAIの説明で、自分の説明にはならないんですよね。
で、気づいたら夕方。
返事はまだ出せていない。
これが何回か続くと、レビューする側からはこう見え始めます。
この人、AIが書いたものを、そのまま貼って出してるだけなんじゃないか。
そう思われると、PRを出すたびに細かく見られて、大事なタスクは任せてもらえないままです。
「一旦調べます」になる人は、2種類いる
「一旦調べます」と返してしまう人は、2種類に分かれます。
1つは、そもそもデータの流れや処理の流れが分かっていない人。
既存の実装がどうなっていて、だからこう書いた、という意図がないまま
AIが書いたものを出しています。
なので、聞かれても答えようがありません。
そしてもう1つは、自分なりに既存のコードを読んで、理解したうえで出している人。
問題ないと思って出しているのに、その場で答えて、怒られたりキツく言われたりするのが怖い。
だから、この人が何を気にしてコメントしているのかを
あとでゆっくり考えてから、ちゃんと返したい。
それで「一旦調べます」と言ってしまう。
「なんでこう書いたの？」が聞いていること
じゃあ、レビューの「なんでこう書いたの？」は、何を聞いているのか。
それは、「どういう意図があって、こう書いたの？」ってことです。
たとえば、今回のタスクだけならこう書けばいい。
でも、ほかの場所でも同じ処理を使っていたから共通化した、とか。
他にも、今はFirestoreを使っているけど、そのうちRDBに移行する予定がある。
だから、すぐRDBに切り替えられるようにデータを読み書きする所だけ分けて書いた、とか。
こういう、いろんな意図があると思うんですよ。
レビューで聞かれているのは、それです。
要は、責めているわけじゃなくて、考え方を聞きたい。
コミュニケーションを取って議論して、この仕事をちゃんと終わらせて、システムを不具合なくどんどん良くしていきたい。
だから二人で議論して、抜け漏れをチェックしたり、いろんな視点で見たりして
いい修正にしていきたい。
少なくとも僕がレビューする側の時は、こう思ってるわけですよ。
だからこそ、自分の意見が大事になるんです。
技術を調べても答えにならないのは、そのためです。
答えはAIの説明でもネットの記事でもなく、自分がどう考えてこう書いたか、なので。
まあ、単純にムカついて「こいつダメだな、どうせ何も考えてないんだろ？」と決めつけて言っている人も、中にはいるかもしれません。
それは、こっちからは分かりません。
でも、どっちだったとしても返すことは同じで、自分がどういう意図で書いたかです。
ただ、レビューする側の書き方も悪いんですよね。
「なんでこう書いたの？」だけだと、責められている気がするので。
本当は
「ここって、こういう意図があって書いたのかなと思ったけど
僕はこう思うから、こう書いた方がいいと思うんだけど、どう？」
って書くべきだと思います。
とはいえ、相手を変えるのはすごく大変なので、こっちが変わるしかないです。
僕がAIのコードを出す前に決めていること
僕も、現場でAIはゴリゴリ使っています。
ただ、AIが書いたコードは必ず自分で読みます。
AIが書いたコードでも、PRを出すのは自分なので、意図を聞かれるのも自分なんですよね。
だから、「なぜこう書いたか」を説明できない状態では、レビューに出しません。
PRの説明をAIに書かせてもいい。
でも、そこに書かれた意図が自分の考えと合っているかは、自分で読まないと分からない。
説明できる状態で出していれば、聞かれて答えようがない、ということは起きません。
怖くて返せない人は、どういう意図で書いたかが、もう自分の中にあるはずです。
だったら、その意図をそのまま、その場で一言返せばいい。
「ほかにも同じ処理があったので、共通化しました」
こんなふうに意図を一言返すだけで、相手が気にしている所が違っていても、そこから話が進むんですよ。
ただ、ここまで読んで
「いや、AIが書いたコードに、自分の意図なんてないんだけど」
と思った人もいると思います。
正直、それが一番まずい状態です。
意図を持つには、まず今のコードがどう動いているかを知っている必要があります。
たとえば「共通化しました」と言えるのは、同じような処理がほかの場所にもあると知っているからです。
今のコードがどう動いているかを知らないまま出している限り、何回聞かれても「一旦調べます」から抜け出せません。
答えられるようになると、レビューの空気が変わる
で、ここに答えられるようになると、空気が変わります。
「この人が出すPRなら大丈夫」と思われると、レビューも早く通る。
そのうち、作る前の段階で「これ、どう作ったらいいと思う？」と相談されるようになります。
で、さっきの「今のコードがどう動いているか」ですが、処理の流れをAIに追わせるのは全然いいです。
ただ、その説明が本当にコードどおりかは、自分で確かめないと分からない。
とはいえ、データの流れや処理の流れを、どこからどう追えばいいのかを自分だけで判断するのは難しいですよね。
それに、意図を一言返すにも、相手が何を気にしてコメントしているのかが読めないと
返した一言がズレてしまう。
分からないまま出し続けると、レビューのたびに同じところで止まって、任される仕事も変わらないままです。
既存のコードのどこから、どの順番で読めば、「なんでこう書いたか」を自分の言葉で言えるようになるのか。
レビューの「なんで？」に、何をどう返せば話が前に進むのか。
既存のコードを読む手順と、レビューへの返し方。この2つは、メルマガで紹介している教材で扱っています。
コメント