SLIDE 1
January 02, 2009
O b j e c t O r i e n t e d S o f t w a r e E n g i n e e r i n g
366
Refactoring Lecture 7 January 02, 2009 O b j e c t O r i e n t e - - PowerPoint PPT Presentation
Refactoring Lecture 7 January 02, 2009 O b j e c t O r i e n t e - - PowerPoint PPT Presentation
366 Refactoring Lecture 7 January 02, 2009 O b j e c t O r i e n t e d S o f t w a r e E n g i n e e r i n g Refactoring: problem statement O b j e c t O r i e n t e d S o f t w a r e E n g i n e e r i n g Add a htmlStatment method
SLIDE 2
SLIDE 3
SLIDE 4
SLIDE 5
SLIDE 6
January 02, 2009
O b j e c t O r i e n t e d S o f t w a r e E n g i n e e r i n g
377
Refactoring: problem statement
Add a htmlStatment method which returns a customer statement string containing html tags. ...and there will be some changes to the way movies are classified ...affecting frequent renter points and charging.
SLIDE 7
SLIDE 8
January 02, 2009
O b j e c t O r i e n t e d S o f t w a r e E n g i n e e r i n g
380
Refactoring: step 2
public String statement() { double totalAmount = 0; int frequentRenterPoints = 0; Enumeration rentals = _rental.elements(); String result = ÒRental Record for Ò + getName() + Ò\nÓ; while (rentals.hasMoreElements()) { double thisAmount = 0; Rental each = (Rental) rentals.nextElement(); thisAmount = amountFor(each); frequentRenterPoints ++; if ((each.getMovie().getPriceCode() == Movie.NEW_RELEASE)&& each.getDaysRented() > 1) frequentRenterPoints++; result += Ò\tÓ + each.getMovie().getTitle()+ Ò\tÓ + String.valueOf(thisAmount) + Ò\nÓ; totalAmount += thisAmount; } result += ÒAmount owed is Ò+Sting.valueOf(totalAmount) + Ò\nÓ; result += ÒYou earned Ò+Sting.valueOf(frequentRenterPoints) + Òfrequent renter points\nÓ; return result;
SLIDE 9
SLIDE 10
SLIDE 11
SLIDE 12
January 02, 2009
O b j e c t O r i e n t e d S o f t w a r e E n g i n e e r i n g
389
Refactoring: step 7
class Customer ... public String statement() { double totalAmount = 0; int frequentRenterPoints = 0; Enumeration rentals = _rental.elements(); String result = ÒRental Record for Ò + getName() + Ò\nÓ; while (rentals.hasMoreElements()) { double thisAmount = 0; Rental each = (Rental) rentals.nextElement(); thisAmount = amountFor(each); // add frequent renter points frequentRenterPoints ++; // add bonus for a two day new release rental if ((each.getMovie().getPriceCode() == Movie.NEW_RELEASE)&& each.getDaysRented() > 1) frequentRenterPoints++; //show figures for this rental result += Ò\tÓ + each.getMovie().getTitle()+ Ò\tÓ + String.valueOf(thisAmount) + Ò\nÓ; totalAmount += thisAmount; É
SLIDE 13
January 02, 2009
O b j e c t O r i e n t e d S o f t w a r e E n g i n e e r i n g
390
Refactoring: step 7
class Customer ... public String statement() { double totalAmount = 0; int frequentRenterPoints = 0; Enumeration rentals = _rental.elements(); String result = ÒRental Record for Ò + getName() + Ò\nÓ; while (rentals.hasMoreElements()) { double thisAmount = 0; Rental each = (Rental) rentals.nextElement(); thisAmount = each.getCharge(); // add frequent renter points frequentRenterPoints ++; // add bonus for a two day new release rental if ((each.getMovie().getPriceCode() == Movie.NEW_RELEASE)&& each.getDaysRented() > 1) frequentRenterPoints++; //show figures for this rental result += Ò\tÓ + each.getMovie().getTitle()+ Ò\tÓ + String.valueOf(thisAmount) + Ò\nÓ; totalAmount += thisAmount;
SLIDE 14
SLIDE 15
SLIDE 16
SLIDE 17
SLIDE 18
SLIDE 19
SLIDE 20
SLIDE 21
SLIDE 22
SLIDE 23
SLIDE 24
SLIDE 25
SLIDE 26
SLIDE 27
SLIDE 28
January 02, 2009
O b j e c t O r i e n t e d S o f t w a r e E n g i n e e r i n g
420
Refactoring: step 14
abstract class Price { abstract int getPriceCode(); } class ChildrenPrice extends Price { int getPriceCode(){ return MOVIE.CHILDREN; } } class NewReleasePrice extends Price { int getPriceCode(){ return MOVIE.NEW_RELEASE; } } class RegularPrice extends Price { int getPriceCode(){ return MOVIE.REGULAR; } }
SLIDE 29
SLIDE 30
SLIDE 31
SLIDE 32
SLIDE 33
SLIDE 34
SLIDE 35
SLIDE 36
SLIDE 37
SLIDE 38
SLIDE 39
SLIDE 40
SLIDE 41
January 02, 2009
O b j e c t O r i e n t e d S o f t w a r e E n g i n e e r i n g
447