Mal wieder eine Frage von einem unwissenden…
Bei dem skin flatPlus ist mir im Quellcode aufgefallen, dass CreatePixmap dreim al vorhanden ist und mit dem selben Code gefüllt ist. In baserender .h unter public: und die anderen beiden .h unter private:
Ich frage mich nun nach dem Sinn des ganzen. Wäre es nicht einfacher das nur in einer .h zu haben?
Wer kennt sich da aus. Wie kompliziert wäre das?
Sind aber doch drei verschiedene Rückgabewerte, oder?
Ahhh sorry!
Das waren die Klassennamen, nicht die Rückgabetypen 🙈
Wenn die Implementierungen wirklich exakt gleich sind, müsste man das in eine eigene Klasse auslagern können und die drei bisherigen Aufrufstellen rufen dann alle die neue Methode.
Viel Erfolg!
Ich sehe keinen dringenden Grund, das zu machen. M.E. würde es das ganze komplizieren und ist den Aufwand nicht wert.
Wenn sich die Klassen cComplexContent, cFlatBaseRender und cTextScroll von einer gemeinsamen Klasse ableiten würden, könntest du das einfach da einbauen, aber das sind eigenständige Klassen.
Ansonsten könntest du eine neue Klasse für die pixmap erstellen, darin die Methode ::CreatePixmap einbauen und von den anderen aufrufen lassen.
Die Notwendigkeit darin sehe ich aber nicht. Ganz sauber sind doppelte Codeabschnitte aber natürlich auch nicht.
Gruß
Andreas
Ist nur blöd bei Änderungen; die muss man dann drei mal machen
Ich bin da auch immer bestrebt, redundanten Code zu beseitigen. In Deinem Fall bedeutet das aber, man muss sich richtig gut überlegen, wie man das macht. Durch die Vererbung ist das nicht ganz trivial.
Möglich wäre z.B. so eine Lösung, wie bei skinnopacity. Da ist diese Funktion in eine helpers Datei ausgelagert. Das bedeutet aber auch, man muss diese Funktion etwas anpassen und auch den Aufruf in allen Dateien wo sie vorkommt, und das sind fast alle.
So wie ich das gesehen habe, gibt es in skinflatplus auch noch mehr redundanten Code. Wenn man das angehen will, sollte man sich insgesamt mal überlegen, ob da grundsätzlich eine Überarbeitung sinnvoll wäre.
Grüße
kamel5
Ich dachte eigentlich dass die baserender.c mehr oider weniger der helpers in skinnopacity entspricht.
Das kann man nicht 1:1 vergleichen, ist aber auch nicht weiter schlimm.
Ich würde auch gerne eine tools.[ch] erstellen um da einige Sachen auszulagern.
Im Prinzip kannst Du dafür auch die flat.[c,h] benutzen. da gibt es bereits am Ende "zentrale" Funktionen und sie ist schon überall inkludiert.
Aber mit dem ganzen C-Zeug stehe ich irgendwie auf Kriegsfuß
Aller Anfang ist schwer. 
Ich habe da auch vor einiger Zeit bei 0 angefangen.
Ich würde die Funktion CreatePixmap ohne Klasse erweitert um den Parameter "osd" in der flat.[c,h] Datei am Ende unterbringen. Den Parameter Name brauchst Du nicht, den habe ich nur fürs Debugging drin.
Dann würde ich die Funktion CreatePixmap in den anderen Dateien, es sind baserender.[c,h], complexcontent.[c,h] und textscroller.[c,h] entfernen. Und dann noch in allen Dateien wo CreatePixmap benutzt wird, den Parameter "osd" nachtragen. Ist halt Fleißarbeit. Und wenn man was vergessen hat, meckert der Compiler schon rum.
Grüße
kamel5
Vielen Dank für die Tipps. Werde ich gleich mal testen 
Schade:
g++ -Wall -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -std=c++11 -c -DPLUGIN_NAME_I18N='"skinflatplus"' -DVDRLOGO=\"vdrlogo_default\" -DWIDGETFOLDER='"/usr/lib/vdr/plugins/skinflatplus/widgets"' -I/usr/include/uuid -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/GraphicsMagick -o complexcontent.o complexcontent.c
complexcontent.c: In member function ‘void cComplexContent::CreatePixmaps(bool)’:
complexcontent.c:80:25: error: ‘osd’ was not declared in this scope; did you mean ‘Osd’?
80 | Pixmap = CreatePixmap(osd, 1, Position, PositionDraw);
| ^~~
| Osd
g++ -Wall -g -O3 -Wall -Werror=overloaded-virtual -Wno-parentheses -fPIC -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -std=c++11 -c -DPLUGIN_NAME_I18N='"skinflatplus"' -DVDRLOGO=\"vdrlogo_default\" -DWIDGETFOLDER='"/usr/lib/vdr/plugins/skinflatplus/widgets"' -I/usr/include/uuid -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/GraphicsMagick -o textscroller.o textscroller.c
make[1]: *** [Makefile:102: complexcontent.o] Fehler 1
make[1]: *** Auf noch nicht beendete Prozesse wird gewartet …
textscroller.c: In member function ‘void cTextScroll::SetText(const char*, cRect, tColor, tColor, cFont*, tColor)’:
textscroller.c:18:27: error: ‘osd’ was not declared in this scope; did you mean ‘Osd’?
18 | Pixmap = CreatePixmap(osd, Layer, Position, drawPort);
| ^~~
| Osd
make[1]: *** [Makefile:102: textscroller.o] Fehler 1
make[1]: Verzeichnis „/home/darkwing/src/skin-flatplus/vdr-plugin-skinflatplus-0.6.7+git20230825-4-a2ad76d9“ wird verlassen
dh_auto_build: error: make -j4 returned exit code 2
make: *** [debian/rules:18: build] Fehler 2Da muss ich wohl auch ohne Klasse in die flat auslagern...
Pixmap = CreatePixmap(osd, 1, Position, PositionDraw);
^~~ | Osd
Es steht ja im Prinzip da. In der Datei textscroller.c musst Du Osd anstelle von osd benutzen. Das muss immer zu dem passen, was in der entsprechenden Datei benutzt wird.
Grüße
kamel5
Hab noch ne Weile getestet und gebaut. Scheint zu laufen! Zumindest in meiner Virtualbox.
Ich denke ich werde das noch ein wenig bereinigen (Habe alten Code nur auskommentiert) und ins git stellen. Vielen Dank für die Unterstützung!
Kommen bestimmt noch mehr Fragen von mir wenn ich wieder Zeit zum Basteln habe
Beides ist legitim und kann benutzt werden. CreatePixmap() hat hier gegenüber "osd->CreatePixmap" den Vorteil, das da noch Prüfungen eingebaut sind.
Und ja, Du kannst das im Prinzip ersetzen. Also ausprobieren.
Grüße
kamel5
Sie haben noch kein Benutzerkonto auf unserer Seite? Registrieren Sie sich kostenlos und nehmen Sie an unserer Community teil!