linux c 简单的多进程出问题了，求帮助

x6666f

1. /* 这个是linux下最简单的多进程代码了，相当与简单的命令执行终端，fork产生新的进程，exevcp为新进程产生映像，makeargs函数为exevcp函数提供参数，不知为何makeargs函数提供的参数exevcp函数总不能执行，请各位大侠帮忙看看哈
   
2. */
   
3. #include<stdio.h>
   
4. #include<stdlib.h>
   
5. #include<unistd.h>
   
6. #include<string.h>
   
7. #include<errno.h>
   
8. #include<sys/wait.h>
   
9. #include<sys/types.h>
   
10. #define MAX_PR 10
    
11. #define MAX_ARG 100
    
12. #define DEBUG
    
13. void makeargs(char *buf,char ***args) //为exevcp函数提供参数，就是把buf里的字符串分别提取出来
    
14. {
    
15.   int i,j=0;
    
16.   int flag=0;
    
17.   *args=(char **)malloc(sizeof(char *)*10);
    
18.   if(!args)
    
19.    perror("args memory malloc error");
    
20.   for(i=0;i<10;i++)
    
21.    {
    
22.      (*args)[i]=(char *)malloc(sizeof(char)*100);
    
23.      if(!(*args)[i])
    
24.      perror("args [] memory malloc error");
    
25.    }
    
26.   for(i=0;buf[i]!='\0';)
    
27.   {
    
28.     for(;buf[i]==' '||buf[i]=='\t';i++)
    
29.     {
    
30.        ;
    
31.     }
    
32.     int start=i;
    
33.     for(;buf[i]!=' '&&buf[i]!='\t'&&buf[i]!='\0';i++)
    
34.     {
    
35.       flag=1;
    
36.     }
    
37.     if(flag==1)
    
38.     {
    
39.       int k,l=0;
    
40.       for(k=start;k<i;k++)
    
41.       {
    
42.         (*args)[j][l]=buf[k];
    
43.         l++;
    
44.       }
    
45.       (*args)[j][l]='\0';
    
46.       j++;  
    
47.     }
    
48.    flag=0;
    
49.   }
    
50. (*args)[j]=NULL;
    
51. }
    
52. int main(int argc,char **argv)
    
53. {
    
54.   int pr_count=0,pr_limit=MAX_PR;
    
55.   char buf[MAX_ARG],**args;
    
56.   int i,child[MAX_PR];
    
57.   pid_t myoldid,myid,childid;
    
58.   myid=getpid();
    
59.   myoldid=myid;
    
60.   while(myid==myoldid) //为了防止子进程也运行此循环而设立的条件
    
61.   {
    
62.     fprintf(stdout,"--->");
    
63.     fflush(stdin);
    
64.     fgets(buf,MAX_ARG,stdin);
    
65.     makeargs(buf,&args);
    
66. #ifdef DEBUG
    
67.     fprintf(stdout,"input is :");
    
68.     for(i=0;args[i]!=NULL;i++)
    
69.      fprintf(stdout,"%s ",args[i]);
    
70. #endif
    
71.     if(strcmp(args[0],"exit")==0)
    
72.     {
    
73.        return 0;
    
74.     }
    
75.     if(pr_count==pr_limit)
    
76.      {
    
77.        fprintf(stdout,"please wait...\n");
    
78.        for(i=0;i<pr_count;i++)
    
79.        {
    
80.         childid=waitpid(child[i],NULL,WNOHANG);
    
81.         if(childid==0)
    
82.          pr_count--;  
    
83.        }
    
84.      }
    
85.     childid=fork();
    
86.     child[pr_count]=childid;
    
87.     if(childid==-1)
    
88.      perror("failed to fork");
    
89.     else
    
90.     {
    
91.      myid=getpid();
    
92.      execvp(args[0],args); //为新进程产生映像
    
93.      pr_count++;
    
94.     }
    
95.    }
    
96.   return 0;
    
97. }
    

复制代码

MMMIX

回复 1# x6666f


    兩點建議：

• string 的解析建議用 strtok() 或類似的函數。
• fork() 在 parent process 中返回 child process ID（非 0），在 child process 中返回 0，區分它們一般用這個。
  

hanzhenlll

1.  少一些 必备的打印调试信息....

2. 个人认为在非必要的情况下，不要随便 强转类型，这样会得到一些你意想不到的结果。。。



3. 编译的时候 自己加上 -Wall 参数 先检查下....

x6666f

谢谢2楼三楼的建议，我找到错误了，并且知道了这些的用法习惯。