Document appropriate implementation of `GetHashCode`.
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
Piste de recherche
Aucun fichier ni test n’est nommé dans l’issue. Repérez l’implémentation existante du dépôt pour les avertissements concernant GetHashCode, puis examinez comment sont analysés la mutabilité des membres et les membres virtuels ; le travail est terminé lorsqu’un membre virtuel utilisé par GetHashCode produit un avertissement sans modifier le comportement existant.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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
- Langage dominant
- C#
- Étoiles
- 12
- Forks
- 16
- Merge moyen
- 4 min
- PR mergées (30 j)
- 12
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de IntelliTect/CodingGuidelines
-
Overriding ToString()Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 25/100
IntelliTect/CodingGuidelines#290 ·
Les mainteneurs répondent en général sous 1 jour
-
proposal: ban List<T>.ForEachOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
IntelliTect/CodingGuidelines#253 ·
Les mainteneurs répondent en général sous 1 jour
-
Review naming recommendation for fieldsPeut-être à nouveau libre @MarkMichaelis l’a pris il y a 1332 jours, et aucune pull request n’est ouverte. Ouverte.editorconfig C# coding guidelines proposal
IntelliTect/CodingGuidelines#249 · 6 commentaires · 1 réaction · 2 personnes assignées ·
Les mainteneurs répondent en général sous 1 jour
-
INTL0001 when using recordsOuverteanalyzer C# coding guidelines proposal
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
IntelliTect/CodingGuidelines#231 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
IntelliTect/CodingGuidelines#230 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de IntelliTect/CodingGuidelines
Issues similaires
-
再現済み 要トリアージ 誤判定
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
yksr-melt/Meltype#421 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
Facepunch/sbox-public#12063 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
facioquo/stock-indicators-dotnet#2316 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
eriknihlen/OpenAC#219 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
ObsidianMC/Obsidian#548 ·
Les mainteneurs répondent en général sous 1 jour