Problem mit Subtyping

[Guest]

Hallo,

folgendes Verständnisproblem habe ich:
Gegeben ist der stark vereinfachte Code unten. Wenn ich die Methode do ausgliedere in eine eigene Klasse + eigenes Paket dann wird diese so ziemlich komplett ignoriert. Es wird zwar in die Methode gesprungen aber das Ergebnis nicht weiter verarbeitet.

Wenn ich die Methode so wie sie ist, in der Klasse MainTest ausführen lasse, dann klappt alles prima.

Was mache ich gedanklich falsch bzw was fehlt?

package MainP

public class MainTest {
         ............
	public void mouseClicked(MouseEvent e) {
		Test test = new Test();
		test.do(i);
		...	
	}
}

package TestP
public class Test extends MainTest {
           ............
	public void do(int i) {
		
          // Anweisung ausfuehren

	}
}

Liebe Grüße

 

[Marco13]

Gegeben ist folgender stark vereinfachter Code

class Test {
    public static void main(Sring args[]) {
        ...
    }
}

Beim Ausführen verhält sich das Progamm nicht wie gewünscht. Woran kann das liegen?

Mal im ernst: Vermutlich liegt es daran, dass bei jedem Mausklick eine neue Instanz erstellt wird. Man kann durch "Subtyping" (d.h. durch Vererbung) NICHT ein bestehendes Objekt um neue Funktionen erweitern. Man kann nur eine Klasse erweitern, und dann ein Objekt dieses (erweiterten) Typs anlegen.

In deinem Beispiel

package MainP

public class MainTest {

    String einString = "Hallo"; //--------------- Soll geändert werden
         ............
   public void mouseClicked(MouseEvent e) {
      Test test = new Test();
      test.do(i);
      ...   
   }
}

package TestP
public class Test extends MainTest {
           ............
   public void do(int i) {

         einString = "Welt"; //-------------------------- Bewirkt KEINE Änderung an obigem String

   }
}

Das liegt daran, dass das Objekt, in dem "do" ausgeführt wird, ein anderes Objekt ist, als das, in dem der Aufruf "mouseClicked" ausgeführt wird. Du musst also ggf. von Vornherein ein Objekt vom Type Test erstellen, und nur damit arbeiten. Ansonsten folgt hier der obligatorische Verweis auf die üblichen Grundlagen-Bücher.

 

[Guest]

Ok, Danke erstmal..

Wie kann ich denn auf dein Beispiel bezogen den String in der do - Methode ändern, und in der Klasse MainTest verändert ausgeben? (wenn es denn möglich ist)

So z.B.

package MainP

public class MainTest {

    String [] einString ; 
         ............
   public void mouseClicked(MouseEvent e) {
      Test test = new Test();
      test.do(i);
      ...   
   }
   public void ausgabe(){
      System.out.println(einstring);
  }
}

package TestP
public class Test extends MainTest {
           ............
   public void do(int i) {

         einString = "Welt"; //-------------------------- Soll eine Änderung bewirken, und dieses oben ausgeben

   }
}

 

[Leroy42]

package MainP

public class MainTest {

    String [] einString ; 
         ............
   public void mouseClicked(MouseEvent e) {
      Test test = new Test();
      einString = test.do(i);
      ...   
   }
   public void ausgabe(){
      System.out.println(einstring);
  }
}

package TestP
public class Test extends MainTest {
           ............
   public String do(int i) {
         return "Welt";
   }
}

 

[Guest]

Ok, Danke... Aber komme auf ein Beispiel bezogen nicht weiter. Wie schaffe ich das in diesem Fall?

package MainP

public class MainTest {

    Array [] counter ;
         ............
   public void mouseClicked(MouseEvent e) {
     int i = 5;
     Test test = new Test();
      test.do(i); 
      ...   
   }
   public void ausgabe(){
      System.out.println(i);
  }
}

package TestP
public class Test extends MainTest {
           ............
   public do(int i) {
         counter[i] = 10;
   }
}

Wie müsste ich das zurückgeben?

 

[Marco13]

Du solltest bei einem Mausklick überhaupt keine neue Instanz erstellen. Wie die "beste" Lösung ist, läßt sich ohne Code und geneuere Erklärung, worum es geht, noch schwerer sagen als mit. Aber so wie es jetzt ist, ist es mit ziemlicher Sicherheit NICHT optimal.

 

[Guest]

Das ist schon sinngemäß mehr oder weniger der Code. In der Klasse MainTest will ich meine Fenster darstellen und alles was mit Grafik zu tun hat. Dafür brauche ich auch das Array counter[]. In der Klasse Test möchte ich gerne nun alles implementieren, was mit dem array "angestellt wird". Hochzählen, runterzählen, Wert löschen etc, damit ich nicht eine Klasse so aufblähe. Da ich auch andere Variablen benötige aus der Klasse MainTest dachte ich setze per Subtyping auf diese Klasse auf. Mir fällt halt keine andere Möglichkeit ein.

 

[Marco13]

Naja. Das Ziel "eine Klasse nicht so aufzublähen" ist zwar OK, aber das gepostete nicht der geeignete Weg, das zu erreichen. Was IST denn dein Array? Überleg' dir, was dein Array IST, und was du damit machen willst. Und beschreib' das dann in einer Klasse. Die Klasse repräsentiert dann das Datenmodell, und bietet alle Methoden an, die benötigt werden, um die gewünschten Operationen durchzuführen. Vom Fenster aus wird diese Klasse dann nur "ferngesteuert" - durch Mausklicks usw. Da du anscheinenen nicht konkreter werden WILLST (oder so) mal ein Beispiel

Sowas wohl eher nicht

class Hauptklasse extends JFrame implements IrgendwelcheListener
{
    Counter counter[] = ... 
    
    void mousePressed() {  for (alle counter) counter[i] = 0; } // Alle löschen
    void button1Clicked() {  for (alle counter) counter[i] *= 2; } // Alle mit 2 multiplizieren
    void button2Clicked() {  for (alle counter) counter[i] /= 10; } // Alle durch 10 teilen
}

Stattdessen überlegt man sich, WAS durch diesen Array repräsentiert wird, und legt dafür eine Klasse an, die eine (möglichst kleine) Schnittstelle nach draußen hat

class CounterArray
{
    private Counter counter[];

    public void clearAll() { for (alle counter) counter[i] = 0; }    
    public void multiplyAll(float factor) { for (alle counter) counter[i] = (int)(counter[i]*factor); }        
}

Und die Methoden werden dann von der Hauptklasse aufgerufen

class Hauptklasse extends JFrame implements IrgendwelcheListener
{
    CounterArray counterArray ...
    
    void mousePressed() {  counterArray.clear(); } // Alle löschen
    void button1Clicked() {  counterArray.multiplyAll(2); } // Alle mit 2 multiplizieren
    void button2Clicked() {  counterArray.multiplyAll(0.1f); } // Alle durch 10 teilen
}

Aber je nachdem, was du vorhast, könnte für dich auch ein anderer Ansatz besser sein ... wie man das am besten strukturiert, ist die Frage, für deren Beantwortung Sofware-Ingenieure so viel Kohle bekommen... :wink: