Javascript return enclosing function

Question

У меня есть простой сценарий, в котором я проверяю, существует ли что-то, прежде чем добавлять его, если это так, I return функция (следовательно, выход). Я использую этот шаблон много раз, и я хотел бы отделить его от другой простой функции.

function onEvent(e){ 
    if(this.has(e)) 
      return 
    this.add(e); 
    // More logic different on an event-basis 
} 

Я хотел бы отвязать его следующим образом:

function safeAdd(e){ 
    if(this.has(e)) 
      return 
    this.add(e); 
} 

function onEvent(e){ 
    safeAdd(e); 
    // More logic 
} 

Но очевидно, что сделать это просто return s safeAdd и не выход из onEvent, а остальная часть логики запускается на выполнение в любом случае.

Я знаю, что я мог бы сделать что-то вроде:

function safeAdd(e){ 
    if(this.has(e)) 
      return false 
    this.add(e); 
    return true 
} 

function onEvent(e){ 
    if(!safeAdd(e)) 
     return 
    // More logic 
} 

Но, так как я повторяю это много, я хотел бы быть как можно более кратким.

Answer 1

Вы могли бы вывернуть ее наизнанку с чем-то вроде этого:

function safeAdd(callback) { 
    return function(e) { 
     if(this.has(e)) 
      return false; 
     this.add(e); 
     return callback.call(this, e); 
    }; 
} 

и тогда вы могли бы сделать что-то вроде этого:

var obj = { 
    onEvent: safeAdd(function(e) { 
     console.log('more logic', e); 
    }), 
    onPancakes: safeAdd(function(e) { 
     console.log('pancakes', e); 
    }), 
    has: function(e) { /* ... */ }, 
    add: function(e) { /* ... */ } 
}; 

Демо: http://jsfiddle.net/ambiguous/T6pBQ/

И если вам нужно поддержите больше аргументов в своих функциях, переключите call на apply и используйте arguments вместо e:

function safeAdd(callback) { 
    return function() { 
     if(this.has(arguments[0])) 
      return false; 
     this.add(arguments[0]); 
     return callback.apply(this, arguments); 
    }; 
} 

Демо: http://jsfiddle.net/ambiguous/3muzg/