후니훈의 모바일 세상

 
void printOwing() {
    Enumeration e = _orders.elements();
    double outstanding = 0.0;
 
    // 배너출력
    System.out.println ("**************************");
    System.out.println ("***** Customer Owes ******");
    System.out.println ("**************************");
 
    // 미지불금액 계산
    while (e.hasMoreElements()) {
        Order each = (Order) e.nextElement();
        outstanding += each.getAmount();
    }
 
    // 상세출력
    System.out.println ("name:" + _name);
    System.out.println ("amount" + outstanding);
}

위 코드를 보면 함수명을 보면 미지불 금액을 출력하는데,  배너도 출력 하고 상세내역도 출력 한다.

이 함수를 기능별로 분리하는 방법을 선호 하는데 이유를 생각해 본다면 다음과 같다.

메서드를 분리 한다면, 분리된 함수들에 대한 사용성이 증가한다.  아래 코드를 보면 배너출력과 상세출력을 분리하였다. 함수명만 봐도 한눈에 이해하기 쉽다. 

또한, 추후에 지불된금액에 대해 출력을 할때,  printBanner() 를 재사용 할 수 있다.

 
void printOwing() {
    Enumeration e = _orders.elements();
    double outstanding = 0.0;
    printBanner();
    // 미지불금액 계산
    while (e.hasMoreElements()) {
        Order each = (Order) e.nextElement();
        outstanding += each.getAmount();
    }
    printDetails(outstanding );
}

// 배너출력
void printBanner() {	
    System.out.println ("**************************");
    System.out.println ("***** Customer Owes ******");
    System.out.println ("**************************");
}
// 상세 출력
void printDetails(double outstanding) {
       System.out.println ("name:" + _name);
       System.out.println ("amount" + outstanding);
}

// 추가된 지불된 금액 함수
void printPaidMoney() {
   printBanner(); // 함수 재사용 증가 
}

 
void printOwing() {
	printBanner();
	double outstanding = getOutstanding();
    printDetails(outstanding );
}
    // 미지불금액 계산
double getOutstanding() {
	Enumeration e = _orders.elements();
    double result = 0.0;
    while (e.hasMoreElements()) {
        Order each = (Order) e.nextElement();
        result += each.getAmount();
    }
	return result;
}

// 배너출력
void printBanner() {	
    System.out.println ("**************************");
    System.out.println ("***** Customer Owes ******");
    System.out.println ("**************************");
}
// 상세 출력
void printDetails(double outstanding) {
       System.out.println ("name:" + _name);
       System.out.println ("amount" + outstanding);
}

1. 리펙토링이란?

- 소프트웨어를 보다 쉽게 이해할 수 있어야 하고, 동작변화 없이 내부 구조를 변경하는 것.

2. 리펙토링의 목적?

- 프로그램을 빨리 작성 할 수 있도록 도와준다.

- 코드 디자인을 개선해준다.

- Bad code -> Good code

3.  Bad code 란? 

 같은 작업을 위해 더 많은 코드 사용, 중복이 많고 이해하기 어렵다. 

 유지보수하기에도 어려운 코드.

4. 리펙토링은 언제하는가?

 틈틈히 계속, 기능추가할때, 버그수정할때, 코드 검토시에.

5. 리펙토링을 할 수 없을때는?

 1) 디자인 실수가 있어 마음대로 리펙토링을 할 수 없을때

 2) 현재 설계된 구조가 보안문제, 퍼포먼스 문제등 중요사항으로 리펙토링을 기대할 수 없을 때.

 3) 코드가 처음부터 작성 하는게 나을 정도로 엉망인 경우

 4) 현재 코드가 작동하지 않을 경우

 5) 마감일이 가까울 경우.

6. 리펙토링할 나쁜코드는 왜 발생하는가?

 - Copy & paste 에 의해 중복 코드 발생.

 - 잘못된 변수명, 함수에서 발생.  (일관성이 중요   add,register, put, create )

 - 특정 클래스내의 메소드가 동작을 하기 위해 다른 클래스에 있는 정보를 많이 필요로 한경우 ( 메서드를 이동한다.)

- 나쁜주석

- 너무긴 메서드, 파라미터

7. 어떤식으로 리펙토링을 시작해야 되는가?

찾기 쉬운것 부터 한다.

 측정할 수 있는 것  (주석, 긴메서드, 거대한 클래스, 긴 매개변수)

   메소드가 하는일 설명, 블록이 하는일 설명

 
 // 배열값의 각 요소들이 이에 대응되는 예측 값과 허용범위 내의 차이를 갖는지 체크한다.
  public boolean compare(int[] expected, int[] actual, int clipLimit, int delta) {
    // clipLimit보다 큰 값을 잘라낸다.
    for (int i = 0; i < actual.length; i++)
      if (actual[i] > clipLimit)
        actual[i] = clipLimit;
    // 비교하려는 두 배열값의 길이가 같은지 체크한다.
    if (actual.length != expected.length)
      return false;
    return true;
  }

다음 강으로...

회사에서 하이브리드 앱 을 개발 중이다. 웹이 80% 이상이고, 20% 가 네이티브 앱으로 되어있다.

개발 하면서 문제가 된 부분이 상당 수다.  

1. 네이티브앱에 비해 웹은 느리다. 

2. 브라우저 오류가 있다.  4.4 킷캣에서 발생하지 않는 오류가 4.0 대 ICS 에 발생한다.

    UI 버그가 상당 하다.

3. 제조사 별로 메모리 관리 하는 용량에 대해 제약조건이 다르기 때문에 발생하는 이슈가 있다.

  예를 들어 상당수의 시간이 걸리는 데이터를 Webview 에서 보여줄 때 삼성폰에서는 정상 동작하나, LG,VEGA 등에서는 앱이 죽는 문제가 발생한다. 

장점이라고 하면

1. 수정할 내용이 있다면 web 은 실시간 반영 이 가능하다. (중요)

2. 수수료를 많이 먹는 인앱결제(구글) 을 사용하지 않고  결제시스템 PG MPI 등을 사용 할 수 있다.

3. 개발 인원을 감축 시킬 수 있다.

장점이 다 비용에 관련된 부분이라 기업에서 추친하는 방향이긴 하지만 

앱 개발자가 봤을 때는 반대입장입니다. 수익을 내려면 우선 앱을 사용자 들이 많이 써야한다.

앱을 많이 쓰게 하려면 UI/UX 는 물론이고, 속도도 빨라야 한다.  

친구중에는 웹페이지 로딩하는데 3~5초 걸린다면  앱을 삭제 하는 경우도 봤다.

내가 기획자라면, 기획자로서 스토리 보드를 만든다면 

결제 페이지는 웹페이지로,  나머지는 전부 네이티브로  결정할 것 같다.