C语言替换密码程序异常：部分字符输出乱码求助

问题根源 #

你的代码里两个全局数组letters和word被声明为零长度数组：

char letters[] = {};
char word[] = {};

C语言中，零长度数组没有分配有效内存空间，当你往letters[i]（i从0到25）或word[j]写入数据时，会越界访问内存，破坏程序其他区域的存储内容，这就是为什么前几个字符看似正常，后续出现随机乱码的原因——内存被非法写入后，数据被污染了。

修复步骤 #

1. 固定letters数组的长度：密钥固定为26个字符，直接声明为char letters[26];，确保有足够空间存储完整密钥。
2. 移除冗余的word数组：这个中间数组完全没必要，转换后的字符可以直接打印，省去额外的内存操作。
3. 优化大小写转换逻辑：保证明文的大小写与密文一致，比如明文是小写时，将密钥对应字符转为小写；明文是大写时转为大写，避免密钥混合大小写导致输出异常。
4. 移除冗余的空格判断：else if ((argv[1][1]) == ' ')这个判断没有意义，后续的长度检查和字符类型检查已经能拦截非法输入。

修复后的完整代码 #

#include <cs50.h>
#include <ctype.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

// 声明足够长度的数组存储密钥
char letters[26];

int main(int argc, string argv[])
{
    if (argc != 2)
    {
        printf("Missing command-line argument\n");
        return 1;
    }
    else if (strlen(argv[1]) != 26)
    {
        printf("Key must contain 26 characters.\n");
        return 1;
    }
    
    for (int i = 0, n = strlen(argv[1]); i < n; i++)
    {
        if (!isalpha(argv[1][i]))
        {
            printf("Key must only contain alphabetic characters.\n");
            return 1;
        }
        // 先统一转存为大写，方便后续转换
        letters[i] = toupper(argv[1][i]);
        
        // 检查重复字符
        for (int j = 0; j < i; j++)
        {
            if (letters[j] == letters[i])
            {
                printf("Key must not contain repeat characters.\n");
                return 1;
            }
        }
    }
    
    string ptext = get_string("plaintext: ");
    printf("cyphertext: ");

    for (int j = 0, len = strlen(ptext); j < len; j++)
    {
         if (isupper(ptext[j]))
         {
            int idx = ptext[j] - 'A';
            // 保持大写输出
            printf("%c", letters[idx]);
         }
         else if (islower(ptext[j]))
         {
            int idx = ptext[j] - 'a';
            // 转为小写输出
            printf("%c", tolower(letters[idx]));
         }
         else
         {
              printf("%c", ptext[j]);
         }
    }
    printf("\n");
    return 0;
}

关键修改说明 #

• 把letters数组长度固定为26，彻底避免内存越界问题。
• 直接打印转换后的字符，去掉了word数组这个不必要的中间层，减少内存操作风险。
• 统一将密钥转为大写存储，转换时根据明文的大小写输出对应大小写的密文，逻辑更清晰、结果更符合预期。
• 用isupper/islower替代手动ASCII范围判断，代码更健壮、可读性更强。

内容的提问来源于stack exchange，提问作者Patman