res.send funktioniert nicht richtig

Question

Ich arbeite mit einer einfachen CRUD-App mit jquery ajax und node.js, nur um meine Fähigkeiten mit Knoten und Ajax zu verbessern. Die Sache ist, dass ich eine Post-Anfrage mache, die mit meinem Post-Router auf dem Node-Server gehandhabt wird, und alles funktioniert gut. Es fügt ein weiteres Produkt zu meiner products.json Datei hinzu, aber am Ende wird die Antwort nicht an den Client zurückgeschickt, der letzte res.send ("done") funktioniert nicht und ich weiß nicht warum. .

hier ist der Code:

ajax

$("#create-form").on('submit',function(){ 
    event.preventDefault(); 
    var createIn = $("#create-input").val(); 
    $.ajax({ 
     url: '/products', 
     method:'POST', 
     data:JSON.stringify({name:createIn}), 
     contentType: "application/json", 
     dataType: "json", 
     success: function(data){ 
      console.log(data); 
      $("create-input").val(""); 
      $("get-button").click(); 
     } 
    }); 
}) 

Knoten

app.post('/products',function(req,res){ 
    fs.readFile('products.json','utf8',function(err,data){ 
     var result = JSON.parse(data); 
     var productName = req.body.name; 
     console.log(req.body.name); 
     currentId++; 
     var productId = currentId; 
     var product = { 
      name: productName, 
      id: productId 
     } 
     result.products.push(product); 
     fs.writeFile(__dirname + "/products.json",JSON.stringify(result),'utf8'); 
    }); 
    res.send("post done"); 
}); 

Dies ist nur der wichtige Teil des Codes, es funktioniert und scheitert nur am Ende in der res.send.

Answer 1

Ihr Client-Code ist auf der Suche nach einem json Antwort, aber Sie sind wieder einen String.

$("#create-form").on('submit',function(){ 
    event.preventDefault(); 
    var createIn = $("#create-input").val(); 
    $.ajax({ 
     url: '/products', 
     method:'POST', 
     data:JSON.stringify({name:createIn}), 
     contentType: "application/json", 
     dataType: "json",   <-------------- 
     success: function(data){ 
      console.log(data); 
      $("create-input").val(""); 
      $("get-button").click(); 
     } 
    }); 
}) 

Entweder diese Zeile löschen oder auf der Serverseite hinzufügen

res.send({"message":"post done"}); 

Answer 2

Dies beantwortet Ihre Frage nicht direkt, aber Sie sollten die Antwort im Idealfall nicht zurücksenden, bis Sie wissen, dass die Arbeit erledigt wurde und Sie Fehler behandeln sollten. Mit anderen Worten, Sie sollten die Rückrufe verwenden. (Zu viele Rückrufe kann problematisch sein, und Sie sollten andere Muster untersuchen - zB Versprechungen - Bit keine Notwendigkeit, hier)