Creating a range function that pushes all the numbers to an array

Question

I am creating a function that adds the numbers within the parameters start to end to an array. I have kind of accomplished that but the function adds a few numbers beyond the end number. I have probably overcomplicated things here...

function range(start, end){

  let numbersArray = [];
 
  let counter = 0;
  
  while(counter < end){
    counter++
    
    if (counter < end){
    numbersArray.push(start++)}
 
    };
  return numbersArray
};

console.log(range(4, 11));
//[4, 5, 6, 7, 8, 9, 10, 11, 12, 13]

Mehan Alavi · Accepted Answer · 2021-07-22 13:44:32Z

0

In your case at counter the beginning must be equal start. Because of that there is no need in variable counter, can use start:

function range(start , end){
  let numbersArray = [];
  while(start <= end){ 
    numbersArray.push( start );
    start += 1;
  };

  return numbersArray
};

But while can lead to eternal loop, so better use this answer

edited Jul 22, 2021 at 13:44

Mehan Alavi

3274 silver badges19 bronze badges

answered Jul 22, 2021 at 9:16

zswqa

9347 silver badges15 bronze badges

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

2 Comments

tyskas Over a year ago

Thx! Thats what I was stuck with... I added an equal sign after while(start <= end) so it includes the end as well.

zswqa Over a year ago

Welcome @tyskas at SO! Do not forget to upvote and accept answer if it was useful.

Jesper · Accepted Answer · 2021-07-22 09:20:38Z

0

Is this what you want? https://jsfiddle.net/bagnk3z4/2/

function range(start, end) {
    let array = [];
    while (start <= end) {
        array.push(start++);
    }
    return array;
}

answered Jul 22, 2021 at 9:20

Jesper

1,11613 silver badges27 bronze badges

1 Comment

tyskas Over a year ago

Heyy... thx for the input. The jsfiddle link stuff I havent learned yet but I am sure it works as well. Taking my baby steps :)

Collectives™ on Stack Overflow

Creating a range function that pushes all the numbers to an array

2 Answers 2

2 Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related