Held: ,

Back to Outline 15 - Codes of Ethics, Revisited. On to Outline 17 - UML.

Summary

We explore issues of working with legacy code and how code smells and refactoring will help us not just with legacy code, but with all of our code.

Related Pages

• Catalog of Refactorings

Overview

• Legacy code.
• Smells and refactoring.
• Some examples.

Administrivia

• Wednesday's class will be UML. No reading. Albert will lead.
• Prof. Curtsinger will be teaching Friday's class. Please send your journal reflections to both of us.

Upcoming Work

• Readings for Friday
  • Arvind Narayaan and Shannon Vallor (March 2014). Why software engineering courses should include ethics coverage. Communications of the ACM 57(3): 23-25. http://dl.acm.org/citation.cfm?id=2566966
  • Neil McBride (August 2012). The ethics of software engineering should be an ethics for the client. Communications of the ACM 55(8): 39-41. http://dl.acm.org/citation.cfm?id=2240250
  • Deborah G. Johnson (October 2008). Computer experts: Guns-for-hire or professionals? Communications of the ACM 51(10): 24-26. http://dl.acm.org/citation.cfm?id=1400190
• Reading Journal for Friday (due Thursday at 8pm)
  • Subject: Reading Journal for Class 18 (Your Name)
  • From each of the three new articles about software engineering ethics, select one sentence that you think best captured its thesis. Quote that sentence, giving a page number and approximate location on the page (e.g., "top right").
  • What is one question arising from these articles that you'd like to discuss during class?
  • Brainstorm a few topics or questions that you might address in a 600-word essay on software engineering ethics (the last assignment). Your ideas might arise from the articles or from current events.
  • As is often the case, these questions were inspired by the work of JLND.

Good Things to Do

Academic

• MET Opera Saturday.

Peer

• Christine's show!

Other

• Vote today!

Preparatory Activity

On a sheet of paper (supplied), answer the following questions.

1. What is "legacy code"? (We see lots of definitions within the chapter.)
2. List three "code smells" that you regularly notice.
3. What is SOFA?
4. List three refactoring techniqeus that you regularly use.

Legacy Code

• What, from your perspective, is "legacy code"? (We see lots of definitions within the chapter.)
• Why do we care about it?
• What kinds of maintainance do we worry about for legacy code?
• How do we do this kind of maintainance?

Code Smells

• What are they?
• Why do we care?
• Which ones stand out to you?

Refactoring

• What do you see as the key principles of refactoring?
• What do you see as the key techniques we use to refactor?
• Sam's perspective: There are a few purposes for the list of refactoring techniques.
  • They serve to remind programmers of the kinds of things they can do to make code better.
  • They provide us with a vocabulary to talk about the kinds of changes you might make to a program.
• You should be experienced enough as programmers to have your own list of common "code improvements". What are some of yours?

An Example

From Field et al., pp. 2-4.

class Movie
  REGULAR = 0
  NEW_RELEASE = 1
  CHILDRENS = 2
  attr_reader :title
  attr_accessor :price_code
  def initialize(title, price_code)
    @title, @price_code = title, price_code
  end 
end

class Rental
  attr_reader :movie, :days_rented
  def initialize(movie, days_rented)
    @movie, @days_rented = movie, days_rented
  end 
end

class Customer
  attr_reader :name

  def initialize(name)
    @name = name
    @rentals = []
  end

  def add_rental(arg)
    @rentals << arg
  end

  def statement
    total_amount, frequent_renter_points = 0, 0
    result = "Rental Record for #{@name}\n"
    @rentals.each do |element|
      this_amount = 0

      # determine amounts for each line
      case element.movie.price_code
      when Movie::REGULAR
        this_amount += 2
        this_amount += (element.days_rented - 2) * 1.5 if element.days_rented > 2
      when Movie::NEW_RELEASE
        this_amount += element.days_rented * 3
      when Movie::CHILDRENS
        this_amount += 1.5
        this_amount += (element.days_rented - 3) * 1.5 if element.days_rented > 3
      end

      # add frequent renter points
      frequent_renter_points += 1
      # add bonus for a two day new release rental
      if element.movie.price_code == Movie.NEW_RELEASE && element.days_rented > 1
          frequent_renter_points += 1
      end

      # show figures for this rental
      result += "\t" + element.movie.title + "\t" + this_amount.to_s + "\n"
      total_amount += this_amount
    end

    # add footer lines
    result += "Amount owed is #{total_amount}\n"
    result += "You earned #{frequent_renter_points} frequent renter points"
    result
  end
end