Thread: C Program Help

This program should prompt the user for a series of letters. Then read in the letters and print out which dictionary words can be made from the provided letters. The dictionary is provided as a text file in the folder. But whenever I run it it doesn't give be anything. Can anyone help me figure out why this is happening?

Code:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define SIZE 99
#define NUM_WORDS 100


void find_frequency(char string[], int count[]);
int compare_arrays(int dictionary[], int user[]);


int main()
{
    int total_words=11; //number of words
    char dictionary_words[NUM_WORDS][SIZE]; //store words in directory
    FILE *cfPtr; //dictionary.txt pointer


    if ((cfPtr=fopen("dictionary.txt","r"))==NULL)//try to open file
    {
        puts("File dictionary.txt could not be opened.");
        exit(1);//exit if file doesn't open
    }


    else{ //Read each word from the dictionary and save to array
        char line[SIZE]; //save each word


        {
            while(fgets(line,SIZE,cfPtr)!= NULL)
                  {
                      char*tokenPtr=strtok(line, "\t");
                      while(tokenPtr != NULL)
                        {
                            strcpy(dictionary_words[total_words],tokenPtr);
                            total_words++;
                            tokenPtr = strtok(NULL, "\t" );
                        }


                  }
        }
        }
    fclose(cfPtr);//close file


    char string[11];//get string of characters from user
    int count[26]={0};//store the number of each letter


    printf("Enter letters:\n");
    gets(string);




    find_frequency(string, count);//count of each character entered


    char temp[SIZE];
    int temp_count[26]={0};//convert words into letters
    int i;


    for(i=0; i<=total_words; i++);
    {
        strcpy(temp,dictionary_words[i]);
        find_frequency(temp,temp_count);//convert word to letters in alphabet


        if (compare_arrays(temp_count,count))//compare words with letters entered
        {
            printf("%s:", temp);//print what you can spell
        }
        else
        {
             printf("broken", temp);
        }
       memset(temp_count,0,sizeof(temp_count));//test next word
    }
    return(0);
}//end main


//define function
void find_frequency(char string[],int count[])
{
    int i;
    for(i=0; string[i] != '\0'; i++)
    {
        if (string[i] >= 'a' && string[i] <= 'z')
        {
            count[string[i]-'a']++;
        }
    }
}


int compare_arrays(int dictionary[], int user[])
{
    int j = 0;


    if (user[j] >= dictionary[j])
    {
        j++;
        if (j == 26)
        {
            return 0;
        }
    }
    return 1;
}

You have an extraneous semicolon at the end of the for head on line 66 above.

And your "compare_arrays" routine is kinda weird. It certainly isn't "comparing arrays". It doesn't even have a loop.

This code bears a remarkable resemblance to the code here. I sure hope you didn't pay for the code you posted.

That code looks like its for the same assignment but its not the code that I used. No I did not pay to get this code.

Good to hear. Did you make any progress?

Originally Posted by chrisjn103

That code looks like its for the same assignment but its not the code that I used. No I did not pay to get this code.

If you paid it would probably be working.
At least admit that you stole the non-working code from that site.

Originally Posted by algorism

If you paid it would probably be working.

One would hope.
If not, running to a forum for help would be the next step.

Originally Posted by algorism

At least admit that you stole the non-working code from that site.

The solution is actually heavily censored (requires payment to access), so I don't think they simply copy/pasted it. The (censored) solution code does have enough pieces visible to imply the similarity, especially constant and variable names. But I am willing to give the OP the benefit of the doubt, since I can't definitely verify the code was lifted.

Originally Posted by algorism

If you paid it would probably be working.
At least admit that you stole the non-working code from that site.

Why would I admit to something I did not do? Like I said , both codes are for the same assignment that's why they look similar. If you actually look at the code they are using it really isn't like mine. I have way more lines and most things are written out completely differently.

Originally Posted by Matticus

Good to hear. Did you make any progress?

Somewhat. it seems like something in my compare arrays section isn't working correctly. I added an else statement to it that prints if its broken and it keeps printing that no matter what I enter.

Originally Posted by chrisjn103

Why would I admit to something I did not do? Like I said , both codes are for the same assignment that's why they look similar. If you actually look at the code they are using it really isn't like mine. I have way more lines and most things are written out completely differently.

Obviously it's for the same program.
Why would you steal code for a different program?
I never said you didn't change or add anything.
But look at all the points where it matches your code exactly.
Clearly you took this code as a starting point.

Originally Posted by chrisjn103

Somewhat. it seems like something in my compare arrays section isn't working correctly. I added an else statement to it that prints if its broken and it keeps printing that no matter what I enter.

Keep trying to get it working. If you get absolutely stuck, post your updated code along with information on what's still wrong.

I don't have much time to review your code at the moment, but you should re-read the first sentence in post #2. This seems to make your code do something approaching what it intends.

Wow yeah I thought I had fixed that. That seems to fix the main issue