AIが書いたコードのレビュー

コードレビューとは、書かれたコードを、使う前に別の目で点検することです。AIアシスタントが書いたコードも、自分が書いたものと同じように、レビューしてから使います。

動いて見えるコードにも、条件の抜けや、危ない書き方が混ざることがあります。

レビューは3つの手順で進めます。まず読んで、1行ずつ何をしているかを説明します。次に動かして、実際の答えを見ます。最後に、なぜこの書き方か、ほかの場合はどうかを問います。

読むだけでも、動かすだけでも、見落としが出ます。

読むときは、まず関数の入口を探すと、全体の形がつかめます。次の Run は、ラボのプロジェクトで function という言葉を含む行を、行番号つきで探します。

最初の状態では、3つの関数の行が出ます。中身を読むのは、そのあとです。

docker exec -w /home/node/project agent grep -n function shop.js

例外を全部飲み込む書き方は、失敗を隠します。次のコードは、文字が入ってもエラーにならず、0 を返してしまいます。

年齢が 0 のデータが静かに混ざり、あとで見つけにくい誤りになります。

def read_age(text):
    try:
        return int(text)
    except:
        return 0

# 文字が入っても、エラーになりません。
print(read_age("20"))
print(read_age("二十歳"))

# except: を except ValueError: に変えて、もう一度実行します。