1. Refactoring

2.
메소드

3. 호출의

4. 단순화
cecil

5. Contents
•메소드를

6. 직관적으로

7.
•파라미터

8. 갯수의

9. 조정

10.
•불필요한

11. 정보를

12. 숨김

13.
•생성자를

14. 팩토리

15. 메소드로

16.
•메소드의

17. 예외

18. 선언

19. 메소드를

20. 직관적으로

21.
(기본적인

22. 리팩토링)
•Rename

23. Method

24.
• 메소드의

25. 이름이

26. 그

27. 목적을

28. 드러내지

29. 못하면

30. 이름을

31. 바꿔라

32.
•Add

33. Parameter

34.
• 메소드를

35. 호출하는

36. 부분에서

37. 더

38. 많은

39. 정보를

40. 필요로

41. 한다면

42. 파라미터를

43. 추가하라

44.
• 이

45. 리팩토링을

46. 하지

47. 않도록

48. 하는

49. 것이

50. 중요(대안:

51. 파라미터

52. 오브젝트

53. 도입)

54.
•Remove

55. Parameter

56.
•파라미터가

57. 메소드에서

58. 더

59. 이상

60. 사용되지

61. 않으면

62. 제거하라

63. 파라미터

64. 갯수

65. 조정을

66. 위한

67.
리팩토링
많은

68. 수의

69. 파라미터는

70. 메소드를

71. 복잡하게

72. 만든다

73. Separate

74. Query

75. from

76. Modifier
Customer
getTotalOutstanding

77.
setReadyForSummaries
Customer
getTotalOutstandingAndSetReadyForSummaries
값을

78. 리턴할

79. 뿐만

80. 아니라

81. 상태도

82. 변경하는

83. 메소드는

84. 분리하라
값을

85. 리턴하고,

86. 눈에

87. 띄눈

88. 부작용이

89. 없는

90. 것은

91. 매우

92. 가치

93. 있는

94. 일

95.
예외)

96. 동시성

97. 문제:

98. 동시성이

99. 문제가

100. 될

101. 경우

102. getAndSet

103. 메소드를

104. 가질

105. 필요가

106. 있음.

107. Parameterize

108. Method
몇몇

109. 메소드가

110. 다른

111. 값을

112. 포함하고

113. 있는

114. 것을

115. 제외하고는

116. 비슷한

117. 일을

118. 하고

119. 있다면,

121.
파라미터를

122. 가진

123. 하나의

124. 메소드로

125. 만들어라
Employee
fivePercentRaise

126.
tenPercentRaise
Employee
raise(percentage)

127. Parameterize

128. Method

129.
(cont’d)
공통된

130. 부분이

131. 덜

132. 명확하게

133. 나타나는

134. 경우

135.
파라미터로

136. 넘겨

137. 받을

138. 수

139. 잇는

140. 몇개의

141. 값을

142. 기초로

143. 반복되는

144. 코드를

145. 찾아라!
protected

146. Dollars

147. baseCharge()

148. {

151. double

152. result

153. =

154. Math.min(lastUsage(),100)

155. *

156. 0.03;

165. if

166. (lastUsage()

168. 100)

169. {

176. result

177. +=

178. (Math.min

179. (lastUsage(),200)

180. -

181. 100)

182. *

183. 0.05;

193. }

198.
if

199. (lastUsage()

201. 200)

202. {

203. result

204. +=

205. (lastUsage()

206. -

207. 200)

208. *

209. 0.07;

216.
}
return

217. new

218. Dollars

219. (result);

221.
}

222.
protected

223. Dollars

224. baseCharge()

225. {
double

226. result

227. =

228. usageInRange(0,

229. 100)

230. *

231. 0.03;
result

232. +=

233. usageInRange

234. (100,200)

235. *

236. 0.05;
result

237. +=

238. usageInRange

239. (200,

240. Integer.MAX_VALUE)

241. *

242. 0.07;

245. return

246. new

247. Dollars

248. (result);

250.
}

251.
protected

252. int

253. usageInRange(int

254. start,

255. int

256. end)

257. {

258. if

259. (lastUsage()

261. start)

265. return

266. Math.min(lastUsage(),end)

267. -

268. start;

269. else

270. return

271. 0;

272.
}

274. Replace

275. Parameter

276. with

277. Explicit

278. Method
파라미터의

279. 값에

280. 따라

281. 다른

282. 코드를

283. 실행하는

284. 메소드가

285. 있다면,

286.
파라미터

287. 값에

288. 대한

289. 별도의

290. 메소드를

291. 만들어라
void

292. setValue

