Question

假设我有文件

5f2
3f6
2f1

代码:( printf应该打印第二个数字（即2,6和1），但它不会

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

int main (int argc, char * argv[])
{
    FILE *ptr;
    char str[100];
    char * token;
    int a, b, i;
    int arr[4];

    if(argc > 1)
    {
        ptr = fopen(argv[1],"r");
        if(ptr == NULL)
        {
            exit(1);
        }
    }

   else
   {
       exit(1);
   }

   //And I'm looking to parse the numbers between the "f" so..

   while(fgets(str,100,ptr) != NULL)
   {
       token = strstr(str,"f");
       if(token != NULL)
       {
           a = atol(str); // first number
           b = atol(token+1); // second number
           arr[i] = b; // store each b value (3 of em) into this array

       }
       i++; 
       printf("Values are %d\n",arr[i]); //should print 2,6 and 1
   }
}

我试图将printf移到循环外面，但这似乎打印出更奇怪的结果，我之前看过有关将文件中的整数存储到数组中的帖子，但是因为这涉及到使用strstr，我'我不完全确定程序是否相同。

Answer 1

int i,j=0;
while(fgets(str,sizeof(str),file) != NULL)
{
   size_t n = strlen(str);
   if(n>0 && str[n-1] == '\n')
   str[n-1] = '\0';
   i = str[strlen(str)-1] - '0'; /* Convert the character to int */
   printf("%d\n",i);// Or save it to your int array arr[j++] = i;
}

只需移动到显示的最后一个字符，然后将其打印为整数。

PS：fgets()附带一个换行符，您需要将其压缩，如图所示

Answer 2

您永远不会初始化i，然后您正在阅读arr[i]（只是发生不会在那里崩溃），然后增加i（到“未定义的值+ 1”），然后打印arr[i] - 即，您正在写入并读取未初始化的内存。

此外，您的FILE *为ptr，而不是file。你应养成使用strtol()代替atol()的习惯，因为前者允许你正确检查是否成功（并从错误中恢复）。

将文件的值存储到数组中会导致奇怪的行为

2 个答案: