if文の共通処理をリファクタリングする方法がどうしてもわかりません

Question

前提・実現したいこと

jsでif文を使用して条件分岐させているのですが、
そこで発生している共通処理をどうにかしてリファクタリングしたいのですが、
調べてもどうしても分かりません。

条件分岐からさらに分岐するelse ifのnumにゼロを入れる処理が
三つあり共通処理となっております。

どなたかご教示して頂けましたら幸いです。

該当のソースコード

function test(num, id){
    if(id === 'aiueo') {
        if (num > 4) {
            num = 4;
        }else if(value < 0){
            num = 0;
        }
    }else if(id === 'kakiku'){
        if(num > value){
            num = value;
        }else if(num < 0){
            num = 0;
        }
    }else if(id === 'sasisu'){
        if (num > obj.test) {
            num = obj.test;
        }else if(num < 0){
            num = 0;
        }
    }
    return num;
}

Comments

[maisumakun]

> 三つあり共通処理となっております。

いちばん上と残り2つで条件が違いますが、それは意図したものでしょうか？

[yambejp]

value やobjなどfunction外から持ち込んでいますか？
その辺の情報提示がないとなんとも言えません

Answer 1

さいしょのif(value < 0){がif(num < 0){の間違いだと仮定します。
「0未満だったら0にする」というのは num = Math.max(num, 0) と書けます。この程度の量でしたら無理にコードを共通化する必要はなく、同じコードを3回書いても問題ないのでは。

function test(num, id){
    if (id === 'aiueo') {
        num = Math.min(Math.max(num, 0), 4);
    } else if (id === 'kakiku') {
        num = Math.min(Math.max(num, 0), value);
    } else if (id === 'sasisu') {
        num = Math.min(Math.max(num, 0), obj.test);
    }
    return num;
}