Javaで計算式を作りたい

Question

パラメータ（in）でたとえば、35421　という数字がきたとして、
その35421　を一ずつ取得したいです。
もし、数字が1と4の場合は×３をする。
もし、数字が２，３，５の場合は×９をする。
そして最終的には掛けた数字の一桁のみを表示させたいです。

35421　の場合、
３９＝２７　　//7のみ表示
５９＝４５　　//5のみ表示
４３＝１２　　//2のみ表示
２９＝１８　　//8のみ表示
１*３＝３　　 //3のみ表示

最終的にはSystem.out.printlnで７５２８３を表示させたい。

以下を実行したところSystem.out.println();何もコンソールに表示されませんでした。
どのようにコードを書いたらいいか教えていただきたいです。

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;
}

Comments

[yureighost]

コンパイルエラーが出てます。
int value02=value % 10;
のvalueが定義されていない(value01の間違い？)

new String (out, total)がエラー(String型にString(String, int)の型のコンストラクタはありません)
これはさすがに何やりたいかわからないので、ちゃんとコンパイルが通るソースを提示してください。

[m.ts10806]

コード内にSystem.out.println()がありませんが

[kazuma-s]

与えられた数の中に 0、6～9 があった場合どうするんですか？

Answer 1

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

Comments

[退会済みユーザー]

ありがとうございます！

Answer 2

文字列で来るのなら、１文字づつとってきてintに変換すればいいです

提示のコードではvalueとvalue02が意味不明ですが。
また、コンソールに出力するコードも見当たりませんねー

Comments

[y_waiwai]

しかし、回答後の質問の修正でマイナス食らうってのはどーにかできんもんかｗ

[退会済みユーザー]

え、すいません、気が付かないで押してしまってました。。わざとでないです！失礼いたしました。。m(__)m

[退会済みユーザー]

こんな機能があるなんて知りませんでした、、

[y_waiwai]

いえいえ、teratailのシステム的な不都合もあったりしますんで、質問くんのせいとはいえないです。

＃回答では質問の編集の通知が来ないってのがガンですわな

[退会済みユーザー]

教えていただけてほんとよかったです、、ずっと気づかなかったかもです。。(;'∀')

Answer 3

次のコードを研究してみてください。

import java.util.HashMap;
import java.util.Map;

public class Main {

	private static Map<Character, Character> map = new HashMap<Character, Character>() {
		{
			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);
		}
	}
}

Comments

[退会済みユーザー]

コードありがとうございます！ホントいろいろな書き方があるのですね！
private static Map<Character, Character> map = new HashMap<Character, Character>()
→この書き方が初めて目にしました。。勉強になりました！ありがとうございます！

Answer 4

以下を実行したところSystem.out.println();何もコンソールに表示されませんでした。

質問に書くコードは、実際にコンパイルして実行したコード全体にしてください。
質問への追記・修正依頼にも応えてください。

次のコードはどうでしょうか？

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"));
	}
}

実行結果

7 5 2 8 3 

追記

最終的にはSystem.out.printlnで７５２８３を表示させたい。

こう書いてあったので、数字の間にスペースが要るのかと思ったら、
全角数字で書いてあるからそう見えただけのようでした。

• で文字列を連結したり、StringBuilder を使うよりも char の配列を

使ったほうが効率が良いと思って書き直してみました。

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) に対応する文字への変換表です。

Comments

[退会済みユーザー]

追記・修正依頼ですが、すいません、理解するのに時間がかかってしまい。。

StringBuilder を使うよりも char の配列を
使ったほうが効率が良いと思って書き直してみました。→yureighostさんのコードも見ていて、いろいろググってみたのですが、なぜ配列のほうが効率がいいのかわからないのですが、すいませんが教えていただけないでしょうか？

[退会済みユーザー]

ありがとうございます。
String t = "0387256789";
すいません。。こちらのコードがどういう意味を持つのかがわからないです。。

[momon-ga]

一桁同士の掛け算の答えの１桁目をつなげたものだと思います。
1,4は3倍、2,3,5は9倍、それ以外を1倍にしてと、0～9まで行って
0,3,18,27,12,45,6,7,8,9 の、下一桁をつなげたものが、
"0387256789";になっていて、計算でなく数字を機械的に置き換えるように定義しているみたいですね。

[退会済みユーザー]

1,4は3倍、2,3,5は9倍、それ以外を1倍にしてと、0～9まで行って→
なるほどですね！なんの数字かわかりませんでした。。
ありがとうございます！

Answer 5

こんな感じで。
問題点についてはamiyaさんが大体突っ込んでくれてますが、
forループが10固定なので例で出してる35421を入れるとArrayIndexOutOfBoundsExceptionになります。
またわざわざ一文字ずつ数字を取り出してるのに10で割るのは全く無意味です。
私はchar配列を作るやり方でやってみました。

    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);
    }

Comments

[退会済みユーザー]

ご回答ありがとうございます。質問なんですが、String型ではなく、なぜ配列型にされたのでしょうか？

[yureighost]

Stringから一文字取り出すcharAtと言うメソッドが返すのがchar型であること。
それとStringとchar配列は相互変換できるので。
javaでは文字列操作の方法の一つとしてよく使われます。

[退会済みユーザー]

なるほど！ありがとうございます！

Answer 6

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);

  }
}

Comments

[退会済みユーザー]

ご回答ありがとうございます。
return out;
// return new String (out, total);
結果をoutだけ返すのも、結果をoutとtotalの2つ返すのも結果が同じになったのですが、
これは1つ返すのと２つ返すのと違うのになぜ実行結果が同じになるのでしょうか？