[노개북] "클린코드" 챌린지 7일차

"노개북" 노마드 개발자 북클럽 7일차!!

#코딩 #개발자 #노마드북클럽 #노개북 #노마드챌린지 #클린코드 #3주스케쥴

2 주차

토, 일 | Assignment #07

• 📚 복습
• ✔️ 미션 Mission 

---

• 오늘의 과제: 더러운 코드를 고쳐라!

문제1

 

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

setTimeout(blastOff, 86400000);

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

const milliseconds = 86400000;
let blastOff = ""; //로켓발사
setTimeout(blastOff, milliseconds);


// 어떻게 고쳤는지, 사례에서 무엇을 배워야 하는지 설명해주세요.
86400000 값에 의미를 부여할 변수를 지정했다.
blastOff의 의미는 변하지 않기 때문에 "로켓발사"라는 의미주석을 달았다.​

 

문제2

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

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

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

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

// 어떻게 고쳤는지, 사례에서 무엇을 배워야 하는지 설명해주세요.
yyyymmddstr 라는 변수의 이름이 애매모호 하여, today 이면서, str이라는것을 강조하기 위해 todayStr 로 변수이름을 수정했다.

 

문제3

// 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;
}

// 어떻게 고쳤는지, 사례에서 무엇을 배워야 하는지 설명해주세요.
car 객체의 변수명에 car가 반복되어있어서 수정했다.