2013-04-03 59 views
2

首先,我是C新手,但没有编程;我在PHP中做了很多web开发。对于这个问题...C - sprintf()影响数组声明

我有一个读取一系列二进制文件并输出内容的函数,但是使用sprintf()正在影响后续多维数组的内存分配。请看下图:

FILE *fp; 

for(int i = 1; i <= 16; i++) { 

    char format_1[] = "track_%d.bin"; 
    char filename_1[sizeof format_1+100]; 
    sprintf(filename_1,format_1,i); 
    printf("Filename: %s \n", filename_1); 
    fp = fopen(filename_1, "rb"); 

    if(fp) { 
     fseek(fp, 0L, SEEK_END); 
     int sz = (ftell(fp)/3) - 1; 
     fseek(fp, 0L, SEEK_SET); 

     printf("File size: %d\n", sz); 

     int nums[sz][3]; 

     fread(nums,sizeof(int),sz * 3, fp); 

     for (int k = 0; k < sz; k++) { 
      printf("Entry number %d = %d with extra values of %d and %d\n", k, nums[k][0], nums[k][1], nums[k][2]); 
     } 
    } else { 
     printf("Skipping \n"); 
    } 

    fclose(fp); 
} 

我知道这是sprintf的()引起的问题,因为我简化了功能:

printf("Please enter file number... \n"); 
scanf("%d%c", &num_file, &term); 

printf("Please enter how many instances in this file... \n"); 
scanf("%d%c", &num_instance, &term); 

int nums[num_instance][3]; 

char format_1[] = "track_%d.bin"; 
char filename_1[sizeof format_1+3]; 
sprintf(filename_1,format_1,num_file); 
printf("Filename: %s \n", filename_1); 

fp = fopen(filename_1, "rb"); 

fread(nums,sizeof(int),num_instance * 3, fp); 
fclose(fp); 

for (int k = 0; k < num_notes; k++) { 
    printf("Entry number %d = %d with extra values of %d and %d\n", k, nums[k][0], nums[k][1], nums[k][2]); 
} 

当我移动的sprintf以上(NUMS的声明),它工作正常(即使它只是在它上面),只要我移动下面的sprintf NUMS的声明()输出看起来象下面这样:

Entry number: 0 = 71 with extra values 116 and 158 
Entry number: 1 = -1073743712 with extra values -1073744240 and -1073744304 
Entry number: 2 = -1073743896 with extra values 59983 and -1073744005 

正如你可以从上面的代码片段看,我需要有declarat nums以下的sprintf(),因为我需要知道文件中的条目数量。

编辑:我想如果我提到一个文件包含多维数组可能会有所帮助:array[x][3]

真的难住这个,任何帮助,非常感谢。

+0

@FredrikPihl C99有VLA。 – 2013-04-03 21:17:55

+0

@FredrikPihl'sizeof format_1 + 3'是一个编译时间常量。 – 2013-04-03 21:20:59

+1

您需要检查C中的返回值。这样您可以准确找到代码失败的位置。 – 2013-04-03 21:21:48

回答

2

你的基本问题是,你正在计算文件的大小,以字节为单位,然后尝试从文件中读取多个整数。由于整数是(在大多数机器上)4个字节,这将失败,并且只能读入nums阵列的第一个四分之一。其余的将包含(并打印为)随机垃圾,这是你所看到的。

您应该检查fread的返回值以确保它有意义。如果您将代码更改为

int numread = fread(nums,sizeof(int),sz * 3, fp); 
printf("Read %d integers from file\n", numread); 
for (int k = 0; k < sz; k++) { 
    if (k < numread/3) { 
     printf("Entry number %d = %d with extra values of %d and %d\n", k, nums[k][0], nums[k][1], nums[k][2]); 
    } else { 
     printf("Entry number %d doesn't exist in the file\n", k); 
    } 
} 

您会立即看到问题。

总是检查你的返回值。

+0

感谢您的提示!我想知道如何在@MaximYegorushkin在评论中提到它时检查返回值。这并不是我遇到的问题,但问题实际上发生在我写入文件的函数中,我认为这与我没有在文件中分配足够的空间有关。不知道为什么它用sprinf()表现这种方式。当我在其他机器上时,我会发布代码。 – 2013-04-04 08:18:37