Как избежать дублирования блоков кода в node.js

Question

У меня есть блок кода, содержащий, если else и большая часть кода в if и else одинаковы, но из-за одного вызова функции с обратным вызовом я должен иметь совершенно такой же код в оба блока if и else. Я знаю, что для этой цели может быть создана функция, но мне потребуется передать слишком много параметров. Вот код блок:

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

Я думаю, что лучшее решением является перемещение повторных функций в объявление функции и просто передать его в качестве обратного вызова. Ваш код будет иметь следующие значения:

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"); 
} 

Код намного чище.

Вы также можете вынесем весь createSessionToken вызов в функцию, но вы должны быть осторожны с прохождением sessionData, self и res (в зависимости от их соответствующего объема).

Answer 2

Вы можете использовать асинхронный модуль, чтобы вам не нужно было повторять эту функцию.