Javascript Warunek w warunku ma zwracać odpowiedz do użytkownika lub kontynuować wykonywanie aplikacji

Question

Cześć, mam małą zagwozdkę chcę sprawdzić czy użytkownik wysłał w body konkretny element jeżeli tak to sprawdzam ten element czy istnieje już w bazie danych jeżeli nie to kontynuuj działanie funkcji jeżeli tak to zwróć response do użytkownika.

if(req.body.username){
   return User.findOne({where: {username: req.body.username}})
   .then(record => {
        if(record) return res.status(422).json({success:false, message:"This username already exists"})
    })
   .catch(() => res.status(500).json({success:false,message:"Server error"}))
        }

Jeżeli usunę return przed findone to aplikacja dalej będzie wykonywać koleje polecenia. A znowu z return aplikacjia przestaje wykonywać kolejne polecenia.

Answer 1

Metoda User.findOne zwraca promisa, przy czym w then odsyłasz odpowiedź do klienta. Zakładam więc, że chcesz po prostu poczekać aż odpowiedź będzie wysłana i wtedy wykonać dalszy kod za if(req.body.username) - czekając na całe wykonanie User.findOne. W takim przypadku zamieniłbym:

return User.findOne

na

await User.findOne

+ oznaczyłbym funkcję, w której całość jest wykonywana jako async (żeby można było użyć await).

---

Jeśli nie o to chodzi, to pokaż proszę szerszy kontekst kodu i doprecyzuj na co chcesz "czekać".

Comments

Mój problem jest powiązany z warunkiem if. Cały proces wygląda następująco użytkownik chce edytować swój profile może zmienić nazwę użytkownika,opis, email itd. I problem pojawia się kiedy użytkownik zmienia nazwę. Przed wykonaniem update sprawdzam czy ta nazwa już istnieje jeżeli tak to ma przerwać wykonywanie kolejnych poleceń i zwrócić respone jeżeli nazwa jest wolna to wykonywać kolejne polecenia i zaktualizować wszystkie dane.

if (req.body.username) {
  await User.findOne({ where: { username: req.body.username } })
    .then(record => {
      if (record) return res.status(422).json({ success: false, message: "This username already exists" })
    })
    .catch(() => res.status(500).json({ success: false, message: "Server error" }))
}
await User.findOne({ where: { id: req.session.userId } })
  .then(record => {
    if (!record) {
      throw new Error('No record found')
    }
    const values = {
      username: req.body.username,
      description: req.body.description
    }
    record.update(values)
      .then(updatedRecord => {
        console.log(`updated record ${JSON.stringify(updatedRecord, null, 2)}`)
        return res.status(200).json({ success: true })
      })
  })

Jednak kiedy warunek znajduje się w warunku to ten pierwszy może przerwać wykonywania kolejnych instrukcji.

Jedynym rozwiązaniem które teraz widzę to zadeklarować zmienną user i po srawdzeniu czy nazwa użytkownika istnieje zrobić true or false.

---

O to chodzi?

if (req.body.username) {
  const shouldStop = await User.findOne({ where: { username: req.body.username } })
    .then(record => {
      if (record) {
          res.status(422).json({ success: false, message: "This username already exists" })

          return true;
      }

      return false;
    })
    .catch(() => res.status(500).json({ success: false, message: "Server error" }))

   if (shouldStop) {
       return;
   }
}

 

---

Zgadza się. To było trochę głupie pytanie ale 11h kodowania robi swoje dzięki.

Dzięki