スッキリわかるJava入門実践編の第７章（JVM操作とリフレクション）の練習問題の疑問

Question

###練習問題の内容
以下のようなLauncherクラスを作るのが練習問題です。（一部省略）
0. 番号リストコマンドライン引数から、次の２つの情報を受けとる。
・第一引数・・・起動すべきクラスのFQCN
・第二引数・・・起動の仕方を示す文字。EまたはI
0. FQCNのクラスが、文字列配列の引数を持つpublicメソッドmain()を持つか調べ、もし持っていれば、コマンドライン第二引数によって次のように起動する。
・Eの場合・・・ProcessBuilderで別プロセスとして起動する
・Iの場合・・・リフレクションでmainメソッドを呼び出す。
0. このプラグラムが終了する際には、軌道に成功した場合は０、そうでなければ１を終了コードとする。

答えのソースコード

import java.lang.reflect.*;
public class Launcher {
	public static void main(String[] args) {
		String fqcn = args[0];
		String sw = args[1];
		try {
			Class<?> clazz = Class.forName(fqcn);
			listMethods(clazz);
			if(sw.equals("E")) {
				launchExternal(clazz);
			} else if(sw.equals("I")) {
				launchInternal(clazz);
			} else {
				throw new IllegalArgumentException("起動方法の指定が不正です.");
			}
		} catch(Exception e) {
			System.out.println(e.getMessage());
			e.printStackTrace();
			System.exit(1);
		}
		System.exit(0);
	}

	public static void listMethods(Class<?> clazz) {
		System.out.println("メソッドの一覧を表示します.");
		Method[] methods = clazz.getDeclaredMethods();
		for(Method m : methods) {
			System.out.println(m.getName());
		}
	}

	public static void launchExternal(Class<?> clazz) throws Exception {
		ProcessBuilder pb = new ProcessBuilder("java", clazz.getName());
		Process proc = pb.start();
		proc.waitFor();		// プロセスの終了まで待つ
	}
	
	public static void launchInternal(Class<?> clazz) throws Exception {
		Method m = clazz.getMethod("main", String[].class);
		String[] args = {};
		m.invoke(null, (Object)args);	// 静的メソッド呼び出し時は第一引数は任意
	}
}

疑問点

見出しの「練習問題の内容」の太字になっているところ。答えのソースコードのif文のあたりがその該当箇所であると思うのですが、答えのソースコードでは起動の仕方を指定しないときの対処だけしか書けていない気がします。起動したいクラスが文字列配列の引数を持つpubicメソッドmain()を持つを調べるif文が抜けていませんか。。？

まとめ

どこかにmain()を調べているコードがあれば、ぜひ教えていただきたいです。javaはまだまだですが、どうぞよろしくお願いします。

Comments

[退会済みユーザー]

listMethods の実装を削除しちゃだめよ

[tanaka_hana]

すいません、その通りです。ありがとうございます。

Answer 1

起動したいクラスが文字列配列の引数を持つpubicメソッドmain()を持つを調べるif文が抜けていませんか。。？

「調べる」という言葉の範囲の問題というのもありますが、Class#getMethodは「当該メソッドがなければ例外になる」ので、「あるかどうか調べる」ことも兼用しています。

Comments

[退会済みユーザー]

多分これ listMethods 内部でやってる想定だけど、コード表記されてないからコンパイルエラーよ

[maisumakun]

そっちか…！

[退会済みユーザー]

っとコード全文をフレアリンク社のところでみてきたらこれはひどいな。　仕様と違うからバグかもなE側の場合

[tanaka_hana]

listMethodsを追加してみたのですが、、結局のところどこでmain()があるかの判断をしているでしょうか？そもそも文字列配列の引数を持つpublicメソッドmain()を持つかどうかを調べてから、もし持っていれば起動方法を指定するのに、launchInternal()内のClass#getMethodでは調べられていない気がするのですが。。

[退会済みユーザー]

I の場合は Class<E>#getMethod の場合、メソッドが存在しない例外が投げられるのでエラーとなりますが順番が仕様と違うのでバグです。

Eの場合はエラーにならない（正常終了）するのでバグですね

[tanaka_hana]

ありがとうございます。仕様書の書き方の順序が逆であったと納得できました。
ただ、何度もすいません。。逆にEの場合エラーになると正しいのでしょうか？どうしてでしょうか？

[退会済みユーザー]

３の起動に成功したら０、失敗したら１に反するからエラーコードは１にならなければいけませんが０になってるはずですよ。（机上デバッグ）

[tanaka_hana]

launchExternal()は具体的にどこがバグっているのでしょうか？