Comments on: Generalizing the algorithm http://blogs.jetbrains.com/dotnet/2006/08/generalizing-the-algoritm/ ReSharper for productivity, dotTrace for performance Sat, 20 Mar 2010 10:53:31 +0000 http://wordpress.org/?v=2.6.5 By: dsl http://blogs.jetbrains.com/dotnet/2006/08/generalizing-the-algoritm/#comment-138 dsl Wed, 16 Aug 2006 08:43:41 +0000 http://blogs.jetbrains.com/dotnet/2006/07/generalizing-the-algoritm/#comment-138 Brien, You can do that in two steps: 1. Invoke Make Method Static on GetBonusForOrder You will get: int GetBonusForOrder(Customer c, Order o) 2. Invoke Make Method Non-Static and select parameter 'o' as future this. You will get (in Order class) int GetBonusForOrder(Customer c) You can get rid of parameter 'c' if it is not needed via Change Method Signature refactoring. Bear in mind though, that in case of virtual methods this trick obviuosly won't work. I think we can post a separate article on this refactoring combination. Brien,
You can do that in two steps:
1. Invoke Make Method Static on GetBonusForOrder
You will get:
int GetBonusForOrder(Customer c, Order o)

2. Invoke Make Method Non-Static and select parameter ‘o’ as future this. You will get (in Order class)
int GetBonusForOrder(Customer c)

You can get rid of parameter ‘c’ if it is not needed via Change Method Signature refactoring.

Bear in mind though, that in case of virtual methods this trick obviuosly won’t work.

I think we can post a separate article on this refactoring combination.

]]> By: Brien http://blogs.jetbrains.com/dotnet/2006/08/generalizing-the-algoritm/#comment-137 Brien Tue, 15 Aug 2006 16:10:14 +0000 http://blogs.jetbrains.com/dotnet/2006/07/generalizing-the-algoritm/#comment-137 How would you refactor if you wanted the Bonus calculation to be a method on the order? I always wind up doing this mostly by hand. How would you refactor if you wanted the Bonus calculation to be a method on the order? I always wind up doing this mostly by hand.

]]>