코드를 이렇게 길게 쓰면 더 좋은것인가요?

gurumong의 이미지

아래는 삭제할 물건이 쇼핑카트내에 존재하면 삭제하고 총가격을 새롭게 계산하는 코드인데요
책에서는 위의 방식대로 되어있는데, 이렇게 약간은 풀어 쓰는것이 더 좋은 프로그래밍 습관인가요?

var item:ShoppingCartItem = new ShoppingCartItem(prod);
var sci:ShoppingCartItem = getItemInCart(item);
if (sci != null) {
	cursor.remove();
}
calcTotal();

if (getItemInCart(new ShoppingCartItem(prod)) != null) {
	cursor.remove();
	calcTotal();
}
kyiimn의 이미지

한번 쓰고 다시는 쳐다도 보지 않을 코드라면 되도록 간단하게 쓰는게 나을듯...

하지만 그런 경우는 아마.. 없을듯 하네요^^

xyhan의 이미지


위에 소스는 아무생각없이 볼수 있어서 좋네요..
아래소스는 키보드 조금만 쳐도 되니 좋코요...

============================================================

선한 인간이냐 악한 인간이냐는 그사람의 의지에 달렸다. -에픽테토스-
의지 노력 기다림은 성공의 주춧돌이다. -파스퇴르-

============================================================

============================================================

선한 인간이냐 악한 인간이냐는 그사람의 의지에 달렸다. -에픽테토스-
의지 노력 기다림은 성공의 주춧돌이다. -파스퇴르-

============================================================

Daiquiri의 이미지

스크립트로 쓸 것이 아닌 이상 가독성을 고려하여 코딩하는 것이 좋겠죠.

johan의 이미지

자신의 취향에 따라 코드를 바꾸는 것은 괜찮겠지만, 위의 코드처럼 의미가 달라지면 곤란하죠 - 제 일터라면 코드 리뷰시 바로 지적 당합니다. 두줄 줄이는데 버그 하나 심어놓은 것보다는 그냥 원래대로 놔두는 것이 낫지 않을까요?