为什么这计算的字数不正确？

Question

公共静态无效字（字符串文本）{int numWords=1；

    for (int charNum = 0; charNum < text.length(); charNum++) {
        char letter = text.charAt(charNum);
        
        if(charNum + 1 < text.length()) {
            char nextLetter = text.charAt(charNum + 1);
            if(letter == '.' || letter == ':' || letter == ';' || letter == '?' || letter == '!'|| letter == ' ' || letter == '\n' || letter == '\t') {
                if(nextLetter != '.' || nextLetter != ':' || nextLetter != ';' || nextLetter != '?' || nextLetter != '!'|| nextLetter != ' ' || nextLetter != '\n' || nextLetter != '\t')
                    numWords += 1;
            }   
        }
    }
    
    System.out.println("Number of words: " + numWords);
}

字符串“是this_one_long_word还是几个？？？你觉得怎么样？？太多“应该打印10个字和”！这使用periods.as.word.delimiters，可能很棘手。“应该打印10个单词。

描述如下：一个单词是由一个或多个字符组成的序列，由空格或句子终止符（句号、冒号、分号、问号、感叹号）分隔，无论它是否为实际的英语单词。空白被定义为空格、制表符（'\t'）、新行字符（'\n'）和字符串本身的结尾。这再次给出了一些可能没有意义的结果。例如，文本“我不喜欢使用空格！”只有一个词：我永远不喜欢使用空格。

Answer 1

您在最后一个if（最嵌套的一个）中有无效条件-您需要确保下一个字符（在代码中称为nextLetter）不是您检查的字符，但您使用或代替了和条件，因此它始终为真（因为字符不能同时是两个不同的字符）。

因此，您可以将此如果更改为：

if(nextLetter != '.' && nextLetter != ':' && nextLetter != ';' && nextLetter != '?' && nextLetter != '!'&& nextLetter != ' ' && nextLetter != '\n' && nextLetter != '\t')

它会正确地计算单词。请确保您也计算第一个单词，因为目前如果您将NumWords设置为零，它将为您的示例打印9而不是10-因为第一个单词不被计算在内。所以你也应该数数。

Btw.您的代码可以在几个方面得到显著简化，您也可以使用替代方法，如拆分函数（来自String）。

Answer 2

以下一行始终正确：

if(nextLetter != '.' || nextLetter != ':' || nextLetter != ';' || nextLetter != '?' || nextLetter != '!'|| nextLetter != ' ' || nextLetter != '\n' || nextLetter != '\t')

将其更改为：

if(nextLetter != '.' && nextLetter != ':' && nextLetter != ';' && nextLetter != '?' && nextLetter != '!'&& nextLetter != ' ' && nextLetter != '\n' && nextLetter != '\t')

开始计算int numwords=1 将为您提供不正确的计数。从0开始。

您的第二个输入从开始！向计数中添加一个单词。
您必须检查终止符后面的字符是否是字母，以避免这种行为：

 public static void words(String text) {
        int numWords = 0;
        final String wordTerminators = ".:;?! \n\t";
        for (int charNum = 0; charNum < text.length() - 1; charNum++) {
            char letter = text.charAt(charNum);
           
            char nextLetter = text.charAt(charNum + 1);
            if (wordTerminators.contains(letter + "")) {
                if (!wordTerminators.contains(nextLetter + "") && !(nextLetter+"").matches("[A-Za-z]"))
                    numWords += 1;
            }

        }
        System.out.println("Number of words: " + numWords);
    }

以下是我的解决方案的更清晰版本：

    public static void words(String text) {
        int numWords = 0;
        
        String[] splitted = text.split("[.:;?! \\n\\t]+");
        for (String word : splitted) {
            if (word.length() > 0)
                numWords++;
        }
        System.out.println("Number of words: " + numWords);
    }

两种方法的输出都是：

Number of words: 10
Number of words: 10