293. (String

294. name,

295. int

296. value)

297. {

300. if

301. (name.equals(height))

304. _height

305. =

306. value;

307. if

308. (name.equals(width))

311. _width

312. =

313. value;

316. Assert.shouldNeverReachHere();

317.
}
void

318. setHeight(int

319. arg)

320. {

323. _height

324. =

325. arg;

326.
}
void

327. setWidth

328. (int

329. arg)

330. {

332. _width

333. =

334. arg;

336.
}
장점:

337. 조건에

338. 따른

339. 행동을

340. 피할

341. 뿐만

342. 아니라,

343. 컴파일

344. 타임에

345. 검사

346. 가능

348. ex)

349. Switch.setState(true)

350. -

351. Switch.beOn()

352.
파라미터

353. 값의

354. 변화가

355. 크다면,

356. 이

357. 리팩토링을

358. 적용하면

359. 안됨.

360. Preserve

361. Whole

362. Object
어떤

363. 객체에서

364. 여러개의

365. 값을

366. 얻은

367. 다음

368. 메소드를

369. 호출하면서

370.
파라미터로

371. 넘기고

372. 있다면,

373. 대신

374. 그

375. 객체를

376. 넘겨라
int

377. low

378. =

379. daysTempRange().getLow();

380.
int

381. high

382. =

383. daysTempRange().getHigh();

385.
withinPlan

386. =

387. plan.withinRange(low,

388. high);
withinPlan

389. =

390. plan.withinRange(daysTempRange());
장점:

391. 파라미터

392. 리스트

393. 변경에

394. 유연하게

395. 대처가

396. 가능,

397. 코드의

398. 가독성을

399. 높임.

400.
단점:

401. 파라미터로

402. 넘겨진

403. 객체와

404. 호출된

405. 객체

406. 사이에

407. 종속성을

408. 생성

409.
만약

410. 이

411. 리팩토링이

412. 종속성

413. 구조를

414. 나쁘면

415. 한다면

416. 사용하면

417. 안됨.

418. Preserve

419. Whole

420. Object

421.
(Example)
class

422. Room...
boolean

423. withinPlan(HeatingPlan

424. plan)

425. {

427. int

428. low

429. =

430. daysTempRange().getLow();

443. int

444. high

445. =

446. daysTempRange().getHigh();

458. return

459. plan.withinRange(low,

460. high);

468.
}

470.
!
class

471. HeatingPlan...

472.
boolean

473. withinRange

474. (int

475. low,

476. int

477. high)

478. {

479. return

480. (low

481. =

482. _range.getLow()

498. high

499. =

500. _range.getHigh());

508.
}
private

509. TempRange

510. _range;

512.
class

513. Room...

515. boolean

516. withinPlan(HeatingPlan

517. plan)

518. {

520. return

521. plan.withinRange(daysTempRange());

523. }

524.
!
class

525. HeatingPlan...

526. boolean

527. withinRange

528. (TempRange

529. roomRange)

530. {

533. return

534. (roomRange.getLow()

535. =

536. _range.getLow()

543. roomRange.getHigh()

544. =

545. _range.getHigh());

547. }
class

548. HeatingPlan...

549. boolean

550. withinRange

551. (TempRange

552. roomRange)

553. {

556. return

557. (_range.includes(roomRange));

560. }

561.
!
class

562. TempRange...

563. boolean

564. includes

565. (TempRange

566. arg)

567. {

570. return

571. arg.getLow()

572. =

573. this.getLow()

579. arg.getHigh()

580. =

581. this.getHigh();

583. }

584. Replace

585. Parameter

586. with

587. Method
객체가

588. 메소드를

589. 호출한

590. 결과를

591. 다른

592. 메소드에

593. 파라미터로

594. 넘기는

595. 상황에서,

596.
호출된

597. 메소드가

598. 이

599. 메소드를

600. 호출할

601. 수

602. 있다면,

603. 직접

604. 호출하도록

605. 변경
int

606. basePrice

607. =

608. _quantity

609. *

610. _itemPrice;
discountLevel

611. =

612. getDiscountLevel();
double

613. finalPrice

614. =

615. discountedPrice

616. (basePrice,

617. discountLevel);
int

618. basePrice

619. =

620. _quantity

621. *

622. _itemPrice;
double

623. finalPrice

624. =

625. discountedPrice

626. (basePrice);
파라미터는

627. 필요할

628. 때만

629. 사용하라.

630. Replace

631. Parameter

632. with

633. Method

634.
(Example)
public

635. double

636. getPrice()

