为何我的else条件触发expression result unused [-Werror,-Wunused-value]错误？

你的代码报错核心原因是最后一个分支的语法完全不符合C语言规范：

else  (plainchar == ' ' || plainchar == ',' || plainchar == '!' );
{
    cipher = plainchar;
}

这里存在两个致命问题：

1. else不能直接跟随条件表达式，带条件的分支必须使用else if(...)语法
2. 末尾的分号会把后面的大括号变成独立代码块，和前面的else完全无关；同时中间的条件表达式计算后没有被使用，编译器开启-Werror后会将这个“未使用表达式结果”的警告升级为错误

之前没报错可能是因为未开启-Werror，但这段代码的逻辑本身就是错误的——不管条件是否成立，大括号里的代码都会执行，且else的写法违反语法标准。

修复后的完整代码 #

char ciphertext(char plainchar, string key)
{
    char cipher;
    // 处理大写字母
    if (isupper(plainchar)) 
    {
        int c = plainchar - 'A'; // 用字符常量代替魔法数字，可读性更好
        cipher = key[c];
        if (islower(cipher))
        {
            cipher = toupper(cipher);
        }
        return cipher;
    }
    // 处理小写字母
    else if (islower(plainchar))
    {
        int c = plainchar - 'a';
        cipher = key[c];
        if (isupper(cipher))
        {
            cipher = tolower(cipher);
        }
        return cipher;
    }
    // 处理数字与指定符号
    else if (isdigit(plainchar) || plainchar == ' ' || plainchar == ',' || plainchar == '!')
    {
        cipher = plainchar;
        return cipher;
    }
    // 处理其他所有未匹配字符（可根据需求调整逻辑）
    else
    {
        cipher = plainchar;
        return cipher;
    }
}

主要修改点说明 #

• 移除了无用变量a和b，直接用plainchar计算，简化代码结构
• 用'A'、'a'代替魔法数字65、97，提升代码可读性与可维护性
• 修正最后一个分支的语法错误，将错误写法改为标准的else if分支
• 每个分支都添加return语句，避免变量未初始化的风险，逻辑更清晰
• 将数字和符号的判断合并到一个分支，简化代码层级

关于调试器崩溃的说明 #

先确保代码编译通过（修复上述语法错误后），调试器崩溃大概率是因为编译阶段存在错误，导致生成的二进制文件异常；如果编译通过后仍崩溃，再检查key的长度是否为26（避免数组越界访问），或调试器的配置问题。

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