Der Server wusste, was du zuletzt gesehen hast — und sonst nichts
Für „wie viel habe ich letzte Woche geschaut" gab es keine Daten. `UserItemState` ist eine Zeile je Titel: der letzte Stand. Wer einen Film zweimal sieht, überschreibt den ersten Durchgang; wann etwas lief, wusste der Server nur fürs jeweils letzte Mal. Nachträglich erfinden lässt sich das nicht. Jetzt gibt es ein Sehprotokoll: **eine Zeile je Nutzer, Tag und Titel.** Keine Sitzungen — die brauchen ein Ende, und das kommt oft nie an, wenn der Browser zugemacht oder das Telefon zugeklappt wird. Tagesbündel haben diesen Lebenslauf nicht, es wird nur addiert. **Gezählt wird, was wirklich lief.** Der Zuwachs zwischen zwei Meldungen, gedeckelt auf die Zeit, die dazwischen verstrichen ist (mal zwei als Toleranz für doppelte Geschwindigkeit und ausgefallene Meldungen). Sonst ergäbe ein Sprung von Minute 5 auf Minute 90 anderthalb Stunden Sehzeit in zehn Sekunden. Gemessen an fünf Meldungen im Abstand von elf Sekunden: 88 Sekunden statt 190. Dabei fiel ein zweiter Fall auf: war **gar keine** Zeit verstrichen, griff kein Deckel statt eines engen — eine frisch angelegte Zeile hätte ihre Anfangsstelle voll mitgezählt. Wer einen Film öffnet und sofort auf Minute fünf springt, hat keine fünf Minuten gesehen. Im Profil stehen drei Zahlen, und jede sagt, woher sie kommt: die aufgezeichnete Zeit fängt heute an, die Zahl gesehener Titel steht seit jeher fest. Dazu eine Wochenkurve, die häufigsten Genres und was zuletzt lief. **Jeder sieht nur sich selbst, auch ein Administrator** — es gibt keinen Weg, ein fremdes Konto zu meinen. Zwei Dinge auf dem Weg dorthin: **Eine stellbare Uhr für die Tests.** Zwei Minuten Film lassen sich nicht in fünf Millisekunden sehen, und das ist richtig so — prüfen liess es sich nur, indem die Zeit vergeht, ohne sie zu verbringen. **Ein Namenskonflikt im CSS.** Die leeren Wochenbalken trugen die Klasse `leer`, und die gehört längst der Fläche „hier ist nichts" — mitsamt 24 Pixeln Polster. Jeder leere Balken war dadurch 48 Pixel hoch, egal welche Höhe dastand. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
d8a1633cf4
commit
579a49b73b
@@ -66,6 +66,10 @@ public static class PlaybackEndpoints
|
||||
var ergebnis = ProgressRule.Apply(
|
||||
TimeSpan.FromSeconds(anfrage.PositionSeconds), laenge, regeln);
|
||||
|
||||
// **Erst zählen, dann überschreiben.** Der Zuwachs ergibt sich aus
|
||||
// der alten Stelle; eine Zeile später gibt es die nicht mehr.
|
||||
await Mitzaehlen(db, benutzer, zustand, ergebnis.Position, clock);
|
||||
|
||||
// **Das Häkchen wird gesetzt, nicht genommen.** Wer einen
|
||||
// gesehenen Film erneut anfängt, hat ihn trotzdem gesehen — und
|
||||
// soll beides haben: das Häkchen und die neue Stelle.
|
||||
@@ -524,6 +528,74 @@ public static class PlaybackEndpoints
|
||||
/// <see cref="UserItemState.InProgress"/> schreibt — gäbe es eine zweite,
|
||||
/// wäre die Spalte wieder ein zweites Urteil statt eines festgehaltenen.
|
||||
/// </remarks>
|
||||
/// <summary>Schreibt die tatsächlich gesehene Zeit ins Protokoll.</summary>
|
||||
///
|
||||
/// <remarks>
|
||||
/// <para>
|
||||
/// <b>Der Zuwachs, gedeckelt auf die verstrichene Zeit.</b> Zwischen zwei
|
||||
/// Meldungen liegen zehn Sekunden; wer in dieser Zeit von Minute 5 auf
|
||||
/// Minute 90 springt, hat nicht 85 Minuten gesehen. Ohne Deckel erfände
|
||||
/// jedes Vorspulen Sehzeit — und ein Film, den man einmal durchgespult
|
||||
/// hat, stünde in der Statistik wie zweimal gesehen.
|
||||
/// </para>
|
||||
/// <para>
|
||||
/// <b>Das Doppelte als Toleranz</b>, nicht genau die verstrichene Zeit:
|
||||
/// eine ausgefallene Meldung, ein kurzer Aussetzer im Netz, doppelte
|
||||
/// Abspielgeschwindigkeit — all das ist echtes Schauen und soll zählen.
|
||||
/// Erst darüber wird gekappt.
|
||||
/// </para>
|
||||
/// <para>
|
||||
/// <b>Rückwärts zählt nichts.</b> Wer zurückspringt und dieselbe Stelle
|
||||
/// noch einmal sieht, bekommt sie nicht abgezogen und auch nicht doppelt
|
||||
/// gutgeschrieben: der Zuwachs ist dann null, bis er die alte Stelle
|
||||
/// wieder erreicht. Das ist die freundlichere Ungenauigkeit von beiden.
|
||||
/// </para>
|
||||
/// </remarks>
|
||||
private static async Task Mitzaehlen(
|
||||
FlaroContext db, Guid benutzer, UserItemState zustand,
|
||||
TimeSpan neueStelle, TimeProvider clock)
|
||||
{
|
||||
if (zustand.MovieId is null && zustand.EpisodeId is null) return;
|
||||
|
||||
var jetzt = clock.GetUtcNow();
|
||||
var verstrichen = jetzt - zustand.UpdatedAt;
|
||||
var zuwachs = (neueStelle - zustand.Position).TotalSeconds;
|
||||
|
||||
// **Ohne verstrichene Zeit gibt es nichts gutzuschreiben.** Das ist
|
||||
// die erste Meldung einer Zeile — sie legt den Nullpunkt, mehr nicht.
|
||||
// Stünde hier stattdessen „kein Deckel", zählte eine frisch angelegte
|
||||
// Zeile ihre Anfangsstelle voll mit: wer einen Film öffnet und sofort
|
||||
// auf Minute fünf springt, hätte fünf Minuten gesehen.
|
||||
if (zuwachs <= 0 || verstrichen <= TimeSpan.Zero) return;
|
||||
|
||||
zuwachs = Math.Min(zuwachs, verstrichen.TotalSeconds * 2);
|
||||
if (zuwachs <= 0) return;
|
||||
|
||||
// Ortszeit des Servers: wer um halb eins zu Ende sieht, hat das am
|
||||
// Abend getan.
|
||||
var tag = DateOnly.FromDateTime(jetzt.ToLocalTime().DateTime);
|
||||
|
||||
var zeile = await db.Sehtage.FirstOrDefaultAsync(z =>
|
||||
z.UserId == benutzer && z.Tag == tag
|
||||
&& z.MovieId == zustand.MovieId && z.EpisodeId == zustand.EpisodeId);
|
||||
|
||||
if (zeile is null)
|
||||
{
|
||||
db.Sehtage.Add(new Sehtag
|
||||
{
|
||||
UserId = benutzer,
|
||||
MovieId = zustand.MovieId,
|
||||
EpisodeId = zustand.EpisodeId,
|
||||
Tag = tag,
|
||||
Sekunden = zuwachs,
|
||||
});
|
||||
}
|
||||
else
|
||||
{
|
||||
zeile.Sekunden += zuwachs;
|
||||
}
|
||||
}
|
||||
|
||||
private static void Nachfuehren(UserItemState zustand, Wiedergaberegeln regeln) =>
|
||||
zustand.InProgress = ProgressRule.BelongsInContinueWatching(zustand, regeln);
|
||||
|
||||
|
||||
Reference in New Issue
Block a user