int型配列の中身を取り出し、カンマ区切りで出力する方法

Question

以下のコードにおいて、配列の値を、カンマ区切りで表示する際、もう少し簡潔に記載する方法があれば教えて欲しいです。

package sample;
public class sample4 {
	public static void main(String[] args) {

		int[] arrays1 = {96,45,31,29,84,77};
		
		
		display(arrays1,false);

	}
	public static void display(int[] ar,boolean isSorted) {
		String msg = "";
		String num_list = "";
		if (isSorted == false) {
			msg = "***並び替え前***";
		}else if (isSorted == true) {
			msg = "***並び替え後***";
		}
		
		for(int i = 0; i < ar.length; i++) {
			if(i == ar.length - 1) {
				num_list += Integer.toString(ar[i]);
				break;

			}
			num_list += Integer.toString(ar[i]) + ",";


		}
		
		System.out.println(msg);
		System.out.println(num_list);

	}
}

displayメソッドに配列を渡しその中のfor文で該当の処理を行ない、System.out.println()で表示を行なっているのですが、こちらより簡潔に表示を行う方法はありますでしょうか？
最終的な表示結果は以下のようにしたいと考えています。

***並び替え前***
96,45,31,29,84,77

Answer 1

出力形式が質問記載のものじゃないので答えじゃないです。

[1, 2, 3]のような文字列になりますが
(docs.oracle.com)Arrays.toString()が配列の内容を表示するだけなら手っ取り早いです。

println(Arrays.toString(arrays1));
で出力できます。

むりやり質問記載の出力にするなら（やらない方がいいけど。。。）
println(Arrays.toString(arrays1).replaceAll("[[] ]", ""));
とかするとなるんじゃないかな。
これやるくらいならほかの方の回答にある通りArrays.stream使うべき。

import java.util.Arrays;

public class sample4 {
    public static void main(String[] args) {

        int[] arrays1 = {96,45,31,29,84,77};
        String msg = Arrays.toString(arrays1);
        System.out.println(msg);

    }
}

結果

$ java sample4
[96, 45, 31, 29, 84, 77]

Comments

[aae_11]

ご回答ありがとうございます。
＞println(Arrays.toString(arrays1));
こちらのやり方で試してみたのですが、結果的にはリストに文字列(カンマを含む)が格納された形となるのですね。
こちらも用途によっては、使っていけたらと考えています。
自分はjavaの経験が浅く、Arrays.streamについてはわからない部分が多い為、調べつつ使用していけたらと考えています。

[Y.H.]

> 結果的にはリストに文字列(カンマを含む)が格納された形となるのですね。
いいえ。文字列として"[96, 45, 31, 29, 84, 77]"が返ります。
回答に追記しておきました。

[aae_11]

返信ありがとうございます。
リスト型ではなく、カッコを含んだ文字列として返されるといった訳だったのですね...

Answer 2

本題から外れますが、isSorted を false や true と比較しているのが気持ち悪いので
そこを簡潔に書いてみます。

package sample;

public class sample4 {
    public static void main(String[] args) {
        int[] arrays1 = { 96, 45, 31, 29, 84, 77 };
        display(arrays1, false);
    }

    public static void display(int[] ar, boolean isSorted) {
        System.out.println(isSorted ? "***並び替え後***" : "***並び替え前***");
		int n = ar.length - 1;
		for (int i = 0; i < n; i++) System.out.print(ar[i] + ",");
		System.out.println(ar[n]);
    }
}

カンマ区切りの方は次のようにも書けます。

		for (int n = ar.length - 1, i = 0; i <= n; i++)
			System.out.print(ar[i] + (i < n ? "," : "\n"));
}

ループの中で i < n の判定を何度も行うのが気に入りませんが。

Comments

[aae_11]

ご回答ありがとうございます。
>「for (int i = 0; i < n; i++) System.out.print(ar[i] + ",");」
こちらのように、for文って一行で書くことも出来るんですね。
>(i < n ? "," : "\n"));
こちらは短縮構文みたいなやつですかね？javaは経験が浅くわからない部分も多い為、勉強になりました。

Answer 3

swordoneさんとほぼ同じですが、区切り文字がカンマなので。
java8以上であれば使えます。

package sample;

import java.util.Arrays;
import java.util.stream.Collectors;

public class sample4 {

    public static void main(String[] args) {

        int[] arrays1 = {96,45,31,29,84,77};


        display(arrays1,false);

    }
    public static void display(int[] ar,boolean isSorted) {
        String msg = "";
        if (isSorted == false) {
            msg = "***並び替え前***";
        }else if (isSorted == true) {
            msg = "***並び替え後***";
        }

        String num_list = Arrays.stream(ar)
        	.mapToObj(String::valueOf)
        	.collect(Collectors.joining(","));

        System.out.println(msg);
        System.out.println(num_list);

    }
}

Comments

[aae_11]

ご回答ありがとうございます。
自分の勉強不足かとは思うのですが、「.mapToObj(String::valueOf)」こちらの部分なのですが、mapToObjメソッドに、String型のvalueOfメソッドを指定しているといった形でしょうか...?

[yureighost]

ここら辺はjava8のstreamという機能を調べてもらうのが一番なんですが、
mapToObjメソッド内部にはint型を引数として受け取り、String型を返すという関数を渡します。
String::valueOfの記述はこのメソッドの他に処理が無い時に簡略的に書けるメソッド参照という物です。

[aae_11]

ご返信ありがとうございます。
javaの経験が浅い為、streamに関しましては、難しく理解できない部分も多い為、取り敢えずは、このような形式で書くものだという認識でも良いかもしれないですね...

Answer 4

Streamを使って。

String result = Arrays.stream(ar)
                      .mapToObj(Integer::toString)
                      .collect(Collectors.joining(","));

Comments

[aae_11]

ご回答ありがとうございます。
メソッドの大方な理解なのですが、「.mapToObj(Integer::toString)」こちらでint型からString型の配列へ変換をし、「 .collect(Collectors.joining(","));」ここで変換した配列の中身をカンマ区切りにする形でしょうか...?

[swordone]

正確には「配列」ではないですが、そんな感じです。

Answer 5

こちらより簡潔に表示を行う方法はありますでしょうか？

という質問でしたら「あります」

あとは検索がんばってください。