코드 중복과 보이스카우트 규칙
캠핑 장은 처음 왔을 때보다 더 깨끗하게 해놓고 떠나라!
보이스카우트 규칙 (클린 코드, p50)
때로는 아주 사소한 코드 중복이 큰 문제를 만드는 경우도 있다.
var fnCallback = $Fn(function(bSuccess, sMessage, oData) {
//...
if (acrobat.isInstalled) {
this.printUrl = oData.redirectURL + "&isSilent=true";
} else {
this.printUrl = oData.redirectURL;
}
}, this).bind();
중복 코드에 민감한 개발자라면 당장에 중복된 부분이 눈에 들어올 것이다. 4째 줄과 6째 줄에 this.printUrl = oData.redirectURL 부분이 중복이다. 단순한 코드라서 언듯 보기에는 별 문제가 없어 보일는지 모르겠다. 그러나 꼭 그렇지만은 않다. 문제가 생길 가능성이 있다면 꼭 문제가 생긴다.