C语言中单词、句子和字母的计数程序中的问题

Question

我需要制作一个程序，通过用户的输入来计算单词、句子和字母的数量。在我给出的输入是多行的之前，这个程序工作得很好。如果输入长度超过终端窗口中可容纳的文本长度，程序将开始忽略所有句号/问号/感叹号。我不知道为什么，我需要一些帮助。如果文本可以放在终端窗口的一行中，则不会发生这种情况。我还打印了程序读取的每个字符，但这也忽略了所有的句号/问号/感叹号。这些字符都不会被打印出来。为了澄清，一个句子只是句号/问号/感叹号的数量，单词的数量只是文本中空格的数量加上1。这是我的密码：

#include <stdio.h>
#include <ctype.h> //for the isalpha() function
#include <cs50.h>  //for the get_string() function

int main(void)
{
    int sentences = 0, letters = 0;
    int words = 1;
    char character;
    string text = get_string("Enter Text: \n");

    char x = 0;

    while (text[x] != '\0')
    {
        character = text[x];
        switch (character)
        {
        case ' ':
            words++;
            break;
        case '.':
            sentences++;
            break;
        case '?':
            sentences++;
            break;
        case '!':
            sentences++;
            break;
        default:
            if (isalpha(character))
            {
                letters++;
            }
        }
        x++;
    }

    printf("\n");
    printf("WORDS: %d, LETTERS: %d, SENTENCES: %d\n", words, letters, sentences);
}

我对c相当陌生，但我有大约一年的Python经验。谢谢你抽出时间。

Answer 1

我将提出一些建议。

首先，不要使用get_string¹（或scanf，或fgets）。对于这样的过滤程序，实际上不需要存储输入来处理它；使用getchar（或fgetc）一次读取一个字符，并在此基础上循环：

int c;  // getchar returns int, not char
...
puts( "Enter Text:" );
while ( ( c = getchar() ) != EOF )
{
  // test c instead of text[x]
}

这种方法将处理任何长度的输入（例如，如果您将文件重定向为输入），并且它避免了在注释中发现的潜在溢出问题Weather Vane。缺点是，您必须从控制台手动发出信号EOF以进行交互式输入（根据您的平台使用Ctrl-z或Ctrl-d）。

您可以在开关中折叠一些测试，例如

case '.' :     // Each of these cases "falls through"
case '!' :     // to the following case.
case '?' :
  words++;     // the end of a sentence is also the end of a word
  sentences++;
  break;

您需要添加案例来处理换行符和制表符：

case ' ' :
case '\n' :
case '\t' :
  words++;
  break;

除非您不希望碰撞单词计数器以重复空白字符，或者如果上一个非空白字符是标点符号字符。因此，您需要一个额外的变量来跟踪以前读取的字符的类：

enum {NONE, TEXT, PUNCT, WHITE} class = NONE;
...
while ( ( c = getchar() ) != EOF )
{
  switch( c )
  {
    case ' ' :
    case '\n' :
    case '\t' :
      if ( class == TEXT )
        words++;
      class = WHITE;
      break;
    
    case '.' :
    case '!' :
    case '?' :
      if ( class == TEXT )  // Don’t bump the word counter
        words++;            // if the previous character was
                            // was whitespace or .! ?
      if ( class != PUNCT ) // Don’t bump the sentence counter
        sentences++;        // for repeating punctuation
      
      class = PUNCT;
      break;
      ...
  }
}

仍然会有一些奇怪的情况，这不会给出一个完全准确的计数，但对于大多数输入来说应该足够好了。

你应该能从那里找到剩下的。

1. CS50字符串处理和I/O例程（如get_string）非常巧妙，但是它们严重歪曲了C实际的工作方式。typedef的string特别令人震惊，因为它的别名不是字符串。请注意，这些工具在CS50课程之外是不可用的，所以不要过于依赖它们。