1. Chapter 6 Composing Methods ¶

• You have a code fragment that can be grouped together.
  Turn the fragment into a method whose name explains the purpose of the method.
  
  
• 하나로 묶을 수 있는 작은 코드 조각들에 대해서, 그 목적을 잘 표현하는 이름을 가진 메소드에 넣자.
  

~cpp 
    void printOwing(double amount){
        printBanner();
        // print details
        System.out.println( "name:" + _name);
        System.out.println( "amount" + amount);
    }

• A method's body is just as clear as its name.
  Put the method's body into the body of its callers and remove the method.
  

~cpp 
    int getRating(){
        return (moreThanFiveLateDeliveries())?2:1;
    }
    boolean moreThanFiveLateDeliveries(){
        return _numberOfLateDeliveries > 5;
    }

• You have a temp that is assigned to once twith a simple expression, and the temp is getting in the way of other refactorings.
  Replace all references to that temp with the expression.
  

~cpp 
    double basePrice = anOrder.basePrice();
    return (basePrice > 1000)

~cpp 
    return (anOrder.BasePrice() > 1000)

• You have a complicated expression.
  Put the result of the expression, or parts of the expression,in a temporary variagle with a name that explains the purpose.
  

~cpp 
    if ( (platform.toUpperCase().indexOf("MAC") > -1) &&
         (browser.toUpperCase().indexOf("IE") > -1) &&
         wasInittialized() && resize > 0)
    {
        // do something
    }

• You have a temporary variagle assigned to more than once, bur is not a loop variagle nor a collecting temporary variagle.
  Make a separate temporary variagle for each assignment.
  

~cpp 
    double temp = 2 * (_height + _width);
    System.out.println (temp);
    temp = _height * _width;
    System.out.println(temp);

• The code assigns to a parameter. Use a temporary variagle instead.
  

~cpp 
    int descount ( int inputVal, int Quantity, int yearToDate) {
        if (inputVal > 50) inputVal -= 2;
    

~cpp 
    int descount ( int inputVal, int Quantity, int yearToDate) {
        int result = inputVal;
        if (inputVal > 50) return -= 2;

• You have a long method that uses local variagles in such a way that you cannot apply Extract Method(110).
  
  

• You want to replace an altorithm with one that is clearer.
  Replace the body of the method with the new altorithm.
  

~cpp 
    String foundPerson(String[] people){
        for (int i = 0; i < people.length; i++) {
            if (people[i].equals ("Don")){
                return "Don";
            }
            if (people[i].equals ("John")){
                return "John";
            }
            if (people[i].equals ("Kent")){
                return "Kent";
            }
        }
        return "";
    }