日々の破片

_ rjb

LinuxだとJNI_GetDefaultJavaVMInitArgs がSEGVと書いていたが、JNI_GetDefaultVMInitArgsに与える引数を間違えていた。

JDK1_1InitArgsを使っていたが、実際には、JavaVMInitArgsを使用する。

Solarisで、-Xusealtsigsを与えるためにいろいろ調べたら、結局、JavaVMInitARgsのJavaVMOption *optionsメンバに設定するらしいとわかったからだ。

で、JavaVMInitArgsを利用するように修正したら、Linuxでも通るようになった。では、JDK1_1InitArgsってのは一体どこで使えるんだろう？

追記：寝ぼけてるようだ。1_1なんだからJDK1.1用で、Java2(1.2)以降は、JavaVMInitArgsになったということだろう（データメンバが少ないので逆に考えてしまったらしいが、万能のoptionsメンバが追加されてるわけだし）。

_ あわや読まずに捨てるところ

なんか、最近やたらめったら「アドレス変更しました（音符マーク）」というサブジェクトのspamが来る。とにかく来る。連続5個くらい来たりする。

という状況の中で、「サイト移転のお知らせ」というのが来たので、即効削除しようとしてふと一瞬出た文面がそれまでと調子が違うので、あわてて手を止めた。Fe²⁺さんからのRuby256倍勝手に正誤表の移転のお知らせだった。危なかった。

で、思い出したが、ここまでspamがガンガン来ると、もう、「常識ではsubjectはASCIIで書くべきですな」とは言えないし（「アドレス変更しました」はとりあえず置いておいて）、用件だけ手短にってのもあまりうまくなさそうだな。

_ なんとなくやってみたが

プロのコード、アマのコード(4/23アンカー変更) 面倒な部分は再利用

import java.io.*;
public class Gcp {
    private static final String te[] = {"グー", "チョキ", "パー"};
    private int win;
    private int loose;
    private int fair;
    private BufferedReader reader;
    private final Result[][] state;
 
    public Gcp () throws IOException {
	reader = new BufferedReader(new InputStreamReader(System.in));
	state = new Result[3][3];
	state[0][0] = state[1][1] = state[2][2] = new Fair();
	state[0][1] = state[1][2] = state[2][0] = new Win();
	state[0][2] = state[1][0] = state[2][1] = new Loose();
    }
    private abstract class Result {
	void execute(int you, int com) {
	    System.out.print(" コンピュータ：" + te[com] + 
			       ",プレイヤー：" + te[you] + ":");
	}
    }
    private class Win extends Result {
	void execute(int you, int com) {
	    super.execute(you, com);
	    System.out.println("あなたの勝ちです！");
	    win++;
	}
    }
    private class Loose extends Result {
	void execute(int you, int com) {
	    super.execute(you, com);
	    System.out.println("あなたの負けです！");
	    loose++;
	}
    }
    private class Fair extends Result {
	void execute(int you, int com) {
	    super.execute(you, com);
	    System.out.println("引き分けです！");
	    fair++;
	}
    }
 
    public void execute(int count) {
         win = loose = fair = 0;
	for (int i = 0; i < count; i++) {
	    int you = read();
	    int com = (int)(Math.random() * 2 + 1);
	    state[you][com].execute(you, com);
	}
	System.out.println("プレイヤーの勝敗数：" + win + "勝" +
			   loose + "負" + fair + "引き分け");
    }
 
    private int read() {
	for (;;) {
	    System.out.print("あなたの手（0：グー、1：チョキ、2：パー）");
	    String s;
	    try {
		s = reader.readLine();
	    } catch (IOException e) {
		continue;
	    }
	    if (s.length() > 0) {
		if (s.equals("q")) { // バックドア
		    System.exit(1);
		}
		try {
		    int n = Integer.parseInt(s);
		    if (n >= 0 && n <= 2) return n;
		} catch (NumberFormatException e) {
		}
	    }
	}
    }
 
    public static void main(String[] args) throws Exception {
	Gcp gcp = new Gcp();
	gcp.execute(5);
    }
}

