編集履歴

回答編集履歴

余計な } を削除(3か所)

2020/10/03 23:53

投稿

スコア8224

test CHANGED Viewed

@@ -284,8 +284,6 @@
 	}
-}
 ```
 次に積(product)の総和(sum)を求めるコードを追加します。
@@ -354,8 +352,6 @@
 	}
-}
 ```
 **追記2**
@@ -390,8 +386,6 @@
 	}
-}
 ```
 sum の最後の2桁については、ちょっと違うやり方にしてみました。

char[] ではなく String を使うコードを追加

2020/10/03 23:53

投稿

kazuma-s

スコア8224

test CHANGED Viewed

@@ -357,3 +357,41 @@
 }
 ```
+**追記2**
+「効率」とか「ひらめき」とかはよくないですね。
+char[] out ではなく、素直に String out を使うコードを示します。
+```Java
+	public String encrypt(String str) {
+		int[] mul = { 1, 3, 7, 9, 7, 3, 1, 3, 9, 7, 1 };
+		String out = "";
+		int sum = 0;
+		for (int i = 0; i < 11; i++) {
+            int product = (str.charAt(i) - '0') * mul[i];
+			sum += product;
+			out += (char)(product % 10 + '0');
+		}
+		String t = "0987654321";
+		return out + t.charAt(sum / 10 % 10) + t.charAt(sum % 10);
+	}
+}
+```
+sum の最後の2桁については、ちょっと違うやり方にしてみました。

indent の再修正

2020/10/03 15:38

投稿

kazuma-s

スコア8224

test CHANGED Viewed

@@ -334,7 +334,7 @@
 		for (int i = 0; i < 11; i++) {
-      		int product = (out[i] - '0') * mul[i];
+            int product = (out[i] - '0') * mul[i];
 			out[i] = (char)(product % 10 + '0');

インデントの修正

2020/10/03 13:50

投稿

kazuma-s

スコア8224

test CHANGED Viewed

@@ -296,7 +296,7 @@
 		for (int i = 0; i < 11; i++) {
-   		int product = (out[i] - '0') * mul[i];
+            int product = (out[i] - '0') * mul[i];
 			out[i] = (char)(product % 10 + '0');

「日本語にする」とはどういうことかを追加

2020/10/03 13:49

投稿

kazuma-s

スコア8224

test CHANGED Viewed

@@ -159,3 +159,201 @@
 }
 ```
+**追記**
+やりたいことを次のような日本語で書いたとします。
+```text
+・文字列を変換して新しい文字列を作る。
+・入力文字列は、11個の数字である。
+・出力文字列は、13個の数字である。
+・変換表 { 1, 3, 7, 9, 7, 3, 1, 3, 9, 7, 1 } が用意されている。
+・入力文字列の各数字について、その値に変換表の同じ位置の値を掛け算して積を作る。
+・その積の一の位の数を出力文字の各文字とする。
+・また、その積の総和を求め、十の位の値を A、一の位の値を B とする。
+・A も B も 0 でなければ、それらの値を 10 - A、10 - B の値に置き換える。
+・A と B を出力文字の 12文字目と 13文字目とする。
+```
+プログラミング言語を知らない人でも、
+この手順書にしたがって、やりたいことができると思いませんか？
+さて、これを Java のプログラムに変換します。
+メソッド名は何でもいいんですが、暗号化だから encrypt にしましょう。
+入出力が文字列だから、`String encrypt(String str) {` で始めます。
+変換表は数値の並びなので、`int[] mul = { 1, 3, 7, ..., 1 }` です。
+掛け算に使うので multiply から mul にしましたが、何でもいいです。
+出力文字列をどうやって作るか考えないといけません。
+String out = ""; と空の文字列を用意し、ここに変換後の各文字を追加していく
+こともできますが、文字列の連結は効率が悪いので、
+char[] out = new char[11]; のように char の配列を用意して、その各要素に
+文字を入れていき、最後に out を String に変換しましょう。
+入力文字列から各文字を取り出して整数値にするのをどうするか考えないといけません。
+```Java
+	for (int i = 0; i < 11; i++) {
+		int val = str.charAt(i) - '0';
+		この val を変換して out[i] に入れる。
+	}
+```
+次のようにも書けますが、これより上のコードの方が簡単でしょう。
+```Java
+	for (int i = 0; i < 11; i++) {
+		int val = Integer.parseInt(str.substring(i, i+1));
+		この val を変換して out[i] に入れる。
+	}
+```
+ここで次ようなことがひらめきました。
+String である入力文字列 str を char[] にしておけば、
+charAtメソッドの呼び出しをせずに配列要素として文字を取り出せます。
+それには出力文字列用の out が流用できます。
+`char[] out = str.toCharArray():` とすればいいんです。
+こうすれば、out[i] から取り出した文字を変換して out[i] に戻すだけです。
+ここまで考えれば、次のようなコードが書けます。
+```Java
+	public String encrypt(String str) {
+		int[] mul = { 1, 3, 7, 9, 7, 3, 1, 3, 9, 7, 1 };
+		char[] out = str.toCharArray();
+		for (int i = 0; i < 11; i++) {
+            int product = (out[i] - '0') * mul[i];
+			out[i] = (char)(product % 10 + '0');
+		}
+		return String.valueOf(out);
+	}
+}
+```
+次に積(product)の総和(sum)を求めるコードを追加します。
+```Java
+		int sum = 0;
+		for (int i = 0; i < 11; i++) {
+   		int product = (out[i] - '0') * mul[i];
+			out[i] = (char)(product % 10 + '0');
+			sum += product;
+		}
+```
+積の総和の十の位と一の位の値を求めるコードは
+```Java
+		int a = sum / 10 % 10;
+		if (a != 0) a = 10 - a;
+		int b = sum % 10;
+		if (b != 0) b = 10 - b;
+```
+全部をまとめて、
+```Java
+	public String encrypt(String str) {
+		int[] mul = { 1, 3, 7, 9, 7, 3, 1, 3, 9, 7, 1 };
+		char[] out = str.toCharArray();
+		int sum = 0;
+		for (int i = 0; i < 11; i++) {
+      		int product = (out[i] - '0') * mul[i];
+			out[i] = (char)(product % 10 + '0');
+			sum += product;
+		}
+		int a = sum / 10 % 10;
+		if (a != 0) a = 10 - a;
+		int b = sum % 10;
+		if (b != 0) b = 10 - b;
+		return String.valueOf(out) + a + b;
+	}
+}
+```