비동기 함수의 ESLint 일관 반환 오류를 전달하려면 false를 반환해도 괜찮습니까?

Question

저는 ExpressJS를 사용하고 있으며 코드를 실행하고 ESLint 규칙을 전달해야했습니다. 다음과 같은 코드에 대해 throw되는 "consistent-return"규칙이 있습니다.

function getUsers(req, res, next){ 
    if(req.userIds.length === 0){ 
    return next(); 
    } 

    collection.find({"_id": {$in: req.userIds}}, function(err, doc){ 
    if(err){ 
     return next(); 
    } 

    req.users = doc; 
    return next(); 
    });//find() 
}//getUsers() 

이제이 코드는 'consistent-return'오류를 표시합니다. 이 오류를 제거하는 2 가지 가능한 변경 사항이있는 것 같습니다. 올바른 것이 확실하지 않습니다.

function getUsers(req, res, next){ 
    if(req.userIds.length === 0){ 
    return next(); 
    } 

    collection.find({"_id": {$in: req.userIds}}, function(err, doc){ 
    if(err){ 
     return next(); 
    } 

    req.users = doc; 
    return next(); 
    });//find() 

    return false; // adding return false passes the ESLint error 
}//getUsers() 

또는

function getUsers(req, res, next){ 
    if(req.userIds.length === 0){ 
    return next(); 
    } 

    // adding return in below line passes the ESLint error. 
    return collection.find({"_id": {$in: req.userIds}}, function(err, doc){ 
    if(err){ 
     return next(); 
    } 

    req.users = doc; 
    return next(); 
    });//find() 
}//getUsers() 

그것에 대해 갈 수있는 올바른 방법이다 알려 주시기 바랍니다. 감사.

Answer 1

여기에있는 규칙은 실제 일관성 문제를 나타냅니다. 때로는 값을 반환하는 것처럼 보이기도하고 그렇지 않은 경우도 있습니다. 반환 된 값이 무시되기 때문에 작동합니다. return 구문의 유일한 포인트는 나머지 기능을 실행하지 않는 것입니다.

여기의 코드는 오해의 소지가 있으며 가독성이 떨어집니다.

교체

if(err){ 
    next(); 
    return; 
} 

전체 없습니다 혼란 코드

if(err){ 
    return next(); 
} 

:

function getUsers(req, res, next){ 
    if(req.userIds.length === 0){ 
     next(); 
     return 
    } 
    collection.find({"_id": {$in: req.userIds}}, function(err, doc){ 
     if(!err){ 
      req.users = doc; 
     } 
     next(); 
    });//find() 
}//getUsers() 

Answer 2

몇 가지 세부 사항이 밖으로 제거 우리는, 코드 비트를 분석해 보겠습니다 :

function getUsers(req, res, next){ 

    if(req.userIds.length === 0){ 
    return next(); 
    } 

    collection.find({"_id": {$in: req.userIds}}, function(err, doc){ 
    // lotsa code here 
    }); 

} 

여기에서 함수 getUsers() 안에는 오류 조건 (req.userIds이 비어있을 때) 또는 두 가지 코드 경로가있을 수 있습니다.

오류 조건이 만족되는지 여부에 따라 getUsers() 함수는 무언가 (next()) 또는 아무것도 반환하지 않습니다 (실제로는 undefined을 암시 적으로 반환 함). collection.find()은 두 번째 경우에서 실행되며, 그 자체가 무언가를 반환 할 수도 있지만 getUsers()은 그것에 대해 알 수 없습니다.

이제 ESLint docs 상태에서 함수는 무언가 또는 아무것도 반환하지 않을 수도 있지만 (암시 적 undefined) 둘 다 반환 할 수는 없습니다 (이는 일관성없는 동작입니다). 그렇지 않으면 consistent-return 오류가 발생합니다.

따라서 오류를 해결하려면 Denys '코드와 사용자 자신의 두 번째 가능한 변경이 모두 작동합니다. Denys의 코드에서 두 코드 경로는 아무 것도 반환하지 않고 (암시 적 undefined) 귀하의 경우 모두 반환합니다.