Gewichteter Mittelwert über eine Zeitperiode

[Fohnbit]

Hallo,

ich sitze nun schon einige Zeit vor folgendem Problem:
Ich möchte den gewichteten Mittelwert von Messdaten über eine Zeit errechnen.
Beispiel:
Software startet mit dem Wert 20, Time 15:00
Nun kommt ein neuer Wert 80, Time: 15:01
(Der Wert 20 war also 1 Minute lang aktiv)
Nun kommt ein neuer Wert 120, Time 15:03
(Der Wert 80 war also 2 Minuten lang aktiv)

Als Ergebnis sollte ich 60 bekommen:
(1*20)+(2*80)/(1+2)=60

Jedoch wie bilde ich das mit einer Methode ab?

 

[stg]

Wo genau liegt das Problem? Du kennst die Werte und die Gewichte und weißt ja offenbar auch, wie du daraus nun den gesuchten Wert berechnest ....

(Dir fehlen übrigens Klammern in deiner Darstellung...)

 

[Fohnbit]

Hallo,

ja, aber bin mir nicht sicher, ob die Aufrufe einigermaßen in Ordnung sind und ob das Ergebnis auch immer paßt.
Mir scheint es eventuell zu kompliziert geschrieben zu sein?

public class Message {
	public long lastTimeStamp;
	public long durationValue1;
	public long durationValue2;
	public double value1;
	public double value2;
	public double max;
	public double min;
	

}

public static Message addValues(long timestamp, double value, Message msg) {
		if (value > msg.max) {
			msg.max = value;
		}

		if (value < msg.min) {
			msg.min = value;
		}

		if (msg.lastTimeStamp == 0) {
			msg.value1 = value;
			msg.lastTimeStamp = timestamp;
			return msg;
		}
		if (msg.durationValue1 == 0) {
			msg.value2 = value;
			msg.durationValue1 = timestamp - msg.lastTimeStamp;
			msg.lastTimeStamp = timestamp;
			return msg;
		}

		msg.durationValue2 = timestamp - msg.lastTimeStamp;

		msg.value1 = ((msg.value1 * msg.durationValue1) + (msg.value2 * msg.durationValue2))
				/ (msg.durationValue1 + msg.durationValue2);

		msg.durationValue1 = msg.durationValue2 + msg.durationValue1;
		msg.lastTimeStamp = timestamp;
		msg.value2 = value;
		return msg;

	}

		Message msg = new Message();
			msg = addValues(System.currentTimeMillis(), 20, msg);
			System.out.println(msg.value1);
			Thread.sleep(1000);

			msg = addValues(System.currentTimeMillis(), 40, msg);
			System.out.println(msg.value1);
			Thread.sleep(1000);
			msg = addValues(System.currentTimeMillis(), 90, msg);
			System.out.println(msg.value1);
			Thread.sleep(1000);
			msg = addValues(System.currentTimeMillis(), 90, msg);
			System.out.println(msg.value1);

 

[stg]

Grundsätzlich ist der Ansatz schon OK so. Ein paar kleine Anmerkungen jedoch:

- Ich würde dir jedoch zu einer adäquateren Namensgebung raten. Für mich war nicht sofort ersichtlich, was du da eigentlich genau machst.
- Ich finde es nicht so gelungen die alten, kumulierten Daten zusammen mit dem neuen Datensatz in ein und demselben Objekt abzulegen. Das sind verschiedene Dinge, die nicht unmittelbar miteinander zusammenhängen und daher würde ich das trennen. Eventuell ist später auch einmal die gesamte Historie interessant (z.B. wenn du den Verlauf graphisch darstellen möchtest). Bei deinem jetzigen Design ist das nicht möglich und auch keine kleinere Änderung. Wenn du die Datenjedoch voneinander trennst sind solche Änderungen später viel einfacher möglich.