Code-Review in TypeScript: Was prüfen und wie man Feedback gibt

Code-Review in TypeScript: Was prüfen und wie man Feedback gibt

Code-Review ist nicht nur eine Formalität, sondern ein mächtiges Werkzeug zur Verbesserung der Code-Qualität und zum Erfahrungsaustausch im Team. Für angehende TypeScript-Entwickler kann dieser Prozess komplex und einschüchternd wirken. In diesem Artikel analysieren wir, worauf bei der Überprüfung von TypeScript-Code zu achten ist und wie man Feedback gibt, das nützlich und konstruktiv ist.

1. Warum ist Code-Review für TypeScript wichtig?

TypeScript fügt JavaScript eine statische Typisierung hinzu, die bereits hilft, viele Fehler während der Kompilierung zu vermeiden. Das Code-Review ermöglicht es jedoch, Probleme zu identifizieren, die der Compiler nicht sieht: logische Fehler, unleserlicher Code, falsche Verwendung von Typen oder architektonische Mängel.

Die Hauptziele des Code-Reviews in TypeScript:

\n
• Verbesserung der Code-Qualität — Erkennung von Bugs und potenziellen Problemen vor dem Einsatz in der Produktion.
\n
• Lernen — Anfänger lernen anhand der Beispiele erfahrenerer Kollegen.
\n
• Einheitlicher Stil — Pflege einer sauberen und verständlichen Codebasis.
\n
• Sicherheit — Prüfung auf Schwachstellen und falsche Verwendung von Drittanbieter-Bibliotheken.
\n

2. Was ist in TypeScript-Code zu prüfen: Wesentliche Aspekte

Bei der Überprüfung von TypeScript-Code ist es wichtig, nicht nur auf die Logik zu achten, sondern auch darauf, wie die Sprachfunktionen genutzt werden. Hier sind die wichtigsten Punkte, die zu prüfen sind:

2.1. Typisierung und Verwendung von any

Eines der Hauptziele von TypeScript ist es, any loszuwerden. Wenn im Code any vorkommt, ist das ein Grund, die Frage zu stellen: "Warum wird hier kein konkreter Typ verwendet?"

// Schlecht: any löscht alle Vorteile von TypeScript\nfunction processData(data: any) {\n  return data.name.toUpperCase();\n}\n\n// Gut: Verwendung eines Interfaces\ninterface User {\n  name: string;\n  age: number;\n}\n\nfunction processData(data: User): string {\n  return data.name.toUpperCase();\n}

Eine Ausnahme ist die Arbeit mit dynamischen Daten, z. B. beim Parsen von JSON. Verwenden Sie in solchen Fällen unknown anstelle von any und überprüfen Sie den Typ.

2.2. Korrektheit von Typen und Interfaces

Überprüfen Sie, ob Interfaces und Typen nicht redundant sind und sich nicht widersprechen. Stellen Sie sicher, dass die Typen die Datenstruktur korrekt beschreiben.

// Schlecht: Redundantes Interface\ninterface User {\n  name: string;\n  age: number;\n  email?: string;\n  address?: string;\n  phone?: string;\n  // ... viele weitere Felder\n}\n\n// Gut: Aufteilen in kleine Interfaces\ninterface User {\n  name: string;\n  age: number;\n  contact: ContactInfo;\n}\n\ninterface ContactInfo {\n  email?: string;\n  phone?: string;\n}

2.3. Fehlerbehandlung und null/undefined

TypeScript hilft, Fehler im Zusammenhang mit null und undefined zu vermeiden, aber nur, wenn der Entwickler optionale Typen und Prüfungen korrekt verwendet.

// Schlecht: Keine Prüfung auf null\nfunction getUserName(user: User | null): string {\n  return user.name; // Fehler! user kann null sein\n}\n\n// Gut: Verwendung von optional chaining und Prüfung\nfunction getUserName(user: User | null): string {\n  return user?.name ?? 'Anonymous';\n}

2.4. Verwendung von Generics

Generics ermöglichen das Schreiben von wiederverwendbarem Code. Überprüfen Sie, ob Logik dupliziert wird, die verallgemeinert werden könnte.

// Schlecht: Duplizierung von Funktionen\nfunction getStringArray(items: string[]): string[] { return items; }\nfunction getNumberArray(items: number[]): number[] { return items; }\n\n// Gut: Verwendung eines Generics\nfunction getArray<T>(items: T[]): T[] {\n  return items;\n}

3. Wie man Feedback gibt: Regeln für Anfänger

Auch wenn Sie viele Fehler gefunden haben, ist es wichtig, sie richtig zu präsentieren