狠狠撸

?? ?? ??? ??
????? ?? ??(SIG) ?? ??
Hong Hyo Sang
???? : ??? ?? ?????? ??
???? ??? ?? ??
?? : https://www.refactoring.com/catalog/, https://refactoring.guru/refactoring
“ ???? ??? ? ?? ??? ???? ? ? ??.
??? ?????? ??? ??? ? ?? ??? ??.”
???? ??? ???? ?? ??? ?? ?? ?

II. ????? ?? ??(SIG) ??
Contents
1. ?? ??? ?? ??
2. ?? ??? ???? ??
3. ??? ??? ????
4. ?? ?????? ?? ??
5. ???? ??? ?? ? ?? ??
1. Reliabilty(???), Scalabilty(???)
2. Maintainability(?????)
I. ???? ?? ? ????

Content
I. ????

???? ??? ?????? ????? ???? ???? ???? ???? ???? ??? ??? ? ??? ??? ???? ?? ?.
?? : ?? : ??? ?? ?????? ??
? ???? ??
????? ??, ? ??, ?? ?
- ????? ????? ?? ??? ??(redundancy)? ????
??
? ??? RAID ??
? ?? ?? ???? ??
? ?? ??? ?? ??? ??
? ????? ??
?? ??, ?? ??? ???? ?? ???? ?? ?
- ?? ?? ???, ???? ??, ???? ?? ???? ??
???? ??? ?? ?? ?? ??
- ? ??? ???, ???? ??, ?? ????? ?? ????? ?
??? ??? ?? ??? ??
? ?? ????
? ??? ?? ?? ?
- ??????? ?? ? ?? ??
- ?? ??? ?
? scaling up / scailing out

????? ??? ???? ?? ??? ?? ???? ??? ?? ?.
?? : ?? : ??? ?? ?????? ??, https://devforperson.tistory.com/entry/ISO-25010-%ED%92%88%EC%A7%88-%ED%8A%B9%EC%84%B1
? ??? (Operability)
???? ???? ???? ?? ??? ? ??? ??. ? ??? ??? ?.
? ??? (Simplicity)
???? ???? ??? ??? ?? ???? ?? ??? ?? ??? ????? ??? ???? ??? ?
???? ???? ????? ? ???? ??? ???? ?? ?? ??? ? ?? ???
? ?? ??? (Functional suitability), ?? ??? (Performance efficiency), ??? (Compatibility), ??? (Usability), ??? (Reliability), ?? (Security), ?
???? (Maintainability), ??? (Portability) ??? ? ?? ???
? ??? (Modularity) : ??? ??? ??? ??? ?? ?? ?? ??? ???(SW)? ??? ??
? ???? (Reusability) : ??(??)? ? ? ??? ????? ??? ? ???, ?? ??? ??? ? ?? ??
? ??? (Analyzability) : ??(???)? ??? ????, ??? ??? ???? ????? ???? ??? ????? ?? ??? ???? ?? ??
? ?? ??? (Modifiability) : ??(???)? ??? ?, ?? ??? ??? ?????? ??? ????? ???? ????? ????? ??? ? ?? ??
? ?? ??? (Testability) : ??(???, ????)? ??? ??? ????? ??? ? ?? ??
? ??? (Portability) : ??(???, ????)? ?? ??? ??(SW/HW/Network) ???? ??? ??? ??
? ??? (Adaptability) : ??(???)? ?? SW(HW)? ????? ????? ????? ??? ? ?? ??
? ?? ??? (Installability) : ??(???)? ????? ??/??? ? ?? ??
? ??? (Replaceability) : ??? ??? ???? ??? ??? ?? ?? ??? SW ???? ??? ? ?? ??

????? ??? ???? ?? ??? ?? ???? ??? ?? ?.
?? : http://i-bada.blogspot.com/2012/07/blog-post_3628.html
? ??? ??? ??? ???? ?? ?????? ????.
? ?????? ???? ?? ??? ??? ?? ???? ?? ???? ??? ??? ?? ??? ???? ??.
- ?? ??? ?? ??? ?? -> ?? ?? ?? ??
? ?????? ??? ? ???? ????? ???? ?????? ?? ??.

Content
1. ?? ??? ?? ??
2. ?? ??? ???? ??
3. ??? ??? ????
4. ?? ?????? ?? ??
5. ???? ??? ????
6. ?? ??
II. ????? ?? ??(SIG) ??

