Kriterien für Junior-Entwickler als Einzelkämpfer in Start-ups zum Abschluss von Sprints ohne Überstunden
Wer ohne Mentor alleine Frontend-Entwicklung betreibt, wird oft von technischen Zweifeln gehetzt. Man hat das Buch zu Clean Code neben sich liegen, grübelt über die perfekte Abstraktion und wiederholt am Ende das Muster, kurz vor Sprint-Ende in den Burnout zu geraten. Was es braucht, ist keine fehlerfreie Architektur, sondern realistische Kriterien, um funktionierende Produkte rechtzeitig auszuliefern.
Falsche Abstraktion kostet mehr als duplizierter Code
Sandy Metz, Expertin für Software-Design, weist darauf hin, dass die Kosten für das Beseitigen einer falschen Abstraktion weitaus höher sind als die Kosten für die Pflege von dupliziertem Code. Tatsächlich hat auch Dan Abramov davor gewarnt, dass übertriebene Code-Wiederverwendung die Flexibilität von Komponenten zerstört.
Probleme durch einfaches Kopieren von Code lassen sich in der Regel innerhalb von rund 50 Minuten in der jeweiligen Datei beheben. Voreilig erstellte gemeinsame Hooks oder Vererbungsstrukturen hingegen erzeugen schon bei minimalen Änderungen an den Anforderungen unzählige if-else-Verzweigungen. Am Ende dauert allein die Suche nach Bugs über zwei Wochen.
Die Entscheidung, ob refaktorisiert wird, stützt sich auf Martin Fowlers Quadranten der technischen Schulden und Adams Donahues Hotspot-Analyse.
| Bereich |
Kriterium (Komplexität und Änderungshäufigkeit) |
Vorgehensweise |
| Sofortige Korrektur |
Hoher geschäftlicher Einfluss × Hohe Änderungshäufigkeit |
Tests schreiben und Schnittstellen bereinigen unmittelbar nach Funktionsimplementierung, dann veröffentlichen |
| Optionale Korrektur |
Hoher geschäftlicher Einfluss × Geringe Änderungshäufigkeit |
Als Schulden-Ticket im Backlog registrieren |
| Direkt veröffentlichen |
Geringer geschäftlicher Einfluss × Hohe Änderungshäufigkeit |
Nur minimales Verhalten verifizieren und veröffentlichen, keine Abstraktion |
| Arbeit zurückstellen |
Geringer geschäftlicher Einfluss × Geringe Änderungshäufigkeit |
Korrekturen selbst bei unsauberem Code weglassen |
Ablauf der Code-Klassifizierung
- Prüfen, ob die aktuell bearbeitete Komponente zu einer Kernlogik (Hotspot) wie Zahlung oder Authentifizierung gehört oder eine einfache Promotionsansicht ist.
- Bei einer Promotionsansicht keinen gemeinsamen Custom Hook entwerfen, sondern inline innerhalb der Komponente schreiben.
- Code kopieren und einfügen, bis dasselbe UI-Verhalten mindestens dreimal auftritt.
Durch Anwendung dieser Kriterien lässt sich die Zeit für unnötige Verallgemeinerungsarbeiten reduzieren und die gesamte Entwicklungsgeschwindigkeit steigern.
Statische Analyse zur Reduzierung von Code-Review-Hinweisen
Stilistische Hinweise wie Variablenamen, Zeilenumbrüche oder Regelverstöße gegen den Linter sollten durch Tools herausgefiltert werden.
Laut einer Studie von SmartBear, die 2.500 Code-Reviews von Cisco Systems analysierte, werden 70 bis 90 Prozent der Fehler entdeckt, wenn der auf einmal überprüfte Code-Umfang unter 200 Zeilen liegt. Wenn der Autor als Kommentar im PR direkt den Grund für die Änderung hinterlässt, sinkt die Fehlerdichte im Durchschnitt um 30 Prozent.
Ablauf für statische Analyse und Selbstprüfung
- React- und TypeScript-Regeln in ESLint 9 Flat Config registrieren.
- Husky und lint-staged verknüpfen, um Formatierungsprüfungen zum Commit-Zeitpunkt zu erzwingen.
- Vor dem Erstellen des PRs sicherstellen, dass die reine Logik innerhalb von 300 Zeilen liegt, und Selbst-Review-Kommentare für komplexe Verzweigungen hinterlassen.
`javascript
// eslint.config.js
import js from "@eslint/js";
import globals from "globals";
import tseslint from "typescript-eslint";
import pluginReact from "eslint-plugin-react";
import pluginReactHooks from "eslint-plugin-react-hooks";
import eslintConfigPrettier from "eslint-config-prettier/flat";
export default tseslint.config(
{ ignores: ["dist/", "node_modules/", "build/"] },
js.configs.recommended,
...tseslint.configs.recommended,
pluginReact.configs.flat.recommended,
eslintConfigPrettier,
{
files: ["**/*.{js,jsx,ts,tsx}"],
languageOptions: {
ecmaVersion: "latest",
sourceType: "module",
globals: globals.browser,
},
plugins: {
"react-hooks": pluginReactHooks,
},
rules: {
"react/react-in-jsx-scope": "off",
"react/prop-types": "off",
"react-hooks/rules-of-hooks": "error",
"react-hooks/exhaustive-deps": "warn",
"@typescript-eslint/no-unused-vars": ["error", { argsIgnorePattern: "^_" }],
"@typescript-eslint/no-explicit-any": "warn",
},
settings: {
react: { version: "detect" },
},
}
);
`
Wird diese Konfiguration im Repository angewendet, lassen sich stilbezogene Review-Rückmeldungen reduzieren und der Fokus auf die eigentliche Logikprüfung legen.
Timeboxing zur Trennung von Implementierung und Bereinigung
Wenn man während der Funktionsentwicklung gleichzeitig die Struktur umbaut, geht der Arbeitskontext verloren. Einer Engineering-Studie von Google zufolge verkürzt sich die Wartezeit auf Code-Reviews von 48 auf 4 Stunden, wenn PRs klein aufgeteilt und die Arbeitsabsichten explizit angegeben werden.
Die tägliche Arbeitszeit wird in steuerbare Sitzungen unterteilt:
- 70 % des Arbeitstages fließen in die Funktionsimplementierung. Der Fokus liegt darauf, dass Bildschirmanforderungen und Datenfluss reibungslos funktionieren, wobei duplizierter Code erlaubt ist.
- 20 % des Arbeitstages dienen der Code-Bereinigung. Es werden lediglich Variablennamen im Scope angepasst, ungenutzte Imports bereinigt und Typen ergänzt – größere strukturelle Umbauten werden nicht angerührt.
- 10 % der wöchentlichen Arbeitszeit werden auf Freitagnachmittag für eine Sitzung zum Abbau von Schulden angesetzt. Nach Abschluss der Deployments wird die Struktur der im Backlog registrierten Hotspot-Module verbessert.
PR-Vorlage zur Angabe des Toleranzbereichs für Mängel
- Im Hauptteil des PRs die zugelassenen technischen Schulden aufführen.
- Bereiche, deren Funktionalität einwandfrei ist, bei denen das Refactoring jedoch aufgeschoben wurde, als Checkliste hinterlassen.
- Diese Punkte als Tickets für die Freitagssitzung registrieren und anschließend den PR einreichen.
`markdown
Überblick
- Arbeitsinhalt: Benutzerprofil bearbeiten und API-Integration
- Verwandtes Issue: #104
Akzeptierte Mängel und registrierte Schulden
- Akzeptierte Schulden: Duplizierte Stil-Logik in der Profilkomponente (unter dreimaliger Duplizierung)
- Akzeptierte Schulden: Alert-Verarbeitung bei Fehlerantwort (Integration der Toast-Komponente verschoben)
- Erforderlicher Review-Bereich: Laufzeitfehler und Geschäftsdaten-Verarbeitungslogik
`
Dies lenkt die Aufmerksamkeit der Reviewer weg von nebensächlichen Stilen hin zur Kernlogik und verkürzt die Freigabezeit.
Graduelle Bereitstellung zur Minimierung von Side Effects
Das Umschreiben von Alcode in einem einzigen großen Schritt erhöht das Risiko von Ausfällen in der Produktionsumgebung. Michael Feathers rät, bei der Modifikation von Legacy-Code zunächst Charakterisierungstests zu schreiben, die das aktuelle Ein- und Ausgabeverhalten fixieren, anstatt die gesamte Struktur umzubauen.
Um zu verhindern, dass bei einem Laufzeitfehler der gesamte Bildschirm weiß bleibt und einfriert, werden Error Boundaries eingerichtet.
`typescript
// components/ErrorBoundary.tsx
import React, { Component, ErrorInfo, ReactNode } from "react";
import * as Sentry from "@sentry/react";
interface Props {
children: ReactNode;
fallbackUI?: ReactNode;
}
interface State {
hasError: boolean;
}
export class GlobalErrorBoundary extends Component<Props, State> {
public state: State = { hasError: false };
public static getDerivedStateFromError(_: Error): State {
return { hasError: true };
}
public componentDidCatch(error: Error, errorInfo: ErrorInfo) {
Sentry.captureException(error, { extra: { componentStack: errorInfo.componentStack } });
}
public render() {
if (this.state.hasError) {
return (
this.props.fallbackUI || (
Beim Laden der Benutzeroberfläche ist ein Fehler aufgetreten.
Bitte aktualisieren Sie die Seite oder versuchen Sie es später noch einmal.
)
);
}
return this.props.children;
}
}
`
Ablauf der schrittweisen Bereitstellung
- Vor dem Anfassen der Legacy-Logik Tests anbringen, die den aktuellen Rückgabewert validieren, und die Schnittstelle mit einer Facade-Schicht umschließen.
- Den Status der API-Kommunikation in der Preview-Umgebung prüfen und neue Funktionen über Feature Flags zunächst einer kleinen Gruppe von Benutzern zugänglich machen.
- Übersteigt die Fehlerrate nach dem Deployment im Sentry-Monitoring 5 %, sofort den Rollback-Befehl ausführen.
`bash
git revert HEAD --no-edit
git push origin main
`
Arbeitseinheiten klein halten und eine Umgebung schaffen, in der man im Fehlerfall zurückkehren kann – das macht die Angst vor Code-Änderungen kontrollierbar. Ein funktionierendes System rechtzeitig zu liefern ist wichtiger als eine perfekte Struktur; das ist das Fundament praktischer Ingenieurskunst.