C语言向指定PID发送信号的实现代码正误判断与优化问询

你提供的代码完全不符合需求，存在多处语法错误、逻辑错误和功能偏差，具体问题如下：

• 语法错误：自定义的signal_handler函数中引用了未定义的全局变量a、b，且函数末尾语句错误使用逗号代替分号，代码无法通过编译
• 头文件引入错误：引入了无用的<wait.h>，同时缺少atoi需要的<stdlib.h>、进程相关接口需要的<sys/types.h>、<unistd.h>，以及错误处理需要的<errno.h>
• 参数校验逻辑错误：需求要求传入2个参数（目标PID、信号编号），加上程序名总参数计数argc应为3，你的校验条件会漏判仅传1个参数的错误场景，此时访问argv[2]会触发空指针异常
• 功能完全偏离需求：需求要求向用户指定的PID发送指定信号，你的代码是给自己进程发送固定的SIGUSR1信号，完全没有使用用户输入的两个参数
• 退出码不规范：参数错误场景返回了代表执行成功的0，不符合程序设计惯例
• 缺少结果反馈：没有判断kill接口的执行结果，无法告知用户操作是否成功，不符合需求要求
• 信号处理逻辑完全多余：你实现的是发信号的工具程序，不需要注册信号处理函数，且信号处理函数中调用printf属于不可重入操作，本身就不符合信号处理的安全规范

修正后的可运行代码 #

#include <stdio.h>
#include <stdlib.h>
#include <sys/types.h>
#include <signal.h>
#include <errno.h>
#include <string.h>

int main(int argc, char *argv[]) {
    // 参数校验：必须传入目标PID和信号编号两个参数
    if (argc != 3) {
        printf("使用方法：%s <目标进程PID> <信号编号>\n", argv[0]);
        return 1;
    }

    pid_t target_pid = atoi(argv[1]);
    int sig_num = atoi(argv[2]);

    // 校验参数合法性
    if (target_pid <= 0 || sig_num <= 0) {
        printf("错误：PID和信号编号必须为正整数\n");
        return 2;
    }

    // 发送信号并判断结果
    int ret = kill(target_pid, sig_num);
    if (ret == 0) {
        printf("操作成功：已向进程%d发送信号%d\n", target_pid, sig_num);
        return 0;
    } else {
        printf("操作失败：向进程%d发送信号%d失败，错误原因：%s\n", target_pid, sig_num, strerror(errno));
        return 3;
    }
}

核心改进点 #

• 删除了所有无用的信号处理逻辑和多余头文件，补充了必要的依赖头文件
• 修正了参数校验逻辑，新增了清晰的使用提示
• 完全对齐需求实现功能：直接使用用户输入的PID和信号编号调用系统接口发送信号
• 新增了系统调用返回值判断，结合错误码输出明确的操作结果，符合需求的结果告知要求
• 规范了退出码：不同错误场景返回不同的非0退出码，成功返回0，符合POSIX程序设计规范
• 代码整体简化，删除冗余逻辑，可读性大幅提升

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