[Nomad Coder] 노개북 클린코드 챌린지 2기 Assignment 7

 

mission1

// BAD 더러운 코드 😣
// Hint❕ : 검색하기 쉬운 이름을 사용하세요.
// blastOFF는 로켓 발사를 의미. 86400000은 하루의 밀리초 (milliseconds) 의미. 

// What the heck is 86400000 for?
setTimeout(blastOff, 86400000);

// GOOD 😎
// 위 코드를 깨끗하게 다시 작성해 주세요.
const secondsOfDay = 86400000;

setTimeout(blastOff, secondsOfDay);



// 어떻게 고쳤는지, 사례에서 무엇을 배워야 하는지 설명해주세요.

 상수는 밖으로 빼서 관리하는 것이 좋습니다.

mission2

// BAD 더러운 코드 😣
// Hint❕ : 의미있는 이름을 사용해주세요.

const yyyymmdstr = moment().format("YYYY/MM/DD");

// GOOD 😎
// 위 코드를 깨끗하게 다시 작성해 주세요.

const date = moment().format("YYYY/MM/DD");



// 어떻게 고쳤는지, 사례에서 무엇을 배워야 하는지 설명해주세요.

 변수는 읽기 쉽고, 의미가 명확해야 한다.

mission3

// BAD 더러운 코드 😣
// Hint❕ : 불필요하게 반복하지 마세요.

const Car = {
  carMake: "Honda",
  carModel: "Accord",
  carColor: "Blue"
};

function paintCar(car, color) {
  car.carColor = color;
}

// GOOD 😎
// 위 코드를 깨끗하게 다시 작성해 주세요.

const Car = {
  make: "Honda",
  model: "Accord",
  color: "Blue"
}

function paintCar(car, color) {
  car.color = color;
}



// 어떻게 고쳤는지, 사례에서 무엇을 배워야 하는지 설명해주세요.

  굳이 클래스의 이름을 attribute에 넣을 필요는 없다.
  불필요하게 계속 반복되기 때문이다.