aalhour/C-Sharp-Algorithms

IsAnagram logic needs to be fixed

Fermée

#11 ouverte le 20 août 2015

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (1 personne assignée)C# (1 409 forks)batch import
buggood first issue

Métriques du dépôt

Stars
 (5 736 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez l'implémentation actuelle de IsAnagram, identifiez le défaut et remplacez la par une solution LINQ utilisant OrderBy et SequenceEqual ou un dictionnaire de fréquence.
Stack technique
csharp
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#LINQ
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75