Все правильно работает, но ответ не принимает

Вопрос по задаче

Обсуждается

Почему не принимает, вроде все валидно, где ошибка? 🙄🙄🙄.

В этой задаче тебе предстоит реализовать метод getPrediction() в классе MagicBall, который будет работать как шар предсказаний.
Он случайным образом возвращает фразу-ответ на заданный вопрос. В методе getPrediction() нужно использовать метод nextInt(int) класса Random, который должен возвращать значения от 0 до 7 включительно.
В зависимости от результата работы метода nextInt(int) возвращается один из вариантов ответа:
0 -> Бесспорно
1 -> Определённо да
2 -> Вероятнее всего
3 -> Хорошие перспективы
4 -> Спроси позже
5 -> Попробуй снова
6 -> Мой ответ — нет
7 -> Весьма сомнительно
Иначе вернуть null.

Требования:

Реализуй метод getPrediction() согласно условию.

package com.javarush.task.pro.task08.task0805; import java.util.Random; public class MagicBall { private static final String CERTAIN = "Бесспорно"; private static final String DEFINITELY = "Определённо да"; private static final String MOST_LIKELY = "Вероятнее всего"; private static final String OUTLOOK_GOOD = "Хорошие перспективы"; private static final String ASK_AGAIN_LATER = "Спроси позже"; private static final String TRY_AGAIN = "Попробуй снова"; private static final String NO = "Мой ответ — нет"; private static final String VERY_DOUBTFUL = "Весьма сомнительно"; public static String getPrediction() { String[] answers = new String[8]; Random r = new Random(); int x = r.nextInt(answers.length + 1); answers[0] = CERTAIN; answers[1] = DEFINITELY; answers[2] = MOST_LIKELY; answers[3] = OUTLOOK_GOOD; answers[4] = ASK_AGAIN_LATER; answers[5] = TRY_AGAIN; answers[6] = NO; answers[7] = VERY_DOUBTFUL; if(x <= 7){ return answers[x]; } else{ return null; } } }

Комментарии (3)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

KaiRev0

Уровень 18

10 октября, 10:58

Исправь

int x = r.nextInt(answers.length + 1);

на

int x = r.nextInt(answers.length);

Должно получиться, потому что условие требует значения от 0 до 7 включительно. У тебя же получается от 0 до 8 включительно, потому что nextInt считает от первого числа (ноль) до последнего указаного числа не включительно. То есть, ты указываешь answers.length + 1 = 9, значит случайные числа будут от 0 до 8, а нам надо от 0 до 7, исправляем на answers.length = 8, тогда и получим новый диапазон: 0..7, 8 не включително по определению random.nextInt(); Надеюсь, помог, а так другой комментатор прав, что тут желательно использовать именно switch case, а не массив. Например:

return switch (r.nextInt(8)) {
            case 0 ->CERTAIN;
            case 1 ->DEFINITELY;
            case 2 ->MOST_LIKELY;
            case 3 ->OUTLOOK_GOOD;
            case 4 ->ASK_AGAIN_LATER;
            case 5 ->TRY_AGAIN;
            case 6 ->NO;
            case 7 ->VERY_DOUBTFUL;
            default -> null;
        };

Гофф

Уровень 3

Master

10 октября, 07:06

Если бы правильно работало, тогда приняло бы... Похоже вот тут собака порылась: answers.length + 1

Violetta Barsukova

Уровень 11

8 октября, 09:08

попробуй switch case