Метод вхождения слова не считая

1

У меня есть метод, который должен учитывать появление слова в предложении и возвращать результаты. По какой-то причине это не считается. Код всегда возвращает 0.

Вот код.

public static int countOccurance (String word, String sentence) {
int count = 0;
for (int i = 0; i != sentence.length()-word.length()-2; i++) {
    if (sentence.substring(i, i+word.length()-1).equalsIgnoreCase(word)) {
        count++;
    }
}
return count;
}

Может ли кто-нибудь сказать мне, что случилось с моим методом?

  • 1
    Что он тогда делает?
  • 1
    образец ввода и вывода?
Теги:

7 ответов

1
Лучший ответ

Я просто взял ваш код и удалил ваши -1 и -2, так как я не понимал, почему они были там.

public static int countOccurance (String word, String sentence) {
    int count = 0;
    for (int i = 0; i != sentence.length()-word.length(); i++) {
        if (sentence.substring(i, i+word.length()).equalsIgnoreCase(word)) {
            count++;
        }
    }
    return count;
}

Пробовал это.

String sentence = "yakkity yak yak yak attack";
String word = "yak";

int wc = countOccurance (word, sentence);

System.out.println("The string \""+sentence+"\" contains the word "+word+" "+wc+" times.");

Результат:

The string "yakkity yak yak yak attack" contains the word yak 4 times.
Поделиться
1

Ваша подстрока выключена, и ваша проверка положения циклы также неверна. Когда я изменю свой метод,

public static int countOccurance(String word, String sentence) {
    int count = 0;
    for (int i = 0; i + word.length() < sentence.length(); i++) {
        if (sentence.substring(i, i + word.length()).equalsIgnoreCase(
                word)) {
            count++;
        }
    }
    return count;
}

public static void main(String[] args) {
    System.out.println(countOccurance("Hello", "hello hello world"));
    System.out.println(countOccurance("Fi", "Fee Fi Fum"));
}

он выводит ожидаемые

2
1
Поделиться
0

Лучше всего проверить "пространство", как каждое пространство, с которым нужно столкнуться... count++..... и вы также можете проверить "\n" в чеке и завершить цикл at\0 строка null знак !!

Поделиться
  • 0
    посмотрите на \ 0 символ? это не персонал переменного тока?
0

Вы можете попробовать этот код:

public static void main(String[] args){
    String sentence = "Hello truc, Hello machi, and Hello all !";
    final String word = "hello";

    System.out.println(count(sentence.toLowerCase(), word.toLowerCase()));
}

public static int count(String sentence, String word){
    int cpt = 0;
    while(sentence.contains(word)){
        cpt++;
        sentence = sentence.substring(sentence.indexOf(word) + word.length());
    }
    return cpt;
}
Поделиться
  • 0
    Хм .. у этого кода много накладных расходов на очень простую задачу.
0

Я бы использовал regex

    Matcher m= Pattern.compile("\\b" + word + "\\b").matcher(sentence);
    while(m.find()) {
        count++;
    }
Поделиться
0

Использование метода Collections.frequency() в List слов обойдует всю потребность в отслеживании счета:

 public static int countOccurance (String word, String sentence) {
     List<String> words = Arrays.asList(sentence.split(" "));
     return Collections.frequency(words,word);
 }
Поделиться
0

Потому что другие уже объяснили, почему ваш текущий код не работает, я хотел бы использовать этот ответ, чтобы сделать рекомендацию. Я лично рекомендовал бы использовать Java regex API для этой проблемы. Используя что-то вроде следующего;

public static int countOccurance (String regex, String sentence) {
    Pattern p = Pattern.compile(word);
    Matcher m = p.matcher(sentence);
    int count = 0;
    while(m.find()) count++;

    return count;
}

Надеюсь, это поможет.

Поделиться

Ещё вопросы

Сообщество Overcoder
Наверх
Меню