Может ли кто-нибудь помочь мне с этим массивом сравнения?

Question

Im новый для C, и у меня возникают проблемы с этим кодом, который я пишу. Я почти на 100% уверен, что из-за моей секции массива сравнения, но я действительно не знаю, что изменить. Может ли кто-нибудь помочь? Если вам нужен весь мой код, я могу также опубликовать его. Код должен сравнивать буквы, введенные пользователем со словами в документе .txt и посмотреть, можете ли вы произнести любые слова с этими буквами.

#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"); 
    scanf("%s", 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; 

    while (user[j] >= dictionary[j]) 
    { 
     j++; 
     if (j == 26) 
     { 
      return 0; 
     } 
     else 
     { 
      printf("also broken"); 
     } 
    } 
    return 1; 
} 

Answer 1

Не уверен, что это правильный ответ, как это трудно догадаться, что вы на самом деле пытаетесь сделать, но вы, вероятно, хотите этого, или что-то подобное:

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

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

Answer 2

Вы возвращаете неправильный результат ,

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

    while (user[j] >= dictionary[j]) 
    { 
     j++; 
     if (j == 26) 
     { 
      // You have checked all 26 letters and for all of them condition is true. Therefore a word can be made from user entered letters. 
      return 1; 
     } 
    } 
    return 0; //Word can not be made from user entered letters 
} 

Если вы хотите, чтобы заботиться о чувствительности к регистру,

void find_frequency(char string[],int count[]) 
{ 
    int i; 
    for(i=0; string[i] != '\0'; i++) 
    { 
     //If letter is in upper case, it will be converted to lower case before checking. 
     if (tolower(string[i]) >= 'a' && tolower(string[i]) <= 'z') 
     { 
      count[tolower(string[i])-'a']++; 
     } 
    } 
} 

UPDATE 1:

Ошибка в tokenizing. 1) int total_words=11; //number of words Вы используете эту переменную в качестве индекса массива. Поэтому он должен быть инициализирован равным нулю. Или вы объявляете другую переменную для индекса. int index=0;

2) strtok вернет адрес начала токена. Таким образом, вы пишете слова с токеном, а не копируете нулевой ограничитель.

char *prevTokenPtr = line; 
    while(fgets(line,SIZE,cfPtr)!= NULL) 
    { 
     char*tokenPtr=strtok(line, "\t"); 
     while(tokenPtr != NULL) 
     { 
      /* Copy from last token to this token. */ 
      int lengthToCopy = (tokenPtr - prevTokenPtr)/sizeof(char); 
      strncpy(dictionary_words[index], prevTokenPtr, lengthToCopy); 
      dictionary_words[index][lengthToCopy] = '\0'; 

      printf("dictionary_words[%d] is [%s]\n", index, dictionary_words[index]); 
      index++; 

      prevTokenPtr = tokenPtr + 1; //Neglect '\t' 
      tokenPtr  = strtok(NULL, "\t"); 
     } 

     /* Copy the last word. */ 
     if(NULL != prevTokenPtr) 
     { 
      strcpy(dictionary_words[index], prevTokenPtr); 
      printf("dictionary_words[%d] is [%s]\n", index, dictionary_words[index]); 
      index++; 
     } 
    } 

Обратите внимание:

1) Я предполагаю, что вход, как это. "word1" \ t "word2" \ t "word3" \ t ... \ t "wordN"

2) Я не тестировал этот код. Отпечатки должны помочь вам отладить дальнейшие действия.