Javaで計算式を作りたい

Question

パラメータ（in）でたとえば、35421　という数字がきたとして、
その35421　を一ずつ取得したいです。
もし、数字が1と4の場合は×３をする。
もし、数字が２，３，５の場合は×９をする。
そして最終的には掛けた数字の一桁のみを表示させたいです。

35421　の場合、
３*９＝２７　　//7のみ表示
５*９＝４５　　//5のみ表示
４*３＝１２　　//2のみ表示
２*９＝１８　　//8のみ表示
１*３＝３　　  //3のみ表示


最終的にはSystem.out.printlnで７５２８３を表示させたい。

以下を実行したところSystem.out.println();何もコンソールに表示されませんでした。
どのようにコードを書いたらいいか教えていただきたいです。

```java
public String Num(String in){
   String out;
   int total=0;

for(int i=0;i< 10; i++){
    int value01=Integer.parseInt(in.substring(i,i+1));
    int value02=value01 % 10;

    out += String.valueOf(value02);

    switch(value02){
      case 1:
      case 4:
        total = value02 * 3;
        break;
      case 2:
      case 3:
      case 5:
        total = value02 * 9;
        break;

    }
  }
  return out;
}
```

Accepted Answer

こんな感じで。
問題点についてはamiyaさんが大体突っ込んでくれてますが、
forループが10固定なので例で出してる35421を入れるとArrayIndexOutOfBoundsExceptionになります。
またわざわざ一文字ずつ数字を取り出してるのに10で割るのは全く無意味です。
私はchar配列を作るやり方でやってみました。
```java
    public static void main(String[] args) {
        System.out.print("入力してください> ");
        Scanner sc = new Scanner(System.in);
        String str = sc.nextLine();
        System.out.println(Num(str));
    }

    public static String Num(String in){
    	char[] out = new char[in.length()];

    	for(int i=0;i< in.length(); i++){
    	    int value01=in.charAt(i) - '0';

            String total = "";
    	    switch(value01){
    	      case 1:
    	      case 4:
    	        total = String.valueOf(value01 * 3);
    	        break;
    	      case 2:
    	      case 3:
    	      case 5:
    	        total = String.valueOf(value01 * 9);
    	        break;
    	    }
    	    out[i] = total.charAt(total.length() - 1);
    	}
        return new String(out);
    }
```

Answer

次のコードを研究してみてください。 ```java import java.util.HashMap; import java.util.Map; public class Main { private static Map map = new HashMap() { { put('0', '0'); put('1', '3'); // 1 * 3 -> 3 put('2', '8'); // 2 * 9 -> 18 - > 8 put('3', '7'); // 3 * 9 -> 27 - > 7 put('4', '2'); // 4 * 3 -> 12 -> 2 put('5', '0'); // 5 * 9 -> 45 -> 5 put('6', '6'); put('7', '7'); put('8', '8'); put('9', '9'); } }; private static int task(int num) { String ans = ""; String s = "" + num; for (int i = 0; i < s.length(); i++) { ans += map.get(s.charAt(i)); } return Integer.parseInt(ans); } public static void main(String[] args) { int data[] = { 35421 }; for (int num : data) { int ret = task(num); System.out.println(ret); } } } ```

Answer

> 以下を実行したところSystem.out.println();何もコンソールに表示されませんでした。質問に書くコードは、実際にコンパイルして実行したコード全体にしてください。質問への追記・修正依頼にも応えてください。次のコードはどうでしょうか？ ```Java class Test { public static String Num(String in) { StringBuilder sb = new StringBuilder(); String t = "0387256789"; for (int i = 0; i < in.length(); i++) sb.append(t.charAt(in.charAt(i) - '0') + " "); return sb.toString(); } public static void main(String[] args) { System.out.println(Num("35421")); } } ``` 実行結果 ```text 7 5 2 8 3 ``` **追記** > 最終的にはSystem.out.printlnで７５２８３を表示させたい。こう書いてあったので、数字の間にスペースが要るのかと思ったら、全角数字で書いてあるからそう見えただけのようでした。 + で文字列を連結したり、StringBuilder を使うよりも char の配列を使ったほうが効率が良いと思って書き直してみました。 ```Java class Test { public static String Num(String in) { int n = in.length(); char[] out = new char[n]; String t = "0387256789"; for (int i = 0; i < n; i++) out[i] = t.charAt(in.charAt(i) - '0'); return String.valueOf(out); } public static void main(String[] args) { System.out.println(Num("35421")); } } ``` **追記2** for文に、無意味な `k = 0, ` が入っていたのに気付きませんでしたか？削除しました。 out[i] = '3'; は、単なる代入です。 sb.append('3'); は、StringBuilderクラスのインスタンス sb についての appendメソッドの呼び出しです。sb や '3' を引数として渡す作業が入ります。 StringBuilder sb は内部にバッファを持っていて、今回の append でそれがバッファオーバーフローしないかどうかチェックしてから '3' をバッファに入れます。その後、this すなわち sb を返すという作業も必要です。 StringBuilder を使うほうが効率が悪いのは明白です。 String t = "0387256789"; ですが、"0123456789" と比べてみてください。 '1' → '3' '2' → '8' '3' → '7' '4' → '2' '5' → '5' という対応になっています。入力文字列の i番目の文字 in.charAt(i) に対応する文字への変換表です。

Answer

問題点をいくつか。
ループの中で一文字取り出して、整数にしよう、というのはよいのですが、
ループが10回固定では、10文字ちょうど以外の文字列が来た時におかしくなると思います。
また、1文字を数値に変換しているのですから、%10する処理は不要です。
そして、計算を行う前に、outに繋いでしまっているので、計算結果が結果に反映されません。
(JAVAでは文字列に変換せずとも、Stringとintの+演算は自動的にStringの連結して処理されます)
計算結果を1桁だけにする処理も必要です。

Answer

```java
public class test {

  public static String Num(String in) {
    String out = "";
    int total = 0;

    for (int i = 0; i < in.length(); i++) {
      int value01 = Integer.parseInt(in.substring(i, i + 1));
      int value02 = value01 % 10;

      switch (value02) {
        case 1:
        case 4:
          total = value02 * 3;
          break;
        case 2:
        case 3:
        case 5:
          total = value02 * 9;
          break;
        default:
          total = value02;
      }

      String totalString = String.valueOf(total);
      String lastKeta = totalString.substring(totalString.length() - 1);
      out += lastKeta;
    }

    return out;
    // return new String (out, total);
  }

  public static void main(String[] args) {
    String num = Num("35421");
    System.out.println(num);

  }
}
```

Answer

文字列で来るのなら、１文字づつとってきてintに変換すればいいです

提示のコードではvalueとvalue02が意味不明ですが。
また、コンソールに出力するコードも見当たりませんねー

関連した質問