system()使用遇到的问题及解决方式
一、遇到的问题：
问题1，函数单独调用system()时，使用没有问题。当做性能压力测试时，对调用函数进行快速的多次调用，发现system()是性能瓶颈。
问题2，在某些情况下，system()调用会卡住线程，导致程序运行阻塞。
二、分析原因：
1，源码
int system(const char * cmdstring)
{
pid_t pid;
int status;
if(cmdstring == NULL)
{
return (1);
}
if((pid = fork())<0)
{
status = -1;
}
else if(pid = 0)
{
execl("/bin/sh", "sh", "-c", cmdstring, (char *)0); -exit(127); //子进程正常执行则不会执行此语句 }
else
{
while(waitpid(pid, &status, 0) < 0)
{
if(errno != EINTER){ status = -1; break;
}
}
}
return status;
} 1，system()会调用fork()产生子进程，由子进程来调用/bin/sh-c string来执行参数string字符串所代表的命令，此命令执行完后随即返回原调用的进程。
2，在调用system()期间SIGCHLD 信号会被暂时搁置，SIGINT和SIGQUIT 信号则会被忽略。 返回值 =-1:出现错误 =0:调用成功但是没有出现子进程 >0:成功退出的子进程的id 如果system()在调用/bin/sh时失败则返回127，其他失败原因返回-1。若参数string为空指针(NULL)，则返回非零值。如果system()调用成功则最后会返回执行shell命令后的返回值，但是此返回值也有可能为 system()调用/bin/sh失败所返回的127，因此最好能再检查errno 来确认执行成功。
这样看实际上system()函数执行了三步操作：
1.fork一个子进程；
2.在子进程中调用exec函数去执行command；
3.在父进程中调用wait去等待子进程结束。 对于fork失败，system()函数返回-1。 如果exec执行成功，也即command顺利执行完毕，则返回command通过exit或return返回的值。 （注意，command顺利执行不代表执行成功，比如command："rm debuglog.txt"，不管文件存不存在该command都顺利执行了） 如果exec执行失败，也即command没有顺利执行，比如被信号中断，或者command命令根本不存在，system()函数返回127. 如果command为NULL，则system()函数返回非0值，一般为1.
2，分析
从源码可以分析出，system()实际上是调用fock()创建子进程执行命令，然后waitpid()回收子进程。
问题1，是不停的对system()进行调用，从源码可以看出，如果这样使用的话确实会因为waitpid()的存在而导致程序执行效率的减慢
问题2，我看过调用栈，基本会阻塞到waitpid，具体原因未知。可能跟waitpid()本身有关，后续会介绍waitpid()相关内容
三、问题处理
提供两种思路：
1，尽量不要使用system()函数，用popen()函数替代。根据网上其他人的说法，popen()返回值的判断要比system()简单。我使用popen()函数替代掉system()后，再压力测试时性能有明显提升，具体原因未分析。
2，对system()进行封装
如果不想深究system()为什么会执行失败，或者挂死、阻塞线程可以使用以下函数封装你的system()调用。原理是调用system()期间SIGCHLD 信号会被暂时搁置，SIGINT和SIGQUIT 信号则会被忽。
int my_system(const char *cmd_line) ;
typedef void (*sighandler_t)(int);
int my_system(const char *cmd_line)
{
int ret = 0;
sighandler_t old_handler;
old_handler = signal(SIGCHLD, SIG_DFL);
ret = system(cmd_line);
signal(SIGCHLD, old_handler);
return ret;
} 也可以把popen()封装成int my_system(const char * cmd)
int my_system(const char * cmd)
{
FILE * fp;
int res; char buf[1024];
if (cmd == NULL)
{
printf("my_system cmd is NULL!\n");
return -1;
}
if ((fp = popen(cmd, "r") ) == NULL)
{
perror("popen");
printf("popen error: %s/n", strerror(errno)); return -1;
}
else
{
while(fgets(buf, sizeof(buf), fp))
{
printf("%s", buf);
}
if ( (res = pclose(fp)) == -1)
{
printf("close popen file pointer fp error!\n"); return res;
}
else if (res == 0)
{
return res;
}
else
{
printf("popen res is :%d\n", res); return res;
}
}
}