-1

我要将文件从输入的源复制到输入的目标。输入目的地后大约 2 秒，我收到分段错误错误。输出文件已创建，因此fopen()可以正常工作。

我上网查了一下，看到很多c=getc(fp)。对于这个问题，我更喜欢fread()andfwrite()因为它更基本一些。

代码

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

//void rmnewline(char *string);
void remove_last_newline(char *str);

int main()
{
  char x[3];
  x[0]='y';
  int ch;

  while(x[0]=='y'||x[0]=='Y')
  {
    char source[256], destination[256];
    int *a;

    printf("Enter a source file: ");
    fgets(source, 256, stdin);
    if (source[254] == '\n' && source[255] == '\0') { while ( (ch = fgetc(stdin)) != EOF && ch != '\n'); }
    remove_last_newline(source);

    printf("Enter a destination file: ");
    fgets(destination, 256, stdin);
    if (destination[254] == '\n' && destination[255] == '\0') { while ( (ch = fgetc(stdin)) != EOF && ch != '\n'); }
    remove_last_newline(destination);

    FILE *sp, *dp;

    sp = fopen(source, "r");
    if (sp == NULL) { printf("ERROR: Could not open source file."); exit(1); }
    dp = fopen(destination, "w");
    if (dp == NULL) { printf("ERROR: Could not open destination file."); exit(1); }
    while(1)
    {
      fread(a, 1, 1, sp);
      if (feof(sp))
      break;
      fwrite(a, 1, 1, dp);
    }

    fclose(sp);
    fclose(dp);

    printf("Run Again?(y/n):");
    fgets(x, 2, stdin);
    while ( (ch = fgetc(stdin)) != EOF && ch != '\n');
 }

}

/*void rmnewline(char *string)
{
    int i, l = strlen(string);

    for(i=0; i<=l; i++)
    {
      if(string[i] == '\0')
         return;
      if(string[i] == '\n')
     {
       string[i] == '\0';
       return;
     }
  }

}*/

void remove_last_newline(char *str) {
    if (*str == '\0') return; /* do nothing if the string is empty */
    while(str[1] != '\0') str++; /* search for the end of the string */
    if (*str == '\n') *str = '\0'; /* if the last character is newline, delete it */
}
4

3 回答 3

2

fgets将读取的换行符存储在缓冲区中。

它可能会阻止fopen打开“指定文件”并fopen可能返回NULL.

如果换行符在缓冲区中，请删除它们。
这可以使用这个函数来完成：

void remove_last_newline(char *str) {
    if (*str == '\0') return; /* do nothing if the string is empty */
    while(str[1] != '\0') str++; /* search for the end of the string */
    if (*str == '\n') *str = '\0'; /* if the last character is newline, delete it */
}

请在向他们读取文件名后将sourceand传递destination给此函数。

于 2015-10-03T14:46:15.507 回答
1

检查 的返回值fopen。 当 SP 为 NULL 时
，程序正在获取 SEGFAULT 。fread(a, 1, 1, sp);由于程序试图只读取一个字节，因此不会有任何溢出的机会。

fopen失败，因为源末尾包含换行符。如果您newline在调用 fopen 之前从源和目标中删除字符，您的程序将正常工作。

于 2015-10-03T15:06:12.513 回答
0

主要问题是留'\n'在文件名中，然后以下fopen()失败，代码没有检查fopen()返回值。其他 2 个答案很好地回答了这个问题，至少 1 个值得接受。

随后的 OP 编辑​​更改int a[1]为int *a导致新故障。建议改成

char a[1];
于 2015-10-03T16:17:31.377 回答