JavaRush/Помощь по Java задачам/Код работает как надо(я думаю) Не проходит валидацию.

35 уровень

08.07.2019
873просмотров
13комментариев

Код работает как надо(я думаю) Не проходит валидацию.

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

User, Loser, Coder and Proger

Java Core, 4 уровень, 8 лекция

Решен

Уже посмотрел коды других кто спрашивал, но не помогло!

Давай напишем программу, которая определит, чем заняться тому или иному человеку.
Для этого нужно:

Ввести [в цикле] с клавиатуры несколько строк (ключей).
Строки (ключи) могут быть такими: "user", "loser", "coder", "proger".
Ввод окончен, когда строка не совпадает ни с одной из выше указанных.
Для каждой введенной строки нужно:
1. Создать соответствующий объект [см. Person.java], например, для строки "user" нужно создать объект класса User.
2. Передать этот объект в метод doWork.
Написать реализацию метода doWork, который:
1. Вызывает метод live() у переданного объекта, если этот объект (person) имеет тип User.
2. Вызывает метод doNothing(), если person имеет тип Loser.
3. Вызывает метод writeCode(), если person имеет тип Coder.
4. Вызывает метод enjoy(), если person имеет тип Proger.

Требования:

Метод main должен считывать строки с клавиатуры.
Метод main должен прекращать считывать строки с клавиатуры, как только введенная строка не совпадает с одной из ожидаемых (user, loser, coder, proger).
Для каждой корректной (user, loser, coder, proger) введенной строки должен быть вызван метод doWork с соответствующим объектом класса Person в качестве параметра.
В классе Solution должен быть реализован метод doWork с одним параметром типа Person.
Метод doWork должен вызывать метод live() у переданного объекта, если этот объект имеет тип User.
Метод doWork должен вызывать метод doNothing() у переданного объекта, если этот объект имеет тип Loser.
Метод doWork должен вызывать метод writeCode() у переданного объекта, если этот объект имеет тип Coder.
Метод doWork должен вызывать метод enjoy() у переданного объекта, если этот объект имеет тип Proger.

package com.javarush.task.task14.task1411; public interface Person { class User implements Person { static void live() { System.out.println("I usually just live."); } } class Loser implements Person { static void doNothing() { System.out.println("I usually do nothing."); } } class Coder implements Person { static void writeCode() { System.out.println("I usually write code."); } } class Proger implements Person { static void enjoy() { System.out.println("It's a wonderful life!"); } } }

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

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

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

Veryprosto

Уровень 35

8 июля 2019, 20:15

Нашел тут у одного товарища, сам пока ни хрена не понял но сработало. Убрал статики у классов усер лосер итп. добавил import static com.javarush.task.task14.task1411.Person.*; Почему и зачем мы это не проходили, помогло избавиться от статиков. По моему нужно формулировать задачу как то подробнее. Есть задача, есть решение. Выдаёт именно то что требуется. но блин неправильно!

Николай

Уровень 36

Expert

8 июля 2019, 20:21

Советую немного притормозить и разобраться полностью в чем была проблема

Veryprosto

Уровень 35

8 июля 2019, 20:32

А как разобраться? код работал и без последних изменений. А сейчас я не смогу валидатора заставить проверять мой код. Я не знаю что именно хочет валидатор. Да есть мысль что делать методы статиками нельзя. Импортировать интерфейс - новое для меня решение, интересное. попробую в дальнейшем использовать этот вариант. import static com.javarush.task.task14.task1411.Person.*; судя по всему импортируются классы интефейса персон со статиком. т.е. я явно не делаю классы статическими, но при этом импортирую их статиками))

Николай

Уровень 36

Expert

8 июля 2019, 20:43

не надо ничего импортировать. Проблема была только в статиках в методах.

JustinianJudge в Mega City One

Master

8 июля 2019, 20:46

В программировании очень важно КАК работает. И это очень хорошо, что валидатор не дает тебе сойти на кривую дорожку процедурщины, к коей относится ненормированное употребление статиков. Ну вот в этом случае, какую цель ты преследовал, делая методы статическими? Что это тебе должно было дать и какую выгоду, по сравнению с нестатическими? Также, Идея должна была бы подсказать. В этой строке:

