2017-05-22 2 views
0

Wie Sie aus dem Code sehen können, gebe ich alle Benutzer durch die Abfrage mit einer undefinierten Variable, wenn Parameter nicht definiert ist. Ist das akzeptabel, oder gibt es einen Nachteil, den ich nicht sehe?Ist es akzeptabel, eine Mungo-Datenbank mit undefinierter Variable abzufragen?

get: function(param, callback){ 
    var query; 
    if (param) { 
     query = {username: param} 
    } 
    User.find(query, (err, result) => { 
     if (err) { 
      callback(err, null); 
      return; 
     } 
     callback(null, result); 
    }); 
} 
+0

Hmm. Aus Gründen der Vernunft würde ich zumindest das 'var query = {}' machen und wenn Sie wirklich nur Ergebnisse für einen "Benutzernamen" wollen, dann würde ich einfach dorthin zurückkehren, wo kein Parameter angegeben wurde. –

Antwort

0

Wie Sie aus the source Mungo kümmert sich um diese für Sie sehen:

Model.find = function find(conditions, projection, options, callback) { 
    if (typeof conditions === 'function') { 
    callback = conditions; 
    conditions = {}; 

Sie können Pass undefinierten als conditions

Wie in den Kommentaren vorgeschlagen, aber so, Ich würde wechseln zu:

var query = {} 

Sie geben also immer ein leeres Objekt weiter.

+0

Danke, das ist genau das, was ich gesucht habe. –

Verwandte Themen