подсчет числа частот слова в тексте

Question

Я написал функцию для подсчета частоты конкретного слова в тексте. Эта программа каждый раз возвращает ноль. Как я могу ее улучшить?

while (fgets(sentence, sizeof sentence, cfPtr)) 
{ 
for(j=0;j<total4;j++) 
     { 
      frequency[j] = comparision(sentence,&w); 
      all_frequency+=frequency[j]; 
}} 
. 
. 
. 
int comparision(const char sentence[ ],char *w) 
{ 
    int length=0,count=0,l=0,i; 
    length= strlen(sentence); 
    l= strlen(w); 
    while(sentence[i]!= '\n') 
    if(strncmp(sentence,w,l)) 
     count++; 
    i++; 
    return count; 
    } 

Answer 1

У меня есть корректура вашего кода и комментарии к стилю кодировки и именам переменных. Там по-прежнему является недостатком, который я оставил с условным, что связано с тем, что он не повторяет предложение .

Вот ваш код, размеченный:

while(fgets(sentence, sizeof sentence, cfPtr)) { 
    for(j=0;j<total4;j++){ 
     frequency[j] = comparision(sentence,&w); 
     all_frequency+=frequency[j]; 
    } 

} 

// int comparision(const char sentence[ ],char *w) w is a poor variable name in this case. 

int comparison(const char sentence[ ], char *word) //word is a better name. 
{ 

    //int length=0,count=0,l=0,i; 

    //Each variable should get its own line. 
    //Also, i should be initialized and l is redundant. 
    //Here are properly initialized variables: 

    int length = 0; 
    int count = 0; 
    int i = 0; 

    //length= strlen(sentence); This is redundant, as you know that the line ends at '\n' 

    length = strlen(word); //l is replaced with length. 

    //while(sentence[i]!= '\n') 

    //The incrementor and the if statement should be stored inside of a block 
    //(Formal name for curley braces). 

    while(sentence[i] != '\n'){ 
     if(strncmp(sentence, word, length) == 0) //strncmp returns 0 if equal, so you  
      count++;        //should compare to 0 for equality 
     i++; 
    } 
    return count; 
}