Merge pull request #5 from rihib/valid_anagram

Valid Anagram

Author: rihib

pullrequests/valid_anagram/step1.go

@@ -0,0 +1,45 @@
+//lint:file-ignore U1000 Ignore all unused code
+package validanagram
+
+/*
+かなり前に解いたものなので、詳細については忘れてしましました。
+小文字のアルファベットのみが入力として与えられる場合と、Unicode文字も含まれる場合の２つの解法が含まれています。
+
+基本的にはそれぞれの文字列に含まれる文字の種類と数が一致しているかを見ています。
+下の解法について、40行目で0未満になった場合のみfalseを返すのは一見おかしいように見えますが、30行目で文字数が等しいことを確認しているため、一方より多い文字があれば何かの文字は少なくなるはずであるため、ロジック的には問題ないと思います。
+*/
+func isAnagramStep1(s string, t string) bool {
+	if len(s) != len(t) {
+		return false
+	}
+
+	var freq [26]int
+	for i := 0; i < len(s); i++ {
+		freq[s[i]-'a']++
+		freq[t[i]-'a']--
+	}
+	for _, v := range freq {
+		if v != 0 {
+			return false
+		}
+	}
+	return true
+}
+
+func isAnagramUnicodeStep1(s string, t string) bool {
+	if len(s) != len(t) {
+		return false
+	}
+
+	freq := make(map[rune]int)
+	for _, r := range s {
+		freq[r]++
+	}
+	for _, r := range t {
+		freq[r]--
+		if freq[r] < 0 {
+			return false
+		}
+	}
+	return true
+}

pullrequests/valid_anagram/step2.go

@@ -0,0 +1,41 @@
+//lint:file-ignore U1000 Ignore all unused code
+package validanagram
+
+/*
+よりシンプルにわかりやすく書くようにしました。例えばfreqのように省略するのではなくfrequencyにしました。
+
+上については、Goの場合、s[i]のようにアクセスするとバイト列にアクセスすることになるので、今回の場合はたまたまascii文字しか入力として与えられないので問題ないのですが、あまりよろしくないと思ったので文字としてアクセスするように変更しました。
+
+下の解法についてはStep1のロジックがわかりづらかったので、より明確になるように変更しました。
+*/
+func isAnagramStep2(s string, t string) bool {
+	var frequency [26]int
+	for _, r := range s {
+		frequency[r-'a']++
+	}
+	for _, r := range t {
+		frequency[r-'a']--
+	}
+	for _, n := range frequency {
+		if n != 0 {
+			return false
+		}
+	}
+	return true
+}
+
+func isAnagramUnicodeStep2(s string, t string) bool {
+	frequency := make(map[rune]int)
+	for _, r := range s {
+		frequency[r]++
+	}
+	for _, r := range t {
+		frequency[r]--
+	}
+	for _, n := range frequency {
+		if n != 0 {
+			return false
+		}
+	}
+	return true
+}

pullrequests/valid_anagram/step3.go

@@ -0,0 +1,32 @@
+//lint:file-ignore U1000 Ignore all unused code
+package validanagram
+
+func isAnagramStep3(s string, t string) bool {
+	return frequency(s) == frequency(t)
+}
+
+func frequency(s string) [26]int {
+	var f [26]int
+	for _, r := range s {
+		f[r-'a']++
+	}
+	return f
+}
+
+// ちゃんとUnicodeに対応させるなら結合文字などを考慮する必要がある
+// https://github.com/rihib/leetcode/pull/5#discussion_r1706198268
+func isAnagramUnicodeStep3(s string, t string) bool {
+	frequency := make(map[rune]int)
+	for _, r := range s {
+		frequency[r]++
+	}
+	for _, r := range t {
+		frequency[r]--
+	}
+	for _, n := range frequency {
+		if n != 0 {
+			return false
+		}
+	}
+	return true
+}

pullrequests/valid_anagram/step4.go

@@ -0,0 +1,32 @@
+//lint:file-ignore U1000 Ignore all unused code
+package validanagram
+
+func isAnagramStep4(s string, t string) bool {
+	return frequencies(s) == frequencies(t)
+}
+
+func frequencies(s string) [26]int {
+	var f [26]int
+	for _, r := range s {
+		f[r-'a']++
+	}
+	return f
+}
+
+// ちゃんとUnicodeに対応させるなら結合文字などを考慮する必要がある
+// https://github.com/rihib/leetcode/pull/5#discussion_r1706198268
+func isAnagramUnicodeStep4(s string, t string) bool {
+	frequencies := make(map[rune]int)
+	for _, r := range s {
+		frequencies[r]++
+	}
+	for _, r := range t {
+		frequencies[r]--
+	}
+	for _, n := range frequencies {
+		if n != 0 {
+			return false
+		}
+	}
+	return true
+}