2013-02-07 84 views
0

我遇到以下代码的问题。我正在使用叉子(创建流程)进行实验室任务。它是一个简单的程序,它应该从键盘读取输入,然后将其读取/写入FIFO并显示其写入的内容和字节。我的fork fifo c代码无法正常工作

当我运行它时,一切似乎都很好,直到我输入一些文本。父打印消息显示正常,但是从不显示子打印消息,直到我输入了第二条消息,它总是说它写了80个字节,尽管我知道它没有,并且有一些奇怪的到处都是特殊字符

这里是关于程序应该如何可执行文件: 的Linux:http://www.mediafire.com/?6806v24q6lz7dpc QNX:http://www.mediafire.com/?a9dhiwmrlx2ktkp

到目前为止我的代码:

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

int main(int argc, char *argv[]) { 

    char FifoName[] = "fifoDan"; 
    int fd; 
    pid_t retval; 
    int size_read; 
    char buff[80]; 
    int size_written; 

    mknod(FifoName, S_IFIFO | 0666, 0); 

    // Check if its not equal to zero (ie: child process = 0) 
    if (retval = fork()) { 

      printf ("Parent: Waiting for writers \n"); 

      if(fd = open(FifoName, O_RDONLY) == -1) { 
          perror("Could not read the FIFO"); 
        return EXIT_FAILURE; 
      } 

      printf ("Parent: Received a writer \n"); 

      do { 

          int strsize; 

          size_read = read(fd, buff, sizeof(buff)); 

          printf("Parent: read %d bytes: %s \n", size_read, buff); 
          fflush(stdout); 

          strsize = strlen(buff); 

          // put a '\0' at the end of the data 
          buff[strsize] = '\0'; 


        } while(size_read > 0); 


        close(fd); 
        waitpid(retval, NULL, NULL); 

        if(unlink(FifoName) != -1) { 
          return EXIT_SUCCESS; 
        } else { 
          return EXIT_FAILURE; 
        } 


    } else { 


     printf ("Child pid %d waiting for readers \n", getpid()); 
     fflush(stdout); 

     if(fd = open(FifoName, O_WRONLY) == -1) { 
          perror("Could not read the FIFO"); 
        return EXIT_FAILURE; 
      } 

     printf ("Child: Got a reader, enter some stuff:\n"); 
     fflush(stdout); 

     while(fgets(buff, 80, stdin) != NULL) { 

        int strsize; 

        strsize = strlen(buff); 

        if(strsize < 80) { 
          buff[strsize] = '\0'; 
        } 

        size_written = write(fd, buff, sizeof(buff)); 

        printf ("Child: wrote %d bytes \n", size_written); 
        fflush(stdout); 

      } 

      close(fd); 

    } 


} 

回答

4

这条线(例如)不会做什么你认为它确实如此:

if(fd = open(FifoName, O_WRONLY) == -1) 

相等运算符有更高的precedence比赋值运算符,让你做什么,其实是这样的:

if(fd = (open(FifoName, O_WRONLY) == -1)) 

,当你想这样的:

if((fd = open(FifoName, O_WRONLY)) == -1) 

与缓冲区中读取的问题/写是因为你写整个缓冲区:

size_written = write(fd, buff, sizeof(buff)); 

更改为仅使用strsize代替实际字符串。

+0

+1。这是我27年来一直在写C的经验,虽然在if语句中写作业是合法的,但它也是创建错误的绝妙方式。一个简单赋值的额外代码行不会减慢编译速度。 :) –

+0

感谢您的帮助。它解决了我遇到的问题。但现在它说,父母和孩子每次输入内容时都会读/写80个字节,而不是计算从键盘输入的字节数。也许我传递了错误的价值? – user1930558

+0

@ user1930558这是因为你_write_ 80字节。由于在'write'调用中使用'sizeof(buff)',因此您可以编写整个缓冲区。 –