Is there a way to add elements with .splice() during a loop in javascript?

Question

I have an array like this:

[0, 24.646213151927437, 27.737256235827665, 29.08326530612245]

I want to fill every space between the elements which are greater than 4 with random numbers. Like this:

var arr = [0, 24.646213151927437, 27.737256235827665, 29.08326530612245];
var length = arr.length;
for (var j = 0; j < length;) {
  var spacebetween = arr[j + 1] - arr[j];
  if (spacebetween > 4) {
    // fill the space until the sum of numbers is greater than space between
    var m = Math.random() * (3 - 1 + 1);
    for (k = m; k < spacebetween;) {
      arr.splice(j, 0, parseFloat(k));
      m = Math.random() * (3 - 1 + 1);
      k = k + m;
    }
  }
  j++;
}
console.log(arr);

The problem is with splice(), the index is shifting and I have no idea how to solve this. The new numbers should stay in ascending order within the array.

So, sort the array once you're done?

Heretic Monkey
– Heretic Monkey

2020-06-09 12:25:57 +00:00
Commented Jun 9, 2020 at 12:25 — Heretic Monkey
– Heretic Monkey, Commented Jun 9, 2020 at 12:25
please add a possible result with comments.

Nina Scholz
– Nina Scholz

2020-06-09 13:03:30 +00:00
Commented Jun 9, 2020 at 13:03 — Nina Scholz
– Nina Scholz, Commented Jun 9, 2020 at 13:03

Katherine R · Accepted Answer · 2020-06-09 12:30:17Z

1

Instead of looping through the array (with a changing length), you can use a reduce to loop through the original array and push updates to the accumulator

var arr = [0, 24, 27, 29];
var newArr = arr.reduce((accumulator, cur, i) => {
    var needTempArr = arr[i + 1] && (arr[i + 1] - cur);

    // if need temp array, generate it; otherwise, empty array
    var tempArr = needTempArr
        ? Array(needTempArr).fill(0).map(() => Math.random())
        : [];


    return accumulator.concat(cur, tempArr);
}, []);

answered Jun 9, 2020 at 12:30

Katherine R

1,1581 gold badge10 silver badges22 bronze badges

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

1 Comment

Tobi Over a year ago

thx for that approach. but i need to have the newly generated random numbers added until the SUM is bigger than the "gap". now its adding a random number for every 'int' of the gap.

Mister Jojo · Accepted Answer · 2020-06-09 13:57:04Z

0

Is there a way to add elements with .splice() during a loop in javascript?

Yes, the solution is to simply start from the end and decrement, the new entries will no longer disturb the process:

changed with no gap greater than 4

let arr = [ 0, 24.646213151927437, 27.737256235827665, 29.08326530612245 ]

for (let j=arr.length; --j;)   // inside loop j start to (arr.length-1) to value==1  ( value zéro mean false)
  {
  let vRef = arr[j-1]
   ,  vMin = arr[j] -4  
   ;
  while (vMin > vRef) 
    {
    vMin = (Math.random()*4) + vMin
    arr.splice( j, 0, vMin )
    vMin -= 4
    }
  }

console.log( JSON.stringify( arr,0,2 ))

.as-console-wrapper { max-height: 100% !important; top: 0; }

edited Jun 9, 2020 at 13:57

answered Jun 9, 2020 at 12:35

Mister Jojo

22.9k6 gold badges28 silver badges45 bronze badges

1 Comment

Mister Jojo Over a year ago

@Tobi changed with no gap greater than 4

Collectives™ on Stack Overflow

Is there a way to add elements with .splice() during a loop in javascript?

2 Answers 2

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related