중복 코드 리팩토링

kenu commented 3 years ago

진단
처방(리팩토링) @hochan222

hochan222 commented 3 years ago

방안 1

Promise.all 로 중복된 코드를 없애고, 병렬로 request 요청하는 방안

setTimeout도 clear해주어야한다.

const timeout = new Promise((resolve, reject) => {
setTimeout(() => resolve(‘timeout’), 1000);
});

Promise.all([axios.get(uri, headers), get42UserCoalition(username, headers), timeout])
.then((data)=>{
    const one = {…data[0], 'coalition’: data[1]}
    ObjectUtils.calcDiff(one.projects_users, 'marked_at');
    await User.save(one);
    res.render('user', { 
      user: one, 
      updatedAt: dayjs().format('YYYY/MM/DD HH:mm:ss'), 
      dayjs 
    });

})
.catch(e=>{
    const error = new Error(e.message);
    console.log(e.message);
    error.status = e.response.status;
    if (error.status === 401) {
      res.redirect('/login/42');
      return;
    }
    next(error);
})