198. House Robber

[nittoco]

URL: https://leetcode.com/problems/house-robber/description/

You are a professional robber planning to rob houses along a street. Each house has a certain amount of money stashed, the only constraint stopping you from robbing each of them is that adjacent houses have security systems connected and it will automatically contact the police if two adjacent houses were broken into on the same night.

Given an integer array nums representing the amount of money of each house, return the maximum amount of money you can rob tonight without alerting the police.

[hayashi-ay]

分かります。まあDP全般そうですよね〜。

[hayashi-ay]

resultだとあまり意味を含んでいないので他の命名が良いかなと思いました。max_amount_so_farとか、のようにresultの部分をもう少しカラフルにすると良いかなと思います。

[hayashi-ay]

自分なら二次元配列にしないで、インデックスで表される家に盗みに入るのと入らないのと2つの配列を作るかなと思いました。

[hayashi-ay]

なんか変数名が全体的にふわっとしていて読むのが難しかったです。たぶんrecentlyというワードチョイスも良くないと思います、なにを指しているのかが曖昧なので、結局コードを全部読む必要がありました。

[nittoco]

ありがとうございます。recentlyは実態と違っており良くなかったと今改めて見て思いました。lastの方が適切でした。
あと2つはmax_amount_so_farと、max_amount_prevですかねえ。max_amount_prevが特に、これでわかりやすいかが迷います。

[oda]

skipped_previous, robbed_previous とかどうでしょうか。

[liquo-rice]

[1]を2変数で書き直すと[2]になります。@nittocoさんの方法と同じだと思います。多重代入を使わないと一時変数が必要になります。[3]は変数の意味が、元の@nittocoさんの方法とは少し異なります。
[1]

class Solution {
public:
    int rob(vector<int>& nums) {
        if (nums.size() == 1) {
            return nums[0];
        }
        nums[1] = max(nums[0], nums[1]);
        for (int i = 2; i < nums.size(); ++i) {
            nums[i] = max(nums[i - 1], nums[i] + nums[i - 2]);
        }
        return nums.back();
    }
};

[2]

class Solution {
public:
    int rob(vector<int>& nums) {
        int max_amount = 0, previous_max_amount = 0;
        for (int num : nums) {
            int next_previous_max_amount = max_amount;
            max_amount = max(max_amount, previous_max_amount + num),
            previous_max_amount = next_previous_max_amount;
        }
        return max_amount;
    }
};

[3]

class Solution {
public:
    int rob(vector<int>& nums) {
        int skipped_previous = 0;
        int robbed_previous = 0;
        for (int num : nums) {
            int next_skipped_previous = max(skipped_previous, robbed_previous);
            robbed_previous = skipped_previous + num;
            skipped_previous = next_skipped_previous;
        }
        return max(skipped_previous, robbed_previous);
    }
};

[nittoco]

皆さんありがとうございます。
そうですよね、skipped_previous, robbed_previous という変数名だと、Step3で書いたコードだと違ってくるかなと思いました(Step1を2変数にしたのに近い)
max_amount, previous_max_amount, skipped_previousあたりですかね(難しい)

[thonda28]

max_amount_can_rob という変数名だと「max amount（主語）は奪うことができる」という風に読めるため、個人的には違和感がありました。（can が使われていると bool をイメージします）
max_robbed_amount や max_robbery_amount くらいで、can の要素を明示的に入れなくても伝わりそうです。

また2次元配列の部分ですが、クラス変数を使って index を扱うのは（少なくとも僕は）見たことがないです。初見で何をしているかがわかりづらかったので、避けたほうがよさそうです。hayashi-ay さんが書いているように2つに分けるか、クラスを定義してあげるとわかりやすくなりそうです。（クラス定義はやり過ぎ感があるかもですが）

[nittoco]

max_amount_can_rob という変数名だと「max amount（主語）は奪うことができる」という風に読めるため、個人的には違和感がありました。（can が使われていると bool をイメージします） max_robbed_amount や max_robbery_amount くらいで、can の要素を明示的に入れなくても伝わりそうです。

なるほど、確かにそう読めますね。
なかなかまだ読む人の心のモデルを構築できないです(英語力の問題もあるかも)

また2次元配列の部分ですが、クラス変数を使って index を扱うのは（少なくとも僕は）見たことがないです。初見で何をしているかがわかりづらかったので、避けたほうがよさそうです。hayashi-ay さんが書いているように2つに分けるか、クラスを定義してあげるとわかりやすくなりそうです。（クラス定義はやり過ぎ感があるかもですが）

なるほど、「見たことない」のはかなり判断の基準になります。ありがとうございます。

[TORUS0818]

配列に違和感を感じる理由を考えていたんですけど、順序に意味がないからなのかなと思いました。
例えば今回の場合numsは泥棒に入ろうとしている順に家のお金が並んでいますが、max_amount_can_robの要素の配列の中身の順序は、特に意味がないのかなと思います。

もっと極端な例を考えると、順序のない状態をn個考えて、それらを配列で管理すると、配列のインデックスとの対応が任意（書く人の決めの問題）なので、かなり分かりにくくなるんじゃないでしょうか。
要は、マジックナンバーを避けるために変数を定義するくらいなら、最初から辞書やデータクラスでキーアクセスする方が自然なんじゃないかなと。

[nittoco]

これ納得感がありすっきりしました。ありがとうございます。
0, 1のDPというのはAtcoderとかだとテクニックとしてあるんですが、実用コード書く時は少し慎重にしてもいいかもですね。

[thonda28]

passed_house_count は変数名の長さに対してわかりやすくはなっていない（むしろ読みにくくなっている）ように思いました。「通過した軒数」≒「何番目の軒数」と言えそうなので、個人的には i で十分な気がします。（for ループの時点で nums を順番に見ていくことはわかりそうです）

[nittoco]

ありがとうございます。
そうですねえ、ただ for i in range(nums) は、問題文の意味からちょっとわかりにくい気が個人的にしてます。
LeetCode側の問題ですが、numsが良くない気がします(house_to_money_amountとかならiでわかる気がします。

[Yoshiki-Iwasa]

Yoshiki-Iwasa/Arai60#50 (comment)

このodaさんのコメント、自分はしっくり来ました。
あとから他のDPを解き直すときもリアルな情景をイメージすると自然に解法までたどり着く印象があります

[nittoco]

ありがとうございます。
そうですね。確かに、リアルな状況を思い浮かべると、変数名も付けやすい気がしてます。(自分も変数名なかなかうまくつけれてないですが…)

[Mike0121]

見ました。自分もすでに議論済みの変数名の部分が気になりましたが、それ以外は良いと思いました。