aalhour/C-Sharp-Algorithms

IsAnagram logic needs to be fixed

Fechada

#11 aberto em 20 de ago. de 2015

Ver no GitHubGuia do colaborador
 (1 comentário) (0 reação) (1 responsável)C# (1.409 forks)batch import
buggood first issue

Métricas do repositório

Stars
 (5.736 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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.

Guia do colaborador

Direção de pesquisa
Revise a implementação atual de IsAnagram, identifique a falha e substitua por uma solução baseada em LINQ usando OrderBy e SequenceEqual ou um dicionário de frequência.
Pilha de tecnologia
csharp
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#LINQ
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75