Question

#include <stdio.h>

int main()
{
    char text[1000], alpha;
    int n;

    printf("Please type in text:\n");
    scanf("%[^\n]s", text);

    printf("\nRotation number:  "); // rotates letters to the right.
    scanf("%d",&n);
    printf("\n");

    n = n % 26; // to wrap around alphabet.

    int i = 0;
    while (text[i] != '\0')
    {
        if((text[i] >= 'a' && text[i] <= 'z'))
        {
            alpha = text[i];

            text[i] += n;

这是我不明白为什么它不起作用的部分：

            if(text[i] > 'z')

            {
                text[i] = 'a' + (n - (26 % (alpha - 'a')));
            }

直到字母'd'为止。 'f'只给'\ 200'。

关于为什么我的代码不起作用的任何想法？

        }
        i++;
    }

        printf("Encrypted text:\n%s", text);

    return 0;
}

Answer 1

这部分你不明白为什么不起作用：

if(text[i] > 'z')
{
    text[i] = 'a' + (n - (26 % (alpha - 'a')));
}

将简单地用

解决

if(text[i] > 'z')
{
    text[i] -= 26;
}

更新您正在使用char whick可能已签名，因此将密码（例如20）添加到z将生成一个＆gt; 128，即否定。

我建议这个修正案

int alpha;   // changed from char

//...

alpha = text[i] + n;
if (alpha > 'z')
    alpha -= 26;
text[i] = alpha;

Answer 2

我认为你想要的是

text[i] = (text[i] - 'a' + n) % 26 + 'a';

执行此操作

text[i] - 'a'  // converts text[i] to a number between 0 and 25
+ n            // add the cipher value
% 26           // wrap as necessary so the value is between 0 and 25
+ 'a'          // convert back to a letter between 'a' and 'z'

所以循环应该看起来像这样

for ( int i = 0; text[i] != '\0'; i++ )
{
   if ( text[i] >= 'a' && text[i] <= 'z' )
      text[i] = (text[i] - 'a' + n) % 26 + 'a';
}

凯撒的C密码：似乎无法包裹后面的字母

2 个答案: