日々の破片

_ フィールドオーバーライド

いや、本物を見るとは思わなかったがすさまじく強力でびびった。ものはセッタインジェクションがからんでいる超弩級のオーバーライドだったのだが、以下はそれを甘ちゃん化したもの。

public class Root {
    private String name;
    public void setName(String newName) {
        name = newName;
    }
    public void printName() {
        System.out.println("my name is " + name);
    }
}

public class Extended extends Root {
    private String name;
    public void setName(String newName) {
        name = newName;
    }
    public static void main(String[] args) {
        Extended e = new Extended();
        e.setName("watashi ha nakano");
        e.printName();
    }
}

で、問題のクラス。実行すると当然、次の結果となる。

$ javac *.java
$ java -cp . Extended
my name is null
$

こういうものをどうすれば作らせないですむかをちょっとディスカッション。

プログラマが100人の村だとして、たった1人でもこういうとんでもないのが紛れ込んでいたら、全部staticメソッド（フィールド利用禁止）にしようホトトギス、と考えたくもなるがそれは無理なので実装継承禁止だホトトギス。

いや、直して見せようホトトギス。でもどうやって？ 全部チェックできるのか？ そんなときのFndBugsFindBugsだホトトギスとか。

フィールドを直接参照禁止だホトトギス。常にゲッタを使えば済むんじゃないか。いえいえそれは無理でしょう。フィールドを直接さわりたいからオーバーライドしたのじゃないかなホトトギス。privateが問題なのだよホトトギス、パッケージプライベートにしとけばこんなことはしなかったんじゃないのかな（と、これはまじめな考察である）。

結局、コピペできなくすれば良いのだホトトギス。DOSで開発させようホトトギス。それは何がなんでも非効率ではなかろうか？ いや、こんなプログラムを量産されるよりは効率的だぞホトトギス。そうは言ってもホトトギス、一生懸命基底クラスの内容をDUPするかもホトトギス。copy Root.java Extended.javaとかしてから編集するかもホトトギス。と、世にぬすっとのネタは尽きマジ。

というか、フィールドオーバーライドだけなら良くはないけどまだ救いはあるのに、なぜセッタまでオーバーライドしたんだろうか(セッタをオーバーライドしなければRootクラス側のフィールドに入る。とは言えExtended側でもフィールドを参照しているメソッドが実際にはあるからだめなんだけど)。とまじめに考察してみる。クラスをまたがってリファクタリングして重複しているセッタとフィールドを基底クラスに汎化して、しかし特化クラス側から削除を（好意的に取れば）忘れた、あるいはもったいないお化けに取り憑かれたか。だろうな。リファクタリングに情けは無用。移動したならばっさり削除。というか、ユニットテストしてないだろうな。それはリファクタリングではなく、バグの培養というべき行為だ。

_ 型がない言い方

手続き型を関数言語と呼ぶのはあ、突っ込まれたほうの言い方だったのですか。それは残念というか。