Question

我编写了两个程序，第一个有一个switch case并创建一个命名管道“pipeselect”来读取用户的值switch，第二个用命名管道读取值。但无论我输入1还是2，第二个程序也会显示“选择选项1”。我的脚本有什么问题？

计划1：

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <fcntl.h>

int main()
{
 char pipeselect[] = "/tmp/pipeselect";
 int bufs[2];
 int fds;
 int select1;

 /* Pipe Creation */
 if (access(pipeselect, F_OK) == -1) {
  fds = mkfifo(pipeselect, 0700);
  if (fds != 0) {
   printf("Pipe creation error\n");
   exit(1);
  }
 }

  printf("1. Option 1\n");
  printf("2. Option 2\n");
  printf("Please select an option: ");
  scanf("%d", &select1);

if (select1 == 1 || select1 == 2)
{
  if ((fds = open(pipeselect, O_WRONLY)) < 0) {
    printf("Pipe open error\n");
    exit(1);
  }

  bufs[0] = select1;     // put value entered by user into buffer
  write(fds, bufs, 1);   // write 1 byte from the buffer
  close(fds);

  printf("Option %d is selected\n", select1);
}
else {
  printf("Wrong Input!\n");
}

unlink(pipeselect);
exit(0);
}

计划2：

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <fcntl.h>

int main()

{
 char pipeselect[] = "/tmp/pipeselect";
 int bufs[2];
 int fds;
 int select1;

  if ((fds = open(pipeselect, O_RDONLY)) < 0) {
    printf("Pipe open error\n");
    exit(1);
  }

  select1 = read(fds, bufs, 1);   // write 1 byte from the buffer
  printf("Option %d is selected\n", select1);
  close(fds);

exit(0);
}

Answer 1

这是因为read()会返回成功读取的字节数，在您的情况1中，因为您正在阅读1。猜测事情是如何运作的并不是一个好主意，有一半时间你会错。

您必须做的是阅读库函数文档以了解它们应该如何使用，例如，您忽略了scanf()的返回值。

你应该尝试这个¹，而不是

write(fds, bufs, 1);

跟着

if (write(fds, &select1, sizeof(select1)) != sizeof(select1))
    fprintf("error writing to pipe: %d:%s\n", errno, strerror(errno));
/* Inlucde <errno.h> and <string.h> for this error message to work */

在另一个程序中，更改

select1 = read(fds, bufs, 1);

到

if (read(fds, &select1, sizeof(select1)) == sizeof(select1))
    printf("Option %d is selected\n", select1);

此外，检查scanf()返回的内容，而不是假设select1已初始化并使用它，以防您的程序未调用未定义的行为。

¹ _{注意，如果在发送这样的数据时字节顺序不一样，它将无法直接工作。在这种情况下它显然没问题，但如果它是netwrok代码，这不是一个很好的方法，因为你应该考虑到endianness。除此之外，这没关系。}

将用户选择的值发送到具有命名管道的其他进程

1 个答案: