Ist hier dynamic_cast ok?

[caster2]

brotbernd schrieb:

D3D9RenderSystem kann nicht jedes RenderWindow behandeln, daher ist diese Abstraktion an dieser Stelle einfach sinnlos.

Stimmt, aber da es nur D3D9RenderWindows kriegen kann, sehe ich damit kein Problem. Vor allem faellt mir nichts eleganteres ein.

@Dot: Ich weiss. Das abstrahieren von Render APIs ist immer ein Schmerz im Hintern und OpenGL ganz besonders. Ich wage zu behaupten, dass ein 100% elegantes Wrappen beider APIs ohne kleinere Hacks unmoeglich ist. Dazu sind sie teilweise einfach zu unterschiedlich. Ich sag nur Contexts vs Swap Chains, Lost Devices etc.

[brotbernd]

caster2 schrieb:

Stimmt, aber da es nur D3D9RenderWindows kriegen kann, sehe ich damit kein Problem. Vor allem faellt mir nichts eleganteres ein.

Und was war mit diesem Vorschlag

class RenderSystem
{
public:
    virtual void MachWas() const = 0;
};

class RenderTarget
{
public:
    void TuWas()
    {
        GetRenderSystem()->MachWas();
        Schwipp();
    }
private:
    virtual void Schwipp() = 0;
    virtual const RenderSystem* GetRenderSystem() const = 0;
};

class D3RenderWindow;
class D3RenderSystem : public RenderSystem
{
public:
    void MachWas() const { std::cout << "D3RenderSystem macht was\n"; }
    void Schwapp(D3RenderWindow& window){}

};

class D3RenderWindow : public RenderTarget
{
public:
    D3RenderSystem mRenderSystem;
    void Schwipp()
    {
        mRenderSystem.Schwapp(*this);
    }
    const RenderSystem* GetRenderSystem() const
    {
        return &mRenderSystem;
    }
};

[dot]

caster2 schrieb:

Ich wage zu behaupten, dass ein 100% elegantes Wrappen beider APIs ohne kleinere Hacks unmoeglich ist.

Ja definitiv.

[caster2]

dot schrieb:

caster2 schrieb:

Ich wage zu behaupten, dass ein 100% elegantes Wrappen beider APIs ohne kleinere Hacks unmoeglich ist.

Ja definitiv.

Im Grunde gehts mir nur darum die Anzahl Hacks zu minimieren. Ganz vermeiden ist unmoeglich.

@brotbernd: Danke! Hab leider im Moment keinen Zugriff auf meinen echten Code und kann deshalb gerade nicht sagen, ob das an irgend einer Stelle Probleme geben koennte. Aber sieht sehr interessant aus!

[dot]

caster2 schrieb:

Ganz vermeiden ist unmoeglich.

Man könnte es vermeiden sinnlose Wrapper zu schreiben, dann braucht man auch keine Hacks und spart unnötigen Overhead

[caster2]

dot schrieb:

caster2 schrieb:

Ganz vermeiden ist unmoeglich.

Man könnte es vermeiden sinnlose Wrapper zu schreiben, dann braucht man auch keine Hacks und spart unnötigen Overhead

Geht nicht. Die Engine soll mit beiden APIs laufen und das erfordert nun mal Abstraktion. Klappt ja bis jetzt auch. Nicht immer schoen, aber es klappt. Schonheitspreise fuer Code gibts eh nicht. Entscheidend ist nur die Qualitaet des Spiels, das man mit dem Code macht.

[dot]

caster2 schrieb:

Die Engine soll mit beiden APIs laufen und das erfordert nun mal Abstraktion.

Ich seh grad nicht wo das Problem liegt, implementier den Renderer der Engine eben einmal mit D3D und einmal mit OpenGL!?

[caster2]

dot schrieb:

caster2 schrieb:

Die Engine soll mit beiden APIs laufen und das erfordert nun mal Abstraktion.

Ich seh grad nicht wo das Problem liegt, implementier den Renderer der Engine eben einmal mit D3D und einmal mit OpenGL!?

Wir drehen uns im Kreis. Lass gut sein.

[Tachyon]

caster2 schrieb:

dot schrieb:

caster2 schrieb:

Die Engine soll mit beiden APIs laufen und das erfordert nun mal Abstraktion.

Ich seh grad nicht wo das Problem liegt, implementier den Renderer der Engine eben einmal mit D3D und einmal mit OpenGL!?

Wir drehen uns im Kreis. Lass gut sein.

Er hat schon recht. Baue jeden Renderer für sich auf, und packe dann Fassaden vor jeden Rendertypen, welche das zusammenfassen, was die unterschiedlichen Renderer wirklich gemein haben.

[caster2]

Und genau diese Fassade ist RenderSystem.

[dot]

Wie gesagt: Dann ist der Abstraktionsgrad dieser Fassade zu niedrig...

[Tachyon]

caster2 schrieb:

Und genau diese Fassade ist RenderSystem.

RenderSystem * rs = RenderSystem3d(...);

//...

RenderWindow rw = AnyAvailableRenderWindowCanBeHere();
rs->method(rw); //wie wird sicher gestellt, dass es ein RenderWindow3D ist bzw. woher weißt Du, dass rs auf ein RenderSystem3d zeigt?

[caster2]

Tachyon schrieb:

RenderWindow rw = AnyAvailableRenderWindowCanBeHere();
rs->method(rw); //wie wird sicher gestellt, dass es ein RenderWindow3D ist bzw. woher weißt Du, dass rs auf ein RenderSystem3d zeigt?

[/cpp]

Garnicht, denn sowas wie AnyAvailableRenderWindowCanBeHere(); habe ich nicht. Wenn das RenderSystem D3D9 ist, gibt es nur D3D9 Render Windows. Ich baue mit Sicherheit nicht einen Wrapper um eine API und darueber dann nochmal einen Wrapper. Halte ich fuer unglaublich haesslich und habe ich bis jetzt auch in noch keiner einzigen Engine gesehen.

[Tachyon]

caster2 schrieb:

Tachyon schrieb:

RenderWindow rw = AnyAvailableRenderWindowCanBeHere();
rs->method(rw); //wie wird sicher gestellt, dass es ein RenderWindow3D ist bzw. woher weißt Du, dass rs auf ein RenderSystem3d zeigt?

[/cpp]

Garnicht, denn sowas wie AnyAvailableRenderWindowCanBeHere(); habe ich nicht. Wenn das RenderSystem D3D9 ist, gibt es nur D3D9 Render Windows. Ich baue mit Sicherheit nicht einen Wrapper um eine API und darueber dann nochmal einen Wrapper. Halte ich fuer unglaublich haesslich und habe ich bis jetzt auch in noch keiner einzigen Engine gesehen.

Dann leuchtet mir mal gar nicht ein, warum RenderSystem eine virtuelle Methode haben muss, welche RenderWindow -Basisklassen bekommt. RenderWindow-Basisklassenzeiger zu übergeben, wenn eigentlich konkrete RenderWindow-Instanzen erwartet werden halte ich fuer unglaublich haesslich und habe ich bis jetzt auch in noch keiner einzigen Engine gesehen. :p
Btw. hast Du gefragt, ob dynamic_cast okay ist. So ziemlich alle meinten nein. Eine Antwort hast Du. Wenn Du es trotzdem so machst, dann war die Frage eigentlich überflüssig.

Da die mögliche Gesamtmenge an Rendersystemen vermutlich begrenzt sein dürfte, werfe ich jetzt zur Behandlung Deiner konkreteten RenderWindows einfach mal das Visitor-Pattern in den Raum.

[caster2]

Tachyon schrieb:

caster2 schrieb:

Tachyon schrieb:

