Is something wrong with my code?

Rozwiązane

I'm pretty sure my code is good but when i run it, it shows me some weird stuff.

public class Artifact {

    int number;
    String culture;
    int century;

    public Artifact(int number){
        this.number = number;
    }
    public Artifact(int number, String culture){
        this.number = number;
        this.culture = culture;
    }
    public Artifact(int number, String culture, int century){
        this.number = number;
        this.culture = culture;
        this.century = century;
    }

    public static void main(String[] args) {
        Artifact artifact1 = new Artifact(212121);
        Artifact artifact2 = new Artifact(213421, "Aztecs");
        Artifact artifact3 = new Artifact(216721, "Romans", 12);

        System.out.println("The first type: " + artifact1);
        System.out.println("The second type: " + artifact2);
        System.out.println("The third type: " + artifact3);

    }
}

Komentarze (5)

Popularne
Najnowsze
Najstarsze

Musisz się zalogować, aby dodać komentarz

Karol Grzeszczak

Poziom 31 , Poland, Poland

1 września 2022, 20:26

Brakuje CI kolego czegoś takiego :)

public String toString(){
    return "Artifact [number=" + number + ", culture=" + culture +", century=" + century+"]";
}

Naygel

Poziom 4 , Poland, Poland

1 września 2022, 20:37

Da się coś z tym zrobić żeby to lepiej wyglądało? Bo w pierwszym przypadku nie ma culture i century, a w drugim century, więc wyświetla się null lub 0.

Naygel

Poziom 4 , Poland, Poland

1 września 2022, 20:49

Poprawiłem ostatnie 3 wersy nie wpisując twojego rozwiązania, jest jakaś duża różnica między tym co mam a tym co napisałeś? :)

System.out.println("The first type: " + artifact1.number);
        System.out.println("The second type: " + artifact2.number + " " + artifact2.culture);
        System.out.println("The third type: " + artifact3.number + " " + artifact3.culture + " " + artifact3.century);

Karol Grzeszczak

Poziom 31 , Poland, Poland

1 września 2022, 20:54

możesz próbować tak jak Ty to robisz ale ogólnie to wina tego, że używasz 3 konstruktorów z jednym dwoma i trzema argumentami bazowa metoda to string nie wie za bardzo jaki obiekt będziesz chciał przedstawić dlatego wyświetla się jak wyświetla i dlatego też musisz ją nadpisać wartości będą null i zero dlatego, że masz wiele konstruktorów a metoda to string jest dla całej klasy Dla prostych przykładów różnicy w sumie nie ma ale gdybyś chciał wyświetlać dziesiątki tysięcy obiektów najlepsza jest rozsądna metoda toString(); wyobraź sobie, że masz 100 tych artefaktów albo nawet 1000 i musisz się tak bawić

Karol Grzeszczak

Poziom 31 , Poland, Poland

1 września 2022, 21:52

ale dla takiej sztuczki jak Ty chcesz możesz sobie coś takiego wstawić

public String toString() {
        if (culture == null && century == 0) {
            return "Artifact [number=" + number + "]";
        } else if (century == 0) {
            return "Artifact [number=" + number + ", culture=" + culture + "]";
        } else return "Artifact [number=" + number + ", culture=" + culture +", century=" + century+"]";
}