String Inhalt wird überschrieben (Subklassen verwenden denselben String)

[ms_green]

Hallo Javabegeisterte!

Ich hoffe, ich hab hier ins richtige Unterforum gepostet...wenn nicht, wars keine Absicht....

Nun zu meinem Problem:

Ich habe einen JUnit Test und muss den dazugehörigen Javacode schreiben, sodass der Test erfolgreich durchläuft.

Der Test sieht so aus:

import static org.junit.Assert.assertEquals;
import org.junit.Test;

public class PetsTest {

    @Test
    public void testPets() {
        Pet pet1 = new Cat("Garfield");
        Pet pet2 = new Dog("Goofy");
        Pet pet3 = new Bird();
        pet3.setName("Tweety");
        assertEquals("Garfield", pet1.getName());
        assertEquals("Goofy", pet2.getName());
        assertEquals("Tweety", pet3.getName());
        assertEquals("Miaow", pet1.makeSound());
        assertEquals("Woof", pet2.makeSound());
        assertEquals("Chirp", pet3.makeSound());
    }
}

Die Superklasse Pet sieht so aus:

class Pet {
    
    String name = "not decided yet";
    String sound;

    
    public String getName() {
        
        return name;
    }

    public void setName(String name) {
        
        this.name = name;
    }

    public String makeSound() {
        
        return sound;
    }

    
    
    
}

Ich habe jetzt für jedes Tier eine Klasse erstellt in der mit

extends

angegeben wird, dass sie Subklassen sind.

Meine Dog Klasse sieht z.B. so aus

public class Dog extends Pet{

    String sound = "Woof";
 
    Dog(String name) {
        System.out.println("I´m a dog, my name is"+" "+name);
    }
    
}

Nun hab ich das Problem, dass der Test bei der Zeile

assertEquals("Garfield", pet1.getName());

einen Fehler ausgibt und zwar

Testcase: testPets(pets.PetsTest):	FAILED
expected:<[Garfield]> but was:<[not decided yet]>
junit.framework.AssertionFailedError: expected:<[Garfield]> but was:<[not decided yet]>

So wie ich das verstehe, wird der vorherige Name immer mit dem nächsten überschrieben, also ist der Name von pet1 nicht mehr Garfield, sondern der des pet3....???:L

Kann mich bitte jemand auf den richtigen Weg führen?

 

[qowaz]

Du musst in den Unterklassen auch den Namen setzen, z.B. im Konstruktor der Klasse Dog:

public Dog(String name){
    this.name = name;
}

 

[Camill]

Außerdem musst du auch noch einen Parameterlosen Konstruktor anlegen (für den JUnit Test nur für die Klasse "Bird()" nötig).

 

[ms_green]

Dankesehr! Funktioniert prima.:applaus:


Hm...
Leider hab ich jetzt in Zeile 17 ein Problem...

junit.framework.AssertionFailedError: expected:<Miaow> but was:<null>

Wieso hat sound keinen Inhalt, wenn es doch in dog einen zugewiesen bekommt?

Ich habs jetzt auch mit

  public String makeSound(String sound) {
        
        return this.sound;
    }

versucht, aber ohne Erfolg...der Compiler sagt mir nur, dass er einen String erwartet und stattdessen "no arguments" gefunden hat...


Ich hab das alles noch nicht ganz durchblickt...

 

[Camill]

Leider hab ich jetzt in Zeile 17 ein Problem...

junit.framework.AssertionFailedError: expected:<Miaow> but was:<null>

Meinst du vielleicht Zeile 15? Auf Zeile 17 trifft die Meldung nicht zu.

 

[ms_green]

Meinst du vielleicht Zeile 15? Auf Zeile 17 trifft die Meldung nicht zu.

Ja, natürlich Zeile 15....da hab ich mich wohl vertippt...

 

[Camill]

Um das Problem zu lösen könntest du in der Klasse "Pet" eine Methode schreiben mit der du den Sound setzt, diese wird dann jeweils von den 3 abgeleiteten Klassen(Dog, Cat, Bird) im Konstruktor mit dem richtigen String als Parameter aufgerufen.

 

[ms_green]

Ich hab gedacht, das passiert mit diesem Code:

   public String makeSound() {
        
         return sound;
    }

Geplant war das so: wenn z.B.: pet1.makeSound() ausgeführt wird, nimmt sound den Wert an, der bei pet1 definiert wurde...also würde die makeSound Methode nach meiner Logik, den richtigen Wert liefern...tatsächlich liefert sie aber gar keinen....???:L


EDIT: Wäre es evtl. ratsam, aus Pets abstract zu machen?

 

[Murray]

In den abgeleiteten Klassen deklarierst eine eigene Variable sound - die hat nichts mit der aus der Basisklasse zu tun.

 

[ms_green]

In den abgeleiteten Klassen deklarierst eine eigene Variable sound - die hat nichts mit der aus der Basisklasse zu tun.

...aber das hab ich doch gemacht?

public class Dog extends Pet{
 
    String sound = "Woof";   //das meinst du doch, oder?
 
    Dog(String name) {
        System.out.println("I´m a dog, my name is"+" "+name);
    }
    
}

;(

 

[qowaz]

Dir fehlt dort aber noch die Methode makeSound()

 

[Camill]

...aber das hab ich doch gemacht?

public class Dog extends Pet{
 
    String sound = "Woof";   //das meinst du doch, oder?
 
    Dog(String name) {
        System.out.println("I´m a dog, my name is"+" "+name);
    }
    
}

;(

Wahrscheinlich hast du "Murray" falsch verstanden, er wollte durch seinen post sagen das die Variablendeklaration

String sound = "Woof";

in der Klasse "Dog" NUR innerhalb dieser Klasse gültig ist - Nicht in der "Pet" Klasse.

 

[ms_green]

Dir fehlt dort aber noch die Methode makeSound()

Ah.....!
Jetzt versteh ich....

public class Dog extends Pet{

    String sound = "Woof";
 
    Dog(String name) {
        this.name = name;
        System.out.println("I´m a dog, my name is"+" "+name);
    }
 
    
    @Override
    public String makeSound() {
         
         return sound;
    }
}

Ich hab das jetzt so gelöst, dass ich eine neue makeSound methode bei jeder Tierklasse geschrieben habe, die jene aus der Pet Klasse überschreibt...uuuuuund: jetzt ist der Testbalken endlich grün

Ich glaub, das ist eh das, was ihr mir sagen wolltet, nur bin ich ziemlich auf der Leitung gestanden^^

Danke euch allen, ohne euch wäre das vermutlich noch ein etwas längerer Abend geworden...

 

[Camill]

Mein Lösungsweg war ein anderer aber durch das überschreiben der "makeSound" Methode geht das ganze natürlich auch.