((Person.Proger) person).enjoy();

Идея бы подсветила ярким оранжевым и дала надпись (при наведении курсором) о том что статические методы вызываются на инстансе. А в этом ненужном импорте:

import static com.javarush.task.task14.task1411.Person.*;

идея должна была бы выделить серым и сказать что он не используется, если ты там конечно еще не наделал модификаторов. import static это аналог обычного import. Это инструмент который позволяет обращаться к членам другого класса без указания полного пути класса (пакет, имя класса). Вот это без использования импорта код:

java.io.BufferedReader bufferedReader = new java.io.BufferedReader(new InputStreamReader(System.in));

А это с импортом:

import java.io.BufferedReader;
BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

Это просто добавляет удобства, ничего иного с модификаторами статик/не статик не происходит, это сокращение записи. Так что будь осторожен копипастя чужие ошибки :) И тем более будь осторожен в выводах что помогло, а что нет.

Veryprosto

Уровень 35

9 июля 2019, 17:17

На счет процедурщины ты прав, я именно этим в этой задаче занимался. Но вот как раз ИДЕА меня натолкнула на это. Когда я написал цикл и попытался его запустить, ИДЕА как раз и сказала что то про то что статик в не статике иллегал.... я чтобы её удовлетворить сделал все методы статиками. ИДЕА удовлетворилась, код работал, но валидатор не пускал без объяснения ПОЧЕМУ. Потом почитал помощь других и нашел интересное(новое для меня решение - убрать статики и импортировать интерфейс. Все заработало. Сейчас попробовал убрать статики и этот импорт все работает. К сож не знаю прошёл бы валидатора так как "неисповедимы пути валидатора"! Единственно что мне до сих пор не понятно почему код if (person instanceof Person.User) { ((Person.User) person).live(); } - правильный а код if (person instanceof Person.User) { Person.User.live(); } не правильный. Для чего нужно приведение к типу Юзер?

Tanarri

Уровень 39

9 июля 2019, 17:37

полезный

верхний код сужает объект до Person.User объекта и вызывает метод на объекте. Нижний пытается вызвать статический метод класса. Приведение же нужно, потому что твой интерфейс Person не имеет метода live(), в то время как этот метод есть у Person.User

JustinianJudge в Mega City One

Master

9 июля 2019, 18:39

полезный

Статические методы в Джава как правило утильные, а в основном методы вызываются на конкретном созданном объекте. А чтобы вызывать на конкретном объекте метод, нам нужно сначала привести тип (как ты помнишь, тип объекта определяет перечень доступных для использования методов) к нужному.

Анатолий

Уровень 28

8 июля 2019, 20:14

while (true){
           key = reader.readLine();
           if (key.equals("user")) person = new Person.User();
           else if (key.equals("loser")) person = new Person.Loser();
           else if (key.equals("coder")) person = new Person.Coder();
           else if (key.equals("proger")) person = new Person.Proger();
           else break;
           doWork(person); //вызываем doWork
       }

Попробуй немного по другому. while (true) { key = reader.readLine(); if (key.contains("user")) { person = new Person.User(); doWork(person); } else if (key.contains("loser")) { person = new Person.Loser(); doWork(person); и тд. В конце break P.s. Операторные скобки не забывай расставлять когда if-else пишешь, это полезная привычка. ИМХО.

Veryprosto

Уровень 35

8 июля 2019, 20:19

в этом варианте если в консоль написать userman? Скобки не расставляю если одна строчка. зачем четыре раза doWork.. цикл не закончился если правильный ввод вполне можно после ифов.

Анатолий

Уровень 28

8 июля 2019, 20:32

Не помню сейчас уже почему именно так решил, но у меня решение прошло.

Николай

Уровень 36

Expert

8 июля 2019, 20:05

Может из за того, что методы класса Person статические?

Veryprosto

Уровень 35

8 июля 2019, 20:20

может. но идеа требовала. нашел решение см. выше