1. Refactoring
Chapter 09 - 조건문 간결화

2. Decompose Conditional(조건문 쪼개기)
복잡한 조건문(if-then-else)이 있을 때는 if, then,
else 부분을 가각 메서드로 빼내자.
if 절을 별도의 메서드로 빼내자.
then 절과 else 절을 각각의 메서드로 빼내자.

3. If

4. (date.before(SUMMER_START)

5. ||

6. date.after(SUMMER_END))

8. charge

9. =

10. quantity

11. *

12. _winterRate

13. +

14. _winterServiceCharge;

15.
else

17. charge

18. =

19. quantity

20. *

21. _summerRate;
If

22. (notSummer(date))

24. charge

25. =

26. winterCharge(quantity);

27.
else

29. charge

30. =

31. summerCharge(quantity);

32. Consolidate Conditional
Expression(중복 조건식 통합)
여러 조건 검사식의 결과가 같을 때는 하나의 조건문으로 합
친 후 메서드로 빼내자.
모든 조건문에 부작용이 없는지 검사하자.
하나라도 부작용이 있으면 X
여러 개의 조건문을 논리 연산자를 사용해 하나의 조건문으로
바꾸자.
합친 조건문에 메서드 추출 적용을 고려하자.

33. double

34. disabilityAmount()

