// 1)
class Font
{
Font(name, size);
};

// 2)
class Font
{
Font();
Create(name, size);
};

Ersteres hat den Vorteil, das man sich darauf verlassen kann (/können sollte), dass die Schrift initialisiert ist (-> weniger Kontrollarbeit) und im Prinzip brauche ich weniger Methoden zum Setzen von Eigenschaften schreiben (wie zum Beispiel Schriftgroesse), weil man dann auch gleich eine neue Schrift erzeugen könnte. Da hoert es aber wohl auch schon auf. Denn ich muss eine Schrift dann eigentlich immer dynamisch erzeugen (imho nicht so toll...).
Die zweite Version ist im Prinzip das genau Gegenteil, was sich auch auf die Vor- und Nachteile bezieht.

Frage: Was haltet Ihr für besser?

Ich tendiere eigentlich zur zweiten Variante (auch weil ich noch in Delphi programmiere und man da eh jedes Objekt über Create initialisiert (was da aber wiederum der Konstruktor ist - *ahhhhh*)), aber bin halt irgendwie noch unentschlossen

Gruss,
DeSoVoDaMu

Create()/Init()-Methoden machen nur Sinn wenn Konstruktion und Initialisierung getrennt ablaufen sollen - scheint mir im gegebenen Fall nicht sinnvoll.

Wie du darauf kommst, dass du dann nur noch dynamisch Schriften erzeugen kannst, weiß ich nicht:

// Viel weniger dynamisch geht imho nicht:
const Font myFont ( "Arial", 10 );

MfG SideWinder

Solche create()-Sachen machen in meinen Augen nur dann Sinn, wenn anhand von Parametern ein mir nicht bekannter Typ zurück gegeben wird. Also wenn es polymorph ist. Wenn du nur Font hast und der User weiß, er muß Font instanzieren, lass die create()-Sache weg. Macht keinen Sinn. Mach nen Ctor und gut ist.

Dann hast du aber eine Factory und create() als static-Methode. das ist hier nicht der Fall. Hier gehts um eine init_object()-Methode.

MfG SideWinder

Vielleicht habe ich die Gruende fuer die dynamische Erzeugung noch nicht exakt erlaeutert:

1. Ich wuerde die Font dann als Membervariable in einer Klasse nutzen und da wuerde ich den Pfad zu der Schrift ggf. noch nicht zur Konstruktion wissen. Ich wuerde sie also erst im weiteren Verlauf erzeugen, wenn der Pfad/Name klar ist (also dynamisch).
2. Wenn ich die Schriftgrösse aendern moechte, muesste ich ja im Prinzip wieder eine neue Schrift erzeugen, wenn ich keine Create Methode habe (also auch dynamisch).

Um funktionslokale Objekte ging es mir jetzt weniger.

Das sind im Prinzip die Zwei Gruende fuer die 2. Variante.

Gruss,
DeSoVoDaMu

EDIT: ok, man koennte bei ersterer Version sagen, dass es schlechtes Design ist (man muesste den Pfad/Namen halt schon vorher wissen) und bei zweiter koennte man ja eine private/protected Methode zur Initialisierung nutzen, die dann beim setzen einer neuen Schriftgroesse ueber eine Set-Methode einfach aufgerufen wirde. Waere vielleicht die beste Loesung...

Danke für die schnellen Antworten.

Vielleicht habe ich die Gruende fuer die dynamische Erzeugung noch nicht exakt erlaeutert:

1. Ich wuerde die Font dann als Membervariable in einer Klasse nutzen und da wuerde ich den Pfad zu der Schrift ggf. noch nicht zur Konstruktion wissen. Ich wuerde sie also erst im weiteren Verlauf erzeugen, wenn der Pfad/Name klar ist (also dynamisch).

class Foo
{
    public:
        void initFont(string path); // zum Laden der Schrift
    private:
        Font* myFont; // im echten leben ist das logisch ein SmartPointer ;)
        // ...
};

void Foo::initFont(string path)
{
    myFont = new Font(path);
}

Damit verschiebst du das Problem aber nur von "init-Funktion in der Schriftart" zu "init-Funktion beim Benutzer der Schriftart". Denn solche "init"-Funktionen sind in der Regel ein Problem!!! Du kannst dich nie drauf verlassen dass deine Schriftarten initialisiert sind. Und das ist einfach schlechtes Design! Da solltest du eine bessere Loesung finden!

2. Wenn ich die Schriftgrösse aendern moechte, muesste ich ja im Prinzip wieder eine neue Schrift erzeugen, wenn ich keine Create Methode habe (also auch dynamisch).

Ja und? Dann erstellst du eine neue Schrift, das ist doch kein Problem. Ich wuerd sogar soweit gehen zu sagen, dass es logisch ist: Ich will eine neue Schriftart, also erstell ich eine neue Schriftart.

...
Ersteres hat den Vorteil, das man sich darauf verlassen kann (/können sollte), dass die Schrift initialisiert ist (-> weniger Kontrollarbeit) und im Prinzip brauche ich weniger Methoden zum Setzen von Eigenschaften schreiben (wie zum Beispiel Schriftgroesse), weil man dann auch gleich eine neue Schrift erzeugen könnte. Da hoert es aber wohl auch schon auf. ...

Zwei Vorteile und kein Nachteil - was gibt es da noch zu überlegen ?

DeSoVoDaMu schrieb:

...
Die zweite Version ist im Prinzip das genau Gegenteil, was sich auch auf die Vor- und Nachteile bezieht....

Genau: Zwei Nachteile und kein Vorteil.

DeSoVoDaMu schrieb:

...
...Ich tendiere eigentlich zur zweiten Variante (auch weil ich noch in Delphi programmiere ...

Das scheint mir das Problem zu sein: Du hast Dich an eine Schwäche von Dephi gewöhnt und nun scheint es Dir komplizierter, es besser zu machen, als weiterhin schlechter.....

DeSoVoDaMu schrieb:

...
2) Wenn ich die Schriftgrösse aendern moechte, muesste ich ja im Prinzip wieder eine neue Schrift erzeugen, wenn ich keine Create Methode habe (also auch dynamisch)....

Du könntest auch eine "changeSize()"-Funktion spendieren. Damit drückst Du dann aus, dass eine Font seine Größe ändern kann. Ohne "setter" sagst Du aus, dass Name und Groesse "Identifizierende" Deines Fonts sind - sprich: Andere Größe => Anderer Font ! und da liegt nichts näher, als dass man auch ein neues Objekt anlegt.

Mir scheint aber Blue-Tigers Hinweis schon wichtig: Evtl. hast Du noch eine Designschwäche - es klingt mi danach, als bräuchtest Du neben dem Font noch einen FontCreator, der Initfiles interpretiert und entsprechende Fonts erzeugt.

Gruß,

Simon2.

Oder was haltet Ihr von der 2. Idee mit der internen Create Methode?

class Font
{
private:
  m_font;

  CreateFont(name, size)
  {
    m_font = OSFunktionZumErstellenEinerSchrift(name, size);
  }

public:
  Font(name, size)
  {
    CreateFont(name, size);
  }

  SetSize(new_size)
  {
    CreateFont(name /*ja, muss irgendwo gespeichert werden*/, new_size);
  }
};

Gruss,
DeSoVoDaMu

Klar kannst du jetzt hier noch fragen, ob auch die internas gut gemacht sind... und es ist so vom Prinzip her auch i.O.