計算文本中某個單詞的頻率數

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; 
}