問題はこっちだと思う。前者は多かれ少なかれどこにでもあるものだが、後者はその開発スタイル自体がすでに割れ窓に近い。
ぐちゃぐちゃなコードベースができてしまった場合、影響範囲を局所化して、確実に安全だと分かる小さな変更を積み上げながら直していくように移行ステップを立てるのがセオリーだ。延期できないバグ修正や機能追加ならまだ理解できるが、割れ窓直し程度のことでリスクのある変更を日常的に繰り返させる体制はやばい。実際に事故ってるし。
今回のケースは他人が埋めた地雷を踏んだようなものだから、気に病みすぎることはない。
ただ、レビュアーがバグチェックをしてるとしても、実装者としての責任を持つ気持ちはあったほうがいいと思う。責任を自信と言い換えてもいいけど、自分が手を動かした仕事に責任を持てるように動いていかないと、結果として遠回りになる。
その辺りでようやくうっすら気づいてきた。コードレビューはデグレが起きないようにしてくれるものじゃないって。コードにより引き起こされるバグの責任はレビュアーじゃなくて俺にあるんだって。
たとえばこのあたり、メンバーの中でコードレビューの目的が共有できてないのは組織の落ち度もあるけど、何かずれてるかもと思った時点で自分からすり合わせしたほうがいい。
無職が転生してプログラマになったんですよ。運よく雇ってくれる会社が見つかった。 それまで一人でしかプログラムを書いたことがなかったから、コードレビューをしてもらえるとい...
本当の問題はコードレビューじゃないな。 diffを読んでも影響範囲が分からなくて、デグレがあるのかないのか判断できないようなコードベース そこに新人を無策で投入して、不安...
ただ、レビュアーがバグチェックをしてるとしても、実装者としての責任を持つ気持ちはあったほうがいいと思う。責任を自信と言い換えてもいいけど、自分が手を動かした仕事に責...
デグレって何?
それまで動いていた機能が動かなくなること。
幼女戦記の主人公
無職転生がかるてっとに参加するってことかな
デーモン小倉の略
個人の努力であらゆる不具合を防止できると思うのがそもそもの誤解 銀行だって暗号化プログラムだってしょっちゅう不具合出してる 不具合はあるものとして受容するしかない
とすると、不具合を出してもメンタル折れない程度にはメンタル強いのがプログラマの適性なんかな?
重要な修正箇所からは全速力で逃げる能力
そのコードはなかなか大規模なもので、その全てを把握するなんて到底無理なものだった。 いやいや、理解しろや。保守担当が理解せんで誰が理解するねん。 レビュアーに責任なしっ...
確かに保守担当だった俺が把握してなかったらというのはあるが、俺よりもずっと長い間件のコードに関わってたはずのレビュアーも全貌を把握し切れていなかったと思われる時点でち...
IEEEがコードレビューに関する論文を出していて、それを読む限りではコードレビューはバグの検出にはそれほど効果がないとのことでした。 (品質の維持/向上やレビュー者間へのシス...
どの論文?
これやで https://www.microsoft.com/en-us/research/publication/code-reviews-do-not-find-bugs-how-the-current-code-review-best-practice-slows-us-down/
ありー
新人がデグレさせるのを止めるのもコードレビューの役目だぞ。 筆者のプロジェクトの詳しい状況は知らんが、恐らくドキュメントどころかテストコードすらまともに書かれていないん...
いや、単体テストはそこそこ書かれてたし、バグ修正の際はリグレッションテストもちゃんと書いてた。 だから全てのテストも通ったからヨシ!的な流れでテストに含まれていなかった...
デグレって初めて聞いたわ 最近使われてる言葉?
アップグレードの逆でしょ