学習71日目 読みやすいコードを書くために

こんにちはKATUUUNです。

本日はリファクタリングについてアウトプットします

リファクタリングとは
実装した機能に影響を与えず、ソースコードを読みやすく改善することです。
実装したコードは自分だけでなく、コードレビューをもらったりと、いろんな人が見る可能性があります。自分が理解できて終わりではなく、誰がみてもわかりやすいコードを記述しましょう。

例えば、
変数名、関数名は理解しやすいものにしましょう

const one = 1;

これではoneという変数名の役割がピンときません

const userId = 1;

これでIdを表していることがわかります。

他には、
ロジックの単純化

const getYear = (year) => {
  if (year % 4 == 0){ 
    if (year % 100 == 0 && year % 400 != 0){
      console.log(`${year}年は閏年ではありません`);
    } else {
      console.log(`${year}年は閏年です`);
    };
  }else {
    console.log(`${year}年は閏年ではありません`);
  };
};

これだとネストの中にネストが入って、分かりにくい

ネストの中のネストを避けて

const isLeapYear? = (year) => {
  if (year % 400 == 0) {
      return true;
  };
  if (year % 100 == 0) {
      return false;
  };
  if (year % 4 ==0) {
      return true;
  };
  return false;
};

if (isLeapYear?(year)) {
  console.log(`${year}年は閏年です`);
} else {
  console.log(`${year}年は閏年ではありません`);
};

これで見やすいコードに変身しました