011
23.10.2006, 08:15 Uhr
ao
(Operator)
|
| Zitat von xXx: |
Machs am besten mit ner struct oda so.... hab mal kurz was zusammen geschrieben .. ist aber ungetestet obs richtig ist ...
|
So, dann vielleicht mal wieder zur Sache zurück.
Ich habs auch nicht getestet, aber es sieht richtig aus. Der Ansatz, Zähler und Nenner zu einem Struct oder einer Klasse zusammenzufassen ist auf jeden Fall schon mal gut, und soweit ich sehe, rechnet die Summenfunktion auch richtig.
Die Addierfunktion hätte ich aber so gebaut:
| C++: |
FRACTION Add (const FRACTION & f1, const FRACTION & f2);
|
Das mit den Pointer-Argumenten würde ich weglassen; so eine Rechenfunktion ist eben nur für intakte Fraction-Objekte definiert und nicht für irgendwas, was auch NULL sein kann.
Gültigkeitsprüfungen (Nenner != 0 und so) würde ich eher da unterbringen, wo neue Fraction-Objekte entstehen, also im Konstruktor, Zuweisungsoperator und so.
Zur Rückgabe: FRACTION ist ein kleines, einfaches Objekt, das kann man ruhig mit copy-by-value zurückgeben. Und, eine Summenfunktion hat immer ein gültiges Ergebnis (*), also braucht man den Fall, dass man nichts zum Zurückgeben hat, nicht zu berücksichtigen.
Wer aber dogmatisch meint, Funktionen, die Objekte zurückgeben, sind böse, kann es auch so schreiben:
| C++: |
void Add (const FRACTION & f1, const FRACTION & f2, FRACTION & result);
|
(*) vorausgesetzt die Eingaben sind gültig, aber wenn sie das nicht wären, würde man das schon bei der Konstruktion der Argumente merken.
Grundsätzlich: Eine Bruchklasse ist eine beliebte Übungsaufgabe zum Thema Methodendesign, weil man hier gut untersuchen kann, wie sich verschiedene Design-Entscheidungen auf die Benutzbarkeit der Klasse im Anwendungskontext auswirken. Man kann also viel dabei lernen, wenn man ein bisschen mit den verschiedenen Varianten herumspielt. Dabei ist die Implementierung der einzelnen Funktionen meist einfach, so dass man sich weitgehend aufs Interface konzentrieren kann.
Soweit erstmal mein Senf
ao
Dieser Post wurde am 23.10.2006 um 08:17 Uhr von ao editiert. |
