392. Is Subsequence

[katsukii]

問題

https://leetcode.com/problems/is-subsequence/description/

言語

Java

次の問題

139. Word Break

[Yoshiki-Iwasa]

手作業でどうやるかを考えるといいかもしれません

https://docs.google.com/document/d/11HV35ADPo9QxJOpJQ24FcZvtvioli770WWdZZDaLOfg/edit?tab=t.0#heading=h.7n6wwffw10hb

[katsukii]

以前同様のフィードバックを受けたのですが、頭から抜けていたようです。癖付けないとですね。ありがとうございます。

[Yoshiki-Iwasa]

問題文下のFollow upも考えて見るといいかもです

Follow up: Suppose there are lots of incoming s, say s1, s2, ..., sk where k >= 109, and you want to check one by one to see if t has its subsequence. In this scenario, how would you change your code?

[oda]

tIndex が配列外アクセスしそうですね。(そもそもこの行がいらない気がします。)

[oda]

ここ、上の while との関係で実行されません。while の後ろに。

[oda]

この条件、ループの後ろに回すと条件が減らせませんか。

[katsukii]

ありがとうございます。これで十分でしたね。

while (true) {
    if (sIndex == sLength) return true;
    if (tIndex == tLength) return false;
    if (s.charAt(sIndex) == t.charAt(tIndex)) sIndex++;
    tIndex++;
}

[oda]

この変数は置かないほうがいいかもしれません。
sIndex < s.length()
のほうが意味がはっきりしていませんか。

[katsukii]

@oda
t.length()はたしかに変数化する必要はないですね。
一方で、複数回呼び出す処理（ここでは s.length() ）は無思考的に変数に格納してしまっておりました。この程度のオーバーヘッドは無視していいと言われればそのとおりと思いつつ、このあたりのバランス感覚が乏しいので判断基準などあればアドバイスいただけないでしょうか。

[oda]

もちろん、状況次第ではあります。

ただ、オーバーヘッドは、関数呼び出しは一回数クロック、ナノ秒のオーダーです。

一方、「現在の文字列の長さ」を変数に格納すると「その時点での文字列の長さ」になりますね。よって、読み手は、「変数の意味」を記憶していて、「変数への格納の後に文字列が変更される経路がないこと」に注目し続けているときに「現在の文字列の長さ」だと分かります。これは結構コストが高いです。

[katsukii]

なるほど、理解しました。オーバーヘッドを過剰見積もりしていたと同時に、変数を読み手が追跡し続ける必要があるコストを考えると確かになと思いました。以後もう少し柔軟に適用していきたいと思います。

[nittoco]

もしご覧になってなかったら、この辺を一通り見て、実際いろんな場合を書いてみるといいかもしれません
https://discord.com/channels/1084280443945353267/1225849404037009609/1243290893671465080

[katsukii]

@nittoco 承知しました。ありがとうございます。見てみます。