spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Zitat von »David Scherfgen«

- Wenn man eine Instanz der Klasse als Member benutzt oder als Parameter (by Value)

Für Parameter braucht man die Definition nicht.

Ansonsten habe ich die gleichen Punkte auch im verlinkten Artikel genannt.

Forward Declarations machen auch dann Sinn, wenn Abhängigkeiten zwischen Dateien aufgehoben werden sollen, das kann bei Bibliotheken beispielsweise schon als einziger Grund genügen, sie zu verwenden. So kann man zum Beispiel Klassen teils modifizieren, ohne gleich jede einzelne Datei, die die Klasse verwendet, neu kompilieren zu müssen.

Man sollte halt IMMER dann forward declarations benutzen, wenn sie reichen. Genauso wie man IMMER const benutzen sollte, wenn es möglich ist. Oder Methoden statisch machen sollte, wenn sie auf keine Member zugreifen. Oder Variablen so lokal wie möglich halten sollte. Alles mehr oder weniger das selbe Prinzip, also erhöhen von Effizienz oder vermeiden von Fehlern.

Zitat von »Jonathan_Klein«

Man sollte halt IMMER dann forward declarations benutzen, wenn sie reichen. Genauso wie man IMMER const benutzen sollte, wenn es möglich ist.

Prinzipiell ist das eine gute Daumenregel. Aber ich find es eher unangenehm wenn ich z.B. eine kleine Library hab und dann ist da eine Klasse die Hand in Hand mit einer anderen Klasse geht und ich will sie verwenden, kann ich aber nicht, bevor ich mich durch die ganzen Header gewühlt hab um rauszufinden, was ich für diese zweite Klasse noch inkludieren muss.

Darum halt ich es z.B. bei Interfaces eher so, dass deren Header direkt die Header aller anderen benötigten Interfaces inkludieren, was der Compiletime nicht besonders schaden sollte, das ganze aber sehr viel angenehmer zu benutzen macht.

Zitat von »Jonathan_Klein«

Oder Methoden statisch machen sollte, wenn sie auf keine Member zugreifen.

In dem Fall würd ich mir zuallererst mal überlegen, ob das überhaupt eine Memberfunktion sein soll

Kann dot nur zustimmen. Ich bin kein Freund von übermäßigen Forwards, weil man dann echt keinen guten Überblick mehr hat und der Code schwer zu warten ist (gerade wenn es nicht der eigene ist). Zirkuläre Abhängigkeiten sind meiner Meinung nach auch oft Hinweis auf ein schlechtes Design.

Statische Methoden sehe ich auch wie dot, denn wenn sie auf keine Member zugreifen, dann sind es sehr oft entweder Utility-Funktionen oder Factory-Methoden. Ersteres sollte man auslagern, bei letzterem kann man drüber nachdenken. Ich werfe mal "Lack of Cohesion" in diesem Zusammenhang mit in den Raum

Ich würde meinen sowas hier.

Jopp

Zitat von »dot«

Zitat von »Jonathan_Klein«
Oder Methoden statisch machen sollte, wenn sie auf keine Member zugreifen.

In dem Fall würd ich mir zuallererst mal überlegen, ob das überhaupt eine Memberfunktion sein soll

Ja, denn kapseln macht immer Sinn.

[EDIT]; "" bedeutet bei mir meist Sarkasmus, nur, dass das hier keiner in den falschen Hals bekommt.