Suche nach Datenbankproblemen

Question

Ich habe Code geschrieben, um Credits von einem Benutzer auf einen anderen zu übertragen, ich habe MongoDB als Datenbank ausgewählt. Aber mir wurde gesagt, dass ich Probleme mit der Datenbank in der Funktion transferCredits habe. Ich habe gesucht, konnte aber kein Problem finden.

function transferCredits(from, to, amt) { 
    var fromAccount = db.game_accounts.findOne({"name": from},{"credits": 1}); 
    var toAccount = db.game_accounts.findOne({"name": to},{"credits": 1}); 
    if (fromAccount.credits < amt) { 
    throw new BalanceError("not enough balance to transfer credits"); 
    } 
    db.game_accounts.update({name: from}, {$set: {credits: fromAccount.credits - amt}}); 
    db.game_accounts.update({name: to}, {$set: {credits: toAccount.credits + amt}}); 
} 

db.game_accounts.insert({name: "John", credits: 1000}); 
db.game_accounts.insert({name: "Jane", credits: 1000}); 

// John transfers credits to Jane 
transferCredits("John", "Jane", 100); 

Answer 1

Sie sagen uns nicht, was das Problem ist, so können wir nur spekulieren ...

Was ich sehe, ist, dass dieser Teil falsch sein könnte: weil

db.game_accounts.update({name: from}, {$set: {credits: fromAccount.credits - amt}}); 
db.game_accounts.update({name: to}, {$set: {credits: toAccount.credits + amt}}); 

Dies ist

Es könnte eine weitere Transaktion zwischen dem findOne() und dem update() stattfinden. Sie verwenden besser $inc, die den gleichen Vorgang ausführen wird, ohne sich jedoch den Rennbedingungen zu öffnen.

Beachten Sie, dass Subtraktion I einfach mit -1 multipliziert, um die Zahl negativ zu machen.

Wenn der Benutzer versucht, einem anderen Benutzer -10 Guthaben zu geben, kann er stattdessen 10 Guthaben STEHLEN ... Mehrere andere Überprüfungen müssen so durchgeführt werden, um eine gültige Transaktion zu gewährleisten.

Nebenbei gesagt, MongoDB ist nicht wirklich für diese Art von Transaktion gedacht, bevorzugen Sie vielleicht eine andere Art von DB, die sicherstellen kann, dass die Daten auf dem neuesten Stand sind und alle Operationen verfolgt und erledigt werden.