C系统调用open（）

Question

我在使用open()时遇到麻烦。它总是返回-1，我不知道代码出了什么问题。它一直在说：

r1: No such file or directory

但是txt文件与C程序位于同一目录中。

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

#define BUFFSIZE 256

char upper(char c);

int main(void){

    int fd1, read1, fd2, write2;
    char *buffer[BUFFSIZE];

    fd1 = open("minuscole.txt", O_RDONLY);
    if (fd1 < 0) { perror("r1"); exit(1); }
    read1 = read(fd1, buffer, BUFFSIZE);
    close(fd1);

    printf("%d", read1);
    if(fd2 = open("maiuscole.txt", O_WRONLY | O_CREAT | O_TRUNC, 0644) > 0){
        write2 = write(fd2, buffer, BUFFSIZE);
    }
    close(fd2);

    return 0;
}

我希望它创建一个名为："maiuscole.txt"的文件并写入其中的内容："minuscole.txt"。

Answer 1

尽管您说输入文件与源文件位于同一位置，但这并不意味着您也在同一位置运行可执行文件。错误消息清楚地表明您不是。

要修复，请确定您从中运行可执行文件的目录，请确保您的输入文件位于该目录中，并且您有权在其中创建输出文件。

如注释中所述，您对fd2的赋值语句不正确，因为赋值运算符的precedence低于比较运算符。为了保持一致性，建议您更改代码以使其与fd1所使用的样式匹配。请注意，0也是有效的文件描述符。

fd2 = open("maiuscole.txt", O_WRONLY | O_CREAT | O_TRUNC, 0644);
if(fd2 >= 0){
    //...

read和write的第二个参数分别取void * / const void *，因此它掩盖了您在传递char *[]（衰减为char **）时遇到的错误职能。这不是您的意图，因为char **只能存储sizeof(char *)字节。您打算最多存储BUFFERSIZE个字节。将buffer更改为char的数组，而不是char *的数组。

char buffer[BUFFSIZE];

从输入文件中读取数据后，read调用的返回值是读取了多少字节。您只应将那么多字节写入输出文件，而不是整个缓冲区。

    write2 = write(fd2, buffer, read1);

如果仅读取10个字节，则缓冲区中前10个字节之后的字节未初始化，因此不应将其写入输出中。

还有其他情况和错误检查供您考虑。我不会提供所有解决方案，但您应该考虑：

• 您的输入文件可能大于BUFFSIZE。
• 即使文件大于read，对BUFFSIZE的调用也可能返回小于BUFFSIZE的状态。
• 对write的呼叫返回的可能少于read1。

最后，是关于close的注释。您并不孤单地忽略close的返回值，即使在生产代码中，这也是很常见的事情。但是，检查close是否成功仍然是一种好习惯。在某些情况下，它可能会失败，因为描述符之前已关闭。您可能想知道是否会发生这种情况，因为它表明程序中存在某种逻辑错误。尽管关闭无效的文件描述符是无害的，但将来，您可能会意外关闭该程序的其他部分正在使用的文件描述符。