Javascriptでreturn文を含む共通ロジックを抽出する方法は？ここ

Question

は私のコードです：

function function1() { 
    var ids = GetIds(); // predefined function 
    if (ids.length === 0) { 
     alert("Please select at least one item to procees."); 
     return; 
    } 

    ... 
} 

function function2() { 
    var ids = GetIds(); // predefined function 
    if (ids.length === 0) { 
     alert("Please select at least one item to procees."); 
     return; 
    } 

    ... 
} 

function function3() { 
    var ids = GetIds(); // predefined function 
    if (ids.length === 0) { 
     alert("Please select at least one item to procees."); 
     return; 
    } 

    ... 
} 

共通部分を抽出するためにどのように？コードを再因子化する方法は？ return文は非常に扱いにくいです。このコードを再因子に関連付けるパターンはありますか？

ありがとうございます！

if (ids.length === 0) { 
     alert("Please select at least one item to procees."); 
     return; 
    } 

Answer 1

できません。あなたができる唯一のことはalert()をGetIds()に移動することです。しかし、この場合、関数名はちょっと混乱します。

var ids = GetIds(); 
if(!ids) return; 

それともあなたはこのようにそれを変更することができます：

GetIds(function(ids) { 

}); 

あなたGetIds機能は、その後、例えばなりますこのように：

function GetIds(callback) { 
    var ids = ....; 
    if(!ids) alert('...'); 
    else callback(ids); 
} 

Answer 2

あなたは試みることができる：

は

function checkId(ids) { 
    if (ids.length === 0) { 
     alert("Please select at least one item to procees."); 
     return false; 
    } 
    return true; 
} 

function function1() { 
    var ids = GetIds(); 
    if (!checkId(ids)) return; 

    // More code here 
} 

Answer 3

これはdecorator patternに適しています。その後、

function decorateGetId(f) { 
    return function() { 
    var ids = GetIds(); 
    if (ids.length === 0) { 
     alert("Please select at least one item to process."); 
     return; 
    } else { 
     return f(ids); 
    } 
    }; 
} 

いずれか

function function1(ids) { 
    // ... 
} 
function1 = decorateGetId(function1); 

または私の好み、functionステートメントは、巻上で奇妙なことをしているため。

var function2 = decorateGetId(function (ids) { 
    // function 2 body ... 
});