JavaRush/Помощь по Java задачам/С этим решением у меня не загружает результат, слишком до...

18 уровень

10.07.2021
349просмотров
11комментариев

С этим решением у меня не загружает результат, слишком долго

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

Решен

Без преувеличений не меньше 3х часов просидел за задачей и поэтому хочу знать, хотя бы туда смотрю?

В классе Solution константа INVENTION_DAY содержит дату изобретения нового популярного гаджета — карманного телепорта.

Реализуй метод isTeleportInvented, который в качестве параметра принимает дату и сравнивает ее с INVENTION_DAY. Если переданная дата следует после даты изобретения, нужно вывести надпись: "Пожалуйста, можете забрать ваш карманный телепорт. Спасибо за ожидание!" — и вернуть true.
Если переданная дата предшествует дате изобретения, вывести: "Извините, телепорт еще не изобрели, приходите через 10 лет." — и вернуть false.

В методе main в цикле вызови isTeleportInvented и передай currentDay в качестве параметра. Если метод возвращает false, прибавь к currentDay 10 лет, иначе — выйди из цикла.

Требования:

Метод isTeleportInvented должен вывести соответствующую надпись и вернуть true, если переданная дата больше INVENTION_DAY.
Метод isTeleportInvented должен вывести соответствующую надпись и вернуть false, если переданная дата меньше INVENTION_DAY.
Каждый раз, когда isTeleportInvented возвращает false, нужно увеличивать currentDay на 10 лет.
Метод isTeleportInvented должен вызываться до тех пор, пока он не вернет true.

package com.javarush.task.pro.task16.task1605; import java.util.Calendar; import java.util.GregorianCalendar; /* Карманный телепорт */ public class Solution { public static final String INVENTED = "Пожалуйста, можете забрать ваш карманный телепорт. Спасибо за ожидание!"; public static final String NOT_INVENTED = "Извините, телепорт еще не изобрели, приходите через 10 лет."; public static final Calendar INVENTION_DAY = new GregorianCalendar(2192, Calendar.JUNE, 12); public static void main(String[] args) { Calendar currentDay = Calendar.getInstance(); //напишите тут ваш код while (currentDay.getTime().before(INVENTION_DAY.getTime())) { isTeleportInvented(currentDay); } } static boolean isTeleportInvented(Calendar currentDay) { //напишите тут ваш код boolean isWork = false; currentDay.add(Calendar.YEAR, 10); System.out.println(NOT_INVENTED); if (!currentDay.getTime().before(INVENTION_DAY.getTime())) { isWork = true; System.out.println(INVENTED); } return isWork; } }

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

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

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

KIRA

Уровень 41

Expert

10 июля 2021, 18:03

решение

У вас полная каша в методах, они делают не то что сказано в задании К примеру год 2190 в методе мейн

while (currentDay.getTime().before(INVENTION_DAY.getTime())) { //2190 раньше 2192 true
   isTeleportInvented(currentDay); //вызвали метод
}

В методе

currentDay.add(Calendar.YEAR, 10); //прибавили 10 лет стало 2200
System.out.println(NOT_INVENTED); //вывели приходите через 10 лет
if (!currentDay.getTime().before(INVENTION_DAY.getTime())) { // 2200 до 2192? false и инверсия дает true
  isWork = true;
  System.out.println(INVENTED); //вывели что телепорт готов
}

Вам нужно переписать логику и перенести действия согласно требованиям

KIRA

Уровень 41

Expert

10 июля 2021, 18:14

решение

Метод main должен в цикле вызывать метод isTeleportInvented и в зависимости от результата либо прекратить работу (true) либо прибавить 10 лет (false) Метод isTeleportInvented должен сравнить даты и если переданная дата до INVENTION_DAY вывести соответствующее сообщение и вернуть false иначе вывести другое сообщение и вернуть true

Дмитрий

Уровень 18

10 июля 2021, 19:17

public static void main(String[] args) {
        Calendar currentDay = Calendar.getInstance();
        //напишите тут ваш код
        while (currentDay.getTime().before(INVENTION_DAY.getTime())) {
            isTeleportInvented(currentDay);
        }
    }

    static boolean isTeleportInvented(Calendar currentDay) {
        //напишите тут ваш код
        boolean isWork = false;
        currentDay.add(Calendar.YEAR, 10);
        if (currentDay.getTime().after(INVENTION_DAY.getTime())) {
            isWork = true;
            System.out.println(INVENTED);
        }
        else if (currentDay.getTime().before(INVENTION_DAY.getTime())) {System.out.println(NOT_INVENTED);}

        return isWork;
    }
}

Попытался исправить, хотя думаю стоит продолжить уже завтра

KIRA

Уровень 41

Expert

10 июля 2021, 19:38

решение

метод isTeleportInvented не должен прибавлять 10 лет, это должно происходить в методе main если метод isTeleportInvented вернет false метод main не должен сравнивать даты, это делает метод isTeleportInvented Задача метода main вызвать метод и в зависимости от того, что он вернет либо завершить работу, либо прибавить 10 лет currentDay Вы никак не используете результат работы метода isTeleportInvented

Дмитрий

Уровень 18

13 июля 2021, 14:24

После этих слов я понял свою главную ошибку. Всегда, читать, условия, до конца. Надеюсь что хотя бы с виду этот вариант рабочий

public static void main(String[] args) {
        Calendar currentDay = Calendar.getInstance();
        //напишите тут ваш код
        while (!isTeleportInvented(currentDay)) {
            currentDay.add(Calendar.YEAR, 10);
            isTeleportInvented(currentDay);
        }
    }

    static boolean isTeleportInvented(Calendar currentDay) {
        //напишите тут ваш код
        boolean isWork = false;
            if (currentDay.getTime().after(INVENTION_DAY.getTime())) {
                isWork = true;
                System.out.println(INVENTED);
            } else if (currentDay.getTime().before(INVENTION_DAY.getTime())) {
                System.out.println(NOT_INVENTED);
            }
        return isWork;
    }
}

KIRA

Уровень 41

Expert

13 июля 2021, 14:37

решение

в main методе вы дважды вызываете метод isTeleportInvented(currentDay); соответственно и выводов в консоль будет в 2 раза больше 2 вызов лишний У вас получается вызов while(isTeleportInvented(currentDay)) если false то прибавили 10 лет и снова будет проверяться условие цикла while т.е. еще раз будет вызван метод и тд

Дмитрий

Уровень 18

13 июля 2021, 18:02

После последнего исправления я уже задумался о своих способностях к этому делу, в любом случае буду продолжать и огромное спасибо за помощь, задание принято валидатором

Никита

Уровень 31

Expert

10 июля 2021, 17:18

А что выдает валидатор?

Дмитрий

Уровень 18

10 июля 2021, 17:40

Что программа загружается слишком долго и после этого валидация просто заканчивается. Понимаю что может со стороны оптимизации у меня беда. И все таки черт побери, почти 4 часа времени, я просто хочу закончить это

Никита

Уровень 31

Expert

10 июля 2021, 17:52

while (currentDay.getTime().before(INVENTION_DAY.getTime())) {
           isTeleportInvented(currentDay);
       }

значит есть варианты когда этот цикл уходит в бесконечный. хотя честно скажу пока не знаю этот вариант. но если ввести дату изначально больше чем INVENTION_DAY то программа вообще ничего не выдаст

Дмитрий

Уровень 18

10 июля 2021, 19:15

Дело в том что цикл не бесконечный но одну ошибку в логике нашел, там был один лишний "прогон" но видимо еще есть проблема(ы)