printf两次打印相同的语句

时间:2018-03-14 22:49:12

标签: c printf fork

下一个代码应该写入“file.txt”PID号,1表示父进程,0表示子进程。

我不确定代码是否正常工作但是我遇到了一个奇怪的问题,因为Printf()会造成麻烦。 我不明白为什么,但是printf两次打印相同的声明。

代码:

#include <stdio.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>


void print_pids(int fd,int n){
int i,p;
char msg[99];

for(i=n;i>0;i--){
    p=fork();
    if(p>0){
        sprintf(msg,"My generation is 1.My pid is %d\n",getpid());
        write(fd,msg,33);
        wait();
    }
    if(p==0){
        sprintf(msg,"My generation is 0.My pid is %d\n",getpid());
        write(fd,msg,33);
    }
    if(p<0){
        printf("cannot fork");
        exit(0);
    }
}


}

void main(){
    int fd;
    char buf[99];
    fd=open("file.txt",O_WRONLY,700);
    print_pids(fd,1);
    close(fd);
    fd=open("file.txt",O_RDONLY,700);
    read(fd,buf,35);
    printf(" %s\n",buf);
    close(fd);
    return;


}

而不是打印

 My generation is 1.My pid is 8022

打印

 My generation is 1.My pid is 8
 My generation is 1.My pid is 8

为什么会这样?

谢谢!

1 个答案:

答案 0 :(得分:3)

孩子不会退出print_pids(),因此会返回main()并打开文件,读取,打印,然后退出。父母也是如此,但只有在孩子死后才会这样做。如果您打印了执行打印操作的过程的PID,您将获得更好的信息。

使用具有固定大小缓冲区的write()也令人担忧。并且没有错误检查。

以下是您的代码的固定版本 - 更相关的标题,正确调用wait()(您不熟悉您的代码没有崩溃),打印额外的诊断信息,编写消息的全长,读取和打印消息的全长(即使没有空终止符),使用八进制数(0600)而不是十进制数(700 )用于权限等。

#include <fcntl.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/wait.h>
#include <unistd.h>

static void print_pids(int fd, int n)
{
    int i, p;
    char msg[99];

    for (i = n; i > 0; i--)
    {
        p = fork();
        if (p > 0)
        {
            sprintf(msg, "My generation is 1. My pid is %d\n", getpid());
            write(fd, msg, strlen(msg));
            int status;
            int corpse = wait(&status);
            printf("Child %d exited with status 0x%.4X\n", corpse, status);
        }
        if (p == 0)
        {
            sprintf(msg, "My generation is 0. My pid is %d\n", getpid());
            write(fd, msg, strlen(msg));
        }
        if (p < 0)
        {
            printf("cannot fork");
            exit(0);
        }
    }
}

int main(void)
{
    int fd;
    char buf[99];
    fd = open("file.txt", O_WRONLY|O_CREAT|O_TRUNC, 0600);
    print_pids(fd, 1);
    close(fd);
    fd = open("file.txt", O_RDONLY);
    int nbytes = read(fd, buf, sizeof(buf));
    printf("%.5d: %.*s\n", (int)getpid(), nbytes, buf);
    close(fd);
    return 0;
}

示例输出:

33115: My generation is 1. My pid is 33112
My generation is 0. My pid is 33115

Child 33115 exited with status 0x0000
33112: My generation is 1. My pid is 33112
My generation is 0. My pid is 33115

请注意获取完整长度的消息有助于您了解正在发生的事情。您的消息正在截断输出,因此您没有看到完整的PID。并且两个进程都写入文件(总共约72个字符)。 (可能会有一些时间问题要改变所看到的内容 - 我至少得到一个异常结果,其中只有一条“我的一代”消息,但我无法可靠地再现这些消息。)