`Erasing state when props change` shouldn't be called "Anti-pattern"
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 11.8k
- Forks
- 7.9k
- Ø Merge
- 1 T. 11 Std.
- Gemergte PRs (30 T.)
- 11
Beschreibung
If we rename mail to defaultMail in that example, I think the component's behavior makes sense. And this pattern also has no conflict with Alternative 1 and Alternative 2 both. Even it's more convenient, because we don't have to update id or key when passing a new default value.
In fact， this pattern doesn't cause any new problem.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem Dokumentationsbeispiel mit dem Titel "Erasing state when props change" und vergleiche seine Formulierung mit Alternative 1 und Alternative 2. Prüfe, ob das Beispiel defaultMail verwenden sollte und ob es korrekt ist, das Muster als Anti-Pattern zu bezeichnen; als erledigt gilt die Aufgabe, wenn Beispiel und Anleitung die vereinbarte Terminologie widerspiegeln.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, react
- Bereich
- documentation
- Issue-Typ
- Dokumentation
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100