Polymorphie - OO Problem

[LennyB]

Hallo mal wieder!
Weil mein Sourcecode zu umfangreich und kompliziert wäre, habe ich das Problem das ich habe in einem Stück Beispielcode dargestellt. Fall Syntaxfehler drin sind, hat das damit zu tun, daß dieser Code nie einen Compiler gesehen hat und einfach nur in Notepad getippt wurde. Ich hoffe er ist dennoch einigermassen fehlerfrei. Soweit zum Disclaimer

Ok, hier erstmal das Szenario:

// --- DATA CLASSES ---
class Data
{
  public Data(int dtype)
  {
    type = dtype;  
  }

  public int getType()
  {
    return type;
  }

  private int type;


  //some data
}


class AData extends Data
{
  public AData()
  {
    super(0);
  }

  //some additional data
}


class BData extends Data
{
  public BData()
  {
    super(1);
  }

  //some additional data
}

// --- PANELS ---
class PanelBase
{
  public setData(Data d)
  {
    // doing something with the data
  } 
}


class APanel
{
  public setData(AData d)
  {
    super.setData(d); 
    // doing something with the data
  } 
}

class BPanel
{
  public setData(BData d)
  {
    super.setData(d); 
    // doing something with the data
  } 
}

// Programmfragment

public someclass
{
  public void doStuff(Data d)
  {
    switch (d.getType())
    {
      case 0:
        panel = new APanel();      
      break;
      case 1:
        panel = new BPanel();      
    }

    // >>> HIER IST DAS PROBLEM !!! <<<
    panel.setData(d);
  }


  private PanelBase panel;
}

Ok, das sind also zwei Hierarchien:
1. Daten
2. Panels in denen diese Daten dargestellt werden sollen

und dann ist da noch die Klasse "SomeClass", die mit diesen arbeitet. Das Problem das ich habe ist, daß in meiner realen Anwendung, egal welchen Wert d.getType() in SomeClass.doStuff() liefert also auch egal welche Panelklasse erzeugt wird, IMMER nur PanelBase.setData(d) aufgerufen wird. Eigentlich möchte ich, daß entweder APanel.setData(d) oder BPanel.setData(d) aufgerufen wird.

Kann mir jemand erklären wo mein Denkfehler liegt und wie man das beheben könnte.

Danke
Lenny

 

[Landei]

Kein Denkfehler, das geht nicht ganz so einfach. Willkommen in der wunderbaren Welt der Generics!

Ebenso ungetestet:

// --- DATA CLASSES ---
class Data
{
  public Data(int dtype)
  {
    type = dtype;  
  }
 
  public int getType()
  {
    return type;
  }
 
  private int type;
 
 
  //some data
}
 
 
class AData extends Data
{
  public AData()
  {
    super(0);
  }
 
  //some additional data
}
 
 
class BData extends Data
{
  public BData()
  {
    super(1);
  }
 
  //some additional data
}
 
// --- PANELS ---
class PanelBase<D extends Data>
{
  public setData(D d)
  {
    // doing something with the data
  } 
}
 
 
class APanel extends PanelBase<AData>
{
  public setData(AData d)
  {
    super.setData(d); 
    // doing something with the data
  } 
}
 
class BPanel extends PanelBase<BData>
{
  public setData(BData d)
  {
    super.setData(d); 
    // doing something with the data
  } 
}
 
// Programmfragment
 
public someclass
{
  public void doStuff(Data d)
  {
    switch (d.getType())
    {
      case 0:
        APanel aPanel = new APanel();     
        aPanel.setData((AData) d); 
        panel = aPanel; 
      break;
      case 1:
        BPanel bPanel = new BPanel();      
        bPanel.setData((BData) d);
        panel = bPanel;
    }
 
    // >>> HIER IST DAS PROBLEM !!! <<<
    //hier geht es immer noch nicht, denn der Typen Data von d und PanelBase von panel sind "zu allgemein" 
    //panel.setData(d); 
  }
 
 
  private PanelBase<?> panel;
}

 

[LennyB]

Hi Landei, erstmal danke soweit. Generics ist wenn ich das richtig sehe sowas ähnliches wie Templates in C++... oder habe ich da jetzt was total falsch verstanden? Ich habe den bösen Verdacht, daß sich Dein Beispiel in meinem wirklichen Code nicht so einfach umsetzen lässt wie in diesem kleinen Beispiel und zwar weil:

//hier geht es immer noch nicht, denn der Typen Data von d und PanelBase von panel sind "zu allgemein"

Der Typ muss leider so allgemein sein. Ich werde es trotzdem mal versuchen

Bin für weitere Ansätze offen

 

[ThreadPool]

Fall Syntaxfehler drin sind, hat das damit zu tun, daß dieser Code nie einen Compiler gesehen hat und einfach nur in Notepad getippt wurde. Ich hoffe er ist dennoch einigermassen fehlerfrei.

So so und du lädst hier einfach einfach deinen kaputten Quellcode ab und hoffst das wir noch zusätzlich Syntaxfehler fixen?

Nebenbei, ich finde keine Polymorphie in deinem Programm, falls APanel und BPanel noch von Panelbase erben sollen sehe ich da nur Funktionsüberladung.

 

[Empire Phoenix]

du weißt das es möglich ist sachen wie <? extends data> anzugeben oder so ähnlich?

alternativ könnte man auch beim panel erstellen gleich
anhand des typs von data das machen if(data.getClass() == BData.class)

zudem finde ich es sinnvoll data abstract zu machen, da du ja sagst das das nicht ausreicht, und dementsprechen wohl auch nie alleine vorkommt

 

[Schumi]

So so und du lädst hier einfach einfach deinen kaputten Quellcode ab und hoffst das wir noch zusätzlich Syntaxfehler fixen?

Er hatte doch extra geschrieben, dass das nur Beispielcode ist um das Problem zu verdeutlichen und man Syntaxfehler getrost ignorieren kann. Um irgendwelche Korrekturen oder ähnliches wurde doch gar nicht gebeten.

ETA: kannst Du nicht setData im BasePanel abstrakt machen, die Logic dort in eine protected setBaseData auslagern. Dann implementierst Du setData in den jeweiligen Panelklassen und rufst dort jeweils die setBaseData auf?

 

[LennyB]

Also zu ThreadPool sage ich mal lieber garnichts... ausser: Wer lesen kann ist im Vorteil

Schumi: DANKE! Das wird funktionieren! Ich hatte gestern schon ein wenig zuviel Codematsch im Hirn, sonst wäre ich vielleicht selbst drauf gekommen. Aber da bin ich mir auch nicht so sicher. Danke, danke, danke! Jetzt kann ich endlich die krüppelige Notlösung rauskicken