Javaで最後の演算子を削除したいです。

Question

はじめまして。
現在Javaを勉強している者です。

拡張forを使って足し算をし結果を出力する式を作成しているのですが、最後の数字にも「+」がついてしまい、どう削除するのか悩んでいます。

この場合、if文で分岐して「numの個数が最大の場合+を削除する」などの処理をした方がいいのでしょうか。

実行結果は以下の通りです。

【実行結果】
--- 合計
式は:1+2+=3
--- 合計
式は:1+2+3+=6
--- 合計
式は:2+4+6+8+=20
--- 合計
式は:1+3+5+7+9+=25

【ソースコード】
class MyClass {
public void printStuff(int...values) {
// 文字列を表示
System.out.println("--- 合計");
// 文字列を表示
System.out.print("式は:");

	int sum = 0;

	// 拡張forでのループ処理
	for (int num : values) {
		System.out.print(num + "+");

		sum += num;
	}
	System.out.println("=" + sum);
}

}

// メインクラスを作成
class task3_04 {
// メインメソッドを定義
public static void main(String[] args) {

	// task3_04クラスをインスタンス化
	MyClass mc = new MyClass();

	mc.printStuff(1, 2);
	mc.printStuff(1, 2, 3);
	mc.printStuff(2, 4, 6, 8);
	mc.printStuff(1, 3, 5, 7, 9);
}

}

皆様のご回答をお待ちしております。
よろしくお願いいたします。

Comments

[fuzzball]

拡張forを使わないといけないのでしょうか？あと、コードは ``` で囲って下さい。

Answer 1

加算する数が正の数だけだというなら(負の数入ってたらどのみち変な式になるが)
「和が0の時、つまり最初に+を出力しない」という方法も取れますし

int sum = 0;

// 拡張forでのループ処理
for (int num : values) {
    if (sum != 0) {
        System.out.print("+");
    }
    System.out.print(num);
    sum += num;
}
System.out.println("=" + sum);

逐一出力せず、StringJoinerを使うという方法もあります。

int sum = 0;
StringJoiner joiner = new StringJoiner("+");

// 拡張forでのループ処理
for (int num : values) {
    joiner.add(String.valueOf(num));
    sum += num;
}
System.out.println(joiner + "=" + sum);

Answer 2

2 つの方法をかいてみました。
2 番目の方法では、 負の値があった場合でも
式が 1+-2 でなく、 1-2 のようになるようにしています。

package teratail;

import java.util.ArrayList;
import java.util.List;
import java.util.stream.Collectors;

class MyClass {
    public void printStuff(int... values) {
        // 文字列を表示
        System.out.println("--- 合計");
        //  int sum = Arrays.stream(values).sum();

        List<Integer> list = new ArrayList<Integer>();
        for (int num : values) {
            list.add(num);
        }
        int sum = list.stream().mapToInt(i -> i).sum();
        String exp = list.stream().map(Object::toString).collect(Collectors.joining("+"));
        System.out.println("式は:" + exp + "=" + sum);
        
        String str = "";
        for (int num : values) {
            str += (num < 0) ? "" + num : "+" + num;
        }
        if (str.startsWith("+")) {
            str = str.substring(1);
        }
        System.out.println("式は:" + str + "=" + sum);
    }
}

// メインクラスを作成
public class task3_04 {
    // メインメソッドを定義
    public static void main(String[] args) {

        // task3_04クラスをインスタンス化
        MyClass mc = new MyClass();
        mc.printStuff(1, 2);
        mc.printStuff(1, 2, 3);
        mc.printStuff(2, 4, 6, 8);
        mc.printStuff(1, 3, 5, 7, 9);
        mc.printStuff(-1, 1, 2, -2, 0);
    }
}

実行例：

参考情報

• Javaでカンマ区切りで文字列連結するいくつかの方法

https://qiita.com/shisama/items/b27d16b3aeb1baf055b1

• ArrayListを文字列に変換する最も良い方法

https://code.i-harness.com/ja/q/92479

Comments

[TomoShimizu]

ご丁寧な回答ありがとうございます。

String Joinerを使うのですね！
全く思いつかなかった書き方でした・・・。

参考情報もありがとうございます。
とても助かります！

Answer 3

最初の要素だけ特別扱いすればわりあいすっきりします。

int sum = values[0];
System.out.print(values[0]);
    
for(int i = 1; i < values.length; ++i) {
    System.out.print("+" + values[i]);
    sum += values[i];
}
System.out.println("=" + sum);

拡張for文を使いたかったら Arrays.copyOfRange が使えそうです。

コードの書き方について

teratailには、コードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押してください。

Answer 4

以前同様の問題(拡張for文で最後の要素だけ処理を変更したい)にぶつかった時、当方は以下を用いました。

JavaのListの繰り返しで一番最後の要素だけ処理を変えたい

見ての通り拡張for文ではなく通常のfor文なのですが、
カウント用の変数なんかを持たせることなく最後の要素の時だけ処理を変更することができます。