aalhour/C-Sharp-Algorithms

IsAnagram logic needs to be fixed

クローズ

#11 opened on 2015/08/20

GitHub で見るコントリビューターガイド
 (1 件のコメント) (0 件のリアクション) (1 人の担当者)C# (1,409 件のフォーク)batch import
buggood first issue

Repository metrics

Stars
 (5,736 個のスター)
PR merge metrics
 (30d に merged PR はありません)

説明

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.

コントリビューターガイド

調査方針
現在のIsAnagramの実装を確認し、欠陥を特定し、OrderByとSequenceEqualまたは頻度辞書を使用したLINQベースのソリューションに置き換えます。
技術スタック
csharp
領域
backend
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
C#LINQ
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
75