簡化 Javascript 中的 if 語句

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

Tân 的版本是您問題的正確答案。 但是，使用最新的 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);
};