Englisch im Code Review

Warum braucht Code Review besonderes Englisch?

Code Review ist nicht nur technisches Feedback — es ist Teamkommunikation. Unpassende Formulierungen koennen:

Arrogant oder sarkastisch wirken
Die Motivation von Kollegen verletzen
Missverstaendnisse verursachen (besonders in interkulturellen Teams)
Den Eindruck erwecken, Sie wuerden “herumnergeln” statt aufrichtig zu helfen

Die gute Nachricht: Es gibt einen Satz von Standardausdruecken, die alle verstehen.

10 unverzichtbare Code-Review-Ausdruecke

1. Nit (Kleinigkeit)

“Nit” kommt von “nitpick” (Haare spalten), ist aber im Code Review ein positiver Begriff: “Das ist klein, beeinflusst nicht das grosse Ganze, aber ist verbesserungswuerdig.”

❌ "This variable name is bad"
✅ "Nit: could we rename this to `userCount`?"
✅ "Minor: I'd prefer `userCount` for clarity"

Wann verwenden:

Benennung, Formatierung, Stilfragen
Refactoring-Vorschlaege (nicht zwingend, aber besser)
Dokumentationsergaenzungen

2. LGTM (Looks Good To Me)

Abkuerzung, die bedeutet: Sie haben das Review abgeschlossen und stimmen dem Merge zu.

LGTM! 👍

Oder formeller:

LGTM. Great implementation.

3. Blocker (Blockierendes Problem)

Schwerwiegendes Problem, das vor dem Merge behoben werden muss.

Blocker: This will cause a race condition in `handleUserLogin`.
Here's why: ...

4. Nice catch (Gut erkannt)

Wenn jemand einen Bug, ein Performance-Problem oder eine versteckte Anforderung findet und Sie bestaetigen, dass sie Recht haben.

Nice catch! I didn't think about the edge case where `null` ...

5. Could you elaborate (Koennten Sie das naeher erlaeutern)

Wenn Sie nicht verstehen, warum etwas so geschrieben wurde, oder die Designentscheidung nachvollziehen moechten.

Could you elaborate on why we're using `Map` instead of `Object`?

Freundlichere Version:

I'm curious about the choice to use `Map` here. Could you walk me through the reasoning?

6-10. Weitere wichtige Ausdruecke

Nitpick: Staerkere Version von “nit”
Looks good: Teilweise Zustimmung zu einem Abschnitt
We should probably: Sanfter Vorschlag
Have you considered: Offene Frage zu Alternativen
Ship it: Informellste Zustimmung — bereit fuer die Veroeffentlichung

Die Hoeflichkeitsformel fuer Code Reviews

Beim Hinweisen auf Probleme

Formel: Kontext → Problem → Vorschlag

❌ "This code is inefficient"

✅ "In `processUserData`, iterating `O(n²)` might be slow with large datasets.
   Could we use a `Set` to get `O(n)` instead?"

Bei unterschiedlichen Meinungen

❌ "That's wrong"

✅ "I see your approach. One concern: `Math.random()` isn't cryptographically
   secure for auth tokens. Should we use `crypto.getRandomValues()`?"

Zustimmen und trotzdem fragen

❌ LGTM

✅ LGTM! Curious: did you benchmark this against the regex approach we discussed?
   I'm wondering if it's noticeably faster for production scale.

Express-Modus im Code Review

Ihr Gedanke in der Muttersprache → Drei englische Optionen → Die passendste waehlen

Szenario 1: Auf einen Bug hinweisen

Sie moechten sagen: “Hier gibt es eine Race Condition, mehrere Anfragen kollidieren”

Basic (Direct):
  "There is a race condition here, multiple requests will conflict"

Intermediate (Natural):
  "Multiple concurrent requests could cause a race condition"

Native (Idiomatic):
  "Concurrent requests might race here. Consider using a lock or queue."

Szenario 2: Hoeflich eine Umbenennung vorschlagen

Sie moechten sagen: “Dieser Variablenname ist zu lang, koennte man ihn kuerzen”

Native (Idiomatic):
  "Nit: `getCurrentUserAuthenticationStatusFlag` is a bit verbose.
   How about `isAuthenticated`?"
  Tipp: Ein konkreter Vorschlag ist hilfreicher als nur "kuerzen"

Haeufige Code-Review-Fehler

Typische Nicht-Muttersprachler-Interferenz

Fehlerhafter Ausdruck	Grund	Bessere Version
”This code is not good”	Zu direkt, wirkt arrogant	”Could we refactor this for clarity?"
"You must change this”	Klingt wie ein Befehl	”We should probably change this"
"This is wrong”	Verletzend	”This might cause X in scenario Y"
"The code of the function is bad”	Uebermaessiges “of"	"The function could be clearer”

Schnellreferenztabelle

Situation	Ausdruck
Zustimmung + Kleinigkeit	LGTM, just one nit: …
Anderer Meinung bei einer Entscheidung	I see your approach, but have we considered…?
Schwerwiegendes Problem	Blocker: This will cause…
Kleiner Vorschlag	Nit: Could we…?
Zustimmung + Lernwunsch	Nice! Could you explain why you chose…?
Unsicher	I’m not sure about this. Could we discuss?
Vollstaendige Zustimmung	LGTM! Ship it.