時間、分、秒を変換する - クリーナーコード

Question

javascriptでこの機能を作成しました。それは非常に多くの巣のif-elseの呼び出しをdoens'tようconvertTimes(90,{symbol:"min"},{symbol:"hour"})リターン1.5

function convertTimes(oldTimeValue, oldTimeUnits, newTimeUnits){ 
    const newTimeUnitSymbol = newTimeUnits.symbol 

    if (oldTimeUnits.symbol === newTimeUnitSymbol) { return oldTimeValue } 

    if (oldTimeUnits.symbol === "hour") { 
     if (newTimeUnitSymbol === "min") { return oldTimeValue * 60 } 
     else if (newTimeUnitSymbol === "sec") { return oldTimeValue * 3600 } 

    } else if (oldTimeUnits.symbol === "min") { 
     if (newTimeUnitSymbol === "hour") { return oldTimeValue/60 } 
     else if (newTimeUnitSymbol === "sec") { return oldTimeValue * 60 } 

    } else if (oldTimeUnits.symbol === "sec") { 
     if (newTimeUnitSymbol === "hour") { return oldTimeValue/3600 } 
     else if (newTimeUnitSymbol === "min") { return oldTimeValue/60 } 
    } 
} 

を呼び出すたとえば
は、あなたがこのコードを清掃/リファクタリングの方法を提案してもらえますか？

Answer 1

var conversions = { 
hour: { 
    min: 60, 
    sec: 3600 
}, 
min: { 
    hour: 1/60, 
    sec: 60 
}, 
sec:{ 
    hour: 1/3600, 
    min: 1/60 
} 
} 

return oldTimeValue * conversions[oldTimeUnits.symbol][newTimeUnitSymbol] 

Answer 2

比率にシンボルの組み合わせをマッピングする方法はありますか？

const CONVERSION = { 
    "hour:min":60, 
    "hour:sec":3600, 
    "min:hour":1/60, 
    "min:sec":60, 
    "sec:hour":1/3600, 
    "sec:min":1/60 
} 

function convertTimes(oldTimeValue, oldTimeUnits, newTimeUnits){ 
    // TODO validate symbols 
    if (oldTimeUnits.symbol === newTimeUnits.symbol) { return oldTimeValue } 
    return oldTimeValue * CONVERSION[oldTimeUnits.symbol+":"+newTimeUnits.symbol]; 
}