dotnet/runtime

Prefer Dictionary.Remove(key, out value) over Dictionary.this[key], followed by Dictionary.Remove(key)

Open

#33.800 aperta il 19 mar 2020

Vedi su GitHub
 (7 commenti) (0 reazioni) (0 assegnatari)C# (5445 fork)batch import
api-approvedarea-System.Collectionscode-analyzercode-fixerenhancementhelp wanted

Metriche repository

Star
 (17.886 star)
Metriche merge PR
 (Merge medio 12g 11h) (661 PR mergiate in 30 g)

Descrizione

Dictionary.ContainsKey<K, V(key), followed by Dictionary<K, V.this[key], followed by Dictionary<K, V>.Remove(key) can be combined into a single Remove call, using the overload accepting out TValue value.

Category: Performance

Guida contributor

Direzione di ricerca
Identifica gli usi del pattern `if (dict.ContainsKey(key)) { var v = dict[key]; dict.Remove(key); ... }` e sostituisci con `if (dict.Remove(key, out var v)) { ... }`.
Tech stack
csharp
Dominio
backend
Tipo issue
Performance
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#.NET
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75