1. ?? ??? ?? ??
??? ??? 15??? ???? ?? ???? ?? ???? ??(??)??, ?????, ????? ?? ??.
? ??? ????? ??? ??? ???? ??.
public void test() {
if (true) {
log.debug(“step 1”);
}
if(!actionA()) {
for ( …. ) {
action ….
}
}
if(!actionB()) {
for ( …. ) {
action ….
}
}
}
if (true) {
}
actionA_B()
}
private void actionA_B() {
if(!actionA()) {
for ( …. ) {
action ….
}
}
if(!actionB()) {
for ( …. ) {
action ….
}
}
}
if (true) {
}
actionA_B()
}
private void actionA_B() {
runActionA();
runActionB();
}
private void runActionA() {
if(!actionA()) {
for ( …. ) {
action ….
}
}
}
private void runActionB() {
if(!actionB()) {
for ( …. ) {
action ….
}
}
}
?? ??? ??? ???? ??? ?? ??? ???
??? ???? ?? ??.
? ???? ???, ???? ??? Extract Method
? ??? ? ?? ???? ???? ? ??? ?? ??? ??? ?? ????? ? ???
??? ??? ??. ??? ?? ???? ?? ?? ?? ?? ???? ??? ? ??.
???? ?? ?? ??? ?? ???? ?? ?? ???? ?? ? ??? ??.
1.? ?? ?? ??
2.?? ?? ??
3.??? ???? ??? ????? ?, ??? ?? ????
???
?? : ?? ???? ????
? ??? ?? ??? ??? ??? ??? ??? ?? ?
- ?????? ??? ?? ??? ??? ??? ??? ???? ?? ? ??
- ??? ?? ??? Loop? ?? ?? ??? ???? ?? ( JVM? ???? )
? ??? ?? ??? ??? ??? ???
- ?? ???? ??? ?? ?? ?? ?? ?? ?? ? ? ?? ??? ?
? ??? ???? ???? ?? ??
- ? ??? ?? ?? ?? ??? ??? ??? ??? ?? ?? ?? ??? ?
? ??
* SIG ?? ?? ?? ??? ?? ?? ? ? 4?? ?? ?? ???
60 ?? ?? 6.9% ??
30 ?? ?? 22.3% ??
15 ?? ?? 43.7% ??
15 ?? ??? 56.3%

2. ?? ??? ???? ??
?? ???? ???? ?? ? ?? ? ???? ?? ??. (?? ? ???? 4? ??? ???? ??? ??? ? ?? ??? ?? ?? ?? ?? ??
?? ?? ? ???? ?? ??.)
– ???? ??? ??? CASE? ???
?? ???? : ?? ??? ??? ? ??? ?? ????? ???? ??? ?? ????
?? ???? ?? ??
?? ?? : ? ??? ?? ?? ??? ?? ??? ???? ?? ?? ??? ?? ?? ??
?? ?? ?? : ??? ??? ??? ?? ?
?? ??? ( ??? ??? ) : ??? + 1 ?? “??? ???? 5? ?? ??“ ?? ???
????? ??? ???? ??? ??? ???.
switch (a) {
case A :
…
break;
case B :
…
break;
case C :
…
break;
case D :
…
break;
case E :
…
break;
default :
…
break;
}
?? ??? : 6
Case ?? ?? ? ? ?? ???? ?? ?
? JAVA ???
- if,
- case,
- ?,
- &&, ||
- while
- for
- catch
If-then-else ?? ?? ?? ???
- ???? ??
- ??? ??? ?? ??
- ???? ??? ?
??? ?? : ??? ???? ???? ??? ??? ???? ??
??? ?? : ??? ?? ?? ??? ???? ?? ??? ??? ?? ??? ?? ?
???? ??
?? ? ??? ?? ?? : ???? ?? ???? ??? ??? ??? ??? ?? ??
?? ??? ?? : ?? ????? ?? ???? ?? ??? ?? ?? break, continue??
??. ??
?? ? ??? ?? ?? ??? : ?? ? ??? ??? ??? ???? ?? ?? ??? ?
? ?? ??? ?? ?? ??? CASE???? ?? ?? ?? ?? ?? ?? ?,
???? ????? ??? : ?? ?? ?? ??? ??? ??? ??? ?? ?? ???
??? ?? ?? ?? ???? ???. ?? ???? ?? ???? ?? ??? ??
? ???? ???? ???? ?? ?? ???
- ???? ???? ???? ?? ??? ??? ??? CASE? ?? ?? ?? ???, ?
?? ??? 4? ??? ??? ??? ?? ?? ???? ?? ?? ??? ?? ????
?? ??? ???? ?? ??? ? ?? ?? ? ? ??.
* SIG ?? ??? ?? ??? ?? ?? ? ? 4?? ?? ?? ???
> ??? 25 ?? ?? 1.5% ??
> ??? 10 ?? ?? 10.0% ??
> ??? 5 ?? ?? 25.2% ??
<= ??? 5 ?? ??? 74.8%

