パスワードを忘れた? アカウント作成
この議論は賞味期限が切れたので、アーカイブ化されています。 新たにコメントを付けることはできません。

緊急地震速報の「誤報」はプログラム中の「単位」間違えが原因と判明」記事へのコメント

  • いや単純に、各種試験の検証漏れだと思いますけど。
    改修対象外機能のモジュールがリリースされてることに気がつけないあたりは、受け入れ担当のボーンヘッドもあるかもしれませんが。
    • 検証以前 (スコア:1, すばらしい洞察)

      by Anonymous Coward on 2009年08月26日 14時13分 (#1629484)
      開発側が「善意」で仕様にないことを追加して、そこがバグってたって話でしょ?
      そういうことを(する/できる)体制ってどうなのよ
      親コメント
      • by goji (949) on 2009年08月26日 17時04分 (#1629571) ホームページ 日記
        前任者の汚いけど良くテストされたコードをリファクタリングしちゃいました!!
        親コメント
        • by Yoh2 (6924) <yoh2@d2.dion.ne.jp> on 2009年08月27日 0時34分 (#1629761) 日記

          前任者の汚いけど良くテストされたコード

          1. 当初はミリメートル単位の値を扱うつもりでコードが書かれていた。
          2. テストしたらバグ発覚。
          3. 調査してみると、バグが潜んでいたのは1の部分ではなくその周辺だった。
          4. ところが、そこにミリメートル単位の値の代わりにマイクロメートル単位の値を入力すると、全体として期待通りに動くことが判明。周辺の根本的なバグを修正するより対処が簡単だったため、ここにマイクロメートル単位の値を入力よう変更することで修正完了とした。
          5. ところが、変数名、関数名、コメント等はミリメートルのまま残してしまった。
          6. 今回、当該コードを読んだ担当者が、「うわバグじゃん」と、ミリメートル単位の値を入力するように「修正」してしまった。
          7. 問題発生。

          というストーリーを妄想してしまいました。

          --
          巧妙に潜伏したバグは心霊現象と区別が付かない。
          親コメント
      • by Anonymous Coward
        問題「内緒で改修されたものをどう検出するか?」

        これ、結構難しいと思うなぁ。今回のは修正後のテストに含まれない筈の話だから。
        どんな修正でも全部テストするというのは非現実的だろうしさ。(コスト度外視はもってのほか)
        • by mogya (27057) on 2009年08月26日 19時42分 (#1629645) ホームページ

          変更点についてコードレビューを経ないとコミットできないというルールでどうでしょう?
          単位が違うくらいならコードレビューで落とせそうな気がします。

          #そうすると今度は、レビューしたのと違うコードをコミットしてしまうミスとかが発生するんですけどね。

          親コメント
          • by Anonymous Coward
            コミットしてからレビューすれば?
            そうすれば差分の情報がちゃんと残るから、こっそり改変は無理でしょう
          • by Anonymous Coward

            問題は委託側がコードレビューできるか、によると思います。
            開発側に任せては意味がないです。

            たしかに"単位が違うくらい"なら分かる可能性もありますが、大概無理ではないでしょうか。
            単位だけを直したわけじゃないでしょうし。

          • by Anonymous Coward

            それは、プロモーション管理のやり方が間違っているよ。

            コミットしたものにタグを打ち、タグで取得してからレビュー&テスト。
            合格したものに対してやはりタグを打ち、タグで取得してリリース。

        • by SteppingWind (2654) on 2009年08月26日 17時48分 (#1629599)

          そういう時のための多数決方式だと思う.

          親コメント
      • by Anonymous Coward

        > そういうことを(する/できる)体制ってどうなのよ

        否定でも反論でもなく賛成なんですけど、
        今回のような事例を防げるようなしっかりした体制を
        実際に作るって難しいだろうと思うのですが。
        みなさんの会社では実現できていますか?

        もしくは取引先の企業はしっかり体制が組まれて、
        今回のようについでにコッソリ他の修正すらできない企業は
        結構多いものですか?

Stay hungry, Stay foolish. -- Steven Paul Jobs

処理中...