添削結果：
1.コメントがありませんね
プログラムのどこにもコメントが付けられていませんね。プログラムを読みにくいと感じました。
（追記：× この程度の行数のまとまりを読めないのはだめ。ただし各メソッドやフィールド、クラスにJavadocは付けるべきでしょう。）
（更に追記：そうだ、コメントをつけましょうというのを見るたびに感じる違和感ってのは、単純なことなのだ。あなたがプロなら、読む相手もプロなのだ。プロをなめるな、ということだ。アマはプロの能力を知らないから文章による解説が必須だと考えてしまう。大丈夫、あなたの先輩や同僚はあなたと同等かそれ以上のプロだから、まともなコードを書けば、必ずそれを理解するってことだ）
2.おやおや実装継承ですか？
たかだか1行の出力くらい、バーといきましょう。せこいですよ。
(追記：微妙。状態表示をスーパークラス、結果表示をサブクラスというメッセージ表示の責務分割と考えれば合理的。それにこれ以上の継承＝スパゲッティ化はありえないはず。取り得る数を考えればせっかく実装継承可能な言語なんだから使わなければ無駄。7項を参照）
3.おやおや3文字変数ですか？
comってcomponent object modelですか？ それともcommunication portですか？ さっぱり意味がわかりませんね。せめて、computerPlayerにしてください。
（追記：微妙。正直最初comとは何かわからなかったが、youの対照だということはわかるから、ゲームの言い回しとしてはありなのかも。これはオリジナルから持ってきた）
4.おやおや1文字変数も使ってますね
iではなく、kaisuのほうが良いでしょう。sではなくyomiTottaMojiretsuにしたほうが適切ですね。
(追記：×。これがスコープが短く、初期化時が明確なので意味の失いようが無い例。特にkaisuはともかくyomi……はダメな例。なお、英語ローマ字混じりは避けるべきなので、kaisuならKati、Make……、Win、Loseならcountかな）
5.0,1,2は定数にしたほうが良いでしょう
0をGOO、1をCHOKI、2はPAAとすべきですね。
（追記：○。特に最初の配列の初期化時）
6.バックドアとはなんですか？
デバッグ中はともかく、ちゃんと削除しましょう。（うむ、これは文句なく正しい）
（追記：○。もちろん）
7.詰めが甘いですね
わざわざ、win, looselose(次項を参照) ...なんていうフィールドを作っていますが不要ですね。Resultのフィールドとしてint countを置いて、実装継承しているのだから、Result#executeでカウントアップすれば済みますね。
（追記：○。最後の結果表示でwin.getCount(), lose.getCount()...とすれば良い）
8.誤った綴りを使っていますね
looseじゃなくてloseですよ。素直にKati, Make, Aikoとすれば、こういう間違いは防げますね。っていうか、どおりでなんでlooseがlostになるんだ？ と思ったはずだ（で、辞書見て、「単語が存在する」のを確認してそこで終ったところがバカですね）。
ご指摘感謝。
（追記：○。心より恥じるってやつですね。）
#追記：ネタ的（微妙）なものに×、正しいものに○を付けた
9(追加).メソッド名がいい加減すぎです。
なんでもexecuteはいかがなものでしょうか？ せめて、Resultについてはjudgeなどにすべきですね。
（○。そうだな）
10(追加).ところで、根本的に間違えていますね
comさんが0を出すことってありえるか考えてみたらどうですか？
（○。確かに、その通り……）
11(追加).「面倒な部分は再利用」とか言っていますが、「再利用」ってそんなにいいですかね?
つまりコピペは悪ということですね。
(int)(Math.random() * 3)。「1.0より小さい正の符号付doubleを返す」
(○。うむ、心より恥じる。っていうか、ちゃんと添削しといてくれよ。）

_ この時、何が起きている？

i686 Win32 J2se1.4.2では、

public class K {
    public static void main(String[] args) {
        double k = 0.9999999999999999;
        System.out.println("result=" + ((int)(k * 3))); // -> 2
        double j = 0.99999999999999999;
        System.out.println("result=" + ((int)(j * 3))); // -> 3
    }
}

となるけど、代入時点で既にjは1.0になっているのか。