Caesar密码在C中不起作用

你的代码有两个简单的错误。首先,一个类型的变量 char 占用 8 bits 内存中具有值范围形式 -128 到 127 . 在某些情况下,加密字符的值可能最终大于 127 所以你需要使用 unsigned char 作为数据类型,取值范围为 0 到 255 相反

因此,您的函数声明应该如下所示: char shiftchar(unsigned char ch);

也不需要这行代码: ch = ch; 你也没有从函数中返回结果的加密字符。

最后一个函数应该如下所示:

char shiftchar(unsigned char ch){
    if( ((ch > 64) && (ch< 91)) || ((ch > 96) && (ch < 123)) ){
        ch = ch + shift;
        if(ch > 90 && ch < 104){
            ch = ch - 90 +64;
        }
        else if(ch > 122 && ch < 136){
            ch = ch -122 +96;
        }
    }
    return ch;
}

到目前为止,您的代码将按预期运行,但为了圆满完成所有工作,我们需要修改角色数组的大小。您可以设置数组的大小 char str[] 在里面 main 25 .

永远不要忘记看不见的东西 NULL 在用于初始化数组的任何字符串文字的末尾终止字符。

char str[26] = "This is the original text"; ,您还可以避免忘记 \0 ,您可以这样做: char str[] = "This is the original text"; 因此,将任务留给编译器来确定数组的大小;)

你好像忘了 char 字符串真的被调用了 以null结尾 字节字符串。这个 这一点非常重要,因为所有标准函数都希望知道字符串何时何地结束。

你的阵列 str 太小了,适合终结者。改为使其成为26个元素的数组。或者不要显式指定大小,让编译器设置它。

我测试了你的代码,除了 unsigned 问题,你错过了 return 中的声明 shiftchar 作用

char shiftchar(unsigned char ch){
    if( ((ch > 64) && (ch< 91)) || ((ch > 96) && (ch < 123)) ){
        ch = ch + shift;
        if(ch > 90 && ch < 104){
            ch = ch - 90 +64;
        }
        else if(ch > 122 && ch < 136){
            ch = ch -122 +96;
        }
    }
    else{
    ch = ch;
    }
    return ch;
}