[Cuis-dev] Proposed changes to the Push up / Push down refactors

[Hernán Wilkinson]

Hi Ruben
 I thought I had answered you this email but I haven't.
 I implemented/integrated a lot of the functionality you and Paolo
suggested.
 There are two things that are not in the image:
1) It does not ask you if you want to keep the implementation in the
current class. I think it makes the refactoring more complex and I do not
find it is really useful. If you push up a method is because you want to
remove it from the current class, not to keep the same code and pollute
your code with repeated code. Same for push down.
2) The same with the functionality of sending the message to super when
pushing up a method. Same reason.

Cheers!
Hernan.

On Wed, May 24, 2023 at 6:26 PM Ruben Bohorquez via Cuis-dev <
cuis-dev at lists.cuis.st> wrote:

> Hey! Sorry for the empty file, I was writing the mail in a hurry
> yesterday... This should be the right file.
> In regards to further changes, nothing really comes to mind right now. The
> only thing I've thought about is to also add this behaviour to a "Push Down
> To" method that Paolo (a friend of mine) was working on recently (I think
> he already sent that a few days ago). Haven't looked into it just yet, but
> from what he told me implementing the same options as the existing push
> method refactors for it shouldn't be hard (mostly just copying what's in
> PushDownMethod already).
> Have a nice day everyone!
> --
> Cuis-dev mailing list
> Cuis-dev at lists.cuis.st
> https://lists.cuis.st/mailman/listinfo/cuis-dev
>


-- 

*Hernán WilkinsonAgile Software Development, Teaching & Coaching*
*Phone: +54-011*-4893-2057
*Twitter: @HernanWilkinson*
*site: http://www.10Pines.com <http://www.10pines.com/>*
Address: Alem 896, Floor 6, Buenos Aires, Argentina
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.cuis.st/mailman/archives/cuis-dev/attachments/20230712/17c177ac/attachment.htm>