Refactoring

double potentialEnergy(double mass,double height)  {
   return mass * 9.81 * height;
}

double potentialEnergy(double mass,double height) {
    return mass * GRAVITATIONAL_CONSTANT * height;
}
static final double GRAVITATIONAL_CONSTANT = 9.81;

int discount (int inputVal, int quantity, int yearToDate)  {
  if (inputVal > 50)
    inputVal -=2;
  ...

int discount (int inputVal, int quantity, int yearToDate)  {
  int result = inputVal ;
  if (inputVal > 50)
    result -=2 ;
  ...

int basePrice = quantity * itemPrice;
discountLevel = getDiscountLevel();
double finalPrice = discountedPrice (basePrice, discountLevel);

int basePrice = quantity * itemPrice;
double finalPrice = discountedPrice (basePrice);

void setValue (String name, int vale) {
  if (name.equals("height") )
     height = value;
  if (name.equals ("width") )
     width = value;
  Assert.shouldNeverReachHere() ;
}

void setHeight (int arg) {
   height = arg ;
}
void setWidht (int arg) {
   width = arg ;
}

double temp = 2*(height+width);(1)
System.out.println (temp);
temp = height*width;(1)
System.out.println (temp);

final double perimeter = 2*(height+width);(1)
System.out.println (perimeter);
final double area = height*width;(1)
System.out.println (area);

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

final boolean isMacOs  = plataform.toUpperCase().indexOf("MAC") > -1;
final boolean is IEBrowser  = browser.toUpperCase().indexOf("IE") > -1;
final boolean wasResired  = resize > 0
if (isMacOs && isIEBrowser && wasInitialized()  && wasResired) {
     // do something
}

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

return (anOrder.basePrice() > 1000);

double basePrice = quantity * itemPrice;
if (basePrice > 1000)
    return basePrice * 0.95;
else
    return basePrice * 0.98;

if (basePrice() > 1000)
   return basePrice() * 0.95;
else
   return basePrice() * 0.98;
...
double basePrice() {
   return quantity * itemPrice;
}

double getPayAmount() {
  double result;
  if (_isDead)
    result = deadAmount();
  else {
    if (_isSeparated)
      result = separatedAmount();
    else {
      if (_isRetired)
        result = returedAmount();
      else
        result = normalPayAmount();
    }
  }
  return result;
}

double getPayAmount() {
  if (_isDead)
    return deadAmount();
  if (_isSeparated)
    return separatedAmount();
  if (_isRetured)
    return retiredAmount();
  return normalPayAmount();
}

double desabilityAmount() {
  if (_seniority <2)
    return 0 ;
  if (_monthsDisabled >12)
    return 0 ;
  if (_isPartTime)
    return 0 ;
  ...
  //compute the disability deadAmount

double disabilityAmount() {
  if (isNotEligableForDisability() )
    return 0 ;
  ...
  //compute the disability deadAmount

if (isSpecialDeal()) {
   total = price*0.95;
   send();
} else {
   total = price*0.98;
   send();
}

if (isSpecialDeal() )
  total = price * 0.95 ;
else
  total = prise * 0.98 ;
send();

set done to false
while not done
  if (condition)
     do something
     set done to true
  next step of loop

while not done
  if (condition)
     do something
     break/return
  next step of loop

if (customer == null)
  plan = BillingPlan.basic();
else
  plan = customer.getPlan();

if (date.before(SUMMER_START) || date.after (SUMMER_END))
   charge = quantity * winterRate+winterServiceCharge;
else
   charge = quantity * summerRate;

if (notSummer(date) )
   charge = winterCharge(quantity);
else
   charge = summerCharge(quantity);

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

String foundPerson (String[] people) {
  List candidates = Arrays.asList (new String[] {"Don", "Jhon", "Kent"});
  for (int i=0; i< people.length; i++)
    if (candidates.contains(people[i]))
      return people[i];
  return "" ;
}

int getRating(){
  return (moreThanFiveLateDeliveries())? 2: 1;
}

boolean moreThanFiveLateDeliveries(){
  return numberOfLateDeliveries > 5;
}

int getRating(){
  return (numberOfLateDeliveries > 5)? 2: 1;
}

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

void printOwing(double amount){
  printBanner();
  printDetails (amount);
}

void printDetails (double amount){
  System.out.println ("name:") + name);
  System.out.println ("amount:") + amount);
}

class Order
  double price() {
    double prinaryBasePrice;
    double secondaryBasePrice;
    double tertiaryBasePrice;
    // long computation;
    ....
}

public String  _name

private String name;

public String getName(){
  return this.name;
}

public void setName(String name){
  this.name = name;
}

private int low;
private int high;

public boolean includes (int value) {
  return this.low <= value && value < = this.high;
}

private int low
private int high;

boolean includes (unt value) {
     return this.getLow() <= value && value <= this.getHigh() ;
}

int getLow() {
  return low;
}

int getHigh() {
  return high;
}

Object lastReading()  {
    return readings.lastElement() ;
}

Reading lastReading()  {
   return (Reading) readings.lastElement() ;
}

double getExpenseLimit() {
   // should have either exprense limit or a primary project
   return (_expenseLimit != NULL _EXPENSE) ?
      _expenseLimit:
      _primaryProject.getMemberExpenseLimit() ;
}

double getExpenseLimit()  {
    Assert.isTrue (_expenseLimit !=NULL _EXPENSE | | _primaryProject != null);
    return (_expenseLimit ! = NULL _EXPENSE) ?
       _expenseLimit:
       _primaryProject.getMemberExpenseLimit() ;
}

int withdraw(int amount) {
    if (amount > _balance)
        return -1;
    else  {
        balance -= amount ;
        return 0 ;
    }
}

void withdraw(int amount) throws BalanceExeption {
    if (amount > _balance) throw new BalanceExeption() ;
    balance -=amount;
}

double getValueForPeriod (int periodNumber) {
   try  {
        return _values[periodNumber];
   }  catch (ArrayIndexOutOfBoundsException e) {
      return 0;
   }
}

double getValueForPeriod (int periodNunber) {
   if (periodNumber >= _values.length) return 0;
   return _values(periodNumber);
}

class Manager extends Employee...
   public Manager (String name,String id, int grade) {
       _name = name;
       _id = id;
       _grade = grade;
}

public Manager (String name, String id, int grade) {
      super (name,id) ;
      _grade = grade;
}

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") ;
}

Employee (int type) {
    _type = type;
}

static Employee create (int type) {
    return new Employee (type);
}

int low = deysTempRange().getLow();
int high = deysTempRange() .getHigh();
withinPlan = plan.withinRange (low,high);

withinPlan = plan.withinRange(deysTempRange() );

String [] row = new String[3];
row [0] ="Liverpool";
row [1] ="15";

Performance row = new Performance ();
row.setName ("Liverpool");
row.setWins("15");

Date newStart =new Date (previousEnd.getYear(),
              previousEnd.getMonth(), previousEnd.getDate() +1);

Date newStart =nexDay(previousEnd);

private static Date nextDay(Date arg) {
    return new Date (arg.getYear(),arg.getMonth(),arg.getDate() +1);
    }