упростить оператор if в Javascript

Question

$scope.vergleich = function () {
    if ($scope.relrechtsform.indexOf(dataService.dic.alt.rechtsformKanlei || dataService.dic.neu.rechtsformKanlei ) !== -1) {
        return true
    } else {
        return false;                   }
    }
}

Я сейчас студент, и IntelliJ говорит мне, что мне нужно упростить это if-утверждение, но я понятия не имею, как. Может быть, кто-нибудь может мне помочь.

Answer 1

Упрощение, вероятно, заключается в том, что, если condition - логическое значение, то:

if (condition) { 
  return true;
} 
else { 
  return false;
}

эквивалентно

return condition;

Однако также, по-видимому, быть логической ошибкой в ​​вашем тесте.

$scope.relrechtsform.indexOf(dataService.dic.alt.rechtsformKanlei ||
                        dataService.dic.neu.rechtsformKanlei ) !== -1

Означает ли не то же самое, что и:

$scope.relrechtsform.indexOf(dataService.dic.alt.rechtsformKanlei) !== -1 || 
$scope.relrechtsform.indexOf(dataService.dic.neu.rechtsformKanlei) !== -1

Answer 2

Может быть, вы ищете это:

 $scope.vergleich = function () {
  return $scope.relrechtsform.indexOf(dataService.dic.alt.rechtsformKanlei || dataService.dic.neu.rechtsformKanlei ) !== -1;
};

Answer 3

Версия Тана - правильный ответ на ваш вопрос. Однако с недавним JavaScript вы можете упростить еще больше благодаря array.includes:

$scope.vergleich = () =>
    $scope.relrechtsform.includes(dataService.dic.alt.rechtsformKanlei || dataService.dic.neu.rechtsformKanlei)

Answer 4

Вы можете просто использовать свое условие вместо выражения IF else -:

$scope.vergleich = function () {
    return ($scope.relrechtsform.indexOf(dataService.dic.alt.rechtsformKanlei ||
                        dataService.dic.neu.rechtsformKanlei ) !== -1);
};