Home > Software design >  JS Classic Fibonacci Challenge - Differences between two solutions
JS Classic Fibonacci Challenge - Differences between two solutions

Time:11-24

I have two solutions to the same challenge, this classic fibonacci challenge that everyone knows how to solve (even your pets).

I kindly ask you NOT to suggest any other solutions. I just want to compare these two solutions. Thousands different solutions can be found with searches.

Challenge:

/*
     0  1  2  3  4  5  6  7   8   9 
    [0, 1, 1, 2, 3, 5, 8, 13, 21, 34]

     fib(3) returns 2
     fib(9) returns 34
     and so on...

*/

Both solutions below are working fine. My only question is:

Does solution B run slower than solution A? Because in B we have this line below:

fibArr.push(fibArr[fibArr.length - 1]   fibArr[fibArr.length - 2])

Does the length function go through the entire array to calculate the number of items? Or already returns from immediately?

Solution A:

function fib(n) {
  const fiboArray = [0,1]
  for(let i=2; i <= n; i  ) {
    fiboArray.push(fiboArray[i-2]   fiboArray[i-1])
  }
  return fiboArray[n]
}
console.log(fib(5))

Solution B:

function fib(n) {
  const fibArr = [0, 1, 1]
  
  if(n == 0) {
    return 0
  }

  if(n == 1 || n == 2) {
    return 1
  }


  if (n > 2) {
    for (let i = 3; i <= n; i  ) {
      fibArr.push(fibArr[fibArr.length - 1]   fibArr[fibArr.length - 2])
    }
  }
  
  return fibArr[fibArr.length - 1]
}


console.log(fib(9))

CodePudding user response:

I concur with CertainPerformance, Solution A is better.

In many situations using .length would be just as fast because the browser will pre-compute it and go just as efficiently as if you make a local variable yourself however I think in your case Solution A is better because you use push on the array during the loop so length will be recalculated.

The answer to this post talks about it but he doesn't have push like you do.

CodePudding user response:

@MisterJojo can you please show me a code example to simplify it?

function my_Fibonacci(n)
  {
  let a = 0
    , b = 1
    , r = [0, 1]
    ;
  for(let i=2; i<=n; i  )
    {
    r.push(a b) // new fibonacci value
    a = b       // set a for next addition
    b = r[i]    // set b for next addition
    }
  // return r.join(' - ')
  return b
  }

document.write(my_Fibonacci(9))
<iframe name="sif1" sandbox="allow-forms allow-modals allow-scripts" frameborder="0"></iframe>

  • Related