C变量循环变化

时间:2018-03-22 10:05:56

标签: c loops variables fgets

当尝试遍历文件中的行并获取其中的数字时,我使用了以下代码,并且在循环的第一次迭代之后,lnCount(用于在循环中递增)的值发生了变化: / p>

long nbl = nbLine(filename); // This function works fine

long *allocatedMemoryStartingPos = NULL; 
allocatedMemoryStartingPos = malloc(sizeof(long)*(nbl+1)); 

if (allocatedMemoryStartingPos == NULL) {
    exit(0); // Immediately stops the program
}

long *posPtr = &allocatedMemoryStartingPos[0];

initArrayTo0(allocatedMemoryStartingPos, nbl+1); // Works as well, sets all values 0

char str[] = "";
char spl[] = "";
long val = 0;

FILE* f = NULL;
f = fopen(filename, "r");
if (f != NULL) {
    for (long lnCount = 0; lnCount < nbl; lnCount++) {
        printf("lnCount = %ld\n", lnCount);
        getStartPosFromFile(f, 250, &val, str, spl);
        posPtr = val;
        posPtr++;
    }
}
fclose(f);
free(allocatedMemoryStartingPos);

getStartPosFromFile()的代码如下:

void getStartPosFromFile(FILE* f, int maxSize, long *ret, char str[], char spl[]){
    if (fgets(str, maxSize, f) != NULL) {
        strcpy(spl, extractColumn(str, 7));
        *ret = strtol(spl, NULL, 10);
    } else {
        printf("fgets failed!\n");
        perror("fgets failed!");
    }
}

在上面的代码中,extractColumn也可以正常工作,只需获取当前行的子字符串并将其复制到stringspl)。

此代码的输出如下:

lncount = 0
lncount = 3301218

1 个答案:

答案 0 :(得分:1)

strspl被声明为1个大小的字符数组(仅终止null)。在其中复制多个字符会调用未定义的行为(因为您在内存中写入可以被其他变量使用)。从那时起,一切皆有可能......

您必须声明符合您需求的尺寸:

#define SIZE 1024

char str[SIZE] = "";
char spl[SIZE] = "";

请输入相关值而不是我的1024示例