Ayende @ Rahien

Refunds available at head office

JAOO: Working Effectively with Legacy Code 2 – Michael Feathers

I have a tremendous amount of respect to Michael Feathers, so it is a no brainer to see his presentation.

Michael is talking about why Global Variables are not evil. We already have global state in the application, removing it is bad/impossible. Avoiding global variables leads to very deep argument passing chains, where something needs an object and it passed through dozens of objects that just pass it down. We already have the notions on how to test systems using globals (Singletons). He also talks about Repository Hubs & Factory Hubs – which provide the scope for the usage of a global variable.

  • Refactor toward explicit seams, do not rely on accidental seams, make them explicit.
  • Test Setup == Coupling, excessive setup == excessive coupling.
  • Slow tests indicate insufficient granularity of coupling <- I am not sure that I agree with, see my previous posts about testing for why.
  • It is often easier to mock outward interfaces than inward interfaces (try to avoid mocking stuff that return data)
  • One of the hardest things in legacy code is making a change and not knowing what it is affecting. Functional programming makes it easier, because of immutability.
  • Seams in a functional languages are harder. You parameterize functions in order to get those seams.
  • TUF – Test Unfriendly Feature – IO, database, long computation
  • TUC – Test Unfriendly Construct – static method, ctor, singleton
  • Never Hide a TUF within a TUC
  • No Lie principal – Code should never lie to you. Ways that code can lie:
    • Dynamically replacing code in the source
    • Addition isn’t a problem
    • System behavior should be “what I see in the code + something else”, never “what I see minus something else”
    • Weaving & aspects
    • Impact on inheritance
  • The Fallacy of Restricted Languages
  • You want to rewrite if the architecture itself is bad, if you have issues in making changes rapidly, it is time for refactor the rough edges out.

Comments

Ben
10/05/2009 11:04 AM by
Ben

So many mistakes, so little text.

Slow down there champ. Re-read what you're writing to make sure you've gotten your thoughts down lest your readers have to struggle through debugging your text themselves :(

Ayende Rahien
10/05/2009 11:06 AM by
Ayende Rahien

Ben,

That is what you get when I try to both write & listen at the same time.

Feel free to skip the JAOO posts, if you don't like them.

Mike
10/05/2009 04:45 PM by
Mike

I think aspects are great for non-business behavior like logging, caching, authorization, validation, etc...

Can you elaborate on this?

Ayende Rahien
10/05/2009 04:53 PM by
Ayende Rahien

Not my session, but I think he meant that it is easy to have things more complex if you do complex things in the aspects.

BL, for example.

Colin Jack
10/07/2009 12:34 PM by
Colin Jack

Out of interest what are Repository Hubs & Factory Hubs?

Grimace of Despair
10/07/2009 01:39 PM by
Grimace of Despair

Offtopic and related to Ben's remark: I think Ayende needs a big disclaimer just below the site banner:

*** WARNING: I use my blog to twitter

I mean: other people put much time in composing well-thought posts, while Ayende practically thinks them up in his blog. A bit like what Twitter is actually meant for. Minutes after a post, he can come up with his own rebuttal. From time to time you see people not used to this reacting on a post and sometimes even quite heavily. I mean: I probably spent more time putting this reaction down than Ayende spent making this post.

Comments have been closed on this topic.