aalhour/C-Sharp-Algorithms

IsAnagram logic needs to be fixed

Chiusa

#11 aperta il 20 ago 2015

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (1 assegnatario)C# (1409 fork)batch import
buggood first issue

Metriche repository

Star
 (5736 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

I think the IsAnagram() method doesn't work as expected. For example, it would fail for the below string combinations ("abc","bbb") or ("acdf", "bcde").

I think a simple LINQ implementation would be easy enough to implement here. If you like, I could do that and also implement unit tests for it.

Guida contributor

Direzione di ricerca
Esamina l'implementazione corrente di IsAnagram, identifica il difetto e sostituiscila con una soluzione LINQ che utilizza OrderBy e SequenceEqual o un dizionario di frequenza.
Tech stack
csharp
Dominio
backend
Tipo issue
Bug
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#LINQ
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75