Помогите! strcmp лжет мне, когда кормил результаты strtok - PullRequest
Новая
       30

Помогите! strcmp лжет мне, когда кормил результаты strtok

1 голос
/ 22 января 2010

strcmp, когда скармливает результаты strtok, в следующем коде, кажется, нагло обманывает меня.

int fSize;
char * buffer=NULL;
char * jobToken = "job";
char * nextToken=NULL;
job * curJob=NULL;
struct node * head=NULL;
struct node * parseList(FILE* file){
    fseek(file,0,SEEK_END);
    fSize=ftell(file);
    buffer = (char*)malloc(fSize+1);
    printf("%d chars: reading buffer now:\n",fSize);
    fseek(file,0,SEEK_SET);
    fread (buffer,1,fSize,file);
    nextToken = strtok(buffer, " \n");
    while (nextToken!=NULL){
            printf("**Running  Token: %s**\n",nextToken);
            if (strcmp(nextToken,jobToken)){
                    printf("Accepted %s  as %s\n",nextToken,jobToken);                
            }else{
                    printf("not %s, %s\n",jobToken,nextToken);
            }

            printf("End of state - %s\n",nextToken);

            nextToken = strtok(NULL, " \n");
    }
    free (buffer);
    return NULL;
}

С этим вводом в файле в параметрах parseList: 

job 23
job 10

Дает этот вывод: 

14 chars: reading buffer now:
**Running  Token: job**
not job, job
End of state - job
**Running  Token: 23**
Accepted 23  as job
End of state - 23
**Running  Token: job**
not job, job
End of state - job
**Running  Token: 10**
Accepted 10  as job
End of state - 10

LIES!

Ответы [ 3 ]

6 голосов
/ 22 января 2010

strcmp возвращает 0, если сравниваемые строки равны. Вам нужно использовать if (!strcmp(...)).

1 голос
/ 22 января 2010

Не относится к вашему вопросу, но пара моментов:

  • После fread() вы должны установить buffer[fSize] = 0;, иначе это не строка.
  • Определение количества символов для чтения в текстовом или двоичном файле с использованием fseek(file,0,SEEK_END); не гарантированно работает .

Лично я бы написал malloc() вызов как: 

buffer = malloc(fSize+1);

потому что это предупредит меня, если я забуду #include <stdlib.h>, и его легче читать. Применение не требуется в C.

1 голос
/ 22 января 2010

strcmp возвращает 0, когда строки равны. Смотри: http://www.elook.org/programming/c/strcmp.html

...