2. ?? ??? ???? ??
?? ???? ???? ?? ? ?? ? ???? ?? ??. (?? ? ???? 4? ??? ???? ??? ??? ? ?? ??? ?? ?? ?? ?? ??
?? ?? ? ???? ?? ??.)
Decompose Conditional
if (date.before(SUMMER_START) || date.after(SUMMER_END)) {
charge = quantity * winterRate + winterServiceCharge;
}
else {
charge = quantity * summerRate;
}
if (isSummer(date)) {
charge = summerCharge(quantity);
}
else {
charge = winterCharge(quantity);
}
* ?????? ?? *
1. Extract Method ? ?? ???? ??? ???? ??
2. then? else??? ?? ????? ?? .

2. ?? ??? ???? ??
?? ???? ???? ?? ? ?? ? ???? ?? ??. (?? ? ???? 4? ??? ???? ??? ??? ? ?? ??? ?? ?? ?? ?? ??
?? ?? ? ???? ?? ??.)
Consolidate Conditional Expression
if (anEmployee.seniority < 2) return 0;
if (anEmployee.monthsDisabled > 12) return 0;
if (anEmployee.isPartTime) return 0;
if (isNotEligableForDisability()) return 0;
private boolean isNotEligableForDisability() {
return ((anEmployee.seniority < 2)
|| (anEmployee.monthsDisabled > 12)
|| (anEmployee.isPartTime));
}
* ?????? ?? *
1. ?? ??? ???? ?? and?? or. ?? :
- ?? ? ???? and
- ?? ???? ?? or
2. ??? ??? ?? Extract Method ? ?? ?? ???? ??? ???? ???? ??

2. ?? ??? ???? ??
?? ???? ???? ?? ? ?? ? ???? ?? ??. (?? ? ???? 4? ??? ???? ??? ??? ? ?? ??? ?? ?? ?? ?? ??
?? ?? ? ???? ?? ??.)
Consolidate Duplicate Conditional Fragments
if (isSpecialDeal()) {
total = price * 0.95;
send();
}
else {
send();
}
if (isSpecialDeal()) {
}
else {
}
send();
* ?????? ?? *
1. ?? ?? ?? ??

