javaでどうしても最後のカンマが消せません

Question

前提・実現したいこと

javaでどうしても最後のカンマが消せません。
過去の質問を参考にしてもわからなかったので質問させていただきます。

コードのどこを抜粋して載せてよいかわからず長くなってしまいすみませんが、
よろしくお願いいたします。

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

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

該当のソースコード

import java.util.Arrays;

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

	}
	public static void arraysSort(int[] array, boolean orderType) {
		if(orderType == true) {
			Arrays.sort(array);
		}else {
			for(int i=0;i<array.length;i++) {
				for(int j=0;j<array.length;j++) {
					if(array[i]>array[j]) {
						change(array,i,j);
					}
				}
			}
		}
	}
	public static void display(int[] array, boolean orderType) {
		if(orderType == true) {
			System.out.println("****並び替え後****");
		}else {
			System.out.println("****並び替え前****");
		}
		for(int i:array) {//わからないのがここからです
			int sts = 5;
			while(sts>2) {
				System.out.print(i);
				System.out.print(",");
				sts--;
				break;//ここまで
			}
		}System.out.println("");
	}

	public static void change(int[] array,int i,int j) {
		int tmp = array[i];
		array[i] = array[j];
		array[j] = tmp;
	}
}

試したこと

java Visualizerで見てみたところ、
上記ではsts=5の状態のままになってしまうことが原因のようですが、それ以上はどうすればいいかわからなくなってしまいました。

ほかにも

//前後は省略しました
for(int i:array) {
	System.out.print(i);
	System.out.print(",");
}System.out.println("");

//前後は省略しました
for(int k=0;k<6;k++) {
	if(k<5) {
	System.out.print(i);
	System.out.print(",");
	}else {
		System.out.print(i);
	}

}

なども試してみましたがだめでした。

Answer 1

配列をカンマ区切りの文字列にするというコードの書き方が知りたいということでなければ、StringJoinerというクラスを使うと簡単に実現できると思います。

コードの書き方（考え方）を知りたい（学びたい）ということであれば、以下の回答は無視してください。

int[] array = {96,45,31,29,84,77};
StringJoiner joiner = new StringJoiner(",");
for (int i : array) {
    joiner.add(String.valueOf(i));
}
System.out.println(joiner.toString());
// → 96,45,31,29,84,77

ちなみに、StringJoinerのコンストラクタにはdelimiter(区切り文字)以外にも、prefix(接頭辞)とsuffix(接尾辞)も渡せます。

StringJoiner joiner = new StringJoiner(",", "{", "}");

// 省略

System.out.println(joiner.toString());
// → {96,45,31,29,84,77}

以上、参考になるようでしたら引き続きStringJoinerクラスのJavaDocやString.joinなども調べてみてください。

Comments

[yamasakisan]

調べたけれどもよくわからなくなっていたところだったのでありがたいです。
勉強になりました！ありがとうございます。

Answer 2

例えば,96,45,31,29,84,77を出力したいならば以下のコードでできます.

public class Main {
    public static void main(String[] args) {
       
        int[] numbers = {96,45,31,29,84,77};
        
        for(int i = 0; i < numbers.length; i++){
            System.out.print(numbers[i]);
            if(i < numbers.length - 1){
                System.out.print(",");
            }
        }
    }
}

以下のコードは,先ほどのコードにソートの部分を入れたものです.表示が3回あるのでメソッドで切り離しました.

public class Main {
    public static void main(String[] args) {
       
        int[] numbers = {96,45,31,29,84,77};
        
        System.out.println("****並び替え前****");
        print_result(0,numbers);
        
        for(int i = 0; i < numbers.length; i++){
            for(int j = 0; j <= i; j++){
                int tmp;
                if(numbers[i] < numbers[j]){
                    tmp = numbers[i];
                    numbers[i] = numbers[j];
                    numbers[j] = tmp;
                }
            }
        }
        System.out.println("****並び替え後****");
        print_result(0,numbers);
        System.out.println("****並び替え後****");
        print_result(numbers.length - 1,numbers);
        
    }
    public static void print_result(int n,int[] numbers){
        
        for(int i = 0; i < numbers.length; i++){
            System.out.print(numbers[Math.abs(n - i)]);
            if(i < numbers.length - 1){
                System.out.print(",");
            }
        }
        System.out.println();
    }
}

Comments

[yamasakisan]

ありがとうございます、スッキリしました！
for(int i:array)の部分を生かすことにこだわりすぎていたようです。

Answer 3

コンマを数値出力の前に出力させるようにし、「最初だけ」コンマを出力しないようにしましょう

Comments

[yamasakisan]

ありがとうございます！
最初のカンマのみ、falseを使って出力しないようにすればできるのではないか、と思いましたが、
うまく表現できませんでした。

[y_waiwai]

boolean f=false;
for(int i:array) {
if(f){ System.out.print(","); }
f=true;
System.out.print(i);
}
System.out.println("");

[yamasakisan]

ありがたいです。ありがとうございます。