Funkcje nie zwracają wartości

Question

Witam, niestety moje funkcje nie zwracają żadnych wartości, mimo tego, że przez console.log czasem da się je zobaczyć. Próbowałem już czterech różnych sposobów (poniżej), żaden nie potrafi odesłać wartości do użytkownika:

exports.findRelation = (req, res) => {
    let startId, endId;
    getStation(req.body.start)
    .then(result => {
        startId = result;
    });
    getStation(req.body.end)
    .then(result => {
        endId = result;
    });
    console.log(startId, endId);
    return res.json({startId, endId});
}

async function getStation (name) {
    await Stations.findOne({name: name}, (err, result) => {
        if (err) return console.log(err);
        else return result._id;
    });
}

 

exports.findRelation = (req, res) => {
    const startId = await getStation(req.body.start);
    const endId = await getStation(req.body.end);
    console.log(startId, endId);
    return res.json({startId, endId});
}

async function getStation (name) {
    await Stations.findOne({name: name}, (err, result) => {
        if (err) return console.log(err);
        else return result._id;
    });
}

 

exports.findRelation = async (req, res) => {
    let stationsId = new Array;
    await Stations.findOne({name: req.body.start}, (err, result) => {
        try {
            stationsId.startId = result._id;
        }
        catch {
            stationsId.startId = null;
        }
    });
    await Stations.findOne({name: req.body.end}, (err, result) => {
        try {
            stationsId.endId = result._id;
        }
        catch {
            stationsId.endId = null;
        }
    });
    console.log(stationsId);
    if (stationsId) return res.json(stationsId);
}

 

exports.findRelation = async (req, res) => {
    let global = new Array;
    await Stations.findOne({name: req.body.start}, (err, result) => {
        stationsId.startId = result._id;
    });
    await Stations.findOne({name: req.body.end}, (err, result) => {
        stationsId.endId = result._id;
    });
    res.json(stationsId);
}

 

W czym może tkwić problem?

Comments

domyślam się, ze całość jest w express.js. zapytanie wysyłasz z postmana/insomnii czy przeglądarki. spróbuj w tej ostatniej wersji przes res.json(..) dopisać return.

---

Tak próbowałem, niestety nie działało, ale przed chwilą, po kilku godzinach znalazłem rozwiązanie. Odpowiedź wykonywała się zanim baza danych zdążyła odpowiedzieć.

exports.findRelation = (req, res) => {
    let startId, endId;
    let start = getStation(req.body.start)
        .then(result => {
            startId = result._id;
        });
    let end = getStation(req.body.end)
        .then(result => {
            endId = result._id;
        });
    Promise.all([start, end])
        .then(() => {
            res.send({ startId, endId });
        });
}

function getStation(name) {
    return Stations.findOne({ name: name }).exec();
}

Pewnie da się to jeszcze jakoś odchudzić.

Dzięki za chęć pomocy.

---

zrób wszystko na async await, jest czytelniej i mniej kodu. ja na przykład też zawsze lubię zdestrukturyzować req.body na początku kontrolera, potem jest łatwiej z tego korzystać i się tak nie gubię, ale to moje widzimisię chyba :D

ale jakim cudem baza odpowiadała po wysłaniu response skoro przy zapytaniu do bazy miałeś await?

---

Nie wiem, też się zastanawiam, choć dla mnie async await to czarna magia, zupełnie tego konceptu nie rozumiem, .then/.catch jest o wiele bardziej zrozumiałe.

---

koniecznie przełącz się na async await to "lepsza" praktyka.

---

No rzeczywiście wygląda to lepiej:

exports.findRelation = async (req, res) => {
    let { _id: start } = await getStation(req.body.start);
    let { _id: end} = await getStation(req.body.end);
    res.send({ start, end });
}

Zamiast

exports.findRelation = (req, res) => {
    let startId, endId;
    let start = getStation(req.body.start)
        .then(result => {
            startId = result._id;
        });
    let end = getStation(req.body.end)
        .then(result => {
            endId = result._id;
        });
    Promise.all([start, end])
        .then(() => {
            res.send({ startId, endId });
        });
}

 

---

nie ogarniam za bardzo mongoose ale zobacz czy nie da się pobierać tylko tych właściwości, które potrzebujesz, pewnie się da. jeżeli później nie chcesz edytować zmiennych to użyj const to jest zdecydowanie dobra praktyka. domyślnie zawsze używaj const a jeżeli już nie ma innej opcji to wtedy dopiero let.

