最好的寫法是什么？ 節點.JS

Question

這是我的第一篇文章：）有人可以幫助我改進我的代碼以獲得更好的編寫方式嗎？

Temporada.findAll({
  where: { status: 1 }
}).then(temporadas => {


  Temporada.count({
    where: { status: 1 }
  }).then(temporadasAbertas => {
    if (temporadasAbertas > 0) {
      if (user_logged_acesso == 1) {
        JogoContagem.findAll({
          where: {
            j1Id: user_logged_id,
            status: { [Op.ne]: 0 }
          }
        }).then((blockeds) => {
          console.log(blockeds)
          res.render("painel/jogos/new", { users, user_logged_id, temporadas, blockeds });
        })
        
      } else {
        req.flash("error_msg", "No momento você não tem premissão de cadastrar jogos.");
        res.redirect("/painel");
      }
      
    } else {
      req.flash("error_msg", "No momento não há temporadas em aberto para cadastar jogos.");
      res.redirect("/painel");
    }
  })
  
})

它運行良好，但是我知道存在更好的編寫方法，可能使用帶有異步等待的函數，但我不知道如何收集結果並將結果插入res.render 。 我想我需要用數組創建一個 var ......無論如何我想......如果你知道一種方法讓我知道我會改進它。

觀察：我的代碼有幾部分是用英語和葡萄牙語編寫的，就像有人閱讀並想知道哪種語言一樣。 （\o/來自巴西的你好）

多謝，

親切的問候戴維蘇亞雷斯

Answer 1

這就是你可以用await做到這一點的方法。 在async function 中，執行以下操作：

const temporadas = await Temporada.findAll({
  where: { status: 1 }
});
const temporadasAbertas = await Temporada.count({
  where: { status: 1 }
});
if (temporadasAbertas <= 0) {
  req.flash("error_msg", "No momento não há temporadas em aberto para cadastar jogos.");
  res.redirect("/painel");
  return;
}
if (user_logged_acesso !== 1) {
  req.flash("error_msg", "No momento você não tem premissão de cadastrar jogos.");
  res.redirect("/painel");
  return;
}
const blockeds = await JogoContagem.findAll({
  where: {
    j1Id: user_logged_id,
    status: {
      [Op.ne]: 0
    }
  }
});

res.render("painel/jogos/new", { users, user_logged_id, temporadas, blockeds });

確保.catch錯誤，無論是在try / catch的catch中，還是在.catch調用上的 .catch 中。

Answer 2

更新代碼：

router.get("/painel/jogos/new", authenticator, (req, res) => {
        var json = req.session.user;
        var user_logged = JSON.stringify(json);
        var user_logged = JSON.parse(user_logged);
        var user_logged_id = user_logged.id;
        var user_logged_acesso = user_logged.acesso;

        async function load() {
                try {
                        const loadUsers = await User.findAll({ where: { acesso: 1 }, order: [ ['primeiro_nome' , 'ASC'] ] })
                        const loadTemporadas = await Temporada.findAll({ where: { status: 1 } })
                        if (loadTemporadas[0] == null) { req.flash("error_msg", "No momento não há temporadas em aberto para cadastar jogos."); res.redirect("/painel"); }
                        const loadBlockeds = await JogoContagem.findAll({ where: { j1Id: user_logged_id, status: { [Op.ne]: 0 } } })

                        res.render("painel/jogos/new", { users: loadUsers, user_logged_id, temporadas: loadTemporadas, blockeds: loadBlockeds });
                } catch (err) {
                        console.log(err)
                        req.flash("error_msg", "Ocorreu algum erro ao carregar a página, favor informar o administrador do site. ERROR CODE: /painel/jogos/new - LOAD"); 
                        res.redirect("/painel");
                }
        }
        load()
})