RenderWindow rw = AnyAvailableRenderWindowCanBeHere();
rs->method(rw); //wie wird sicher gestellt, dass es ein RenderWindow3D ist bzw. woher weißt Du, dass rs auf ein RenderSystem3d zeigt?

[/cpp]

Garnicht, denn sowas wie AnyAvailableRenderWindowCanBeHere(); habe ich nicht. Wenn das RenderSystem D3D9 ist, gibt es nur D3D9 Render Windows. Ich baue mit Sicherheit nicht einen Wrapper um eine API und darueber dann nochmal einen Wrapper. Halte ich fuer unglaublich haesslich und habe ich bis jetzt auch in noch keiner einzigen Engine gesehen.

Dann leuchtet mir mal gar nicht ein, warum RenderSystem eine virtuelle Methode haben muss, welche RenderWindow -Basisklassen bekommt. RenderWindow-Basisklassenzeiger zu übergeben, wenn eigentlich konkrete RenderWindow-Instanzen erwartet werden halte ich fuer unglaublich haesslich und habe ich bis jetzt auch in noch keiner einzigen Engine gesehen. :p
Btw. hast Du gefragt, ob dynamic_cast okay ist. So ziemlich alle meinten nein. Eine Antwort hast Du. Wenn Du es trotzdem so machst, dann war die Frage eigentlich überflüssig.

Da die mögliche Gesamtmenge an Rendersystemen vermutlich begrenzt sein dürfte, werfe ich jetzt zur Behandlung Deiner konkreteten RenderWindows einfach mal das Visitor-Pattern in den Raum.

Na 3mal darfst du raten wieso ich diesen Thread erstellt hab;)

[Tachyon]

caster2 schrieb:

Na 3mal darfst du raten wieso ich diesen Thread erstellt hab;)

Trollversuch?

[caster2]

Tachyon schrieb:

caster2 schrieb:

Na 3mal darfst du raten wieso ich diesen Thread erstellt hab;)

Trollversuch?

Klar. War ja so unglaublich provozierend oder polemisch die Frage, ne? Deppen gibts hier...
Immerhin konnte brotbernd und volkard auch was SINNVOLLES beitragen.

[Tachyon]

caster2 schrieb:

Tachyon schrieb:

caster2 schrieb:

Na 3mal darfst du raten wieso ich diesen Thread erstellt hab;)

Trollversuch?

Klar. War ja so unglaublich provozierend oder polemisch die Frage, ne? Deppen gibts hier...
Immerhin konnte brotbernd und volkard auch was SINNVOLLES beitragen.

Der Hinweis auf das Visitor-Pattern ist in Deinem Kontext sinnvoll.

[caster2]

Tachyon schrieb:

caster2 schrieb:

Tachyon schrieb:

caster2 schrieb:

Na 3mal darfst du raten wieso ich diesen Thread erstellt hab;)

Trollversuch?

Klar. War ja so unglaublich provozierend oder polemisch die Frage, ne? Deppen gibts hier...
Immerhin konnte brotbernd und volkard auch was SINNVOLLES beitragen.

Der Hinweis auf das Visitor-Pattern ist in Deinem Kontext sinnvoll.

Das mit dem "Trollversuch" hingegen wenig. Das ist eine ziemliche Frechheit.

[brotbernd]

Visitor ist hier im Prinzip das gleiche Problem wie dynamic_cast. Ich muss keinen konkreten Typen suchen, wenn ich schon vorher weiß welcher es ist. Also cast weg, visitor weg, Abstraktion weg und einfach von vornherein den konkreten Typen benutzen. Wenn jemand nur von einer abstrakten Teilschnittstelle abhängen soll, wie die RenderTarget Basisklasse, dann bekommt diese eine RenderSystem Basisklassenreferenz von der Klasse, die den konkreten Typen kennt; nicht andersherum.
Also, das konkrete RenderTarget kennt sein konkretes RenderSystem. Andere Klassen, die nur von der allgemeinen RenderSystem Schnittstelle abhängen, bekommen nur diese zu sehen.