Thread: searching a text file for a word

My guess from looking at your poor indentation is that you close the file after checking the first word.

Also, there are a number of FAQ points which relate to your code.

I am unsure why your program doesn't work.... but I have recently being doing something similar and used the strstr function.... it seems to be a whole lot easier.

the parameters are as follows:

strstr(string1, string2)

where string 1 is the word to search, and string 2 is the string to search within. The return of the function is the position of the searched string if it is found.

so i have been sitting down for the last couple of hours and ive just found my problem is the strcmp bit i was checking the thing wrong so the if statement after wards was looking for the wrong integer

Code:

ch=strcmp
(tokenptr,argv[2]);
            if(ch==0)

is should be

Code:

ch=strcmp
(tokenptr,argv[2]);
            if(ch==1)

thanks for any ideas

EDIT: sorry for the mistake

errrr those look the same to me?!?

See the problem yet, when you have decent formatting?

Code:

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

int main(int argc, char *argv[])
{
    int ch;
    char string[150];
    char *tokenptr;

    FILE *in_stream;

    if (argc == 3) {
        if ((in_stream = fopen(argv[1], "r")) == NULL) {
            fprintf(stderr, "Could Not Open File!");
            perror("Because");
            return 1;
            fclose(in_stream);  /*!! HOW IS THIS REACHED? */
        } else {
            while (!feof(in_stream)) {
                fgets(string, 150, in_stream);
                printf("%s\n\n", string);
                tokenptr = strtok(string, " ");
                while (tokenptr != NULL) {
                    ch = strcmp(tokenptr, argv[2]);
                    if (ch == 0) {
                        printf("%s was Found In File", argv[2]);
                    } else {
                        printf("%s was not found in the file", argv[2]);
                    }

                    fclose(in_stream);  /*!! one word read, now bomb out */
                    return 0;
                }
            }
        }
        if (argc != 3)  /*!! smooth - it's inside if ( argc == 3 ) */
            return 1;
    }
}