2分探索後、出力するデータの身長が0表示され、困っています。

Question

###前提・実現したいこと

2分探索後、出力するデータの身長が0表示され、困っています。

期待出力）
データ：松富明雄 169 0.8

このソースだと出力）
データ：松富明雄 0 0.8

###発生している問題・エラーメッセージ

エラーメッセージ

###該当のソースコード

import java.util.Arrays;
import java.util.Scanner;
import java.util.Comparator;
class En3_7{
	static class PhyscData{
		private String name;
		private int height;
		private double vision;
		public PhyscData(String name, int heigth, double vision){
			this.name = name; this.height = height; this.vision = vision;
		}
		public String toString(){
			return name + " " + height + " " + vision;
		}
		public static final Comparator<PhyscData> VISION_ORDER = new VisionOrderComparator();
		private static class VisionOrderComparator implements Comparator<PhyscData>{
			public int compare(PhyscData d1, PhyscData d2){
				return (d1.vision > d2.vision) ? 1 : (d1.vision < d2.vision) ? -1 : 0;
			}
		}
	}
	public static void main(String[] args){
		Scanner stdIn = new Scanner(System.in);
		PhyscData[] x = {
			new PhyscData("赤坂忠雄", 162, 0.3),
			new PhyscData("長浜良一", 168, 0.4),
			new PhyscData("加藤富明", 173, 0.7),
			new PhyscData("松富明雄", 169, 0.8),
			new PhyscData("浜田哲明", 174, 1.2),
			new PhyscData("武田信也", 171, 1.5),
			new PhyscData("斉藤正二", 175, 2.0),
		};
		System.out.print("視力いくつの人を探しますか：");
		double vision = stdIn.nextDouble();
		int idx = Arrays.binarySearch(x, new PhyscData("", 0, vision), PhyscData.VISION_ORDER);
		if(idx < 0)
			System.out.println("その値の要素は存在しません。");
		else{
			System.out.println("その値はx[" + idx + "]にあります。");
			System.out.println("データ：" + x[idx]);
		}
	}
}

###試したこと
課題に対してアプローチしたことを記載してください

###補足情報(言語/FW/ツール等のバージョンなど)
より詳細な情報

Answer 1

コンストラクタでint heigthとスペルミスしています。

結果、this.height = heightが、フィールドの方を参照しているため、デフォルトの0のママとなります。

Comments

[gyro16]

コンストラクタでミスタイプしているのでint heightがデフォルトの0のまままんですね。
分かりました。コンストラクタのタイプには気をつけます。

[maisumakun]

フィールドと引数で同じ名前を使うとどちらかわかりづらいですし、「引数には別な名前を使う」というのも1つの方法かなと思います。

Answer 2

質問とは関係ないですが、Comparatorのところは匿名クラスとDouble.compareを使ってこのように書けます。

public static final Comparator<PhyscData> VISION_ORDER = new Comparator<PhyscData>(){
    public int compare(PhyscData d1, PhyscData d2){
        return Double.compare(d1.vision, d2.vision);
    }
};

もしくは、Java8が使えればこんなことも。

public static final Comparator<PhyscData> VISION_ORDER = Comparator.comparingDouble(d -> d.vision);