| T O P I C R E V I E W |
| khb |
Posted - Jun 27 2006 : 07:22:02 AM
I like the new refactoring function "Extract Method", but it sometimes happens to me that I need the reverse function. For example: I had a function that shrinked down to 4 lines of code after some refactoring. I called this function just from 2 locations in the code. So my idea was to remove the function and copy its code directly to the 2 locations. It would be cool if this function would be available from the refactoring menu. Of course, this should only be possible if the function to remove doesn't call itself 
Regards,
Marcus. |
| 4 L A T E S T R E P L I E S (Newest First) |
| support |
Posted - Jun 28 2006 : 11:54:58 AM
FWIW: Refactoring is sometimes called "Inline Method."
http://www.refactoring.com/catalog/inlineMethod.html |
| khb |
Posted - Jun 28 2006 : 02:13:43 AM
@feline: Thanks for putting in a feature request.
@joe: 
Regards,
Marcus. |
| jpizzi |
Posted - Jun 27 2006 : 11:18:19 PM
Here is how you can do this without VA's support.
At the beginning of the line, type the string, "inline "
Of course, if you need want VA's help, you could always create an autotext entry  |
| feline |
Posted - Jun 27 2006 : 4:44:08 PM
i can think of times when working out if the function calls its self would be rather hard to determine. heavy use of overloaded functions plus default parameters is a good example 
my gut instinct is to say "leave the function alone", since i have come to view small functions as good, but i understand why you are saying this. it would require the ability to remove the function, and its declaration.
i have just been sitting here, trying to work out it this presents any obvious problems or challenges that would make this very hard to do. so far though i have not come up with anything that sounds worse than extract method must have been, so i have put in a feature request
case=1493 |
