proposal: ban List<T>.ForEach
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 45/100
- Issue-Typ
- Dokumentation
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- csharp
- Bereich
- documentation
Rechercherichtung
Beginne damit, den Eintrag oder das Dokument in den Coding-Richtlinien zu finden, das List.ForEach behandelt, und überprüfe, wie ähnliche Konventionen dokumentiert sind. Bestätige das vorgeschlagene Verbot und seine Begründung anhand des verlinkten Artikels von Eric Lippert; als erledigt gilt die Aufgabe, wenn die Richtlinie stattdessen eindeutig foreach empfiehlt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
List.ForEach is yucky and there's no scenario where it should be used instead of a foreach loop.
Eric Lippert agrees: https://ericlippert.com/2009/05/18/foreach-vs-foreach/
- Vorherrschende Sprache
- C#
- Sterne
- 12
- Forks
- 16
- Ø Merge
- 5 Min.
- Gemergte PRs (30 T.)
- 12
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus IntelliTect/CodingGuidelines
-
Overriding ToString()Offen
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 25/100
IntelliTect/CodingGuidelines#290 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Review naming recommendation for fieldsEvtl. wieder frei @MarkMichaelis hat das vor 1335 Tagen übernommen, und es ist kein Pull Request offen. Offen.editorconfig C# coding guidelines proposal
IntelliTect/CodingGuidelines#249 · 6 Kommentare · 1 Reaktion · 2 zugewiesene Personen ·
Maintainer antworten meist innerhalb von 1 Tag
-
analyzer C# coding guidelines proposal
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
IntelliTect/CodingGuidelines#231 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
IntelliTect/CodingGuidelines#230 ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 42/100
IntelliTect/CodingGuidelines#229 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in IntelliTect/CodingGuidelines
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
Maintainer antworten meist innerhalb von 1 Tag
-
area:re1 bounty help wanted needs-game
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
StackExchange/StackExchange.Redis#3274 ·
Maintainer antworten meist innerhalb von 1 Tag
-
affected/v17 affected/v18 area/frontend type/bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
umbraco/Umbraco-CMS#24186 ·
Maintainer antworten meist innerhalb von 2 Tagen