【リファクタリングメモ】vol.3 -JavaScript -

内容

以下のswitch文をリファクタリングすること。 冗長で拡張性が低い。

// カードの値を数字に変換する
function switchToValue(card) {
 switch(card) {
   case "A":
     card = 1;
     break;
   case "J":
     card = 11;
     break;
   case "Q":
     card = 12;
     break;
   case "K":
     card = 13;
     break;
 } 
   return card;
}

修正案

ポイント: オブジェクトを使用してカードの値を変換する。

function switchToValue(card) {
    const cardValue = {
        "A": 1,
        "J": 11,
        "Q": 12,
        "K": 13
    } 
   return cardValue[card];
}