49 group anagrams by MA-yo-TA · Pull Request #13 · MA-yo-TA/leetcode

MA-yo-TA · 2026-06-17T12:59:50Z

この問題：https://leetcode.com/problems/group-anagrams/
次の問題：https://leetcode.com/problems/intersection-of-two-arrays/

h-masder · 2026-06-17T15:14:45Z

+
+## step1（まず通す）
+
+「文字を入れ替えて同じになる」という同値関係に基づいて同値類に分割するという話。


正確な説明ですが、同値関係や同値類あたりはソフトウェアエンジニアの世界では普段から使う言葉とは感じないので、避けてもいいかもしれません。

「並べ替えると同じになる文字列を同じグループにまとめる」でいいかと思います。

そうですね、自分の思考過程をメモ書き程度に書いたつもりでしたが他の方も見るので言葉遣いは周りに合わせたほうが良さそうです。

↓のような説明だとだいぶマシでしょうか。
skypenguins/coding-practice#32 (comment)

いいと思います。

効率の良さを議論するときは、それによって何が嬉しいのかも意識すると良いと思います。想像の域は出ませんが、例えば、前者の方法で実装されたプログラムが既に動いているとして、後者へ変更する価値があるのかを考えてみるとよいと思います。実行時間を見積もる意味も、そういった判断材料を得るところにあると私は思っています。

例えば、前者の方法で実装されたプログラムが既に動いているとして、後者へ変更する価値があるのかを考えてみるとよいと思います。実行時間を見積もる意味も、そういった判断材料を得るところにあると私は思っています

まさにそうですね、困ってない（し、これからも困ることはなさそう）なら触らないというのは実務では多いですよね。

実践的なアドバイスありがとうございます！

h-masder · 2026-06-17T15:21:47Z

+class Solution:
+    def groupAnagrams(self, strs: list[str]) -> list[list[str]]:
+        sorted_to_originals: dict[str, list[str]] = {}
+        for string in strs:


~~stringは型名のように感じるので避けたいです。~~
(↑こちらstep2に言及されていました。失礼しました。)
wordなどはいかがでしょうか

ありがとうございます。word もありかなと思いつつ、"" みたいな自然言語の感覚で言うと word じゃない奴もいるよな、みたいな細かいところを気にしてました。

今考えてみると割り切って word は良い名前な気がしてます。

"" みたいな自然言語の感覚で言うと word じゃない奴もいるよな、みたいな細かいところを気にしてました。

おっしゃる通り、場合によってはword以外の変数名も考えたほうが良いかもしれないですね。

h-masder · 2026-06-17T15:27:54Z

+        sorted_to_originals: dict[str, list[str]] = {}
+        for string in strs:
+            sorted_string = str(sorted(string))
+            if sorted_string in sorted_to_originals:


6~10行目はdefalutdictを使うこともできます。
利点は、行数が短くなる、「存在しないキーには空のリストを作り、そこへ要素を追加していく処理」が想像しやすい、あたりです。

from collections import defaultdict class Solution: def groupAnagrams(self, strs: list[str]) -> list[list[str]]: sorted_to_originals = defaultdict(list) for string in strs: sorted_string = str(sorted(string)) sorted_to_originals[sorted_string].append(string) return list(sorted_to_originals.values())

ありがとうございます。 defaultdict をあまり認識していなかったので調べておきます。

h-masder · 2026-06-17T15:30:07Z

+- `frozenset` のように hashable な dict はないのかということが気になるが、調べたら 3.15 から `frozendict` が組み込み型に追加されるらしい。
+  - <https://docs.python.org/ja/3.15/library/stdtypes.html#frozendict>
+  - 現在の安定版の最新は 3.14 なのでまだ一般的に使える感じではなさそう
+


アルゴリズムの実行時間を見積もることをおすすめします。
Yuto729/leetcode#16 (comment)

ありがとうございます。

今回の場合は配列長が最大 10 ^ 4 , 文字列一つ一つの長さが最大 100 なので、
(10 ^ 4 * 100 * log 100) / 10 ^ 7 ~ 0.6 s くらいという計算をして、実際と大分ずれているなという印象ですね、、（leetcode だと 20 ms くらいで終わっている）

（テストケースが最大の大きさのものがないとかかもしれないので最終的には実際に計らないとわからないのだろうと思いつつ、見積もりが雑とか計算間違ってるかもしれません。）

10^4 や 100 * log 100がどこからきているか書いておくと良いと思います
（前者がメモリアクセス、後者がsorted()ですね）
sorted()を使うのであれば、内部実装として、何が使われているか知っておくとよいと思います。以下のあたりを見る限りtimsortのようです。
https://github.com/python/cpython/blob/main/Objects/listsort.txt
https://github.com/python/cpython/blob/main/Objects/listobject.c#L3122

テストケースが最大の大きさのものがないとかかもしれないので最終的には実際に計らないとわからないのだろうと思いつつ

はい、テストケースでだいぶ変わると思います。ゆくゆくはleetcodeにたよらず、自分でテストケースを書く癖をつけていくとよいかと思います。（癖というより、自分でテストケースを書きたくなる気持ちを養うといったほうがいいかもしれないです）

10^4 や 100 * log 100がどこからきているか書いておくと良いと思います

省略しすぎました。サボらず伝わる書き方を心がけます。

自分でテストケースを書きたくなる気持ちを養う

ありがとうございます。まずは想定できるケースのバリエーションを増やす意識で取り組みます。

h-masder · 2026-06-17T15:32:12Z

+
+## step2（整形＆他の人のコードを読む）
+
+`sorted_string` は `sorted_` とかでもいいかも。今回は基本 string の話しかしてなくて string であることは明らかなので。


関数名と紛らわしくなるため、同じ（もしくはほとんど似た）名前を変数名に使うのは避けたほうが良いと思います。元の sorted_string の方が意図が分かりやすいと感じました。

そうですね、書いているうちにそう思うようになりました。
ありがとうございます。

MA-yo-TA added 5 commits June 17, 2026 08:50

step1

af327e6

update note

c232664

step2

6e62d05

modify variable name

1d97299

step3

16a25f8

h-masder reviewed Jun 17, 2026

View reviewed changes


		## step1（まず通す）

		「文字を入れ替えて同じになる」という同値関係に基づいて同値類に分割するという話。


		## step2（整形＆他の人のコードを読む）

		`sorted_string` は `sorted_` とかでもいいかも。今回は基本 string の話しかしてなくて string であることは明らかなので。

Conversation

MA-yo-TA commented Jun 17, 2026

Uh oh!

h-masder Jun 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

MA-yo-TA Jun 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

h-masder Jun 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

h-masder Jun 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

h-masder Jun 17, 2026 •

edited

Loading

MA-yo-TA Jun 18, 2026 •

edited

Loading

h-masder Jun 17, 2026 •

edited

Loading

h-masder Jun 18, 2026 •

edited

Loading