0

我正在尝试实现一些代码来挑选文本文件的最后一行，并且目前正在使用这个示例，但我无法弄清楚如何防止代码在找不到文件时打印出“（null）”，在当它找不到文件的那一刻，它会打印出“文件无法在最后一行打开：没有这样的文件或目录”然后它会在下面打印空的东西，对于这个特定的应用程序，我宁愿它只打印出“文件无法打开”部分，但不打印出空部分，如果有人能为此指出正确的方向，我将不胜感激，谢谢！

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

#ifndef max
#define max(a, b) ((a)>(b))? (a) : (b)
#endif

long GetFileSize(FILE *fp){
    long fsize = 0;

    fseek(fp,0,SEEK_END);
    fsize = ftell(fp);
    fseek(fp,0,SEEK_SET);//reset stream position!!

    return fsize;
}
char *lastline(char *filepath){
    FILE *fp;
    char buff[4096+1];
    int size,i;
    long fsize;
    if(NULL==(fp=fopen(filepath, "r"))){
        perror("file cannot open at lastline");
        return NULL;
    }
    fsize= -1L*GetFileSize(fp);
    if(size=fseek(fp, max(fsize, -4096L), SEEK_END)){
        perror("cannot seek");
        exit(0);
    }
    size=fread(buff, sizeof(char), 4096, fp);
    fclose(fp);
    buff[size] = '\0';
    i=size-1;
    if(buff[i]=='\n'){
        buff[i] = '\0';
    }
    while(i >=0 && buff[i] != '\n')
        --i;
    ++i;
    return strdup(&buff[i]);
}

int main(void){
    char *last;

    last = lastline("data.txt");
    printf("\"%s\"\n", last);
    free(last);
    return 0;
}
4

3 回答 3

1

您可以在该行之后添加这两行last = lastline("data.txt");。打印的原因NULL是因为您正在打印函数的结果lastline()。此函数将NULL在失败时返回。

if( last != NULL )
    printf( "\"%s\"\n", last );
于 2012-05-11T14:12:41.487 回答
1

它来自这里：printf("\"%s\"\n", last);

if (last) {
  printf("\"%s\"\n", last);
  free(last);
}
于 2012-05-11T14:12:44.697 回答
1

您应该检查返回值fread以检查它是否成功。如果fread失败，那么errorno将被设置并且您的变量size不会读取字符数。因此，您随后对变量的使用buff：

buff[size] = '\0';
i=size-1;
if(buff[i]=='\n'){
    buff[i] = '\0';
}

可能会产生意想不到的结果。strdup(buff[i])从函数返回到的 which 也是如此main。

于 2012-05-11T14:26:56.363 回答