while(p = strtok(NULL,“,”))警告:可能是错误的分配

时间:2014-01-02 06:54:49

标签: c++ c

有一个代码,我在

中遇到错误
Line 8: array size too large
line 9: array size too large 
line 28 while (p=strtok(NULL,",")) warning :possible incorrect assignment 

那么请你建议我如何处理这个警告/错误?

代码:

  

/ *将csv数据转换为libsvm / svm-light格式* /

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

char buf[10000000];
float feature[100000];

int main(int argc, char **argv)
{
FILE *fp;

    if(argc!=2) { fprintf(stderr,"Usage %s filename\n",argv[0]); }
    if((fp=fopen(argv[1],"r"))==NULL)
    {
        fprintf(stderr,"Can't open input file %s\n",argv[1]);
    }

    while(fscanf(fp,"%[^\n]\n",buf)==1)
    {
        int i=0,j;
        char *p=strtok(buf,",");

        feature[i++]=atof(p);

        while((p=strtok(NULL,",")))
            feature[i++]=atof(p);

        //      --i;
        /*
        if ((int) feature[i]==1)
        printf("-1 ");
    else
        printf("+1 ");
       */
       //       printf("%f ", feature[1]);
        printf("%d ", (int) feature[0]);
        for(j=1;j<i;j++)
        printf(" %d:%f",j,feature[j]);


        printf("\n");
    }
    return 0;
}

1 个答案:

答案 0 :(得分:2)

您实际上希望将strtok返回的值存储到变量p中,同时检查此值是否为NULL。

我建议你用以下方式编写它,它与目前的编写方式相当,更具描述性。

while( ( p=strtok(NULL,",") )!=NULL )

关于数组大小太大错误,您应该考虑使用mallocfree对C中的两个数组进行动态分配,或者使用std::vector C ++。