Serait-ce efficace? Factoriels avec JavaScript

Question

Même si j'apprends JavaScript, j'ai trouvé cette solution pour Factoriels et j'aimerais obtenir des commentaires.

function factorialize(num) { 
var total= []; 
    if(num!==0){ 
    for(var i=1; i<=num; i++){ 
     total.push(i); 
    } 
    var newTotal = total.reduce(function(a,b){ 
     return a*b; 
    }); 
    return newTotal; 
    }else { 
    return 1; 
    } 
} 
console.log(factorialize(6)); 

Cette autre manière que je trouvais en ligne facile à lire et à comprendre, je pense tout simplement ne peux pas de cette façon.

non récursive

var factorial = function(n) { 
if(n == 0) { 
    return 1 
} else { 
    product = 1; 
    for(i = 1; i <= n; i++) { 
     product *= i; 
    } 
    return product; 
    } 
} 
console.log(factorial(num)); 

récursive:

var factorial = function(n) { 
if(n == 0) { 
    return 1 
} else { 
    return n * factorial(n - 1); 
} 
} 
console.log(factorial(num)); 

Answer 1

Votre solution est raisonnable, bien que ce qui pourrait être critiqué à ce sujet, est qu'il est un peu inutile avec la mémoire. Les autres versions n'ont besoin de conserver qu'une seule valeur intermédiaire en tant qu'état (en supposant une optimisation de la fin de l'appel pour la version récursive), alors que votre algorithme repose sur un tableau de longueur n. Vous pouvez utiliser un générateur au lieu d'une liste pour éviter cela.