ソートについて教えてください

Question

javaを勉強しはじめた初心者です。下記の件でアドバイスをお願いします。

Mapでkey（数値）の小さい順でソートをさせたいのですが、絶対値でソート
されてしまいます。
どこが問題なのかわかりません。宜しくお願い致します。

<期待する結果>
strawberry
orange
apple
melon
pineapple

<実際の結果>
apple
orange
melon
pineapple
strawberry

コード
import java.util.*;
 
public class Sample {
 
    public static void main(String[] args) {
        // Mapの宣言
        Map<Integer, String> mMap = new HashMap<Integer, String>();
        
        // Mapにデータを格納
        mMap.put( 1, "apple");
        mMap.put( -2, "orange");
        mMap.put( 4, "pineapple");
        mMap.put( -5, "strawberry");
        mMap.put( 3, "melon");
        
        Object[] mapkey = mMap.keySet().toArray();
        Arrays.sort(mapkey);
        
        for (Integer nKey : mMap.keySet())
        {
            System.out.println(mMap.get(nKey));
        }
    }
 
}

Answer 1

ベストアンサーついたので別解を、そもそもソートしてもらう案です。

public static void main(String[] args) {
	// Mapの宣言
//	Map<Integer, String> mMap = new HashMap<Integer, String>();
	Map<Integer, String> mMap = new TreeMap<Integer, String>();

	// Mapにデータを格納
	mMap.put( 1, "apple");
	mMap.put( -2, "orange");
	mMap.put( 4, "pineapple");
	mMap.put( -5, "strawberry");
	mMap.put( 3, "melon");

//    Object[] mapkey = mMap.keySet().toArray();
//    Arrays.sort(mapkey);

	for (Integer nKey : mMap.keySet())
	{
	    System.out.println(mMap.get(nKey));
	}
}

Comments

[hhd00144]

momon-gaさん、アドバイスをありがとうございます。考え方によってはこんなに簡潔なコードになるんですね。ありがとうございます。参考にさせていただきます。

Answer 2

mapkeyをソートしていますが出力しているのはmMap.keySet()です。
mapkeyの中身を確認してみてください。

Comments

[hhd00144]

アドバイスをありがとうございました。

Answer 3

import java.util.*;

public class Main{

    public static void main(String[] args) {
        // Mapの宣言
        Map<Integer, String> mMap = new HashMap<Integer, String>();

        // Mapにデータを格納
        mMap.put( 1, "apple");
        mMap.put( -2, "orange");
        mMap.put( 4, "pineapple");
        mMap.put( -5, "strawberry");
        mMap.put( 3, "melon");

        Integer[] mapkey = mMap.keySet().toArray(new Integer[0]);
        Arrays.sort(mapkey);

        for (Integer nKey : mapkey)
        {
            System.out.println(mMap.get(nKey));
        }
    }

}

Comments

[hhd00144]

修正のコードを記述いただきありがとうございました。参考にさせていただきます。