CodeGym/Pomoc z zadaniami Java/Cannot find symbol (User,Loser,Coder,Programmer)

Poziom 27

Warsaw

06.02.2020
1124widoki
13komentarze

Cannot find symbol (User,Loser,Coder,Programmer)

Pytania dotyczące zadania

Dyskutowane

Hi Guys, I cannot work it out, I already tried creating new instances of classes, casting them on person, creating new instances, and using them for method innitalization.

1. Wprowadź kilka ciągów (kluczy) z klawiatury [w pętli].
Dostępne są nastepujące ciągi (klucze): "user", "loser", "coder", "programmer".
Przestań wczytywać ciągi po wczytaniu jakiegokolwiek ciągu, który nie pasuje do żadnego z powyższych.

2. Dla każdego wprowadzonego ciągu:
2.1. Utwórz odpowiedni obiekt [patrz Person.java], np. utwórz obiekt User dla ciągu "user".
2.2. Przekaż ten obiekt do metody doWork.

3. Napisz implementację metody doWork, która:
3.1. Wywołuje metodę live() na przekazywanym obiekcie, jeśli osoba to User.
3.2. Wywołuje metodę doNothing() na przekazywanym obiekcie, jeśli osoba to Loser.
3.3. Wywołuje metodę writeCode() na przekazywanym obiekcie, jeśli osoba to Coder.
3.4. Wywołuje metodę enjoy() na przekazywanym obiekcie, jeśli osoba to Programmer.

Wymagania:

Metoda main musi wczytywać ciągi z klawiatury.
Metoda main powinna przerwać wczytywanie ciągów z klawiatury, gdy tylko wprowadzony ciąg nie pasuje do jednego z dozwolonych ciągów ("user", "loser", "coder", "programmer").
Dla każdego prawidłowo wprowadzonego ciągu ("user", "loser", "coder", "programmer") wywołaj metodę doWork z odpowiednim obiektem Person jako argumentem.
Klasa Solution musi implementować metodę doWork z jednym parametrem Person.
Metoda doWork powinna wywoływać metodę live() na przekazywanym obiekcie, jeśli jest to User.
Metoda doWork powinna wywoływać metodę doNothing() na przekazywanym obiekcie, jeśli jest to Loser.
Metoda doWork powinna wywoływać metodę writeCode() na przekazywanym obiekcie, jeśli jest to Coder.
Metoda doWork powinna wywoływać metodę enjoy() na przekazywanym obiekcie, jeśli jest to Programmer.

package pl.codegym.task.task14.task1411; import java.io.BufferedReader; import java.io.InputStreamReader; /* User, loser, coder i programmer */ public class Solution { public static void main(String[] args) throws Exception { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); Person person = null; String key = null; while (true) { key = reader.readLine(); if (!(key.equals("user") || key.equals("loser") || key.equals("coder") || key.equals("programmer"))) { break; } if ("user".equals(key)) { User user = new User(); person = user; } else if ("loser".equals(key)) { Loser loser = new Loser(); person = loser; } else if ("coder".equals(key)) { Coder coder = new Coder(); person = coder; } else if ("programmer".equals(key)) { Programmer programmer = new Programmer(); person = programmer; } doWork(person); // Wywołuje doWork } } public static void doWork(Person person) { if (person instanceof User) { person.live(); } else if (person instanceof Loser) { person.doNothing(); } else if (person instanceof Coder) { person.writeCode(); } else if (person instanceof Programmer) { person.enjoy(); } } }

Komentarze (13)

Popularne
Najnowsze
Najstarsze

Musisz się zalogować, aby dodać komentarz

Guadalupe Gagnon

Poziom 37 , Tampa, United States

6 lutego 2020, 14:57

przydatny

The big problem is that this code does not use the proper class names for the Person interface classes when called in the Solution class. They would be Person.User, Person.Loser etc. You have 2 options to fix this: 1) go through your code and make sure you use the full qualified names so that the computer knows what you mean; or 2) add the proper import statement to the Solution class that tells the IDE when you type the simple class names you actually mean the full class names. The import statement would be something along the lines of: import pl.codegym.task.task14.task1411.Person.*;

Guadalupe Gagnon

Poziom 37 , Tampa, United States

6 lutego 2020, 15:00

przydatny

your next problem is line 19. This line reads: if key does not equal "user" OR key equals "loser" OR key equals "coder" OR key equals "programmer" If any of those individual statements separated by the OR logic are true, then the whole statement is true. The only value you can enter to make this statement false is "user"

