139. Word Break by hemispherium · Pull Request #37 · hemispherium/LeetCode_Arai60

hemispherium · 2026-04-30T04:28:53Z

この問題：https://leetcode.com/problems/word-break/description/
次に解く問題：https://leetcode.com/problems/coin-change/description/

nodchip · 2026-05-01T09:40:40Z

+class Solution {
+public:
+    bool FindWordRecursively(string s, vector<string>& wordDict, unordered_map<string, bool>& memo) {
+        if (s.size() == 0) {


if (s.empty()) {

のほうがシンプルだと思います。
https://cpprefjp.github.io/reference/string/basic_string/empty.html

ありがとうございます。そちらの方がシンプルそうです。

nodchip · 2026-05-01T09:41:34Z

+            return true;
+        }
+
+        if (memo.count(s)) {


C++20 以降なら std::unordered_map::contains() を使うことをお勧めいたします。
https://cpprefjp.github.io/reference/unordered_map/unordered_map/contains.html

ありがとうございます。こちらのメンバ関数は知りませんでした。

こちらですが過去のPRで複数回同じ指摘があったようです。同じ指摘を受けないように、次回以降のPRで反映させるとより効果的な学習になると思います。

#16 (comment)
#3 (comment)

失礼しました。気をつけます。

nodchip · 2026-05-01T09:44:20Z

@@ -0,0 +1,28 @@
+class Solution {
+public:
+    bool FindWordRecursively(string s, vector<string>& wordDict, unordered_map<string, bool>& memo) {


memo という変数名に、あまり情報がないように思いました。ある接頭辞が breakable かどうかを表す map ですので、 prefix_to_breakable はいかがでしょうか？

ありがとうございます。そちらの名前にしようと思います。

nodchip · 2026-05-01T09:44:48Z

@@ -0,0 +1,28 @@
+class Solution {
+public:
+    bool FindWordRecursively(string s, vector<string>& wordDict, unordered_map<string, bool>& memo) {


string を値渡しすると、毎回コピーが発生します。 const 参照で渡すことをお勧めいたします。

承知しました！

nodchip · 2026-05-01T09:46:06Z

@@ -0,0 +1,28 @@
+class Solution {
+public:
+    bool FindWordRecursively(string s, vector<string>& wordDict, unordered_map<string, bool>& memo) {


unordered_map<string, bool> より、 std::vector として、何文字目以降の接頭辞は breakable かを保持するようにしたほうが、ハッシュの計算が省略でき、軽くなると思います。未調査・breakable・unbreakable の 3 つの値を格納することになると思います。

ありがとうございます！こちらの実装alternative-solution.cppで追加してみました。

liquo-rice · 2026-05-01T14:39:27Z

@@ -0,0 +1,11 @@
+### step1
+
+再帰的に辞書からワードを探す形で実装。TLEしたのでメモ化する部分だけChatGPTに書いてもらった。


DPでも書けますか？

liquo-rice · 2026-05-01T14:42:04Z

@@ -0,0 +1,27 @@
+class Solution {
+public:


アクセス修飾子を意識して使っていますか？このコードでは、どう変更するのが適切そうでしょうか？

参考：#23 (comment)

あまり意識していませんでした。このコードでは再帰関数は公開しなくていいと思うのでprivateにするのがよいでしょうか？

class Solution { private: bool FindWordRecursively(string s, const vector<string>& wordDict, unordered_map<string, bool>& memo) { if (s.size() == 0) { return true; } if (memo.count(s)) { return memo[s]; } for (int i = 0; i < wordDict.size(); i++) { if (s.starts_with(wordDict[i])) { if (FindWordRecursively(s.substr(wordDict[i].size()), wordDict, memo)) { return memo[s] = true; } } } return memo[s] = false; } public: bool wordBreak(string s, vector<string>& wordDict) { unordered_map<string, bool> memo; return FindWordRecursively(s, wordDict, memo); } };

はい、そうですね。

5ky7 · 2026-05-04T05:44:25Z

+            return memo[s];
+        }
+
+        int size = wordDict.size();


wordDict.size()のままで良いと思います．
sizeは1箇所しか用いず，また可読性の面でも大差ないと思うので．

そうですね、特に変数化しなくてよかったかもしれないです。

5ky7 · 2026-05-04T05:46:24Z

+        int size = wordDict.size();
+        for (int i = 0; i < size; i++) {
+            if (s.starts_with(wordDict[i])) {
+                if (FindWordRecursively(s.substr(wordDict[i].size()), wordDict, memo)) {


string型に対するsubstr関数は都度コピーを生成することになり遅いので，string_viewに変換して扱ってはいかがでしょうか．

ありがとうございます！string_viewは知りませんでした。

5ky7 · 2026-05-04T05:51:22Z

+        for (int i = 0; i < size; i++) {
+            if (s.starts_with(wordDict[i])) {
+                if (FindWordRecursively(s.substr(wordDict[i].size()), wordDict, memo)) {
+                    return memo[s] = true;
+                }
+            }
+        }


実際には最悪計算量よりも平均的な計算量のほうが重要でしょうが，今回はそれがわからないので最悪計算量で考えると，今回のケースでは

wordDictのサイズが最大で2000

sのサイズは最大20

ですからwordDictを操作するよりもwordDictをsetかunordered_setに変換してsの部分文字列を操作する方が最悪計算量が小さく抑えられます．

add files

42d3700

hemispherium self-assigned this Apr 30, 2026

nodchip reviewed May 1, 2026

View reviewed changes

liquo-rice reviewed May 1, 2026

View reviewed changes

5ky7 reviewed May 4, 2026

View reviewed changes

add alternative solution

377658c

		@@ -0,0 +1,11 @@
		### step1

		再帰的に辞書からワードを探す形で実装。TLEしたのでメモ化する部分だけChatGPTに書いてもらった。

Conversation

hemispherium commented Apr 30, 2026

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!

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!

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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants