関数名に変数を使用する

Question

ロジックを複製する2つの関数がありますが、私は自分のコードをリファクタリングしたいと思います。型として渡される値は、「day」または「night」です。機能は以下のとおりです。

function doBooking(type, rate, nights) { 
    if (type=="day") { 
    reservation.day(rate, nights, function(err, data) { 
     do (a) 
     do (b) 
     do (c) 
    }); 
    } else { 
    reservation.night(rate, nights, function(err, data) { 
     do (a) 
     do (b) 
     do (c) 
    }); 
    } 
}); 

私は賢いことと、これは以下のようなものにするために改善したいと思いますが、それはうまく動作しませんでした...

function doBooking(type, rate, nights) { 
    ["reservation."+type](rate, nights, function(err, data) { 
    do (a) 
    do (b) 
    do (c) 
    }); 
}); 

Answer 1

このような何か試してみてください：

を

function doBooking(type, rate, nights) { 
    var _type = (type === "day") ? "day" : "night"; 
    reservation[_type](rate, nights, function(err, data) { 
    do (a) 
    do (b) 
    do (c) 
    }); 
}); 

Answer 2

同じビジネスロジックを共有している場合、昼夜の別々の機能を使用するのはどうしてですか？あなたは時間の別の議論を追加できませんでしたか？

function doBooking(type, rate, nights) { 
    reservation.make(type, rate, nights, function(err, data) { 
     // (a), (b), & (c) happen inside the reservation handler 
     // If you do anything unique to day or night, do it here in the callback of the generic handler 
    }); 
} 

これ以外にも、ここで機能を分けることができるはるかにクリーンな方法があります。あなたが知られているタイプを扱うことができますので、予約を処理するために定義されたクラスを開始します。

function Reservation (type, customer) { 
    this.type = type 
    this.customer = customer 
    this.rate 
    this.nights = [] 
    this.isBooked 
    this.make = function(callback) { 
     longRunningBookingOperation(this, function(err, data) { 
      this.isBooked = data.result === "success" && !err 
      callback(err, data) 
     }) 
    } 
} 

そして、それを使用する：

const customer = Customer({firstname: "Tom", lastname: "Jones", id: 2392015572-1827}) 
var reservation = Reservation(ReservationTypeDay, customer) 
reservation.make(err, data) { 
    // handle the result 
})