c++.de :: SoftPixel Sandbox

LukasBanana Mitglied 10:58:47 09.11.2011 Titel:

Hallo Leute,
ich arbeite nun schon seit ein paar Monaten an einem World Editor für meine 3D Engine: die SoftPixel Sandbox.

Vor ein paar Tagen habe ich die erste ALPHA version (0.1) hochgeladen.
Diese World Editor soll auch für die nutzbar sein, die nicht meine 3D Engine verwenden.
Für das LevelFile Format gibt's in meinem Forum eine Spefizikation.

Denkt daran: das ist noch eine ganz frühe ALPHA version und da ist noch einiges dran zu tun.

Den Download gibt's hier.

Gruß,
Lukas

Kóyaánasqatsi Mitglied 11:39:33 09.11.2011 Titel:

Sehr schön, gefällt mir. Weiter so!

314159265358979 Mitglied 13:59:01 09.11.2011 Titel:

Dein Code ist schon viel besser als letztes mal. :live:

Aber:
- Viele Memberfunktionen sind inline.
- Zu viel protected. (Member sollten niemals protected sein, Methoden nur, wenn sie aus der abgeleiteten Klasse aufgerufen werden sollen. Um zu überschreiben, reicht private.)
- Ich sehe zu viele Dtoren. Vermutlich eine Folge von zu viel manueller Speicherverwaltung. Ich habe übrigens beim drübersehen keinen einzigen Smartpointer gesehen.
- Ich sehe zu viele rohe Zeiger. Auch hier sollten bei besitzenden Zeigern Smartpointer her.
- Wozu die FileSystem Klasse? Auch solltest du statt openFile und closeFile RAII verwenden.
- Der Zweck deiner Stringklasse ist mir immer noch ein Rätsel.
- Dein MemoryManager ist immer noch unnötig.

Insbesondere auf Smartpointer haben ich und joke dich im IRC ja hingewiesen, die hast du dir offenbar nicht angesehen

rapso Moderator 14:45:57 10.11.2011 Titel:

es sieht mir bei einigen dingen eher danach aus, als ob es grobe frameworks sind fuer das wofuer sie spaeter benoetigt werden, von daher kann man noch nicht sagen, ob ein memory manager usw. sinn macht.

was mir ein wenig verbesserungswuerdig scheint ist das hardgecodete bei den UI resourcen, bei einem guten editor, moechtest du normalerweise, dass du dinge an der engine aendern kannst, ohne noch n-stellen im code zu suchen um sie daran anzupassen, von daher sollten die elemente deiner engine selbst reflektieren was fuer properties sie haben.
dafuer empfehle ich "3D Games: Realtime rendering and technology" von policarpo
ISBN10: 0201619210
ISBN13: 9780201619218

314159265358979 Mitglied 14:53:45 10.11.2011 Titel:

rapso schrieb:

es sieht mir bei einigen dingen eher danach aus, als ob es grobe frameworks sind fuer das wofuer sie spaeter benoetigt werden, von daher kann man noch nicht sagen, ob ein memory manager usw. sinn macht.

So ein MemoryManager, der überhaupt nichts managed macht keinen Sinn.

Edit: Ich vermute mal, dass es exakt die selbe Klasse (Klasse!) ist, wie die hier: http://softpixelengine.sourceforge.net/docu/classsp_1_1_memory_manager.html

this->that Mitglied 16:17:29 10.11.2011 Titel:

314159265358979 schrieb:

rapso schrieb:

es sieht mir bei einigen dingen eher danach aus, als ob es grobe frameworks sind fuer das wofuer sie spaeter benoetigt werden, von daher kann man noch nicht sagen, ob ein memory manager usw. sinn macht.

So ein MemoryManager, der überhaupt nichts managed macht keinen Sinn.

Selbst dann kann er Sinn machen. Wenn du z.B. Allokationen zentralisierst, kannst du spaeter die Allokationsstrategie ganz leicht austauschen (gegen Pooling etc.).

314159265358979 Mitglied 16:52:33 10.11.2011 Titel:

Sieh dir bitte erstmal seinen Code an, bevor du antwortest. Sein MemoryManager hat nicht im entferntesten was mit dem zu tun, wovon du gerade sprichst.

rapso Moderator 17:10:26 10.11.2011 Titel:

314159265358979 schrieb:

rapso schrieb:

es sieht mir bei einigen dingen eher danach aus, als ob es grobe frameworks sind fuer das wofuer sie spaeter benoetigt werden, von daher kann man noch nicht sagen, ob ein memory manager usw. sinn macht.

So ein MemoryManager, der überhaupt nichts managed macht keinen Sinn.

sorry, aber ich sehe keinen sinn darin ueber etwas unfertiges als sinnvoll/sinnfrei zu urteilen.
vieles in meinen engines faengt genau so an, und wenn der bedarf kommt, bau ich es bedarfsentsprechend aus. Es ist gut von anfang an solche interfaces/wrapper zu haben, die ueberall genutzt werden, denn dann kann man an einem punkt etwas aendern und muss nicht quer durch das projekt refactorn. Gerade gutes memory management ist nicht trivial im nachinein einzubauen.

Zitat:

Edit: Ich vermute mal, dass es exakt die selbe Klasse (Klasse!) ist, wie die hier: http://softpixelengine.sourceforge.net/docu/classsp_1_1_memory_manager.html

sieht doch ok aus, wenn man erstmal alle memory management operationen darueber laufen laesst, kann man das ding gut an die beduerfnisse erweitern/anpassen die man hat.

314159265358979 Mitglied 17:18:32 10.11.2011 Titel:

rapso schrieb:

sorry, aber ich sehe keinen sinn darin ueber etwas unfertiges als sinnvoll/sinnfrei zu urteilen.
vieles in meinen engines faengt genau so an, und wenn der bedarf kommt, bau ich es bedarfsentsprechend aus. Es ist gut von anfang an solche interfaces/wrapper zu haben, die ueberall genutzt werden, denn dann kann man an einem punkt etwas aendern und muss nicht quer durch das projekt refactorn. Gerade gutes memory management ist nicht trivial im nachinein einzubauen.

Offenbar ist das nicht so rübergekommen: Ich rede hier von _seinem_ MemoryManager, nicht von Speichermanagement im allgemeinen. Selbst da würde ich aber auf Smartpointer anstatt irgend eine Singleton-Klasse oder sowas setzen.

rapso schrieb:

sieht doch ok aus, wenn man erstmal alle memory management operationen darueber laufen laesst, kann man das ding gut an die beduerfnisse erweitern/anpassen die man hat.

Das Ding ist ein Design-Fail zum Quadrat.

deleted_2013_01_05 Mitglied 17:21:51 10.11.2011 Titel:

Halt doch bitte einfach mal die Klappe.

Kann ein Mod diesen ganzen Müll bitte löschen?