Document appropriate implementation of `GetHashCode`.
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 35/100
Rechercherichtung
Im Issue werden keine Dateien oder Tests genannt. Lokalisieren Sie die bestehende Implementierung des Repositorys für Warnungen im Zusammenhang mit GetHashCode und untersuchen Sie anschließend, wie die Veränderlichkeit von Membern und virtuelle Member analysiert werden; als abgeschlossen gilt die Aufgabe, wenn ein von GetHashCode verwendetes virtuelles Member eine Warnung erzeugt, ohne das bestehende Verhalten zu beeinträchtigen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
VS and Rider both flag warnings when you use a property in a GetHashCode implementation that is not read-only, however, I recently encountered an issue where one of the properties was virtual. This allowed the property to change in a derived class and alter the base class hash code leading to a failed dictionary lookup.
Solution: flag as warning when virtual member is used in GetHashCode
- Vorherrschende Sprache
- C#
- Sterne
- 12
- Forks
- 16
- Ø Merge
- 4 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
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 45/100
IntelliTect/CodingGuidelines#253 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Review naming recommendation for fieldsEvtl. wieder frei @MarkMichaelis hat das vor 1331 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
Alle Issues in IntelliTect/CodingGuidelines
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Volodymyr-Petrunin/Bankomaten#45 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
AvaloniaUI/Avalonia#22420 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
dotnet/SqlClient#4823 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 2 Tagen
-
type/automation type/tech-debt
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag
-
area-testing p1-recommended
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
Maintainer antworten meist innerhalb von 1 Tag