2023-05-23

anond:20230522093152

本当の問題コードレビューじゃないな。

問題はこっちだと思う。前者は多かれ少なかれどこにでもあるものだが、後者はその開発スタイル自体がすでに割れ窓に近い。

ぐちゃぐちゃなコードベースができてしまった場合、影響範囲局所化して、確実に安全だと分かる小さな変更を積み上げながら直していくように移行ステップを立てるのがセオリーだ。延期できないバグ修正機能追加ならまだ理解できるが、割れ窓直し程度のことでリスクのある変更を日常的に繰り返させる体制やばい。実際に事故ってるし。

今回のケースは他人が埋めた地雷を踏んだようなものから、気に病みすぎることはない。

ただ、レビュアーバグチェックをしてるとしても、実装者としての責任を持つ気持ちはあったほうがいいと思う。責任を自信と言い換えてもいいけど、自分が手を動かした仕事責任を持てるように動いていかないと、結果として遠回りになる。

その辺りでようやくうっすら気づいてきた。コードレビューデグレが起きないようにしてくれるものじゃないって。コードにより引き起こされるバグ責任レビュアーじゃなくて俺にあるんだって

たとえばこのあたり、メンバーの中でコードレビュー目的が共有できてないのは組織の落ち度もあるけど、何かずれてるかもと思った時点で自分からすり合わせしたほうがいい。

記事への反応 -
  • 無職が転生してプログラマになったんですよ。運よく雇ってくれる会社が見つかった。 それまで一人でしかプログラムを書いたことがなかったから、コードレビューをしてもらえるとい...

    • 本当の問題はコードレビューじゃないな。 diffを読んでも影響範囲が分からなくて、デグレがあるのかないのか判断できないようなコードベース そこに新人を無策で投入して、不安...

      • ただ、レビュアーがバグチェックをしてるとしても、実装者としての責任を持つ気持ちはあったほうがいいと思う。責任を自信と言い換えてもいいけど、自分が手を動かした仕事に責...

    • デグレって何?

    • 個人の努力であらゆる不具合を防止できると思うのがそもそもの誤解 銀行だって暗号化プログラムだってしょっちゅう不具合出してる 不具合はあるものとして受容するしかない

      • とすると、不具合を出してもメンタル折れない程度にはメンタル強いのがプログラマの適性なんかな?

    • そのコードはなかなか大規模なもので、その全てを把握するなんて到底無理なものだった。 いやいや、理解しろや。保守担当が理解せんで誰が理解するねん。 レビュアーに責任なしっ...

      • 確かに保守担当だった俺が把握してなかったらというのはあるが、俺よりもずっと長い間件のコードに関わってたはずのレビュアーも全貌を把握し切れていなかったと思われる時点でち...

        • IEEEがコードレビューに関する論文を出していて、それを読む限りではコードレビューはバグの検出にはそれほど効果がないとのことでした。 (品質の維持/向上やレビュー者間へのシス...

    • 新人がデグレさせるのを止めるのもコードレビューの役目だぞ。 筆者のプロジェクトの詳しい状況は知らんが、恐らくドキュメントどころかテストコードすらまともに書かれていないん...

      • いや、単体テストはそこそこ書かれてたし、バグ修正の際はリグレッションテストもちゃんと書いてた。 だから全てのテストも通ったからヨシ!的な流れでテストに含まれていなかった...

    • デグレって初めて聞いたわ 最近使われてる言葉?

記事への反応(ブックマークコメント)

ログイン ユーザー登録
ようこそ ゲスト さん