文字列の右回転、左回転：与えられた文字列ABCを回転1で１つ右に回転、回転−1で１つ左に回転させるための式が正しくなさそうです。

Question

回転(rotation)　による暗号化(encryption), 復号化(decryption) のコードを作っています。
文字列中の文字を一つずつ呼び出す暗号化、復号化のコードは作れたのですが、文字の回転を行うletter operatorのメソッドがうまく機能していません。
文字はアルファベットのみ、大文字小文字を識別して回転したいのですが、下記の間違いがあればご指摘いただけますでしょうか？

letterOperator、rotation = -58でHelloを暗号化しようとすると、ByffiになるはずがB_ffiとなってしまいます。アルファベットの終わりの数文字（v辺りから）になるところの回転がうまくいきません。

private static char letterOperator(char ch, int rotation) {

 if (Character.isUpperCase(ch)) {
	 if ((rotation % 26) + ch > 90) {
		 return (char) ((rotation % 26) + ch - 26);
	 } else {
		 return (char) ((rotation % 26) + ch);
	 }
 } else if (Character.isLowerCase(ch)) {
	 if ((rotation % 26) + ch > 122) {
		 return (char) ((rotation % 26) + ch - 26);
	 } else {
		 return (char) ((rotation % 26) + ch );
	 }
 } else {
	 return ch;
 }
	

お力を貸していただけると助かります。よろしくお願いいたします。

Comments

[dodox86]

「回転による暗号化」と書かれても閲覧した方には話がいきなり過ぎて分からないと思います。例えば"abc"と言う文字列があったら、+1で右方向に回転で"cab", -1で左方向に回転で"bca" 、と言う意味でしょうか。

[linka.]

拙文で失礼致しました。
dodox86様の仰る通りです。
参考に編集させていただきます。

[kazuma-s]

文字列の回転ではなく、文字列の各文字をアルファベット内で回転ですよね。
"abc" で +1 なら "bcd"、-1 なら "zab"。
質問のコードからはそうしたいことが分かります。

Answer 1

こんな風にも書けます。

class Test {
	private static char letterOperator(char ch, int rotation)
	{
		if (!Character.isAlphabetic(ch)) return ch;
		char a = Character.isUpperCase(ch) ? 'A' : 'a';
		return (char)((rotation % 26 + 26 + ch - a) % 26 + a);
	}

	public static void main(String[] args) {
		String s = "Hello";
		StringBuilder sb = new StringBuilder();
		for (int i = 0; i < s.length(); i++)
			sb.append(letterOperator(s.charAt(i), -58));
		String s2 = sb.toString();
		System.out.println(s2);
	}
}

追記
"Hello" が "Byffi" になってほしいのに "B_ffi" になる。
ということは、'e' が 'y' になってほしいのに '_' になる。

質問のコードをトレースしてみればいいでしょう。
なぜやらないのか不思議です。

ch = 'e'、rotation = -58 で letterOperatorメソッドの実行を始めます。

ch は大文字(UpperCase) ではなくて、小文字(LowerCase)なので、
if ((rotation % 26) + ch > 122) に行きます。
rotation % 26 は、-58 % 26 で、-6。
ch は 'e' で 101。
したがって、(rotation % 26) + ch は 95。

95 は 122 より大きくないので
return (char)((rottion % 26) + ch); に行き、95 を返す。
95 は 'y' ではなく、'_' です。

小文字('a'～'z')の範囲は 97～122 です。
95 はその範囲から外れています。

rotation が負の場合、% 26 の結果も 0 か負です。
ch にそれを足すと 'a' より前になることがあるのです。
次のようにしないといけないでしょう。

    if ((rotation % 26) + ch > 'z')
	    return (char) ((rotation % 26) + ch - 26);
    if ((rotation % 26) + ch < 'a')
	    return (char) ((rotation % 26) + ch + 26);
    else 
	    return (char) ((rotation % 26) + ch);

コード中に 90 とか 122 と書くと何のことか分かりません。
'Z' や 'z' と書きましょう。

Comments

[linka.]

ご丁寧に追記までいただきありがとうございます。
まさに、仰る通りで、問題解決できました! 'a'や'z'はASCII 変換前の数字で書いていましたが、そのままの表示で大丈夫なのですね。大変勉強になりました。

Answer 2

マイナスを指定したときの計算が

return (char) ((rotation % 26) + ch );

ではなく、

return (char) ((rotation % 26) + ch + 26);

ではないかと。