2013-11-24 71 views
0

我有一个函数loadsets()(加载设置的缩写),它应该从名为的文本文件加载设置Progsets.txt。 loadsets()在成功时返回0,在检测到致命错误时返回-1。然而,实际上读取的部分代码Progsets.txt(三个fgets()),似乎都失败并返回空指针,因此不会加载任何东西,而是一堆空值。我的代码有问题吗?当我运行代码时,fp是一个有效的指针,我可以打开它阅读。那么,怎么了?fgets()不从文本文件读取?

此代码用于使用cmd加载我非常基本的文本编辑器程序的默认文本颜色。

头:

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

#define ARR_SIZE 100 

struct FINSETS 
{ 
    char color[ARR_SIZE + 1]; 
    char title[ARR_SIZE + 1]; 
    char maxchars[ARR_SIZE + 1]; 
} SETTINGS; 

载入集():

int loadsets(int* pMAXCHARS) // load settings from a text file 
{ 
    FILE *fp = fopen("C:\\Typify\\Settings (do not modify)\\Progsets.txt", "r"); 
    char *color = (char*) malloc(sizeof(char*) * ARR_SIZE); 
    char *title = (char*) malloc(sizeof(char*) * ARR_SIZE); 
    char *maxchars = (char*) malloc(sizeof(char*) * ARR_SIZE); 
    char com1[ARR_SIZE + 1] = "color "; 
    char com2[ARR_SIZE + 1] = "title "; 
    int i = 0; 
    int j = 0; 
    int k = 0; 
    int found = 0; 

    while (k < ARR_SIZE + 1) // fill strings with '\0' 
    { 
     color[k] = title[k] = maxchars[k] = '\0'; 
     SETTINGS.color[k] = SETTINGS.maxchars[k] = SETTINGS.title[k] = '\0'; 
     k++; 
    } 

    if (!fp) // check for reading errors 
    { 
     fprintf(stderr, "Error: Unable to load settings. Make sure that Progsets.txt exists and has not been modified.\a\n\n"); 
     return -1; // fatal error 
    } 

    if (!size(fp)) // see if Progsets.txt is not a zero-byte file (it shouldn't be) 
    { 
     fprintf(stderr, "Error: Progsets.txt has been modified. Please copy the contents of Defsets.txt to Progsets.txt to manually reset to default settings.\a\n\n"); 

     free(color); 
     free(title); 
     free(maxchars); 

     return -1; // fatal error 
    } 

    // PROBLEMATIC CODE: 

    fgets(color, ARR_SIZE, fp);  // RETURNS NULL (INSTEAD OF READING FROM THE FILE) 
    fgets(title, ARR_SIZE, fp);  // RETURNS NULL (INSTEAD OF READING FROM THE FILE) 
    fgets(maxchars, ARR_SIZE, fp); // RETURNS NULL (INSTEAD OF READING FROM THE FILE) 

    // END OF PROBLEMATIC CODE: 

    system(strcat(com1, SETTINGS.color)); // set color of cmd 
    system(strcat(com2, SETTINGS.title)); // set title of cmd 
    *pMAXCHARS = atoi(SETTINGS.maxchars); 

    // cleanup 

    fclose(fp); 
    free(color); 
    free(title); 
    free(maxchars); 

    return 0; // success 
} 

Progsets.txt:

COLOR=$0a; 
TITLE=$Typify!; 
MAXCHARS=$10000; 

编辑:这里是size()函数的定义。由于我只使用ASCII文本文件,因此我假设每个字符都是一个字节,并且可以通过计算字符数来计算文件大小(以字节为单位)。任何可疑的东西?

大小():

int size(FILE* fp) 
{ 
    int size = 0; 
    int c; 

    while ((c = fgetc(fp)) != EOF) 
    { 
     size++; 
    } 

    return size; 
} 
+0

我的程序中没有看到任何'fscanf'。 – haccks

+0

是的。在标题上错字,抱歉。它应该是“fgets()不从文本文件读取?” – user3026735

+1

你的mallocing'char *'为'char **',没有检查mallocs的返回值,并且在获得大小后可能需要倒带文件 – amdixon

回答

0

问题就出在你使用size()功能。它在文件句柄上反复调用fgetc(),直到它到达文件末尾,递增一个值以跟踪文件中的字节数。

这不是一个坏方法(虽然我敢肯定有更好的,不涉及低效基于字符的I/O),但它确实有一个致命的缺陷你似乎都忽略了。

你叫后,您已经阅读了文件中的所有方式来结束,这样任何进一步的读取,如:

fgets(color, ARR_SIZE, fp); 

只会失败,因为你在最后是已经的文件。在从size()返回之前,您可能需要考虑诸如rewind()之类的内容 - 这会将文件指针放回到文件的起始位置,以便您可以再次读取它。