C程序中的字长频率

Question

我正在尝试编写一个简单的 C 程序来输出单词的长度并输出它们的频率。例如，如果用户输入“嘿”，我的程序将输出 Length of word: 3 Occurrences 1，依此类推，输入更大的字符串。我似乎无法正确循环它。当看到分隔符同时计算单词当时的长度及其出现时，我想设置两个计数器，但我还没有找到一种工作方式。如何修复我的循环？我的代码如下。我会很感激任何帮助。我应该包括我的程序只对输入的一个单词正确运行，而不是整个句子或多个句子。

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <ctype.h>
const char delim[] = ", . - !*()&^%$#@<> ? []{}\\ / \"";
const int n_delim = 31;
#define SIZE 1000

int is_delim(int c);
int main(){
    char string[SIZE];
    int wordlength = 0, wl[SIZE];
    int word = 0, i;


    printf("Enter your input string:");
    fgets(string, SIZE, stdin);
    string[strlen(string) - 1] = '\0';

printf("Word Length\tCount\n");
    int seen = 0;
    int l;
    for (i = 0; i < strlen(string); i++){
        if (is_delim(string[i])){
            wl[word++] = wordlength;
            l = wordlength;
            seen++;
            printf("%d\t\t%d\n", l, seen);
            wordlength = 0;
        }
        wordlength++;

    }
    return 0;
}

int is_delim(int c){
    register int i;
    for (i = 0; i < n_delim; i++)
        if (c == delim[i]) return 1;
    return 0;
}

Answer 1

诀窍在于 wl[n] 保存字数 长度为 n。此外，您无需继续调用 strlen() 在每次迭代中，只需检查最后的零字节。 如果您启用它，优化器将为您执行此操作。 奇怪的 for(;1;) 是这样循环计数的 最后一个字，以零字节结束。

memset(wl,0,sizeof(wl));
for(wordStart=maxLength=i=0;1;i++) {
  if(is_delim(string[i]) || string[i]==0) {
    int wordLength= i-wordStart;
    if(wordLength>0)
      wl[wordLength]++;
    if(wordLength>maxLength)
      maxLength= wordLength;
    wordStart= i+1;
  }
  if(string[i]==0)
    break;
}

for(i=1;i<=maxLength;i++) {
  if(wl[i]>0) {
    printf("%d words of length %d.\n",wl[i],i);
  }
}

Answer 2

你真的应该为此使用strtok。现在，您永远不会将最后一个字符串与当前字符串进行比较，因此您无法区分它们。您可以为此使用strcmp。最后，您应该使用strlen，而不是手动测试字符串的长度。这是你的循环的样子

int seen = 0;

pch = strtok(string, delim);
last = pch;
while(pch != NULL) {
  if(strcmp(last, pch) != 0) {
    printf("%s:\t%d\t\t%d\n", last, (int)strlen(last), seen);
    seen = 1;
  }else {
    seen++;
  }
  last = pch;
  pch = strtok(NULL, delim);
}
printf("%s:\t%d\t\t%d\n", last, (int)strlen(last), seen);

注意，你应该在循环之前将变量seen设置为0。