Если инструкция, которая проверяет несколько тернарных операторов Javascript

Question

Мне интересно, почему мое решение не работает. У меня есть следующие:

//tells if type should be included in the row data 
isInReport = {factual: true, eac: false, variance: false} 

//report has hundreds of objects, each with a type of either Plan, Factual, EAC, Variance 
report = [{type: "Plan"}, {type: "Factual"}, {type: "EAC"}, {type: "Variance"}]; 

мне нужно перебрать массив отчетов и сделать что-то всегда, если item.type это «план», или, если это один из других 3-х типов, но только если это true в объекте isInReport. Поэтому в моем примере оператор if должен пройти, если item.type является «Plan» или «Factual»

Почему этот код не работает? Логика кажется мне правдой, даже немного странной. Когда я тестировал, он всегда возвращает все типы независимо от того, что. Спасибо за любую помощь!

report.map(function (item) { 
    if (
    item.type === "Plan" || 
    item.type === (isInReport.factual) ? "Factual" : "Plan" || 
    item.type === (isInReport.eac) ? "EAC" : "Plan" || 
    item.type === (isInReport.variance) ? "Variance" : "Plan" 
) { 
    //do stuff 
    } 
}); 

Answer 1

ли вы хотите сделать:

if (item.type === "Plan" || isInReport[ item.type.toLowerCase() ]) { 
    //do stuff 
} 

Существует комментарий предполагая это не правильно. Можете ли вы подтвердить, что вы ожидаете в результате для 4 пунктов в report?

//tells if type should be included in the row data 
 
isInReport = {factual: true, eac: false, variance: false} 
 

 
//report has hundreds of objects, each with a type of either Plan, Factual, EAC, Variance 
 
report = [{type: "Plan"}, {type: "Factual"}, {type: "EAC"}, {type: "Variance"}]; 
 

 
report.forEach(function(item){ 
 
    if (item.type === "Plan" || isInReport[ item.type.toLowerCase() ]) { 
 
    console.log("Item Type:" + item.type + " PASSED TEST"); 
 
    } else { 
 
    console.log("Item Type:" + item.type + " FAILED TEST"); 
 
    } 
 
});

Если вы хотите, чтобы придерживаться пути вы начали, то вы хотите использовать некоторые скобки, чтобы лучше контролировать порядок или opperations.

//tells if type should be included in the row data 
 
isInReport = {factual: true, eac: false, variance: false} 
 

 
//report has hundreds of objects, each with a type of either Plan, Factual, EAC, Variance 
 
report = [{type: "Plan"}, {type: "Factual"}, {type: "EAC"}, {type: "Variance"}]; 
 

 
report.forEach(function(item){ 
 
    if (
 
    item.type === "Plan" || 
 
    item.type === (isInReport.factual ? "Factual" : "Plan") || 
 
    item.type === (isInReport.eac ? "EAC" : "Plan") || 
 
    item.type === (isInReport.variance ? "Variance" : "Plan") 
 
) { 
 
    console.log("Item Type:" + item.type + " PASSED TEST"); 
 
    } else { 
 
    console.log("Item Type:" + item.type + " FAILED TEST"); 
 
    } 
 
});

Answer 2

Я не вижу ошибку ... я скрипка его здесь: http://jsfiddle.net/Lnkky0fw/

$(document).ready(function() { 
var isInReport = {factual: true, eac: false, variance: false}; 

//report has hundreds of objects, each with a type of either Plan, Factual, EAC, Variance 
var report = [{type: "Plan"},{type: "Factual"},{type: "EAC"},{type: "Variance"}]; 

report.map(function (item) { 
    if (
    item.type === "Plan" || 
    item.type === (isInReport.factual) ? "Factual" : "Plan" || 
    item.type === (isInReport.eac) ? "EAC" : "Plan" || 
    item.type === (isInReport.variance) ? "Variance" : "Plan" 
) { 
    //do stuff 
    alert('ok'); 

    } 
}); 
}); 

Answer 3

Вы отсутствуете запятые между элементами в массиве «отчет».

Answer 4

Я бы создать массив допустимых значений, а затем использовать фильтр. Это облегчит чтение и поддержку, чем многократная вложенная, если/тройная смесь.

var isInReport = { 
 
    factual: true, 
 
    eac: false, 
 
    variance: false 
 
}; 
 

 
var report = [{ type: "Plan" }, { type: "Factual" }, { type: "EAC" }, { type: "Variance" }]; 
 

 
var allowed = ["plan"] 
 
    .concat(Object.keys(isInReport) 
 
    .map(function (key) { 
 
     if (isInReport[key]) return key.toLowerCase(); 
 
    }).filter(function (v) { 
 
     return v; 
 
    }) 
 
); 
 

 
var filtered = report.filter(function (d) { 
 
    if (allowed.indexOf(d.type.toLowerCase()) > -1) return true; 
 
    return false; 
 
}); 
 

 
console.log(filtered);

Answer 5

Вы должны приложить тройные выражения в скобках, чтобы получить ожидаемый результат

if (
    item.type === "Plan" || 
    item.type === ((isInReport.factual) ? "Factual" : "Plan") || 
    item.type === ((isInReport.eac) ? "EAC" : "Plan") || 
    item.type === ((isInReport.variance) ? "Variance" : "Plan") 
) 

(и вы забыли запятые в

report = [{type: "Plan"},{type: "Factual"},{type: "EAC"},{type: "Variance"}];)