eclipse-adt-extract-method
![images/thumbnail.jpg - Thumbnail](/en/2024/06/eclipse-adt-extract-method/images/thumbnail_hu50f147dd30d23fe2f8eec79931fd0797_92577_300x0_resize_q75_box.jpg)
When you have a very large method, you should subdivide it into several smaller ones.
Clean ABAP says that methods should ideally have 3 to 5 commands and suggests a maximum of 20 in extreme cases. If you’re still making 300-line methods, think about it, be ashamed of yourself and go and subdivide them.
Mr Martin Fowler has studied in depth the optimum processes for refactoring programmes and has published his results in an excellent book called Refactoring which I strongly advise you to read. Alternatively, you can consult the website refactoring.guru.
The book Refactoring describes systematic processes for carrying out each of the cases. These are the steps it proposes for the Extract Method technique:
- Create a new method and name it in a way that makes its purpose self-evident.
- Copy the relevant code fragment to your new method. Delete the fragment from its old location and put a call for the new method there instead.
- Find all variables used in this code fragment. If they’re declared inside the fragment and not used outside of it, simply leave them unchanged—they’ll become local variables for the new method.
- If the variables are declared prior to the code that you’re extracting, you will need to pass these variables to the parameters of your new method in order to use the values previously contained in them. Sometimes it’s easier to get rid of these variables by resorting to Replace Temp with Query.
- If you see that a local variable changes in your extracted code in some way, this may mean that this changed value will be needed later in your main method. Double-check! And if this is indeed the case, return the value of this variable to the main method to keep everything functioning.
In SAPGUI
Right. Same old poo. If you want to extract part of the code from one method to another, you have to do it all by hand. There’s no automation here. No wonder people who use SAPGUI tend to make long methods. SAPGUI should be called an IUE instead of an IDE, the U standing for Underdeveloped.
In Eclipse ADT
- Select the code you want to extract
- Right button, Source code > Extract method
- Name the new method and adjust the parameters
- Next, next, finish.
There, that’s it. The 5 steps described above have been carried out automatically. Follow them.
Conclusion
So, if you’re still using SAPGUI and you could be using Eclipse ADT, you’re not the sharpest tool in the shed, are you?
Greetings from Abapinho