Simplify осложнено, если оператор

Question

Этот следующий код взят из Backbone.js Events#off()

Как можно упростить, если часть, так что легко понять?

for (i = list.length - 2; i >= 0; i -= 2) { 
    if (!(callback && list[i] !== callback || context && list[i + 1] !== context)) { 
     list.splice(i, 2); 
    } 
} 

Answer 1

некоторых булеву математику:

!(callback && list[i] !== callback || context && list[i + 1] !== context) 
!(!(!callback || list[i] === callback) || !(!context || list[i + 1] === context)) 
(!callback || list[i] === callback) && (!context || list[i + 1] === context) 

Я думаю, что disjunctive normal form намного легче понять, чем отрицание conjunctive normal form.

Для того, чтобы отдельные части легче понять, использовать говоря имена переменных (я не знаю, масштабы этого, вы можете найти что-то лучше):

var rightCallback = !callback || list[i] === callback; 
var rightContext = !context || list[i + 1] === context; 
if (rightCallback && rightContext) {…} 

Answer 2

var noCallback = callback && list[i] !== callback; 
var noContext = context && list[i + 1] !== context; 
if(! noCallback && ! noContext){...} 

Или удалить отрицания, как советовал в комментариях:

var callbackCheck = !callback || list[i] === callback; 
var contextCheck = !context || list[i + 1] === context; 
if(callbackCheck && contextCheck){...} 

Answer 3

Я предлагаю, чтобы инкапсулировать заявление в функцию с описательным именем (я не понимаю, что вы хотите делать), а затем вызвать

if (true == meaningfulConditionName(list, callback)) 
{ 
    list.splice(i, 2); 
} 

Answer 4

Вот моя версия, прошли все Backbone .js тесты, и будет медленнее, чем оригинал

if(
     // callback defined, context is null 
     // remove callback for all context 
     (callback && list[i] === callback 
     && !context) 

     // callback defined and context is defined 
     // ie remove specified callback for specified context 
     || (callback && list[i] === callback 
      && context && list[i+1] === context) 

     // callback NOT defined and context is defined 
     // ie removed all callback for specific context 
     || (!callback && list[i] !== callback 
      && context && list[i+1] === context) 

     // callback NOT defined and context NOT defined 
     // i.e. removed everything, this is unreachable code 
     // as it is caught by 
     // https://github.com/documentcloud/backbone/blob/master/backbone.js#L112 
     // and https://github.com/documentcloud/backbone/blob/master/backbone.js#L103 
     || (!callback && !context) 

    ) { 

      // remove callback and context 
      list.splice(i, 2); 
    }       

Answer 5

не может быть упрощена больше, чем это:

for (i = list.length - 2; i >= 0; i -= 2) callback && list[i] !== callback || context && list[i + 1] !== context || list.splice(i, 2)