39. Combination Sum #56
Conversation
| for (int i = start; i < candidates.size(); i++) { | ||
| vector<int> added_combination = partial_combination; | ||
| added_combination.push_back(candidates[i]); | ||
| combination_state.push({added_combination, remain - candidates[i], i}); | ||
| } | ||
| } |
There was a problem hiding this comment.
インデントずれていますね。
一つ上のぶら下がり文のせいでしょうか。
https://docs.google.com/document/d/11HV35ADPo9QxJOpJQ24FcZvtvioli770WWdZZDaLOfg/edit?tab=t.0#heading=h.gblcuhn8bpdi
There was a problem hiding this comment.
@oda
レビューありがとうございます。該当箇所みました。
まさに事故を起こしておりました🙇
またこちらの纏めは60問解き終わってからきちんと目を通そうと思います。
| int partial_sum = 0; | ||
| for (int num : partial_candidate) { | ||
| partial_sum += num; | ||
| } |
There was a problem hiding this comment.
@oda
sum + candidates[i] を使った方が素直ですね🙇♂️
再帰から戻ってきた際に、sum -= candidates[i];をしてsumで動くようにしました。
for (int i = start; i < candidates.size(); i++) {
partial_candidate.push_back(candidates[i]);
sum += candidates[i];
GenerateCombinationToTarget(all_candidates, partial_candidate, candidates, target, i, sum);
partial_candidate.pop_back();
sum -= candidates[i];
}
There was a problem hiding this comment.
あら、sum を更新せずに、新しい変数にするか引数に直接和を渡せば、引かなくてもいいのではないでしょうか。
There was a problem hiding this comment.
@oda
sumを足したり引いたりする方が、バックトラッキングぽいのかなと感じました。
実際は、回答になるpartial_candidateだけを増やしたり減らしたりすることかと思いますが。
| public: | ||
| vector<vector<int>> combinationSum(vector<int>& candidates, int target) { | ||
| vector<vector<int>> all_combinations; | ||
| stack<CombinationAndTargetAndIndex> combination_state; |
There was a problem hiding this comment.
targetとremainって意味的に違う気がして、こっちの名称はremainですかね？
また、C++のstructの名称の付け方の慣習が自分よくわかってないのですが、3つの要素をただandで繋げるのはデータ定義見ればわかるよ、と思ってしまいがちなのですが、どうなんでしょう、、、？
There was a problem hiding this comment.
@nittoco
レビューありがとうございます。
3つの要素をただandで繋げるのはデータ定義見ればわかるよ、と思ってしまいがちなのですが、どうなんでしょう、、、？
慣習とかはないと思いあます🙇♂️このデータ構造を表すいい名前が思いつかずとりあえず入っているものを繋げました。
いっそのこと無理にstructを使わず、stackを３つ用いた方が分かりやすいですかね?
There was a problem hiding this comment.
うーん難しいですね、stackが3つなのもそれはそれで煩雑そうですし(はっきりしてなくてすみません)
| struct CombinationAndTargetAndIndex { | ||
| vector<int> combination; | ||
| int remain; | ||
| int index; |
There was a problem hiding this comment.
どういうindexかの情報が個人的には欲しいかもです(appending_indexとか？）
問題へのリンク
https://leetcode.com/problems/combination-sum/description/
問題文(プレミアムの場合)
備考
次に解く問題の予告
Generate Parentheses
フォルダ構成
LeetCodeの問題ごとにフォルダを作成します。
フォルダ内は、step1.cpp、step2.cpp、step3.cpp、loop.cppとmemo.mdとなります。
memo.md内に各ステップで感じたことを追記します。