---

Myślałem nad tym czy nie wybierać domyślnie jednej wartości, ale jednak podobna funkcja będzie mi potrzebna jeszcze wielokrotnie, więc wolę wybrać tak jak to jest zrobione, niż robić funkcję pod każdy przypadek.

---

z tego co widzę w kodzie i tak destrukturyzujesz to co Ci baza zwraca. ja bym chyba tak nie robił. jak baza Ci zwróci Error bo cokolwiek to się wykrzaczy, w szczególności, że zapytanie do bazy nie jest w bloku try catch. imo każde zapytanie do bazy powinno być w try catch.

---

Już tak mam jak piszesz. To co wysłałem to była powiedzmy taka wersja alpha

Answer 1

Jeśli używasz Mongoose lub MongoDB, to podając callback jako ostatni parametr do metody ona nie zwraca promisa, więc nie musisz tego await-ować. Albo await i wtedy bez callbacka, albo callback bez await.

https://mongoosejs.com/docs/api.html#model_Model.findOne

Podpowiedzi zamieściłem w komentarzach w kodzie. Przy okazji - czy któryś z przypadków nie rzuca w konsoli błędem? Funkcje, w których używasz await muszą być asynchroniczne - słówko async - a nie w każdym przypadku u Ciebie tak jest, więc konsola powinna rzucać błędem.

---

Pierwszy przypadek:

exports.findRelation = (req, res) => {
    let startId, endId;
    getStation(req.body.start)
    .then(result => { // [1]
        startId = result;
    });
    getStation(req.body.end)
    .then(result => { // [2]
        endId = result;
    });
    console.log(startId, endId); // [3] <- to i return poniżej wykonają się przed punktami 1 i 2, ponieważ tam oczekujesz na promisa
    return res.json({startId, endId});
}
 
async function getStation (name) {
    await Stations.findOne({name: name}, (err, result) => {
        if (err) return console.log(err);
        else return result._id; // <- tego nie zwracasz poza metodę Stations.findOne
    });
   // <- tutaj powinieneś zwrócić wartość z wywołania Stations.findOne(..)
}

Powinno działać w ten sposób:

exports.findRelation = async (req, res) => {
    const startId = await getStation(req.body.start);

    const endId = await getStation(req.body.end);

    console.log(startId, endId);
    return res.json({startId, endId});
}
 
function getStation (name) {
    return Stations.findOne({name: name});;
}

---

Drugi przypadek wygląda ok, tylko zapomniałeś oznaczyć funkcji, w których używasz await jako async i ponowny błąd z brakiem returna w getStation:

exports.findRelation = async (req, res) => { // <- tutaj async
    const startId = await getStation(req.body.start);
    const endId = await getStation(req.body.end);
    console.log(startId, endId);
    return res.json({startId, endId});
}
 
function getStation (name) {
    return Stations.findOne({name: name});
}

---

Trzeci przypadek - co tutaj jest logowane w console.log(stationsId); pod koniec funkcji? Na oko powinna być tablica z propertisami startId i endId, ale nie w formie indeksowanych elementów, lecz propertisów obiektowych, bo zamiast pushować te zmienne do tablicy, to przypisałeś je jak do obiektu.

---

Czwarty przypadek - czemu tworzysz zmienną global a później operujesz na zmiennej stationsId? Poza tym, podobnie jak w trzecim przypadku.

---

Uwaga ogólna - gdy używasz await, to warto ten kod owinąć w try..catch, żeby móc obsłużyć ewentualny błąd.

Comments

Dzięki za pomoc, już znalazłem rozwiązanie. Async Await był zupełnie niepotrzebny, ale musiałem zamknąć odpowiedź w Promise.all aby wykonywała się dopiero po spełnieniu wszystkich obietnic.

Rozwiązanie jest wyżej, ale wersja z tablicą zamiast osobnych zmiennych nie działa.

---

Rozwiązanie jest wyżej, ale wersja z tablicą zamiast osobnych zmiennych nie działa.

 

Czemu? Jak to obsługujesz? 

---

Podobnie jak w działającym przykładzie, inicjalizuję pustą tablicę na początku i w .then dodaję do niej wartości. Na samym końcu okazuje się pusta.

---

A wstaw jakiś console.log w miejscu, w którym przypisujesz coś do tablicy w callbacku - możliwe, że to się dzieje po tym jak wykonsolujesz tablicę i odeślesz odpowiedź do klienta.