Intellij Idea-Warnung – „Zurückgegebenes Versprechen wird ignoriert“ mit aysnc/await

Question 1

Ich benutze Express.js in meinem Code mit Node.js v7.3. In diesem habe ich eine erstellt User Router die die Anfragen an my weiterleitet User Controller.

Ich verwende async/await in der User Controller asynchrone Aufrufe durchzuführen. Das Problem ist, dass IntelliJ mir eine Warnung gibt, die das sagt

Das von login() zurückgegebene Promise wird ignoriert.

Die Sache ist, ich gebe nicht einmal etwas von der zurück login() Methode.

Hier ist der Code –

UserRouter.js

router.post('/login', function (req, res, next) {
    userController.login(req, res); // I get the warning here
});

UserController.js

exports.login = async function (req, res) {
    try {
        const verifiedUser = await someFunction(req.body.access_code);
        let user = await User.findOrCreateUser(verifiedUser);
        res.status(200).send(user);
    }
    catch (err) {
        res.status(400).send({success: false, error: err});
    }
};

Wenn ich dieselbe Anmeldemethode nur mit nativen Versprechungen schreibe, erhalte ich diese Warnung nicht. Verstehe ich hier etwas falsch oder ist IntelliJ schuld?

BEARBEITEN –

Dank @Stephen verstehe ich, dass eine asynchrone Funktion ein Versprechen zurückgibt, aber wäre es nicht besser, wenn Intellij feststellt, dass nichts von der asynchronen Funktion zurückgegeben wird, und diese Warnung nicht anzeigt, denn wenn ich a verkette .then() nach dem login() Funktion, es bietet eine undefined Objekt in das dann Ergebnis. Das heißt, wenn wir nicht explizit etwas von der async-Funktion zurückgeben, wird undefiniert zurückgegeben?

Question 2

Das userController.login() Die Funktion gibt ein Versprechen zurück, aber Sie tun nichts mit dem Ergebnis des Versprechens, indem Sie its verwenden then() Funktion.

Zum Beispiel:

userController.login(req, res).then(() => {
    // Do something after login is successful.
});

oder in der ES2017-Syntax:

await userController.login(req, res);

Wenn Sie dort eigentlich nichts tun wollen, können Sie die Warnung einfach ignorieren. Die Warnung ist meistens da, weil das nicht verwendet wird then() Funktion auf einem Versprechen ist normalerweise ein Codegeruch.

Question 3

Die Sache ist, dass ich nicht einmal etwas von der Methode login() zurückgebe.

Eine als “async” deklarierte Funktion gibt per Definition ein Promise zurück. Siehe zum Beispiel https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/async_function

Die IDEA-Warnung ist jedoch nur eine Inspektion. Sie können bei der Warnung „alt-enter, right“ drücken und die Inspektionsebene ändern, damit die Warnung verschwindet. Die Inspektion befindet sich in der Kategorie “JavaScript -> Wahrscheinliche Fehler” und heißt “Ergebnis des Methodenaufrufs, der ein Versprechen zurückgibt, wird ignoriert”.

Question 4

router.post('/login', function (req, res, next) {
    void userController.login(req, res); // I get the warning here
});

Sie sollten den Operator “void” verwenden.

Question 5

wenn du wirklich manisch bist wie ich und die then() nicht erforderlich ist, aber die Warnung verschwinden soll, ist eine mögliche Lösung:

functionWithAsync.error(console.error);

Question 6

Eine andere Möglichkeit, die Warnung loszuwerden, ist die Definition eines Leerzeichens then():

userController.login(req, res); // <- Get the warning here

userController.login(req, res).then(); // <- No warning

Question 7

Wenn Sie diese Warnung einfach für eines der JetBrains-Produkte ausschalten möchten. Gehe zu

Preferences > Inspections > JavaScript and TypeScript | Async code and promises | Result of method call returning a promise is ignored und schalten Sie die Einstellung aus.

Question 8

Ich benutze try{} catch(e){} in NodeJs und fand das einfach durch Hinzufügen Error() zum Ende der Funktion wurde die Warnung behoben.

Vollständiger Code:-

someArray.forEach(async (arrayValue) => {
    try {
        const prodData = await myAsyncFunc(arrayValue);
    } catch(e) {
        console.error(`Error: ${e}`);
    }
}, Error());