how to push string into new array?

Question

I am new to JavaScript and is going through the tutorials here on www.codeacademy.com. I am trying to push a string into a new array however i am getting an error when i run the code saying

"Oops, try again! It looks like your second 'for' loop isn't pushing values to the hits array. Make sure it's working properly and that myWord text appears somewhere in the text variable."

I have looked over the code but have no clue as to where the fault is?

var someText = "This is some text and i am trying to push this text into a new string containing the string text.";

var myWord = "text";

var hits = [];

for (var i = 0; i < someText.length; i++) {
    if (someText[i] === myWord[0]) {
        for (var j = i; j < someText[i] + myWord.length; j++) {
            hits = [];
            hits.push("text");
            hits[0]; //equals 'text'

        }
    }
}

any ideas how to get this to work?

someText[i]+myWord.length is the stopping condition for the second loop. perhaps i didn't need to reference the index :S — user2656943
– user2656943, Commented Oct 4, 2013 at 14:43
I am trying to check 'someText' for the first letter in 'myWord' then push (add) the number of characters equal to 'myWord's' length to an array. — user2656943
– user2656943, Commented Oct 4, 2013 at 14:49
Just saying, this condition j < someText[i] + myWord.length is going to evaluate to something like j < t20 — tymeJV
– tymeJV, Commented Oct 4, 2013 at 14:50

Ian · Accepted Answer · 2013-10-04 14:42:57Z

2

You reset hits at each iteration.

Remove hits = [] in your loop.

edited Oct 4, 2013 at 14:42

Ian

51k13 gold badges104 silver badges111 bronze badges

answered Oct 4, 2013 at 14:40

Étienne Miret

6,6795 gold badges28 silver badges41 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

tymeJV · Accepted Answer · 2013-10-04 15:10:04Z

0

Your second loop iteration is doing an illegal comparison j < someText[i] + myWord.length;. Remove that someText[i] from the mix and it'll work: j < myWord.length

answered Oct 4, 2013 at 15:10

tymeJV

105k14 gold badges165 silver badges158 bronze badges

Collectives™ on Stack Overflow

how to push string into new array?

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related