Wie doppelte Codeblöcke in node.js zu vermeiden

Question

Ich habe einen Codeblock, der wenn sonst und der meiste Code in if und anderen gleich ist, aber aufgrund eines Funktionsaufrufs mit Rückruf, muss ich ziemlich einen gleichen Code in sowohl wenn als auch sonst blockieren. Ich weiß, dass eine Funktion für diesen Zweck erstellt werden kann, aber ich muss zu viele Parameter übergeben. Hier ist der Code-Block:

if (results[0].register_type == 'D') { 
       sessionData.register_type = results[0].register_type; 
       UserModel.updategcm_id(req.body, function (err, result) { 
        UserSessionModel.createSessionToken(sessionData, function (err, result, token) { 
         if (err) { 
          res.status(400).send(self.createResponse({}, { 
           success: false, 
           message: err.message 
          })); 
          return false; 
         } 
         res.status(200).send(self.createResponse({ 
          token: token, 
          userid: results[0].id 
         }, { 
           success: true, 
           message: "User has logged in successfully" 
          })); 
         console.log("User has logged in successfully...\n"); 
        }); 
       }); 
      } else { 
       UserSessionModel.createSessionToken(sessionData, function (err, result, token) { 
        if (err) { 
         res.status(400).send(self.createResponse({}, { 
          success: false, 
          message: err.message 
         })); 
         return false; 
        } 
        res.status(200).send(self.createResponse({ 
         token: token, 
         userid: results[0].id 
        }, { 
          success: true, 
          message: "User has logged in successfully" 
         })); 
        console.log("User has logged in successfully...\n"); 
       }); 
      } 

Answer 1

Ich denke, dass die beste Lösung ist, die wiederholte Funktion auf eine Funktionsdeklaration zu bewegen und es nur als Callback übergeben. Ihr Code würde sich ändern in:

if (results[0].register_type == 'D') { 
    sessionData.register_type = results[0].register_type; 
    UserModel.updategcm_id(req.body, function (err, result) { 
     UserSessionModel.createSessionToken(sessionData, validateResponse); 
    }); 
} else { 
    UserSessionModel.createSessionToken(sessionData, validateResponse); 
} 

function validateResponse(err, result, token) { 
    if (err) { 
     res.status(400).send(self.createResponse({}, { 
      success: false, 
      message: err.message 
     })); 
     return false; 
    } 
    res.status(200).send(self.createResponse({ 
     token: token, 
     userid: results[0].id 
    }, { 
     success: true, 
     message: "User has logged in successfully" 
    })); 
    console.log("User has logged in successfully...\n"); 
} 

Der Code ist jetzt viel sauberer.

Sie können auch den ganzen createSessionToken Anruf in eine Funktion ausklammern, aber Sie würden mit geben sessionData vorsichtig sein, self und res (abhängig vom jeweiligen Umfang).

Answer 2

Sie können das Async-Modul verwenden, so dass Sie die Funktion nicht wiederholen müssen.