Eigene Klassen schreiben

[freshman]

Hallo,

ich habe ein kleines Problem und hoffe auf hilfreiche Tips. Hier mein vereinfachter Code:

class Test {
    public static void main(String[] args){
        Spiel [] liste = new Spiel[4];        
        liste [0] = new Spiel("Nintendo","Super Mario",1983);
        liste [1] = new Spiel("","Pacman",1980);
        liste [2] = new Spiel("Sega","Sonic",1990);
        liste [3] = new Spiel("Sony","Gran Turismo",1997);
        print(liste);
    }
    
    static void print (Spiel [] liste){
       for (int i = 0; i < liste.length; i++)
           System.out.println(liste[i].firma + " " + liste[i].name + " " + liste[i].jahr); 
    }
}

class Spiel{
    String name,temp;
    int jahr;
    Hersteller firma;
  
    Spiel(String temp, String name, int jahr){
        if (!temp.equals("")){
             firma = new Hersteller(temp); 
        }
        else  
            this.temp = "<kein Hersteller>";
        this.name = name;
        this.jahr = jahr;
    }
}

class Hersteller{
    String temp;
    
    Hersteller(String temp){
        this.temp = temp;         
    }
}

Die Ausgabe:

Hersteller@6782a9 Super Mario 1983
null Pacman 1980
Hersteller@1b34126 Sonic 1990
Hersteller@17a4989 Gran Turismo 1997

Wie man sieht bekomme ich bei der ersten Variablen nur die Adressen und will natürlich den Inhalt der Felder. Was mache ich falsch?

mfg freshman

 

[marlboro85]

in deiner ausgabe rufst du auf:

liste[i].firma

Das gibt dir aber n Hersteller Objekt zurück, nicht das was in der Klasse Hersteller als temp drin steht.

Mach da mal

liste[i].firma.temp

dann stimmt das schonmal.

In der Spiel musst die bei der IF Abfrage beim else noch

firma = new Hersteller("<kein Hersteller>");

machen.

Wenn du da nur temp = ... zuweisst, kriegste bei liste[1] ne NullPointerException, da ja dort kein
Hersteller angelegt wurde

 

[Marco13]

Da "temp" vermutlich der Name sein soll, wäre 'name' vielleicht ein besserer name als 'temp'. Abgesehen davon: Fields sollten immer private sein! Da Hersteller sich selten umbenennen, braucht man da auch nur eine get-Methode.

class Hersteller
{
    private String name;
   
    Hersteller(String name)
    {
        this.name = name;         
    }
    public String getName()
    {
        return name;
    }
}

Dann kann man den Namen mit
System.out.println(hersteller.getName());
ausgeben. "Besser" wäre es, die toString-Methode zu überschreiben, die automatisch aufgerufen wird, wenn man ein objekt mit System.out.println ausgibt:

class Hersteller
{
    ....
    public String toString()
    { 
        return name;
    }
}

Hersteller hersteller = new Hersteller("Bla");
System.out.println(hersteller); // Gibt "Bla" aus

 

[freshman]

habs verstanden

Vielen Dank!!!