2022-02-27 clean code 활용 코드 리팩토링
1.
setTimeout(blastOff, 86400000);
const dayToMillisec = 2000;
function launchForRocket() {
console.log("launch! not lunch");
};
setTimeout(launchForRocket, dayToMillisec);
setTimeout(blastOff, 86400000);
const dayToMillisec = 2000;
function launchForRocket() {
console.log("launch! not lunch");
};
setTimeout(launchForRocket, dayToMillisec);
일반적으로 잘모르는 단어 대신 보편적으로 사용하는 단어를 사용하고
setTimeout을 자주 사용 하더라도 86400000라는 숫자를 쉽게 바로 알아 차리진 못한다.
변수에 담음으로써 어떤의미에 숫자로 쓰이는지와 자동완성의 사용 및 오타로 인한
오류를 방지했다.
2.
//const yyyymmdstr = moment().format("YYYY/MM/DD");
const generateDate = moment().format("YYYY/MM/DD");
//const yyyymmdstr = moment().format("YYYY/MM/DD");
const generateDate = moment().format("YYYY/MM/DD");
좀 더 바로 알아볼수있는 변수명으로 작명했다.
3.
const Car = {
carMake: "Honda",
carModel: "Accord",
carColor: "Blue"
};
function paintCar(car, color) {
car.carColor = color;
};
const Car = {
Make: "Honda",
Model: "Accord",
Color: "Blue"
};
function paintCar(car, color) {
car.Color = color;
}
const Car = {
carMake: "Honda",
carModel: "Accord",
carColor: "Blue"
};
function paintCar(car, color) {
car.carColor = color;
};
const Car = {
Make: "Honda",
Model: "Accord",
Color: "Blue"
};
function paintCar(car, color) {
car.Color = color;
}
오브젝트를 호출할때 이미 Car. 로 판별이 가능하니까 굳이 CarMake식으로 작명을 안해도 될꺼같다.
Author And Source
이 문제에 관하여(2022-02-27 clean code 활용 코드 리팩토링), 우리는 이곳에서 더 많은 자료를 발견하고 링크를 클릭하여 보았다 https://velog.io/@pastelblue0721/2022-02-27-clean-code-활용-코드-리팩토링저자 귀속: 원작자 정보가 원작자 URL에 포함되어 있으며 저작권은 원작자 소유입니다.
우수한 개발자 콘텐츠 발견에 전념 (Collection and Share based on the CC Protocol.)