Array kopieren -> Stack- oder Heap-Überlauf?

Question

Ich habe ein Array namens globalArrayAllTrades, wie Sie unten sehen. Ich möchte einfach das Datum in eine neue Kopie des Arrays INVERTIEREN. Also ich durchschlinge, ein neues Objekt erstellen und es dem neuen Array hinzufügen - einfach.

Dann funktioniert die Funktion genau wie erwartet. ABER wenn das Array zu viele Objekte enthält, schlägt der Code mit einem "FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - Prozess nicht genügend Arbeitsspeicher".

Mein Laptop hat 8 GB Speicher ... Wenn der NODEJS-Prozess abstürzt, verwendet es etwa 1,5 GB und etwa 70% der gesamten verfügbaren Speichermenge wird verwendet.

Ich führe die NODEJS App mit dem Parameter: --max_old_space_size=5000, die normalerweise alles regelt. Aber nicht dieses und ich habe viele verschiedene Möglichkeiten versucht, um die gleiche Funktion zu programmieren - aber jedes Mal - es scheitert ... es sei denn, das ursprüngliche Array ist kleiner.

Wie kann ich dieses Problem beheben?

function invertTrades(){ 

    var original = globalArrayAllTrades.slice(); 

    globalArrayAllTrades.length = 0; 
    globalListAllTrades.length = 0; 

    for(var i = 0; i < original.length; i++){ 

     var objS = original[i]; 
     var objE = original[original.length-1-i]; 
     var objInv = new TradePoint(objS.number, objS.matchdate, objE.price, objE.size, objE.issell); 

     globalArrayAllTrades.push(objInv); 

     globalListAllTrades[objInv.matchdate] = objInv; 
    } 
} 

Answer 1

Sie etwas Speicher Original enthalten sparen können, indem sie nur die Eigenschaften, die Sie umkehren müssen, nicht das ganze TradePoint Objekt. Dann müssen Sie keine neuen TradePoint Objekte erstellen, Sie können sie an Ort und Stelle ändern.

var original = globalArrayAllTrades.map(function(trade) { 
    return { 
     trade.price, 
     trade.size, 
     trade.issell 
    }; 
}).reverse(); 
globalArrayAllTrades.forEach(function(trade, i) { 
    trade.price = original[i].price; 
    trade.size = original[i].size; 
    trade.issell = original[i].issell; 
}); 

Und da alle Objekte an Ort und Stelle verändert wurden, gibt es keine Notwendigkeit globalListAllTrades zu aktualisieren.

Eine andere Möglichkeit ist es, die price, size und issell Eigenschaften zwischen den Paaren von Elementen zu tauschen:

var midpoint = Math.floor(globalArrayAllTrade.length/2); 
for (var i = 0; i < midpoint; i++) { 
    var objS = globalArrayAllTrades[i]; 
    var objE = globalArrayAllTrades[globalArrayAllTrades.length-1-i]; 

    var temp = objS.price; 
    objS.price = objE.price; 
    objE.price = temp; 

    temp = objS.size; 
    objS.size = objE.size; 
    objE.size = temp; 

    temp = objS.issell; 
    objS.issell = objE.issell; 
    objE.issell = temp; 
} 

Answer 2

Haben Sie darüber nachgedacht?

// Copy array and then reverse it 
var newArray = [].concat(original).reverse(); 

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/reverse

Answer 3

würde ich vermeiden lassen vermuten, dass Array zu kopieren:

function getInverse(i) { 
    var objS = globalArrayAllTrades[i]; 
    var objE = globalArrayAllTrades[globalArrayAllTrades.length-1-i]; 
    var objInv = new TradePoint(objS.number, objS.matchdate, objE.price, objE.size, objE.issell); 
    globalListAllTrades[objInv.matchdate] = objInv; 
    return objInv; 
} 
function invertTrades(){ 
    globalListAllTrades.length = 0; 
    for (var i = 0, l = Math.floor(globalArrayAllTrades.length/2); i < l; i++) { 
     var j = globalArrayAllTrades.length-1-i; 
     var a = getInverse(i); 
     var b = getInverse(j); 
     globalArrayAllTrades[i] = a; 
     globalArrayAllTrades[j] = b; 
    } 
}