javascript - 文字列内の単語を見つけるループ

Question

だから、私は私の名前、アンドリューを識別し、それを配列にプッシュする「for」ループに取り組んでいますが、何か問題があります

/*jshint multistr:true */

var text = ("Andrew is really awesome and Andrew should be working on the project, but there is honestly nothing for Andrew to do.");
var myName = ("Andrew");
var hits = [];
for (var i = 0; i < text.length; i ++) {
    if (text[i] === "A") {
        for (var j = i; i + nyName.length; i ++) {
            hits.push(text[j]);
        }
    }
}

また、2 番目のループは、myName の最後に到達すると停止するはずです。

score 1 · Accepted Answer

JSHINT を使用しているため、エラーメッセージを読むだけで、何が問題なのかが正確にわかります。

エラー:

7 行目:for (var j = i; i + nyName.length; i ++) {

「nyName」は定義されていません。

3 行目:var myName = ("Andrew");

「myName」は定義されていますが、使用されていません。

JSHINT は、それが何を伝えているかに注意を払わなければ、あまり役に立ちません。

また、内側のループが奇妙に見えます。

for (var j = i; i + nyName.length; i ++) {

無限ループになりそうです。あなたはおそらくj別の条件で望んでいます。

score 0 · Accepted Answer

myNamefor ループ構文のスペルを間違えてnyName代わりに入力したため、その行に到達するとすぐにスクリプトが終了する可能性があります。

score 0 · Accepted Answer

参照したい for ループのタイプミスはmyName大きな問題のようです:

for (var j = i; i + nyName.length; i ++)
                    ^

score 0 · Accepted Answer

myName失敗するのはスペルミスだけではありません。i + myName.lengthfor ループは常にに評価されるため、ループを終了することはありませんtrue。また、値を増やす必要があります。jそうしないと、常に index の文字が取得されますi。

これが修正されたループです。

for (var i = 0; i < text.length; i ++) {
    if (text[i] === "A") {
        for (var j = 0; j < myName.length; i++, j++) {
            hits.push(text[i]);
        }
    }
}

javascript - 文字列内の単語を見つけるループ

4 に答える 4

Related

Reference