c - Strcmp nie zwraca 0 dla równych ciągów znaków

Question

Tak więc próbowałem znaleźć rozwiązanie tego problemu, ale mogę tylko znaleźć posty, w których brakuje nowego wiersza lub bajtu null w jednym z ciągów. Jestem prawie pewien, że tak nie jest.

Korzystam z poniższej funkcji, aby porównać słowo z plikiem zawierającym listę słów z jednym słowem w każdym wierszu (słownik w funkcji). Oto kod:

int isWord(char * word,char * dictionary){ 
    FILE *fp; 
    fp = fopen(dictionary,"r"); 
    if(fp == NULL){ 
    printf("error: dictionary cannot be opened\n"); 
    return 0; 
    } 
    if(strlen(word)>17){ 
    printf("error: word cannot be >16 characters\n"); 
    return 0; 
    } 
    char longWord[18]; 
    strcpy(longWord,word); 
    strcat(longWord,"\n"); 
    char readValue[50] = "a\n"; 
    while (fgets(readValue,50,fp) != NULL && strcmp(readValue,longWord) != 0){ 
    printf("r:%sw:%s%d\n",readValue,longWord,strcmp(longWord,readValue));//this line is in for debugging 
    } 
    if(strcmp(readValue,longWord) == 0){ 
    return 1; 
    } 
    else{ 
    return 0; 
    } 
} 

Kod kompiluje bez błędów, a funkcja odczytuje plik słownika grzywny i wydrukuje listę słów, które pojawiają się tam. Problem, który mam, polega na tym, że nawet jeśli dwa ciągi są identyczne, strcmp nie zwraca wartości 0, więc funkcja zwróci wartość false dla dowolnego wejścia.

np uzyskać:

r:zymoscope 
w:zymoscope 
-3 

Jakieś pomysły? Czuję, że brakuje mi czegoś oczywistego, ale nie mogłem znaleźć niczego w moich wyszukiwaniach.

Answer 1

Widzę, że dodajesz newline do ciągów testowych, aby spróbować rozwiązać problem z zatrzymaniem końca linii w numerze fgets(). Znacznie lepiej to naprawić u źródła. Możesz usunąć wszystkie końcowe rzeczy w ten sposób, natychmiast po odczytaniu z pliku.

readValue [ strcspn(readValue, "\r\n") ] = '\0'; // remove trailing newline etc 

Answer 2

Ciąg znaków, który czytasz zawiera znaki końcowe, a więc nie jest taki sam, jak ciąg, z którym go porównujesz.

Usuń końcowy znak nowej linii (i CR, jeśli tam jest); wtedy nie trzeba dodawać żadnego nowego wiersza lub powrót karetki do łańcucha porównywanych:

int isWord(char *word, char *dictionary){ 
    FILE *fp; 
    fp = fopen(dictionary, "r"); 
    if (fp == NULL){ 
    fprintf(stderr, "error: dictionary cannot be opened\n"); 
    return 0; 
    } 
    if (strlen(word) > 16){ 
    fprintf(stderr, "error: word cannot be >16 characters\n"); 
    return 0; 
    } 
    char readValue[50]; 
    while (fgets(readValue, 50, fp) != NULL){ 
    char *ep = &readValue[strlen(readValue)-1]; 

    while (*ep == '\n' || *ep == '\r'){ 
     *ep-- = '\0'; 
    } 
    if (strcmp(readValue, word) == 0){ 
     return 1; 
    } 
    } 
    return 0; 
}