实施“猫”；在c中-wwwmc？ （我的代码有什么问题）

Question

我的代码在某种程度上运行良好。不过我有两个问题。一，在将文件打印到标准输出结束时，它给了我一个分段错误。

第二，我无法使用 fputs 打印数据，因为我立即遇到分段错误。因此，为了修复它，我使用了 put ，它可以很好地打印它，但在每行后面添加一个 '\n' ，使文本单行间隔以及末尾的段错误。

#include <stdio.h>
#include <string.h>

void concat(char *arg){

    char string[256];
    FILE *fp = fopen(arg, "r");

    while(!feof(fp)){
        fgets(string, 256, fp);
        //fputs(string, fp);
        puts(string);
    }

    fclose(fp);

}

void stdincat(){

    char string[256];   
    while(!feof(stdin)){
        fgets(string, 256, stdin);
        fputs(string, stdout);
    }
}

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

    char argvalues[256][40];

    if(argv[1] == NULL)
        stdincat();
    else if(argv[1] != NULL){
        int i;

        for(i=1;i<=(argc);i++){
            concat(argv[i]);
        }
    }

    return 0;
}

Answer 1

您在 concat 中注释掉的对 fputs 的调用正在尝试写入 fp，您打开它只是为了阅读，所以毫不奇怪它不会/不起作用。

除此之外，您的阅读循环： while(!feof(fp)) { （以及类似的，除了来自 stdin 而不是 fp）遵循熟悉的、广泛存在的反模式——像这样的循环不能/不会/不能正常工作。您通常希望在同一操作中阅读并测试是否成功：

while(fgets(string, 256, stdin))
    fputs(string, stdout);

编辑：我还应该提到，我希望避免在 concat 和 stdincat 中重复代码。我宁愿传递 FILE * 作为参数进行读取，因此您可以使用相同的代码从 stdin 或其他任意文件读取：

// Warning: untested code.
// error checking omitted for readability.
void catfile(FILE *file) { 
    char line[256];
    while (fgets(line, sizeof(line), file))
        puts(line);
}

int main(int argc, char **argv) { 
    int i;
    if (argc == 1)
        catfile(stdin);
    else for (int i=1; i<argc; i++) {
        FILE *infile = fopen(argv[i], "r");
        catfile(infile);
        fclose(infile);
    }
    return 0;
}

最后，我需要注意的是，如果您只是要复制整个文件，fgets 可能不是完成这项工作的最有效方法。 fread 可能更合适。当您这样做时，您也可以使用二进制模式读取和写入。

Answer 2

您不能在仅用于读取而打开的流上写入，您应该使用 stdout 作为函数 concat 中的 fput