将一个文件复制到另一个文件（Unix/C）？答案

【问题标题】：Copying one file to another(Unix/C)?将一个文件复制到另一个文件（Unix/C）？
【发布时间】：2016-01-31 21:49:53
【问题描述】：

我编写了以下代码来将一个文件复制到另一个文件。尽管代码有效，但代码仍会打印两条错误消息。为什么是这样？我是 Unix 和 C 编程的完全初学者（虽然我以前使用过 C++），所以任何尽可能详细的帮助都会很棒。谢谢！

int main(int argc, char *argv[])
{
    int n;
    char buf[4096];
    while ((n=read( open(argv[1], O_RDONLY) , buf, 4096))>0)
    {
        if (write(creat(argv[2], S_IREAD | S_IWRITE ), buf, n)!=n)
            printf("Error writing to file.\n");
    }
    if (n<0)
        printf("Error reading from file.\n");
    exit(0);
}

【问题讨论】：

您在每次循环迭代时都重新打开这两个文件。
“代码有效” 但您是否真的检查过目标文件与源文件的内容是否相同？请使用文件大小 > 4096 进行测试。
您认为从哪里开始读取新打开的文件？如果文件已经存在，create 函数会做什么？
read() 和 write() 返回 ssize_t，而不是 int。它们不是一回事。
@WeatherVane 是的，我运行代码后，目的地和原始文件的内容一样。

标签： c unix cp file-copying

【解决方案1】：

您在每次迭代中打开文件并尝试在每次迭代中creat 文件。

因此，除了第一次迭代之外，所有后续写入都将失败。它可能“似乎有效”，因为您的输入文件包含少于 4096 个字节。所以第一次调用 write 是让它看起来像所有东西都被复制了。如果使用超过 4096 个字节的输入，您只会看到前 4096 个字节（假设 read() 和 write() 都不会失败）。

如果write() 一直成功（例如，您在循环之外有creat()），那么open() 调用会不断打开同一个文件，并且可能是无限循环，否则您的系统将耗尽文件描述符并返回一个无效的文件描述符，read() 将失败。

长话短说：不要写那样的代码:)

将两个调用移至循环外的open() 和creat()：

int fd = open(argv[1], O_RDONLY);
if (fd == -1) { 
   perror("open");
   exit(1);
}

int fd2 = creat(argv[2], S_IREAD | S_IWRITE );
if (fd2 == -1) { 
   perror("write");
   exit(1);
}

while ( (n=read( fd , buf, 4096)) > 0 )
{
    if ( write(fd2 , buf, n) != n )
        printf("Error writing to file.\n");
}

【讨论】：

另外，添加一些错误检查。检查 fd（文件描述符）是否为负数。通过命令行的文件路径可能无效。

【解决方案2】：

上面的答案已经发现，但是应该避免使用 creat()，因为 create() 是一个过时的函数。创建（文件名，模式）；相当于打开（文件名，O_WRONLY | O_CREAT | O_TRUNC，模式）；

http://www.gnu.org/software/libc/manual/html_node/Opening-and-Closing-Files.html

代码最好写成：

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

#define BUFSIZE 4096

int main(int argc, char **argv)
{
    int infd, outfd;
    int n;
    char *infile = NULL;
    char *outfile = NULL;
    int src_flags, dest_flags;
    mode_t dest_perms;
    char buf[BUFSIZE];

    if (argc < 3) {
        fprintf(stderr, "At least src and dest files must be specified\n"); /*Never rely on users' inputs*/
        /*Print usage information if possible*/
        exit(EXIT_FAILURE); 
    }

    infile = argv[1];
    outfile = argv[2];  /*Presuming the order*/            

    src_flags = O_RDONLY;
    dest_flags = O_CREAT | O_WRONLY | O_TRUNC;  
    /*creat() is equivalent to open(fname, O_CREAT | O_WRONLY | O_TRUNC, mode)*/
    dest_perms = S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH | S_IWOTH; /*rw-rw-rw-*/

    infd = open(infile, src_flags);
    if (infd == -1) {
        perror("cannot open src ");
        exit(EXIT_FAILURE);
    }

    outfd = open(outfile, dest_flags, dest_perms);
    if (outfd == -1) {
        perror("cannot open dest ");
        exit(EXIT_FAILURE);
    }

    while ((n = read(infd, buf, BUFSIZE)) > 0) {
        if (write(outfd, buf, n) != n) {
            fprintf(stderr, "failed to write buf\n");
            goto exit_failure;
        }
    }

    if (n == -1) {
        fprintf(stderr, "read() failed\n");
        goto exit_failure;
    }

exit_failure:
     if (infd) {
        close(infd);
     }

    if (outfd) {
        close(outfd);
    }

    printf("Copy successful\n");

    exit(EXIT_SUCCESS);
}

【讨论】：