VoQn
2014-01-14 14:04:54
[Java]
すれ違い、それは…
投稿者からのアピールポイント
リリース済みのブツを改修する任に当たった際に発見した可愛らしいコーディングミスなのですが、 問題は last commit (リリース時) から5年以上このまんま製品版として放置されてたっていう事だったんですよねぇ… …どこでココのヘマが回収されてたんでしょうねぇ…
// おそらくはモデルに相当するベリーベリーナイスなクラス
public class XZKaisekiZu {
private Foo foo;
// ...
public Foo setFoo(Foo newFoo) {
if (newFoo == null) {
this.foo = newFoo;
}
}
}
// ...
// おそらくはビューに相当するベリーベリークールなクラス
public class XZZuKaiseki {
private XZKaisekiZu zu; // 解析結果クラス
private XZZuPaper paper; // 印刷紙クラス
// ...
public void init() {
// ...
if (zu.getFoo() != null) {
paper.setFoo(zu.getFoo());
}
}
}
コメント(3)
つまり、今抱えているソースコードは全くのゴミ。という可能性が一番高いと思います。 すぐにエスカレーションしといた方が無難でしょう。
その後の改修がコミットされていないことが発覚して真っ青になった投稿者が偲ばれます。