637. {
int

638. basePrice

639. =

640. _quantity

641. *

642. _itemPrice;
int

643. discountLevel;
if

644. (_quantity

646. 100)

647. discountLevel

648. =

649. 2;
else

650. discountLevel

651. =

652. 1;
double

653. finalPrice

654. =

655. discountedPrice

656. (basePrice,

657. discountLevel);

661. return

662. finalPrice;

671.
}

672.
private

673. double

674. discountedPrice

675. (int

676. basePrice,

677. int

678. discountLevel)

679. {

682. if

683. (discountLevel

684. ==

685. 2)

686. return

687. basePrice

688. *

689. 0.1;

691. else

692. return

693. basePrice

694. *

695. 0.05;

697.
}
public

698. double

699. getPrice()

700. {

701. int

702. basePrice

703. =

704. _quantity

705. *

706. _itemPrice;

707. int

708. discountLevel

709. =

710. getDiscountLevel();

711. double

712. finalPrice

713. =

714. discountedPrice

715. (basePrice,

716. discountLevel);

719. return

720. finalPrice;

721.
}

722.
private

723. double

724. discountedPrice

725. (int

726. basePrice,

727. int

728. discountLevel)

729. {

730. if

731. (getDiscountLevel()

732. ==

733. 2)

734. return

735. basePrice

736. *

737. 0.1;

740. else

741. return

742. basePrice

743. *

744. 0.05;

745.
}

746.
!
private

747. int

748. getDiscountLevel()

749. {

752. if

753. (_quantity

755. 100)

756. return

757. 2;

758. else

759. return

760. 1;

761.
}
public

762. double

763. getPrice()

764. {

765. int

766. basePrice

767. =

768. _quantity

769. *

770. _itemPrice;

771. double

772. finalPrice

773. =

774. discountedPrice

775. (basePrice);

778. return

779. finalPrice;

780.
}

781.
private

782. double

783. discountedPrice

784. (int

785. basePrice,)

786. {

787. if

788. (getDiscountLevel()

789. ==

790. 2)

791. return

792. basePrice

793. *

794. 0.1;

797. else

798. return

799. basePrice

800. *

801. 0.05;

802.
}
private

803. double

804. getPrice

805. ()

806. {

807. if

808. (getDiscountLevel()

809. ==

810. 2)

811. return

812. getBasePrice()

813. *

814. 0.1;

817. else

818. return

819. getBasePrice()

820. *

821. 0.05;

822.
}

823.
!
private

824. double

825. getBasePrice()

826. {

829. return

830. _quantity

831. *

832. _itemPrice;

833.
}
Extract

834.
Method
Replace

835. Parameter

836. with

837. Method
Inline

838.
Method

839. Introduce

840. Parameter

841. Object
자연스럽게

842. 몰려

843. 다니는

844. 파라미터

845. 그룹을

846. 가지고

847. 있다면,

848.
그것을

849. 객체로

850. 바꾸어라.
Customer
amountInvoiceIn(Date

851. start,

852. Date

853. end)

854.
amountReceivedIn(Date

855. start,

856. Date

857. end)

858.
amountOverdueIn(Date

859. start,

860. Date

861. end)
Customer
amountInvoiceIn(DateRange)

862.
amountReceivedIn(DateRange)

863.
amountOverdueIn(DateRange)

864. Preserve

865. Whole

866. Object

867. 와

868. 유사

869.
차이점:

870. 이

871. 리팩토링은

872. 새로운

873. 객체를

874. 도입함으로,

875. 리팩토링

876. 절차가

877. 틀림.

878. 불필요한

879. 정보를

880. 숨기는

881. 리팩토링
•Remove

882. Setting

883. Method

884.
• 어떤

885. 필드가

886. 생성시에

887. 값이

888. 정해지고,

889. 이후

890. 변경되지

891. 않아야

892. 한다면

893. 설정

894. 메소드를

895. 제거하라

896.
•Hide

897. Method

898.
• 메소드가

899. 다른

900. 클래스에서

901. 사용되지

902. 않는다면,

903. private로

904. 만들어라

905.
•Encapsulate

906. Downcast

907.
• 메소드가

908. 그

909. 호출부에서

910. 다운캐스트

911. 될

912. 필요가

913. 있는

914. 객체를

915. 리턴하고

916. 있다면,

917. 다운

919.
캐스트를

920. 메소드

921. 안으로

922. 옮겨라

923. (ex:

924. java

925. 1.4이하

926. 버전의

927. Collection

928. Classes)

929. Replace

930. Constructor

931. With

932. Factory

933. Method
객체를

934. 생성할

935. 때

936. 생성하는

937. 것외에

938. 다른

939. 작업을

