Eine kleine Änderung.
Eine doppelte Zahlung?
Ein Review des Zahlungswegs zeigt, wie Zeilenwerk einen Hinweis aus dem Zusammenhang begründet. Der Code ist ein vereinfachtes Beispiel.
async function createInvoice(items) { const total = items.reduce(sum, 0);+ await chargeCustomer(total);1 return saveInvoice({ total });}Warum diese Stelle?
Der Aufruf belastet zuerst die Zahlungsmethode und speichert danach die Rechnung. Schlägt das Speichern fehl, bleibt die erfolgreiche Zahlung ohne zugehörigen Datensatz. Ein erneuter Versuch kann die Belastung wiederholen.
Der Hinweis bezieht sich auf die Reihenfolge zweier Seiteneffekte. Eine sichere Lösung muss sowohl Wiederholungen als auch unvollständige Vorgänge berücksichtigen.
const key = request.idempotencyKey;
const payment = await chargeCustomer(total, { key });
return saveInvoice({ total, paymentId: payment.id });Die konkrete Umsetzung hängt vom Zahlungsanbieter und Ihrem Datenmodell ab. Ergänzen Sie Tests für Wiederholungen und Fehler beim Speichern.
Welche Tests fehlen?
- Zahlung erfolgreich, Speichern der Rechnung schlägt fehl.
- Derselbe Auftrag wird mit identischem Schlüssel wiederholt.
- Zwei Anfragen erreichen den Zahlungsweg gleichzeitig.
Ein nachvollziehbarer Report.
Exporte enthalten Dateipfad, Zeilenbereich, Erklärung und Prüfschwerpunkt. Markdown eignet sich für die Diskussion im Pull Request, JSON für Ihre eigene Auswertung.
## Hinweis: wiederholte Zahlung
Datei: src/billing/invoice.ts
Zeilen: 30–32
Schweregrad: hoch
Prüfbereich: Logik & Randfälle{
"file": "src/billing/invoice.ts",
"lines": [30, 32],
"severity": "high",
"category": "logic"
}
Was passiert bei einem erneuten Versuch?
Wird das Speichern abgebrochen, kann ein zweiter Aufruf dieselbe Zahlung erneut auslösen. Ein Idempotenzschlüssel schützt den Zahlungsweg.