Así que intenté buscar una solución para esto de manera extensa, pero realmente solo puedo encontrar publicaciones donde falta la nueva línea o el byte nulo de una de las cadenas. Estoy bastante seguro de que ese no es el caso aquí.
Estoy usando la siguiente función para comparar una palabra con un archivo que contiene una lista de palabras con una palabra en cada línea (diccionario en la función). Aquí está el código:
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; } }El código se compila sin errores y la función lee correctamente el archivo del diccionario e imprime la lista de palabras tal como aparecen allí. El problema que tengo es que incluso cuando las dos cadenas son idénticas, strcmp no devuelve 0 y, por lo tanto, la función devolverá falso para cualquier entrada.
por ejemplo, obtengo:
r:zymoscope w:zymoscope -3¿Algunas ideas? Siento que me falta algo obvio, pero no he podido encontrar nada en mis búsquedas.
Veo que está agregando una newline a sus cadenas de prueba para tratar de solucionar el problema de fgets() que retiene los finales de línea. Es mucho mejor arreglar esto en la fuente. Puede eliminar todas las cosas finales como esta, inmediatamente después de leer el archivo.
readValue [ strcspn(readValue, "\r\n") ] = '\0'; // remove trailing newline etcLa cadena que está leyendo contiene caracteres finales y, por lo tanto, no es la misma que la cadena con la que la está comparando.
Elimine la nueva línea final (y CR si está allí); entonces no necesita agregar ninguna nueva línea o retorno de carro a la cadena que se compara:
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; }