デッドコードじゃないんだが到達不能？

Question

前提

既存システムの単体試験をしています。
実装から参画したため、前任者の設計書通りに実装しテストに移ったわけですが、スケジュールの都合上しっかりソースを読める時間がなく、取り掛かったという背景があります。
ちょっと僕の経験内では想定外の既存実装がありまして、タイトルの通りコンパイル上はデッドコードにはならないものの、改修したのに通せない箇所が出てきました。
継承についてなのですが、めちゃくちゃ知見が深いわけではないので、これってどういうこと？つまりどうなるの？的なところをご教示いただけたらと思います・・・。

該当のソースコード

public class aLogic() {
  public void execute() {
    aDao dao = new aDao();
    String hoge = dao.selectHoge(param)
  }
}

public class aDao {
  public String selectHoge(param) {
  // 処理
  }
}

ここまではごく一般的な流れだと思います。
別の起動プロセスから動く以下のコード・・・

public class bLogic() {
  public void execute() {
    aDao dao = new bDao();
    String hoge = dao.selectHoge(param)
  }
}

public class bDao extends aDao {
  public String selectHoge(param) {
  // 処理
  }
}

上記のように、別の起動プロセスを経て動くbLogicがDaoのインスタンス生成時にaDaoを、aDaoを継承したbDaoでnewするといったことになっています。。。

aDao、bDaoどちらのselectHogeも全く同じ処理をしているため、同様に改修したのですが、bLogicから飛ぶのはaDaoのselectHoge。

bDaoのselectHogeが別のところから参照されていればいいのですが、当該メソッド以外では呼び出されておりません。（何ならbDao自体、このインスタンス生成時以外使用されていない・・・）

聞きたい事

class単位でのテストは可能ですが、一連の処理の流れをテストしたいとき、まあどうやってもbDaoのselectHogeに突入することなく、aDaoの方に行ってしまうので、改修したbDaoのselectHogeに到達できず、といったところです。

継承の基本的なところについては理解しているつもりですが、このような使い方を今までにしたことがないので、戸惑っております。
これがどうしてもスタンダードな処理だとは、思えないのですが。

インスタンス生成の時点でおかしなことやってるから**「通らない / 通ってない」って判定で正しいのでしょうか？
それとも継承という観点からみると、処理内容が全く同じであるなら、「通っている」**という判定になるなんてことまさかまさかであったりしますか？

愚問なのかもしれませんが、ご教示頂けますよう、よろしくお願いします。

補足情報（FW/ツールのバージョンなど）

Java6

Comments

[Zuishin]

@Override がついていないのでは？

Answer 1

bDaoが継承しているメソッドが、実は継承していないということはありませんでしょうか。

メソッド名が違う、引数の型が違うなど。

Comments

[swordone]

｢引数の型が違う｣はありそう。
コードに引数の型ないし。

Answer 2

A-pZさんの回答で合っている気がします。

オーバーライドと呼び出し方法のサンプルですが。
理解できていないと想定外の動きになります。

単純にオーバライドするしないの話と、
インスタンスの型でなく宣言の型でメソッドが呼ばれるのがわかると思います。

メソッド名が違う、引数の型が違うなど。

を確認してみては、どうでしょう

public class Hoge {
	public static void main(String[] args) {
		Hoge hoge = new Hoge();
		Hoge who = new Foo();	// who
		Foo  foo = new Foo();

		System.out.println(hoge.say(hoge));	// Hoge:hoge
		System.out.println(hoge.say(who));	// Hoge:hoge
		System.out.println(hoge.say(foo));	// Hoge:hoge
		System.out.println();

		System.out.println(who.say(hoge));	// Foo:hoge
		System.out.println(who.say(who));	// Foo:hoge  not Foo:Foo
		System.out.println(who.say(foo));	// Foo:hoge  not Foo:Foo
		System.out.println();

		System.out.println(foo.say(hoge));	// Foo:hoge
		System.out.println(foo.say(who));	// Foo:hoge  not Foo:Foo
		System.out.println(foo.say(foo));	// Foo:Foo

	}
	public String say(Hoge h) {
		return "Hoge:Hoge";
	}
}
class Foo extends Hoge {
	@Override
	public String say(Hoge h) {
		return "Foo:Hoge";
	}

	public String say(Foo f) {
		return "Foo:Foo";
	}
}

Answer 3

実際に動かしたいのはbDao#selectHogeですよね？
インスタンス化の際にaDao dao = ...としているのでアクセスできるのはaDaoのメソッドだけです。
インスタンス化したdaoはaDaoクラスだからです、という説明で大丈夫でしょうか？

実際のインスタンスには継承したbDaoを含みますが、aDaoとして扱う以上はaDaoクラス以外のフィールドもメソッドも使うことは出来ません。

したがってこの場合はbDao dao = new bDao();とするのが良いでしょう。

上記回答は誤りです。
他の回答者様の回答やコメントの通り、引数違いではないでしょうか。
bDaoに同じメソッド名で別の引数を持つメソッドを作ってしまうと呼び出すことが出来ません。
これはaDao dao = でインスタンス化している為、aDaoのメソッドシグネチャしか認識できない為です。

---

ご指摘を頂いたので再確認。
確かによく考えればオーバーライドされたメソッドが呼ばれますね。
（自分でもインスタンス化でbDaoになってると言いながら…）

Eclipseのハイライト見てA側だなと思って確認せず回答してしまいました。
誤った情報を載せてしまい申し訳ございません。

検証コードを分かりやすく変更
paiza.ioで実行

import java.util.*;

public class Main {
    public static void main(String[] args) throws Exception {

        ADao dao = new BDao();
        System.out.println(dao.aString);
        //System.out.println(dao.bString);
        System.out.println(dao.procA());
        //System.out.println(dao.procB());
        //System.out.println(dao.procA("追加引数"));
    }
    
    public static class ADao {
        public String aString = "AaString";

        public String procA() {
            return "AprocA";
        }
    }

    public static class BDao extends ADao {
        public String aString = "BaString";
        public String bString = "BbString";

        @Override
        public String procA() {
            return "BprocA";
        }
        
        public String procA(String exArg) {
            return exArg;
        }

        public String procB() {
            return "BprocB";
        }
    }
}

Comments

[momon-ga]

> String ret3 = dao.procA(); //○　これはADao#procA
オーバライドされてるなら、BDaoのprocAでしょ？

[S_kawa]

@momon-ga さん
ご指摘ありがとうございます。ごもっともです。
回答は訂正させていただきました。