2. ?? ??? ???? ??
?? ???? ???? ?? ? ?? ? ???? ?? ??. (?? ? ???? 4? ??? ???? ??? ??? ? ?? ??? ?? ?? ?? ?? ??
?? ?? ? ???? ?? ??.)
Replace Control Flag with Break
for (const p of people) {
if (! found) {
if ( p === "Don") {
sendAlert();
found = true;
}
for (const p of people) {
if ( p === "Don") {
sendAlert();
break;
}

2. ?? ??? ???? ??
?? ???? ???? ?? ? ?? ? ???? ?? ??. (?? ? ???? 4? ??? ???? ??? ??? ? ?? ??? ?? ?? ?? ?? ??
?? ?? ? ???? ?? ??.)
Replace Nested Conditional with Guard Clauses
public double getPayAmount() {
double result;
if (isDead){
result = deadAmount();
}
else {
if (isSeparated){
result = separatedAmount();
}
else {
if (isRetired){
result = retiredAmount();
}
else{
result = normalPayAmount();
}
}
}
return result;
}
public double getPayAmount() {
if (isDead){
return deadAmount();
}
if (isSeparated){
return separatedAmount();
}
if (isRetired){
return retiredAmount();
}
return normalPayAmount();
}

2. ?? ??? ???? ??
?? ???? ???? ?? ? ?? ? ???? ?? ??. (?? ? ???? 4? ??? ???? ??? ??? ? ?? ??? ?? ?? ?? ?? ??
?? ?? ? ???? ?? ??.)
class Bird {
// ...
double getSpeed() {
switch (type) {
case EUROPEAN:
return getBaseSpeed();
case AFRICAN:
return getBaseSpeed() - getLoadFactor() * numberOfCoconuts;
case NORWEGIAN_BLUE:
return (isNailed) ? 0 : getBaseSpeed(voltage);
}
throw new RuntimeException("Should be unreachable");
}
}
public interface Bird {
double getSpeed();
}
class European implements Bird {
double getSpeed() {
return getBaseSpeed();
}
}
class African implements Bird {
double getSpeed() {
return getBaseSpeed() - getLoadFactor() * numberOfCoconuts;
}
}
class NorwegianBlue implements Bird {
double getSpeed() {
return (isNailed) ? 0 : getBaseSpeed(voltage);
}
}
private static final Map<Nationallity, Flag> FLAGS = new HashMap<>();
static {
FLAGS.put(EUROPEAN, new European();
FLAGS.put(AFRICAN, new African();
FLAGS.put(NORWEGIAN_BLUE, new NorwegianBlue();
}
public double getSpeed(Nationallity nationallity) {
Flag flag = FLAGS.get(nationallity);
flag = flag != null ? Flag : new DefaultNational();
return flag.getSpeed();
}

3. ??? ??? ????
??? ???? ?? ??? ??? ??? ?? ??? ?????? ??? ?? ???? ?? ??? ?? ???? ???.
- ?? ?? ??? ??.
- ???? ???. : ?? ?? ?? ?
- ???? ??? : ?? ?? ?? ?
- ?? : ?? ???, ?? ???? ?? ???, ?? ?? ???? ?? ?? ???? ???
??? ??
??? 6?? ??? ??? ?? ? ?
- ?? ?? ??, ?? ??
Type 1 Clone : 6?? Clone ??
public void setName(String name) {
this.name = name;
}
this.name = name;
}
Type 2 Clone :
String lastName = “??”;
this.name = name + lastName ;
…
}
String lastName = “??”;
this.name = name + lastName ;
…
}
???? ??? : ??? code? ???? ?? ?? ???? ??? static ?? ??
?? ??? : ??? code? ???? ?? ?? ?? ???? ?? ? extends? ??? ??
? ?? ?? ???? ??? ???? ? ?? ??
- ?? ? ??. ??? ??? ?? ????? ?? ??? ??.
- 1. ?? ?? ?? ??
- 2. ?? ?? ???? ?? ?? ?? ?? ???? ??? ?? ??
? ??? ??? ??? ?? ??.
- ?? ??? ??? ?? ?????? ??? ??? ??? ??
? ? ??? ??, ?? ?? ?? ??.
- ?? ?? ??? ??? ??? ??. ?? ??? ?? ?? ?? ?
* SIG ?? ?? ??? ??? ?? ? 4?? ?? ?? ???
?? ?? 95.4% ??
?? 4.6% ??

4. ?? ?????? ?? ??
????? ?? ??( 4? ?? )? ???? ??, ???, ????? ????. 4? ???? ???? ??? ??
* SIG ?? ?????
??? ???? ?? ? 4?? ?? ?? ???
7? ?? 0.7 % ??
5~6 ? 2.7 % ??
3~4 ? 13.8 % ??
2? ?? 86.2 % ??

5. ???? ??? ????
???? ??? ??(???)? ??? ?? ??? ?? ??? ??? ? ??.
- ???? ???? ???? ?? ??.
- ?? ??? ??? ?? ?? ?? ??? ????? ??? ???.
- ???? ?? ???? ??? ??.
- ??? ???? ?? ??? ?? ???? ??? ?? ???? ???? ?? ?? ? ??
????. ???? ? ? ?? ???? ???? ?? ?? ?? ?? ? ??.
- ??? ?? ??? ??? ???? ??
: ?? ????? ?? ??? ?? ?? ?? ?? ???? ?? ???? ??
- ???? ?? ?? ?? ???? ?? ??? ??? ?? ?? ?? ???? ??? ???
??.
?? ??? ??
?? ???? ????? ?? ??
??? ??? ???? ?????/??? ?? ??
* Java Interface? ??? 2? ??? ???? ?? ?? ?? ??

6. ?? ??
?? ??.
- ?? ? ???? ??, ??, ???? ? ??? ??? ?? ?? ???? ???? ?? ??? ??? ?? ???? ?? ??
- ??. ???? ??? ??? ?? ?? ??? ?? ?? ? ????. ??? ? ? ?? ??? ?? ???? ??? ?? ??
- ?? ???? TODO, ?? ?? ?? ???? ?? ?? ??
- ???? ??? ??? ??? ????? ???? ? ?? ???? ??? ? ??? ??.
- ????? ???? ???? ?? ??? ?? ???? ??? ?? ??? ? ????? ???? ??? ? ?? ???? ??.
?? C. ??" ?? ??? ???? 7? ??
? ?? ??? ?? ??? ??? ??.
? ?? ??? ??? ??.
? ?? ?? ??? ??? ??.
? ?? ??? ??? ??.
? ? ??? ??? ??? ??.
? ?? ??? ??? ??.
? ??? ?? ? ? ??? ??? ??.

www.iabacus.co.kr
Tel. 82-2-2109-5400
Fax. 82-2-6442-5409
THANKS

狠狠撸

Java ???? ??? ?? ??

Recommended

More Related Content

What's hot (19)

Similar to Java ???? ??? ?? ?? (20)

More from Hyosang Hong (20)

Java ???? ??? ?? ??