3
import java.util.Scanner;

public class Task10 {
    public static void main(String[] args) {

        Scanner scanner = new Scanner(System.in);
        System.out.println("Введите строку:");
        String str = scanner.nextLine();

        String zhiWrong = "жы";
        String shiWrong = "шы";
        String zhiRight = "жи";
        String shiRight = "ши";

        str = str.toLowerCase();

        String[] words = str.split(" ");

        for (int i = 0; i < str.length(); i++) {

            //words = str.split(" ");
            System.out.println(words[i]);
            if (words[i].contains(zhiRight) | words[i].contains(shiRight)) {
                System.out.println("Правильно!");
            } else if (words[i].contains(zhiWrong) | words[i].contains(shiWrong)) {
                System.out.println("Неправильно! Жи/Ши пиши с буквой 'и'");
            } else
                System.out.println("Для этого слова правило не применяется");
        }
    }
}

Поменял. Но теперь выходит: Введите строку: жыраф жывет в африке жыраф Неправильно! Жи/Ши пиши с буквой 'и' жывет Неправильно! Жи/Ши пиши с буквой 'и' в Для этого слова правило не применяется африке Для этого слова правило не применяется Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index 4 out of bounds for length 4 at Task10.main(Task10.java:22) - это строка System.out.println(words[i]);

О чем это говорит? Что нужно исправить?

6
  • Попробуйте не next(), а nextLine(). И в цикле надо писать не <=, а строго меньше, потому что в массиве из n элементов последний элемент имеет индекс n-1. Кстати, лучше код вставлять не скрином, а текстом с 4 пробелами в начале каждой строки. 30 апр 2019 в 12:26
  • Интересно. А в чем разница между next() и nextLine()? Я думал, next() - это универсальная вещь
    – Coder8
    30 апр 2019 в 13:09
  • nextLine() читает всю строку, а next() - только то что до пробела. 30 апр 2019 в 13:14
  • Спасибо за ответ) twollingchar, как здесь можно поставить лайк за ответ?
    – Coder8
    30 апр 2019 в 13:24
  • Сейчас я его оформлю, и можно будет поставить лайк. Никак не отучу себя писать ответы в комментах. 30 апр 2019 в 13:26

1 ответ 1

2

Функция next() считывает символы только до пробела. Чтобы считать всю строку, надо использовать nextLine().

А еще я вижу недочет в логике, вот здесь:

if (words[i].contains(zhiRight) | words[i].contains(shiRight)) {
    System.out.println("Правильно!");
} else if (words[i].contains(zhiWrong) | words[i].contains(shiWrong)) {
    System.out.println("Неправильно! Жи/Ши пиши с буквой 'и'");
} else
    System.out.println("Для этого слова правило не применяется");
}

Сначала проверяется наличие правильного написания, и нам выведет "Правильно" даже если там есть неправильное. Проверьте на слове "жыжи". Надо поменять местами первое условие со вторым.

Также вместо | здесь лучше использовать ||, чтобы избежать лишних вычислений (a() || b() не вызовет b() если a() вернуло true).

1

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.