• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

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

VPS Starter Arubacloud
+1 głos
103 wizyt
pytanie zadane 14 lipca 2021 w JavaScript przez Vigo Nowicjusz (190 p.)

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.

1 odpowiedź

+2 głosów
odpowiedź 14 lipca 2021 przez ScriptyChris Mędrzec (190,190 p.)

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ć".

komentarz 14 lipca 2021 przez Vigo Nowicjusz (190 p.)

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.

1
komentarz 14 lipca 2021 przez ScriptyChris Mędrzec (190,190 p.)

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

 

2
komentarz 14 lipca 2021 przez Vigo Nowicjusz (190 p.)
Zgadza się. To było trochę głupie pytanie ale 11h kodowania robi swoje dzięki.

Dzięki

Podobne pytania

–2 głosów
4 odpowiedzi 191 wizyt
pytanie zadane 27 listopada 2015 w C i C++ przez Avernis Nałogowiec (27,400 p.)
0 głosów
1 odpowiedź 122 wizyt
pytanie zadane 3 maja 2021 w JavaScript przez ShockWave Bywalec (2,350 p.)

92,454 zapytań

141,262 odpowiedzi

319,088 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...