1
#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;
}
4

2 回答 2

1

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

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

将简单地解决

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

您正在使用的UPDATEchar可能已签名,因此将密码(例如 20)添加到z将产生 > 128 的数字,即负数。

我建议这个修改

int alpha;   // changed from char

//...

alpha = text[i] + n;
if (alpha > 'z')
    alpha -= 26;
text[i] = alpha;
于 2015-08-22T18:16:02.817 回答
0

我想你想要的是

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';
}
于 2015-08-22T17:56:39.127 回答