無駄な処理を省きたい。

Question

お世話になっております。
現在以下の課題を解き進めております。

/*1から50まで順に数を表示する。
ただし、その数が3の倍数の付く数字の場合、数字の後に!を表示する。

5の倍数の場合は、数字の後に?と表示する。
両方の条件に合致した場合、数字の後に!?と表示する。*
/

僕なりにはそこそこ綺麗に書けたと思ったのですが、
3でも5の倍数でもない数値まで全てString変換するのはロスだと言われました。

しかしifの後に文字列変換を行うと、3の倍数のifで宣言して5でも宣言・・・
となると、3の倍数かつ5の倍数の時に"!"が"?"で上書きされてしまいます。

どうか記号がつかない数字に手を加えず!と?を綺麗に加える方法を教えて下さい。

public class Forked {
	public static void main(String[] args) {
		for (int i = 1; i <=50; i++){
			//String変数numberにiを変換
			String number = String.valueOf(i);

			//3の倍数の時numberに"!"を追加
			if (i % 3 == 0){
				number = number + "!";
			}

			//5の倍数の時numberに"?"を追加
			if(i % 5 == 0){
				number = number + "?";
			}
			//表示
			System.out.println(number);
		}
	}

}

Answer 1

標準出力は文字変換しないといけないのだから、不正解ではないと思いますよ。
と、あっさり言うと変に言いくるめられそうですから、ソースコードを眺めましょう。

Print.io.PrintStream::println(int)
https://docs.oracle.com/javase/jp/6/api/java/io/PrintStream.html#println(int)

これ、ソースを見ると、こうなってます。

    public void println(int x) {
        synchronized (this) {
            print(x);
            newLine();
        }
    }

    public void print(int i) {
        write(String.valueOf(i));
    }

    private void write(String s) {
        try {
            synchronized (this) {
                ensureOpen();
                textOut.write(s);
                textOut.flushBuffer();
                charOut.flushBuffer();
                if (autoFlush && (s.indexOf('\n') >= 0))
                    out.flush();
            }
        }
        catch (InterruptedIOException x) {
            Thread.currentThread().interrupt();
        }
        catch (IOException x) {
            trouble = true;
        }
    }

ちなみにStringの場合は、こう。

    public void println(String x) {
        synchronized (this) {
            print(x);
            newLine();
        }
    }

    public void print(String s) {
        if (s == null) {
            s = "null";
        }
        write(s);
    }

標準出力を行う場合、必然的に文字列に変換して出力を行います。
つまり、3、5で割り切れない数値を文字列に変換するという行為はロスではありません。
変換するタイミングが異なるだけです。

よって、指摘は絶対的な正解とは言い切れません。

Comments

[R.lawliet]

わざわざコメントをありがとうございます。
printのメソッド内の処理は裏で変換されていたのですね。
とても勉強になります。

まだ右も左もわからないので指摘されたらそれが絶対正解と思い込んでしまいがちです。
人に見せる正解と自分の中の正解は別に持っていると良いかもしれませんね。

Answer 2

だとしたら，付ける記号を別に用意しておけばいいのではないでしょうか？

public class Forked {
    public static void main(String[] args) {
        for (int i = 1; i <=50; i++){
            //記号を格納するString変数を用意
            String signal = "";

            //3の倍数の時signalに"!"を追加
            if (i % 3 == 0){
                signal += "!";
            }

            //5の倍数の時signalに"?"を追加
            if(i % 5 == 0){
                signal += "?";
            }
            //表示
            System.out.println(i + signal);
        }
    }

}

余談ですが，printlnメソッドはString型，boolean型，char型以外の引数を取った時
String.valueOfメソッドが内部で実行されるので，ロスかと言われるとやや疑問です．

---

更に余談
数値→文字列変換は色々方法がありますが，その速さに関して面白い実験をした方がいました．
→あえて言うほどではない 数値 ⇔ 文字列変換 Java編

Comments

[R.lawliet]

swordoneさん、返答ありがとうございます。
実はその案はフローチャートの時点で落とされてしまいました。

System.out.printを最後に纏めてみましょう。
とのコメントで、現在のコードに辿り着いております。

質問の段階で書いておくべきでしたね、申し訳ありません。

[R.lawliet]

追記ありがとうございます。
おぉ、シンプルなコードで良いですね♪
BAは譲ってしまいましたがこちらを参考に修正を加えてみたいと思います。
ありがとうございます。

お二人方の解答から中身の無い宣言の方法を学ばせて頂きました。

Answer 3

3の倍数のif分で変換しますが、変数はifの外で持っていれば可能ですよ
5の倍数のif分の中ではStringのインスタンスが生成されていなければ、生成するようにします。

public class Forked {
    public static void main(String[] args) {
        for (int i = 1; i <=50; i++){
            String number = null;
            if (i % 3 == 0){
                number = String.valueOf(i); // String変換
                number = number + "!";
            }
            if(i % 5 == 0){
                if(number == null){ // 3の倍数ではなかったらここでString変換
                    number = String.valueOf(i);
                }
                number = number + "?";
            }
            if(number != null){ // Stringに変換せれていればnumberを出力
                System.out.println(number);
            }else{ // Stringに変換されていなければiを出力
                System.out.println(i);
            }
            
        }
    }

}

これでいかがでしょうか？

Comments

[R.lawliet]

素晴らしいですね!
感動しました。

なるほど、nullで作っておくことでif内での変換が可能になるのですね。
丁寧に書いて頂けたのでとても理解しやすかったです♪
ベストアンサーをお送り致します。

Answer 4

改行を行わない System.out.print() と
改行を行う System.out.println() を使い分けてはいかがでしょうか。

for (int i = 1; i <=50; i++){
	System.out.print(i);
	if (i%3==0) {
		System.out.print("!");
	}
	if (i%5==0) {
		System.out.print("?");
	}
	System.out.println();
}

Answer 5

public class Forked {
    public static void main(String[] args) {
        for (int i = 1; i <= 50; i++) {
            // 3 の倍数なら"!"
            // 5 の倍数なら"?"
            // 5 の倍数なら"!?"                                           
            String optstr = ((i % 3 == 0) ? "" : "!") + ((i % 5 == 0) ? "" : "?");
            System.out.printf("%d%s\n", i, optstr);
        }
    }
}