What's missing here or am I completely off?

JavaScript
#1

Hey guys decided to use while to do this one, but it’s not working out. What am I doing wrong?

Your code so far


function repeatStringNumTimes(str, num) {
var finalResult = ""; //initialize string to hold result
  while (num > 0) { //repeat finalResult.push(str) while num is positive
    return finalResult.push(str); 
    num--; // subtract num by 1
  } else {
    return ""; // else return empty string
  }
}

console.log(repeatStringNumTimes("abc", 3));

Your browser information:

User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.96 Safari/537.36.

Challenge: Repeat a String Repeat a String

Link to the challenge:

#2

There is no such construct as while...else.
Strings have no push() method.
Code after return won’t execute.

1 Like
#3

There are a few things wrong with your code.

  1. You should use the chain of if…else and not while…else. It simply doesn’t exist. Actually, you don’t need else block here. While loop will execute as long as the condition is true. Loop will stop when condition becomes false and nothing will happen. So, you don’t need to pass it explicitly.

  2. return returns a value and stops any further execution. Therefore, you should use it only at the end of a function.

  3. push method pushes element inside an array. It doesn’t work with strings. You should use string concatenation in this case.

Now, here is the solution. Compare it with your code and try to understand.

REDACTED

#4

It is great that you solved the challenge, but instead of posting your full working solution, it is best to stay focused on answering the original poster’s question(s) and help guide them with hints and suggestions to solve their own issues with the challenge.

We are trying to cut back on the number of spoiler solutions found on the forum and instead focus on helping other campers with their questions and definitely not posting full working solutions.

2 Likes
#5

Well, I had explained step by step with comments. I thought that will help him. I am new here so didn’t know the guidelines.

#6

Hi @croy4744 !

We understand that you are trying to help but if the OP wanted answers they could just look it up in the guide.

Allow the OP to digest the steps you provided them and revise their approach and if they are follow up questions then they can ask here in this thread.

It will be a better learning experience if that arrive at the answer on their own with hints instead of reading solutions.

Hope that helps!

3 Likes
#7

I completely understand your point. Will definitely keep it in mind in future. I, myself, am a newbie and struggling with javascript. That was done with the intention to help only.

#8

No worries.

I think your advice was very helpful. :grinning:

1 Like
#9

Hi @am93 !

As the others have mentioned, you do have to make some changes.
But you are not that far off from the solution using a while loop.

Keep tinkering with it and I am sure you will arrive at the solution.

2 Likes
#10

@croy4744 @jwilkins.oboe @JeremyLT thank you. I was able to figure it out with your advice!

2 Likes