c 管道读取字符串为空
c pipe read string is empty
我只想创建 2 个新的分叉(子进程),它们会依次 放置它们的名称。所以首先他们需要一些管道中的字符串来检查一些东西。来看代码:
char myname[] = "ALOAA";
int main ()
{
int fds[2];
pid_t pid;
pipe(fds);
pid = fork();
if(pid == 0)
{
strcpy(myname, "first");
}
else
{
pid = fork();
if(pid == 0)
{
strcpy(myname, "second");
}
}
if(strcmp(myname, "ALOAA") != 0)
{
char readbuffer[1025];
int i;
for (i = 0; i < 2 ; i++)
{
//printf("%s\n", myname);
close(fds[0]);
write(fds[1], myname, strlen(myname));
while(1)
{
close(fds[1]);
int n = read(fds[0], readbuffer, 1024);
readbuffer[n] = 0;
printf("%s-alihan\n", readbuffer);
if(readbuffer != myname)
break;
sleep(1);
}
//printf("%s\n", myname);
}
}
return 0;
}
所以第一个进程会将她的名字写入管道。之后,将检查管道中是否有任何新字符串。第二个也一样。但是我从 read() 函数中得到了空字符串。所以它打印成那样
-alihan
-alihan
我无法解决问题。
However I got empty string from read() function [...] I couldn't get the problem.
@MikeCAT 通过他在评论中的观察指出了这个问题,即每个 child 在尝试读取它之前都会关闭 fds[0]
。之间没有其他文件分配相同的 FD,因此读取失败。你不测试失败。
不测试读取失败是一个严重的问题,因为您的程序不仅无法识别它——结果会表现出未定义的行为。出现这种情况(至少)有两个原因:
read()
将通过 returning -1
指示失败,您的程序将通过尝试 out-of-bounds 写入(到 readbuffer[-1]
).
如果我们忽略由 (1) 产生的 UB,此后我们仍然有程序从完全未初始化的数组 readbuffer
中读取(因为 read()
调用和赋值将设置该数组的任何元素的值)。
总的来说,您需要学习检查库函数调用的 return 值以了解错误情况的原则,至少在任何重要的是否发生错误的地方(对于大多数调用而言)。例如,您对 pipe()
、fork()
和 write()
的使用也存在此问题。在某些情况下,您想要检查 printf()
系列函数的 return 值,并且您通常想要检查输入函数的 return 值——而不仅仅是 read()
,但是 scanf()
、fgets()
等..
第三,您对 read()
和 write()
的用法不正确。您犯了一个常见的错误,即假设(成功时)write()
将可靠地写入所有指定的字节,并且 read()
将读取所有已写入的字节,直到指定的缓冲区大小。尽管这在实践中通常适用于通过管道交换短消息,不能保证。一般来说,write()
可能只执行部分写入,read()
可能只执行部分读取,原因不详,不可预测。
要成功写入,通常必须准备好在循环中重复 write()
调用,使用 return 值来确定从何处(或是否)开始下一次写入。要成功读取完整的消息,通常必须准备好类似在循环中重复 read()
调用,直到已将所需数量的字节读入缓冲区,或者直到满足其他一些终止条件,例如结束正在到达的文件。我想您不会忘记,许多形式的这种情况需要预先了解要读取的字节数。
我只想创建 2 个新的分叉(子进程),它们会依次 放置它们的名称。所以首先他们需要一些管道中的字符串来检查一些东西。来看代码:
char myname[] = "ALOAA";
int main ()
{
int fds[2];
pid_t pid;
pipe(fds);
pid = fork();
if(pid == 0)
{
strcpy(myname, "first");
}
else
{
pid = fork();
if(pid == 0)
{
strcpy(myname, "second");
}
}
if(strcmp(myname, "ALOAA") != 0)
{
char readbuffer[1025];
int i;
for (i = 0; i < 2 ; i++)
{
//printf("%s\n", myname);
close(fds[0]);
write(fds[1], myname, strlen(myname));
while(1)
{
close(fds[1]);
int n = read(fds[0], readbuffer, 1024);
readbuffer[n] = 0;
printf("%s-alihan\n", readbuffer);
if(readbuffer != myname)
break;
sleep(1);
}
//printf("%s\n", myname);
}
}
return 0;
}
所以第一个进程会将她的名字写入管道。之后,将检查管道中是否有任何新字符串。第二个也一样。但是我从 read() 函数中得到了空字符串。所以它打印成那样
-alihan
-alihan
我无法解决问题。
However I got empty string from read() function [...] I couldn't get the problem.
@MikeCAT 通过他在评论中的观察指出了这个问题,即每个 child 在尝试读取它之前都会关闭 fds[0]
。之间没有其他文件分配相同的 FD,因此读取失败。你不测试失败。
不测试读取失败是一个严重的问题,因为您的程序不仅无法识别它——结果会表现出未定义的行为。出现这种情况(至少)有两个原因:
read()
将通过 returning-1
指示失败,您的程序将通过尝试 out-of-bounds 写入(到readbuffer[-1]
).如果我们忽略由 (1) 产生的 UB,此后我们仍然有程序从完全未初始化的数组
readbuffer
中读取(因为read()
调用和赋值将设置该数组的任何元素的值)。
总的来说,您需要学习检查库函数调用的 return 值以了解错误情况的原则,至少在任何重要的是否发生错误的地方(对于大多数调用而言)。例如,您对 pipe()
、fork()
和 write()
的使用也存在此问题。在某些情况下,您想要检查 printf()
系列函数的 return 值,并且您通常想要检查输入函数的 return 值——而不仅仅是 read()
,但是 scanf()
、fgets()
等..
第三,您对 read()
和 write()
的用法不正确。您犯了一个常见的错误,即假设(成功时)write()
将可靠地写入所有指定的字节,并且 read()
将读取所有已写入的字节,直到指定的缓冲区大小。尽管这在实践中通常适用于通过管道交换短消息,不能保证。一般来说,write()
可能只执行部分写入,read()
可能只执行部分读取,原因不详,不可预测。
要成功写入,通常必须准备好在循环中重复 write()
调用,使用 return 值来确定从何处(或是否)开始下一次写入。要成功读取完整的消息,通常必须准备好类似在循环中重复 read()
调用,直到已将所需数量的字节读入缓冲区,或者直到满足其他一些终止条件,例如结束正在到达的文件。我想您不会忘记,许多形式的这种情况需要预先了解要读取的字节数。