35. {

37. if

38. (_seniority

40. 2)

41. return

42. 0;

44. if

45. (_monthDisabled

47. 12)

48. return

49. 0;

51. if

52. (_isPartTime)

53. return

54. 0;

59. …⋯
double

60. disabilityAmount()

61. {

63. if

64. (isNotEligableForDisability())

65. return

66. 0;

71. …⋯

72. Consolidate Duplicate Conditional
Fragments(조건문의 공통 실행 코드 빼기)
조건문의 모든 절에 같은 실행 코드가 있을 때는 같은 부분을 조건문 밖
으로 빼자.
조건에 상관없이 공통적으로 실행되는 코드를 찾자.
공통 코드가 조건문의 앞 절에 있을 땐 조건문 앞으로 빼자.
공통 코드가 조건문의 끝 절에 있을 땐 조건문 뒤로 빼자.
공통 코드가 조건문의 중간 절에 있을 땐 앞뒤의 코드와 위치를 바꿔도 되는지 판단하
자. 그래서 바꿔도 된다면 조건문의 앞이나 끝 절로 뺀 후 앞의 단계처럼 조건문의 앞이
나 뒤로 빼자.
공통 코드 명령이 둘 이상일 땐 메서드로 만들자.

73. if

74. (isSpecialDeal())

75. {

77. total

78. =

79. price

80. *

81. 0.95;

83. send();

84.
}

85. else

86. {

88. total

89. =

90. price

91. *

92. 0.98;

94. send();

95.
}
if

96. (isSpecialDeal())

98. total

99. =

100. price

101. *

102. 0.95;

103.
else

105. total

106. =

107. price

108. *

109. 0.98;

110.
send();

112. Remove Control Flag(제어 플래그 제거)
논리 연산식의 제어 플래그 역할을 하는 변수가 있을 때는 그 변수를 break 문
이나 return 문으로 바꾸자.
break 문이나 continue 문이 있을 때
논리문을 빠져나오게 하는 제어 플래그 값을 찾자.
그 제어 플래그 값을 대입하는 코드를 break 문이나 continue 문으로 바꾸자.
break 문이나 continue 문이 없는 언어에서
로직을 메서드로 빼내자.
논리문을 빠져나오게 하는 제어 플래그 값을 찾자.
빠져나오게 하는 값을 return 문으로 바꾸자.

113. void

114. checkSecurity(String[]

115. people)

116. {

118. boolean

119. found

120. =

121. false;

123. for

124. (int

125. i

126. =

127. 0;

128. i

130. people.length;

131. i++)

132. {

135. if

136. (!found)

137. {

141. if

142. (people[i].equals(“Don”))

143. {

148. sendAlert();

153. found

154. =

155. true;

159. }

163. if

164. (people[i].equals(“John”))

165. {

170. sendAlert();

175. found

176. =

177. true;

181. }

184. }

186. }

187.
}
void

188. checkSecurity(String[]

189. people)

190. {

192. for

193. (int

194. i

195. =

196. 0;

197. i

199. people.length;

200. i++)

201. {

204. if

205. (people[i].equals(“Don”)

206. ||

207. people[i].equals(“John”))

208. {

212. sendAlert();

216. break;

219. }

221. }

222.
}
TODO: try one more step

223. Replace Nested Conditional with Guard
Clauses(여러 겹의 조건문을 감시 절로 전환)
메서드에 조건문이 있어서 정상적인 실행 경로를 파악하기
힘들 땐 모든 특수한 경우에 감시 절을 사용하자.
조건 절마다 감시 절을 넣자.
그 감시 절은 값을 반환하거나 예외를 통지한다.
각 조건 절을 감시 절로 바꿀 때마다 컴파일과 테스트를 실시
하자.
모든 감시 절의 결과가 같다면 중복 조건식 통합 기법을 실시하자.

224. double

225. getPayAmount()

226. {

228. double

229. result;

231. if

232. (_isDead)

233. result

234. =

235. deadAmount();

237. else

238. {

241. if

242. (_isSeparated)

243. result

244. =

245. separatedAmount();

248. else

249. {

253. if

254. (_isRetired)

255. result

256. =

257. retiredAmount();

261. else

262. result

263. =

264. normalAmount();

267. }

269. }

271. return

272. result;

273.
}
double

274. getPayAmount()

275. {

277. if

278. (_isDead)

279. return

280. deadAmount();

282. if

283. (_isSeparated)

284. return

285. separatedAmount();

287. if

288. (_isRetired)

289. return

290. retiredAmount();

292. return

293. normalAmount();

294.
}

295. Replace Conditional with
Polymorphism(조건문을 재정의로 전환)
객체 타입에 따라 다른 기능을 실행하는 조건문이 있을 때는 조건문의 각 절을 하
위클래스의 재정의 메서드 안으로 옮기고, 원본 메서드는 abstract 타입으로 수
정하자.
조건문이 메서드 코드의 일부라면 그 조건문을 따로 떼서 메서드 추출을 적용하자
필요하다면 메서드 이동을 적용해서 조건문을 최상위 클래스로 옮기자.
하위클래스 중 하나를 택해서 그 안에 조건문 메서드를 재정의하는 메서드를 작성하자. 조건문의
해당 절에 있는 코드를 그 하위클래스 메서드로 옮기고 적절히 수정하자.
메서드로 복사해 넣은 조건문 안의 절은 삭제하자.
조건문의 나머지 절의 코드도 마찬가지로 하위클래스 메서드 안으로 옮기자.
상위클래스 메서드를 abstract 타입으로 만들자.

296. class

297. Employee…⋯

298. {

300. int

301. payAmount()

302. {

305. switch

306. (getType())

307. {

311. case

312. EmployeeType.ENGINEER:

317. return

318. _monthlySalary;

322. case

323. EmployeeType.SALESMAN:

328. return

329. _monthlySalary

330. +

331. _commission;

335. case

336. EmployeeType.MANAGER:

341. return

342. _monthlySalary

343. +

344. _bonus;

348. default:

353. throw

354. new

355. RuntimeException(“Exception”);

358. }

360. }

361.
}
class

362. Employee…⋯

363. {

365. int

366. payAmount()

367. {

370. return

371. _type.payAmount(this);

373. }

374.
}

375.
class

376. EmployeeType…⋯

377. {

379. abstract

380. int

381. payAmount(Employee

382. emp);

383.
}

384.
class

385. Engineer…⋯

386.
class

387. Salesman…⋯

388. Introduce Null Object(Null 검사를 널
객체에 위임)
Null 값을 검사하는 코드가 계속 나올 때는 null 값을 널 객체로 만들자.
원본 클래스 안에 널 객체 역할을 할 하위클래스를 작성하자. 원본 클래스와 널 클래스 안에 isNull
메서드를 작성하자. 원본 클래스의 isNull 메서드는 false 를 반환해야 하고, 널 클래스의 isNull 메
서드는 true 를 반환해야 한다.
원본 객체에 요청하면 null 을 반환할 수 있는 부분을 전부 찾자. 그래서 그 부분을 널 객체로 바꾸자.
원본 타입의 변수를 null 과 비교하는 코드를 전부 찾아서 isNull 호출로 바꾸자.
클라이언트가 null 이 아니면 한 메서드를 호출하고 null 이면 다른 메서드를 호출하는 case 문을 전
부 찾자.
각 case 문마다 널 클래스 안의 해당 메서드를 다른 기능의 메서드로 재정의하자.
재정의 메서드를 사용하는 부분에서 조건문을 삭제하고 컴파일과 테스트를 실시하자.

389. if

390. (customer

391. ==

392. null)

393. plan

394. =

395. BillingPlan.basic();

396.
else

397. plan

398. =

399. customer.getPlan();
plan

400. =

401. customer.getPlan();
class

402. Customer

403. {

405. static

406. Customer

407. newNull()

408. {

411. return

412. new

413. NullCustomer();

415. }

417. BillingPlan

418. getPlan()

419. {

422. …⋯

424. }

425.
}

426.
…⋯

427.
Customer

428. getCustomer()

429. {

431. return

432. (_customer

433. ==

434. null)

435. ?

438. Customer.newNull()

439. :

442. _customer;

443.
}

444.
…⋯

445.
class

446. NullCustomer

447. extends

448. Customer

449. {

450.
…⋯

452. Introduce Assertion(어설션 넣기)
일부 코드가 프로그램의 어떤 상태를 전제할 때는 어
설션을 넣어서 그 전제를 확실하게 코드로 작성하자.
어설션 기능을 사용할 수 있는 Assert 클래스를 작성하자.

453. double

454. getExpenseLimit()

455. {

457. //

458. 비용

459. 한도를

460. 두던지,

461. 주요

462. 프로젝트를

463. 정해야

464. 한다.

466. return

467. (_expenseLimit

468. !=

469. NULL_EXPENSE)

470. ?

473. _expenseLimit

474. :

477. _primaryProject.getMemberExpenseLimit();

478.
}
double

479. getExpenseLimit()

480. {

482. Assert.isTrue(_expenseLimit

483. !=

484. NULL_EXPENSE

489. ||

490. _primaryProject

491. !=

492. null);

494. return

495. (_expenseLimit

496. !=

497. NULL_EXPENSE)

498. ?

501. _expenseLimit

502. :

505. _primaryProject.getMemberExpenseLimit();

506.
}