Klassen in C++ Durchschnitt von zwei Zahlen berechnen
-
@DocShoe sagte in Klassen in C++ Durchschnitt von zwei Zahlen berechnen:
Hihi, wir benutzen genau sowas.
Vermutlich nicht, wenn ich dich richtig verstehe. Ihr habt (vermute ich) eine Klassenhierachie für verschiedene Werte, die man berechnen kann. Nicht verschiedene Möglichkeiten, den Mittelwert zu berechnen. Also eher: mach irgendwas mit einer Zahlenfolge. -> Variante 1: Mittelwert. Variante 2: Summe. Oder so. Aber doch nicht x verschiedene Varianten für ein und denselben Wert, die ihrerseits wieder Struktur haben.
Und mir ist klar, dass man abhängig von der Datenmenge manchmal z.B. aufsummierende Varianten haben will, die nicht so exakt arbeiten wie andere (z.B. wenn man die Varianz berechnen will). Aber auch da wäre dann die Frage, ob die wirklich voneinander erben sollten (oder eine abc haben sollen). Beide könnten auch bei dir von der globalen "mach was mit Zahlen"-Klasse erben.
Generell finde ich kleinere / flachere Klassenhierachien immer besser.
-
@wob sagte in Klassen in C++ Durchschnitt von zwei Zahlen berechnen:
Ich halte das für komplett overdone. Klassenhierachie für eine Mittelwert-Berechnung? Ernsthaft?
Ich dachte es ginge um eine Demonstration, wie man so etwas stringent designt. Ob in den Funktionen, die tatsächlich etwas tun, dann ein trivialer Einzeiler oder wer weiß was für komplexe Berechnungen stattfinden, ist ja egal für die Fragestellung, wie man beispielsweise unterschiedliche Strategien für diese Berechnungen sauber implementieren kann.
Um meinen Senf dazu zu geben: Ich hätte hier die Strategien vermutlich als ein Plugin-System auf Templateebene implementiert. So a la
AverageInterface<MovingAverageEngine>undAverageInterface<AccumulatingAverageEngine>. Denn wie auch schon wob und DocShoe sehe ich keinen Grund, wieso sich dies polymorph verhalten sollte. Man wird ja stets nur eine Strategie für eine Aufgabe wählen.@EinNutzer0 sagte in Klassen in C++ Durchschnitt von zwei Zahlen berechnen:
@SeppJ Wie geht es besser (das Design)? Abstrakte Klassen dürfen ja keine Member haben...
Was besser? Weder das Speicherleck noch das komische Interface für deinen
ExactAveragehaben etwas mit exakten Klassen zu tun. Das erste ist einfach eine Frage der korrekten technischen Umsetzung, wie man korrektes C++ programmiert, und das zweite ist halt einfach nur komisch und keiner hier weiß, wieso du das überhaupt so gemacht hast anstatt wie bei der anderen Variante mitzuzählen.
-
@DocShoe sagte in Klassen in C++ Durchschnitt von zwei Zahlen berechnen:
@wob sagte in Klassen in C++ Durchschnitt von zwei Zahlen berechnen:
Ich halte das für komplett overdone. Klassenhierachie für eine Mittelwert-Berechnung? Ernsthaft?
Hihi, wir benutzen genau sowas. Wir haben Messdatenreihen, für die sich der Kunde passende Filter zusammenklicken kann, um sich sein Diagramm zusammenzustellen.
Habe sowas ähnliches auch schon bei Indikatoren im Börsenumfeld benutzt. Es gibt N-Eingangszeitreihen, M-Ausgangszeitreihen und dazwischen ist eine Blackbox. Lässt sich auch sehr gut über eine solche Klassenhierarchie abbilden. Ist im einfachsten Fall auch nur eine fortlaufende Mittelwertberechnung, wenn auch geringfügig mehr als nur Summe / Anzahl.
-
Es ist nicht nötig, alle Werte zu sammeln.
Du hast eine Fallunterscheidung in der Logik, die immer wieder aufgerufen wird, unschön.@DocShoe :
Wo ist meine Klassenhierarchie hin???
Ich hab explizit gesagt, dass ich zwei unterschiedliche Summierungsstrategien umsetzen möchte.Erzählt mir doch nix, private Member abstrakter Klassen sind in von ihr abgeleiteten Klassen nicht sichtbar.....
-
@EinNutzer0 sagte in Klassen in C++ Durchschnitt von zwei Zahlen berechnen:
@DocShoe :
Wo ist meine Klassenhierarchie hin???
Ich hab explizit gesagt, dass ich zwei unterschiedliche Summierungsstrategien umsetzen möchte.Lesen, nachdenken, verstehen, antworten.
Dann bau dir doch eineExactAverageCalculatorKlasse, die vonUnaryNumericOperationerbt. Damit kannste noch mehr Strategien implementieren, nicht nur Mittelwertbestimmung.@EinNutzer0 sagte in Klassen in C++ Durchschnitt von zwei Zahlen berechnen:
Abstrakte Klassen dürfen ja keine Member haben...
ist was anderes als
@EinNutzer0 sagte in Klassen in C++ Durchschnitt von zwei Zahlen berechnen:... private Member abstrakter Klassen sind in von ihr abgeleiteten Klassen nicht sichtbar.....
-
Na dann erklär mal...
-
@EinNutzer0
Stell doch mal ne Frage...
-
@DocShoe sagte in Klassen in C++ Durchschnitt von zwei Zahlen berechnen:
@EinNutzer0
Stell doch mal ne Frage...Bitte nicht :((
-
@EinNutzer0 sagte in Klassen in C++ Durchschnitt von zwei Zahlen berechnen:
Es ist nicht nötig, alle Werte zu sammeln.
Du hast eine Fallunterscheidung in der Logik, die immer wieder aufgerufen wird, unschön.Hmm das glaube ich nicht.
Denn so ein Moving Average (Filter) berechnet den Mittelwert über die letzten N Werte und dieser wird kontinuierlich aufgerufen, heißt man steckt üblicherweise ein Wert in die Klasse und fragt danach den neuen Mittelwert ab. Steckt man also X Werte rein, so bekommt man X gemittelte Werte raus.
Der Moving Average Filter wird dazu benutzt um Datenreihen zu glätten. Ein Beispiel: Reihe {1, 0, 1, 0, 1}, N = 2
Der erste Wert ist 1, gemittelt 1.
Der zweite Wert ist 0, gemittelt (1+0)/2 = 0.5.
Der dritte Wert ist 1, gemittelt (0 + 1)/2 = 0.5.
Der vierte Wert ist 0, gemittelt (1 + 0)/2 = 0.5.
Der fünfte Wert ist 1, gemittelt (0 + 1)/2 = 0.5.Solche geglättete Kurven sieht man z B. manchmal bei der Wetter-Temperaturprognose und bei den Grafiken zu den Corona-Fallzahlen.
