Как сделать эту функцию более эффективной?

Question

У меня есть эта структура данных типов кредитных карт.

Было бы неплохо сделать hasTransFee более эффективным. Если бы я начал добавлять карты хранения и т. Д. В этот список, он мог бы стать довольно большим, и чем быстрее он будет работать, тем лучше.

У кого-нибудь есть предложения?

$scope.creditCards = [ 
    { name: 'VISA DEBIT/DELTA', value: 'DEL', transactionFee: false}, 
    { name: 'VISA CREDIT', value: 'VIS', transactionFee: true }, 
    { name: 'MASTERCARD CREDIT', value: 'MSC', transactionFee: true }, 
    { name: 'MASTERCARD DEBIT', value: 'MCD', transactionFee: false }, 
    { name: 'MAESTRO', value: 'MAE', transactionFee: false }, 
    { name: 'SWITCH', value: 'SWI', transactionFee: false }, 
    { name: 'VISA ELECTRON', value: 'ELC', transactionFee: false }, 
    { name: 'SOLO', value: 'SOL', transactionFee: false } 
    ]; 


var hasTransFee = function(cardType) 
{ 
for (var i=0; i < $scope.creditCards.length; i++) { 
    if($scope.creditCards[i].value==cardType && $scope.creditCards[i].transactionFee == true){ 
    return true; 
    } 
    } 
    return false; 
} 

Answer 1

Кажется, что value является уникальным идентификатором, если это так, вы можете хранить «кредитные карты» в object вместо этого, как это:

$scope.creditCards = { 
    'DEL': { name: 'VISA DEBIT/DELTA', transactionFee: false}, 
    'VIS': { name: 'VISA CREDIT', transactionFee: true }, 
    'MSC': { name: 'MASTERCARD CREDIT', transactionFee: true } 
    }; 

И тогда вам не нужно даже функцию для проверки, если кредитная карта имеет transactionFee, если вы все еще хотите иметь функцию, эта функция будет выглядеть следующим образом:

var hasTransFee = function(cardType){ 
    return $scope.creditCards[cardType].transactionFee; 
} 

Answer 2

Побочное примечание: Это лучше всего подходит для CodeReview

Попробуйте это:

var hasTransFee = function(cardType) 
{ 
    var thecard; 
    $scope.creditCards.some(function(item) { 
     return item.value == cardType && (thecard = item); 
    }); 
    return thecard && thecard.transactionFee; 
}; 

Это только итерацию, насколько это необходимо для того, чтобы найти карту с правильный тип и не далее (ваш исходный код будет продолжать сканировать весь массив, если карта найдена, но не имеет платы).

Внимание: Только ОДИН =&& thecard = item. Он намеренно назначает , необходимый для выполнения окончательного return работы.