Funktions Pointer + Vererbung = Problem

Simon2

abzy schrieb:

Ok, wie kann ich mein Problem dann anders lösen?...

Das hier:

abzy schrieb:

...Das habe ich gemacht indem ich sin bzw cos Funktionen in A angelegt hab und dann mit einem Funktionspointer aufgerufen habe. ...

Klasse A: diverse Rechenoperationen, sin & cos Funktion, "anmelde" Funktion
Klasse B: 7 weitere Funktionen.
...

lassen.

Es sieht ein wenig so aus, als wärst Du dem klassischen "OO ist, wenn man alles in Klassen packt"-Irrtum aufgesessen (oft in Kombination mit "Gute Programmierung ist, wenn man OO (nach obigem Verständnis) macht").

sin, cos und all die anderen Rechenoperationen, sind keine guten Kandidaten für Memberfunktionen, sondern ideal für "freie Funktionen".
Was sollen Deine Klassen eigentlich sein ? Was ist ein A ... und was unterscheidet es von einem anderen A ?
Wenn man einfach Funktionen bündeln will, sind namespaces prima.
Dann kannst Du auch die von Dir gewohnten Funktionspointer ("Nicht-member-Pointer") verwenden.
C++ unterstützt Dich durch overloading und Operatoren ziemlich in dieser Vorgehensweise.

Gruß,

Simon2.

abzy

sin, cos und all die anderen Rechenoperationen, sind keine guten Kandidaten für Memberfunktionen, sondern ideal für "freie Funktionen".
Was sollen Deine Klassen eigentlich sein ? Was ist ein A ... und was unterscheidet es von einem anderen A ?
Wenn man einfach Funktionen bündeln will, sind namespaces prima.
Dann kannst Du auch die von Dir gewohnten Funktionspointer ("Nicht-member-Pointer") verwenden.
C++ unterstützt Dich durch overloading und Operatoren ziemlich in dieser Vorgehensweise.

Also wenn es nach mir ginge würde ich das sicher in keiner eigenen Klasse machen

Aber ich muss mich ja an meine Angabe halten(Klasse A ist zu vergleichen mit CMathFloat und Klasse B mit CMathFloatFlex):

Implementieren Sie eine Klasse CMathFloatFlex, welche von der (modifizierten) Klasse
CMathFloat abzuleiten ist.
Die Klasse CMathFloatFlex hat folgende Zusatzfunktionaliät zu bieten:
• Zahlen sollen neben dem Fließkommaformat auch im wissenschaftlichen Format
<floatzahl> e +/- <exponent> einlesbar sein.
• Die Klasse soll es erlauben, dass man einstelligen oder zweistellige Funktionen
mittels Name und Funktionspointer registrieren lassen kann. Im Programm
mymath2 sind dann mindestens 7 Funktionen zu registrieren, wobei zumindest
die folgenden enthalten sein müssen: „min(x,y)“, „max(x,y)“, „pow(x,y)“,
„abs(x)“, „diff(x,y)“, „sqr(x)“ (Quadrat-Funktion), und „sgn(x)“ (Signum). Die
Klasse soll dann dynamisch prüfen, ob bei einem Funktionsaufruf die passende
Funktion registriert worden ist. Wenn nicht, soll eine Exception geworfen werden.

Vielleicht ist nun klarer warum ich so auf den 2 Klassen rumreite

LG

Simon2

Hmmm ... vielleicht hab ich's nur nicht gefunden, aber ich lese da nichts davon, dass sin&Co Memberfunktionen sein müssten....
Da steht nur, dass sie "registriert" werden können müssen (also sowas wie "Funktionspointer in die Map" eintragen ... wie Du es oben schon gemacht hast).

Ich sehe als "2 Klassen" eigentlich nur eine eigene Repräsentation von Zahlen...

BTW: Unter C++ gehören auf "freie Funktionen" mit zur Schnittstelle ... wie z.B.

struct A;
ostream& operator(ostream&, A const&);

Gruß,

Simon2.

Tachyon

Wieso nicht einfach direkt die mathematischen Funktionen in die Map packen?

typedef double (*mat_function)(double);

class BaseMath
{
public:
    void addFunction(mat_function function, const std::string& name);
    void execute(const std::string& name, double argument);
private:
    typedef std::map<std::string, mat_function> FunctionMapType;

    FunctionMapType m_FunctionMap;
};

void BaseMath::addFunction(mat_function function, const std::string& name)
{
    m_FunctionMap[name] = function;
}

void BaseMath::execute(const std::string& name, double argument)
{
    FunctionMapType::iterator iter = m_FunctionMap.find(name);
    if(iter != m_FunctionMap.end())
    {
        std::cout << "Result: " << (iter->second)(argument) << '\n';
    }
}

Oder ggf. eine Hilfsklasse benutzen, die Funktionsmaps verwaltet.

Simon2

Tachyon schrieb:

Wieso nicht einfach direkt die mathematischen Funktionen in die Map packen?...

Wäre halt eine zusätzliche Kapsel ... aber ob man die braucht, weiß ich nicht. Das könnte man auch genausogut direkt in CMathFloatFlex reinpacken - immerhin soll diese Klasse lt. Aufgabenstellung für Registrierung und Check zuständig sein (wobei ich immer weniger verstehe, was ein "CMathFloatFlex"-Objekt eigentlich sein soll)...

Gruß,

Simon2.

abzy

Den Sinn der übung weiß ich selber nicht so ganz

Aber hab jetzt eigenständig folgenden Weg gefunden: Hab alles was rund um dieses registrieren läuft in die abgeleitete Klasse gepackt. Ist zwar nicht so wie ich es machen wollte aber immerhin ist es so einfacher und funktioniert jetzt.

Danke nochmal, bin auf die idee erst gekommen als ich mir meinen eigenen Forum post durchgelesen habe mit der Angabe nochmal drin.

LG
Daniel

Tachyon

Simon2 schrieb:

[...](wobei ich immer weniger verstehe, was ein "CMathFloatFlex"-Objekt eigentlich sein soll)...

Gruß,

Simon2.

Ich habe so im Urin, dass es sich dabei um so eine Art normalen Taschenrechner handeln soll (CMathFloat) der um wissenschaftliche Funktionen erweitert werden soll/kann (CMathFloatFlex). Aber wenn dem tatsächlich so ist, verschließt sich mir etwas der Sinn dahinter, Funktionen anmelden zu können. Ich würde die dann gleich fest in die Basis bzw. die Ableitung(en) einbrennen.

Simon2

Außerdem finde ich es ziemlich merkwürdig, verschiedene Laufzeitflexibilisierungsmechanismen zu mischen (Vererbung und Map) ...

Gruß,

Simon2.

Tachyon

Simon2 schrieb:

[...]Laufzeitflexibilisierungsmechanismen[...]

Jo, stimmt.

Simon2

Tachyon schrieb:

Simon2 schrieb:

[...]Laufzeitflexibilisierungsmechanismen[...]

Jo, stimmt.

Hast recht ... schrieb sich ganz entspannt, aber zu lesen ist's ungeschmeidig...

Gruß,

Simon2.