DeZilius

Poziom 27 , Warsaw, Poland

12 lutego 2020, 16:42

Thank You for the tips, I have already tried both ways with creating objects (importing and using full name i.e. Person.User()) but now I have different issue. I cannot call functions on person object a.k.a. person-user, person-loser etc. Is there something else I am missing? I am able to call the it on Person.User.live() but no person.live() /////////////////////ERROR pl/codegym/task/task14/task1411/Solution.java:44: error: cannot find symbol person.live(); ^ symbol: method live() location: variable person of type pl.codegym.task.task14.task1411.Person

Guadalupe Gagnon

Poziom 37 , Tampa, United States

12 lutego 2020, 17:29

This is because the variable "person" is an instance of a Person (interface), which does not contain any methods, even though the object contained in the variable is of either Coder, Loser, User, Programmer. This is a concept of polymorphism which is extremely useful in object orientated programming, though requires a little understanding to fully utilize it's power. What you will need to do first is cast the variable to the proper class before calling the method you need to.

DeZilius

Poziom 27 , Warsaw, Poland

14 lutego 2020, 17:38

Found solution, code technically works, but task doesn't approve the way of calling methods. Last 4 requirements are not passing.

Guadalupe Gagnon

Poziom 37 , Tampa, United States

14 lutego 2020, 18:04

what does your code look like now?

Damian Siadaczka

Poziom 20 , Warszawa, Poland

20 lutego 2020, 16:33

I have the same problem

public static void doWork(Person person) {
        // Punkt 3
        if(person instanceof Person.User)
           ((Person.User) person).live();
        if(person instanceof Person.Loser)
            ((Person.Loser) person).doNothing();
        if(person instanceof Person.Coder)
            ((Person.Coder) person).writeCode();
        if(person instanceof Person.Programmer)
            ( (Person.Programmer) person).enjoy();
    }

this method not working but program is compile

Guadalupe Gagnon

Poziom 37 , Tampa, United States

20 lutego 2020, 17:10

cant help unless you post all the code. You should probably make a post.

Damian Siadaczka

Poziom 20 , Warszawa, Poland

20 lutego 2020, 18:05

package pl.codegym.task.task14.task1411;

import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.util.List;
import java.util.ArrayList;


/*
User, loser, coder i programmer
*/

public class Solution {
    public static void main(String[] args) throws Exception {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        Person person = null;
        String key = null;

        // Pętla do wczytywania kluczy. Punkt 1
        List<Person> list = new ArrayList<>();
        while(true)
        {
            key = reader.readLine();
            if(key.equals("user"))
                list.add(new Person.User());
            if(key.equals("loser"))
                list.add(new Person.Loser());
            if(key.equals("coder"))
                list.add(new Person.Coder());
            if(key.equals("programmer"))
                list.add(new Person.Programmer());
            else break;
        }
        for(int i=0; i<list.size(); i++)
        {
            // Tworzy obiekt. Punkt 2
            if(person instanceof Person)
            person = list.get(i);
            doWork(person); // Wywołuje doWork

        }
    }

    public static void doWork(Person person) {
        // Punkt 3
        if(person instanceof Person.User)
           ((Person.User) person).live();
        if(person instanceof Person.Loser)
            ((Person.Loser) person).doNothing();
        if(person instanceof Person.Coder)
            ((Person.Coder) person).writeCode();
        if(person instanceof Person.Programmer)
            ( (Person.Programmer) person).enjoy();
    }
}

Guadalupe Gagnon

Poziom 37 , Tampa, United States

20 lutego 2020, 18:31

get rid of the list and do this:

while(true) {
   key = reader.readLine();
   if(key.equals("user"))
      person = new Person.User();
   if(key.equals("loser"))
      person = new Person.Loser();
   if(key.equals("coder"))
      person = new Person.Coder();
   if(key.equals("programmer"))
      person = new Person.Programmer();
   else break;

   doWork(person);
}

Damian Siadaczka

Poziom 20 , Warszawa, Poland

20 lutego 2020, 19:35

i do it and... nothing the program displays nothing

Guadalupe Gagnon

Poziom 37 , Tampa, United States

20 lutego 2020, 19:45

sorry. those should all be if -else if statments:

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("programmer"))
                person = new Person.Programmer();
            else break;

            doWork(person);
        }

Damian Siadaczka

Poziom 20 , Warszawa, Poland

20 lutego 2020, 21:26

thenk you, now is work