Linux下C语言开发单行客户端-服务器计算器的运算字符串解析问题求助

你的字符串解析思路存在多处可直接导致运行异常的硬错误，具体包括：

• 源地址错位：代码中调用strrev(oper)反转了oper指向的运算串，但后续拷贝操作数时，strncpy的源参数写的是operation。如果两个变量不是指向同一块内存，相当于你计算了反转串的运算符位置长度，实际拷贝的却是完全没修改的原正序串，两者逻辑完全脱节。
• 变量赋值错误：取完第二个操作数的转换结果后，代码依然写的是op1 = atoi(operand2string)，正确逻辑应当赋值给op2，最终会导致op1被第二个操作数覆盖，op2从未被初始化。
• 字符串终止符缺失：使用strncpy按固定长度拷贝操作数字符串时，没有在拷贝内容的末尾手动补'\0'。atoi必须遇到合法的字符串终止符才会停止读取，否则会读到buffer外的随机内存值，计算结果完全不可控。
• 接口可移植性问题：strrev是GNU环境下的扩展函数，不属于C标准库定义，很多编译配置下需要手动开启_GNU_SOURCE宏才能正常链接，且该函数会直接修改原字符串内容，容易破坏现有buffer的可用数据。
• 思路冗余：为了取运算符后面的第二个操作数，把整个字符串反转、再对截取出的内容反转，整个流程绕了远路，额外增加了多步出错概率，完全没有必要。

对于固定单字符运算符、两个整数操作数的简单运算表达式，完全不需要使用拷贝、反转的操作，定位到运算符位置后直接拆分字符串即可，核心逻辑如下：

1. 遍历接收到的运算字符串，定位到+、-、*、/四类运算符的位置，直接拿到指向该运算符的指针。
2. 临时把运算符位置的字符替换成字符串终止符'\0'，此时运算符前面的内容自动成为合法C字符串，直接传入atoi即可得到第一个操作数，不需要额外拷贝。
3. 运算符指针的下一个位置，就是第二个操作数字符串的起始地址，因为整串本来就以'\0'结尾，直接传入atoi即可得到第二个操作数。
4. 按照识别到的运算符分支做整数运算即可。

参考核心实现代码：

// operation为接收到的运算串，确保手动在末尾补好'\0'
char *op_pos = NULL;
char op_type;
const char *valid_op = "+-*/";

// 定位运算符位置
for (int i = 0; operation[i] != '\0'; i++) {
    if (strchr(valid_op, operation[i])) {
        op_pos = &operation[i];
        op_type = operation[i];
        break;
    }
}
// 无合法运算符的分支直接返回错误
if (!op_pos) {
    send(client_fd, "Invalid expression", 18, 0);
    continue;
}

// 拆分两个操作数
char orig_op_char = *op_pos;
*op_pos = '\0'; // 临时截断字符串
int op1 = atoi(operation);
int op2 = atoi(op_pos + 1);

// 按运算符计算结果
int calc_result;
switch(op_type) {
    case '+': calc_result = op1 + op2; break;
    case '-': calc_result = op1 - op2; break;
    case '*': calc_result = op1 * op2; break;
    case '/':
        if (op2 == 0) {
            send(client_fd, "Division by zero", 16, 0);
            continue;
        }
        calc_result = op1 / op2;
        break;
}

• 调用recv()接收客户端传来的运算串后，必须手动在接收长度的下一个位置补'\0'，因为TCP接收函数不会自动为字符串添加终止符，否则后续所有字符串操作都会出现越界读取。
• 可以增加基础的合法性校验：确保运算符前后都存在有效数字，字符串中没有夹杂字母、特殊符号，避免异常输入导致服务端计算逻辑崩溃。

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