猫でもわかるWeb開発・プログラミング

本業エンジニアリングマネージャー。副業Webエンジニア。Web開発のヒントや、副業、日常生活のことを書きます。

『ある日のコードレビュー』匿名ダイアリーについて

以下のはてな匿名ダイアリーが有名になっている。

anond.hatelabo.jp

俺(レビュアー)「ここはMath.minを使えば1行で書けるのでそうしてください」

レビュイー「AIがこのように生成しました」

俺「(だから何だよ…と思いつつ)盲目的にAIに従うのではなく、それを受け入れた以上コードの責任はあなたが持ってください」

レビュイー「しかしこの先コードを読むのもAIになってきている以上、AIが生成するコードはAIにとって読みやすい良いコードなんじゃないでしょうか?そもそもですが前から増田先輩は小さいことにこだわってapproveを遅らせすぎです。さっさとリリースしたほうが顧客価値に繋がるのでは?」

30分ぐらい悩んだが反論できず「確かにそういう考えもありますね」って言ってapproveした。

僕の意見

俺さんの引き際はいい引き際だと思う。私がレビューしててもこの指摘コメントはしそうだけど、「変える気無いです」と言われたら諦める。そこまでこだわる部分では無いから。

「しかしこの先コードを読むのもAIになってきている以上、AIが生成するコードはAIにとって読みやすい良いコードなんじゃないでしょうか?」という部分については疑問がある。

AIが生成するからといって、AIが読みやすいとは限らないとも思うが、そもそも、プログラミング言語は、同じものを何回動かしても基本的には同じ動きをするので、そういったものを解釈するのは AI の得意分野なのでは無いだろうか。なので、AIが読みづらいコードはあまりないのでは?というのが私の思うところである。

個人でやるのであればこういった細かいところを気にしても良いが、チームで仕事をする場合、多少思想が違った人が混ざるもので、全部を自分の思い通りにすることは不可能なので、致命的ではない範囲で受け入れていく必要がある。