322. Coin Change

[potrue]

問題文
https://leetcode.com/problems/coin-change/description/

[nodchip]

範囲 for 文でプリミティブ型変数で回すときは、 const は付けないことが多いように思います。好みの問題かもしれません。

[nodchip]

真偽を反転して continue し、ネストを下げたほうが読みやすくなると思います。

[nodchip]

fewest_coins を -1 ではなく大きな数字で初期化すると、 if 文が消せ、

fewest_coins[i + coin] = min(fewest_coins[i + coin], fewest_coins[i] + 1);

だけで済むようになり、シンプルになると思います。

[tokuhirat]

i よりは合計金額という意味がわかる変数名だとわかりやすいと思いました。

[potrue]

本当はamountという変数名をつけたかったんですが、もともとの関数の引数の名前に取られてたのでiにしました。
ただ今考えたら勝手に変数名変えても別にleetcodeの実行上は支障なかったですね。
おっしゃる通りこちらのほうがわかりやすいと思います！

class Solution {
public:
    int coinChange(vector<int>& coins, int target_amount) {
        vector<int> min_coins(target_amount + 1, -1);
        min_coins[0] = 0;
        for (int amount = 0; amount <= target_amount; ++amount) {
            if (min_coins[amount] == -1) continue;
            for (const int coin : coins) {
                if (coin > target_amount - amount) continue;
                min_coins[amount + coin] = min_coins[amount + coin] == -1
                                        ? min_coins[amount] + 1
                                        : min(min_coins[amount + coin], min_coins[amount] + 1);
            }
        }
        return min_coins.back();
    }
};

[tokuhirat]

枚数としてあり得ない大きい数で初期化したい派です。