940. 하고

941. 있다면,

942.
생성자를

943. 팩토리

944. 메소드로

945. 대체하라
class

946. Employee

947. {

948.
private

949. int

950. _type;
static

951. final

952. int

953. ENGINEER

954. =

955. 0;

959. static

960. final

961. int

962. SALESMAN

963. =

964. 1;

973. static

974. final

975. int

976. MANAGER

977. =

978. 2;

988. Employee

989. (int

990. type)

991. {

995. _type

996. =

997. type;

999. }

1000.
}
static

1001. Employee

1002. create(int

1003. type)

1004. {

1007. switch

1008. (type)

1009. {

1012. case

1013. ENGINEER:

1016. return

1017. new

1018. Engineer();

1021. case

1022. SALESMAN:

1025. return

1026. new

1027. Salesman();

1030. case

1031. MANAGER:

1034. return

1035. new

1036. Manager();

1039. default:

1043. …⋯

1045. }

1046.
}

1047.
Employee.create(ENGINEER);
class

1048. Person...

1049. static

1050. Person

1051. createMale(){

1054. return

1055. new

1056. Male();

1059. }

1061. static

1062. Person

1063. createFemale()

1064. {

1068. return

1069. new

1070. Female();

1072. }

1073.
}

1074.
…⋯

1075.
Person

1076. kent

1077. =

1078. Person.createMale();
문자열을

1079. 이용한

1080. 서브

1081. 클래스

1082. 객체

1083. 생성 명시적

1084. 메소드로

1085. 서브

1086. 클래스

1087. 객체

1088. 생성
etc)

1089. 문자열을

1090. 이용한

1091. 생성

1092. 메소드

1093. 예외

1094. 선언과

1095. 관련된

1096.
리팩토링
테스트와

1097. 예외를

1098. 구분하여

1099. 사용하라

1100. Replace

1101. Error

1102. Code

1103. with

1104. Exception
메소드가

1105. 에러를

1106. 나타내는

1107. 특별한

1108. 코드를

1109. 가지고

1110. 있다면,

1111.
대신

1112. 예외를

1113. 던져라.
int

1114. withdraw(int

1115. amount)

1116. {

1119. if

1120. (amount

1122. _balance)

1125. return

1126. -1;

1129. else

1130. {

1133. _balance

1134. -=

1135. amount;

1138. return

1139. 0;

1142. }

1143.
}
void

1144. withdraw(int

1145. amount)

1146. throws

1147. BalanceException

1148. {

1149. if

1150. (amount

1152. _balance)

1156. throw

1157. new

1158. BalanceException();

1161. _balance

1162. -=

1163. amount;

1164.
}
비검사

1165. 예외를

1166. 사용하는

1167. 경우:

1168. 검사의

1169. 책임이

1170. 호출부에

1171. 있는

1172. 경우

1173.
검사

1174. 예외를

1175. 사용하는

1176. 경우:

1177. 검사의

1178. 책임이

1179. 수신수에

1180. 있는

1181. 경우

1182. Replace

1183. Exception

1184. with

1185. Test
호출부에서

1186. 먼저

1187. 검사할

1188. 수

1189. 있는

1190. 조건에

1191. 대해

1192. 예외를

1193. 던지고

1194. 있다면,

1195.
호출부가

1196. 먼저

1197. 검사하도록

1198. 바꿔라.
double

1199. getValueForPeriod

1200. (int

1201. periodNumber)

1202. {

1205. try

1206. {

1209. return

1210. _values[periodNumber];

1211. }

1212. catch

1213. (ArrayIndexOutOfBoundsException

1214. e)

1215. {

1218. return

1219. 0;

1222. }

1223.
}
double

1224. getValueForPeriod

1225. (int

1226. periodNumber)

1227. {

1228. if

1229. (periodNumber

1230. =

1231. _values.length)

1235. return

1236. 0;

1239. return

1240. _values[periodNumber];

1241.
}
예외가

1242. 조건

1243. 테스트를

1244. 대신하는

1245. 역할을

1246. 하면

1247. 안된다.

1248.
호출부에서

1249. 동작을

1250. 호출하기

1251. 전에

1252. 조건을

1253. 테스트

1254. 할

1255. 수

1256. 있다면

1257. 테스트를

1258. 제공해야

1259. 하고,

1260. 호출부에서

1261. 그것을

1262. 사용해야

1263. 한다.

1264. QA

1265. Reference
• Martin

1266. Fowler.

1267. 리팩토링.

1268. (윤성준,

1269. 조재박역).

1270. 서울:

1271. 대청미디어,

1272.
2003