300. longest increasing subsequence

[5ky7]

問題: 300. Longest Increasing Subsequence

[nodchip]

vector<int> increasing_subsequence = { 1 };

のほうがシンプルに感じました。

[5ky7]

vector<int> increasing_subsequence = {nums[0]};

でしょうか．
確かに1要素ならこちらのがシンプルですね．

[nodchip]

自分なら std::lower_bound() が nums.end() を返した場合は push_back() する、というコードを書くと思うのですが、趣味の範囲だと思います。

[5ky7]

std::lower_bound() が nums.end() を返した場合は push_back() する

nums.end() -> increasing_subsequence.end()でしょうか．
確かに次のようにUpdateSubsequence()を分離せず，かつ効率的なコードが書けますね．

class Solution {
public:
    int lengthOfLIS(vector<int>& nums) {
        if (nums.empty()) {
            return 0;
        }
        vector<int> increasing_subsequence(1, nums.front());
        for (int i = 1; i < nums.size(); ++i) {
            auto it = std::lower_bound(increasing_subsequence.begin(), increasing_subsequence.end(), nums[i]);
            if (it == increasing_subsequence.end()) {
                increasing_subsequence.push_back(nums[i]);
                continue;
            }
            *it = nums[i];
        }
        return increasing_subsequence.size();
    }
};

[nodchip]

nums の要素で new_num 以上の数のうち，最小のものをnew_numで置き換える
ではないでしょうか。

https://cpprefjp.github.io/reference/algorithm/lower_bound.html

イテレータ範囲 [first, last) のうち、指定された要素以上の値が現れる最初の位置のイテレータを取得する。

[5ky7]

勘違いしていました．ありがとうございます！

[dxxsxsxkx]

概ね良いと思いました。

[dxxsxsxkx]

初見の時に同じ勘違いをしました...

[dxxsxsxkx]

max_len は他の箇所で使われていないように思います。

[dxxsxsxkx]

max_len は他の箇所で使われていないように思います。

[5ky7]

ご指摘ありがとうございます．
(最後にコードを通しで見直す癖をつけたいです...)