topic Re: abap program for optimization in Application Development and Automation Discussions

abap program for optimization

Former Member — Fri, 06 Apr 2018 05:51:21 GMT

ztest-program.txt

Hi ABAP,

I am an abap newbie. I have written a code that needs to be optimized.

Not sure if my attached ABAP source code is already optimized.

Please check my source code and provide comments or suggestions if it is optimized or not.

Thank you

Juan

Re: abap program for optimization

matt — Fri, 06 Apr 2018 06:00:11 GMT

No. It's not optimised.

It uses performs, which are obsolete ABAP constructs. You should re-write in an object oriented fashion ,

It's not well modularised. Rule of thumb, no modular unit should be more than about 20 statements. (Rule of thumb so there are exceptions).

You've not used HASHED or SORTED tables.

Looks like you've got nested database selects - you don't have any INNER JOINs.

Rather than ask if your code is optimised, why don't you research the tools that analyse the runtime and apply them. Find out where the bottlenecks are and look for ways to improve.

Re: abap program for optimization

Ganesh_Pandian — Fri, 06 Apr 2018 09:07:57 GMT

I hope these links helps you,

https://www.sap.com/documents/2016/09/0479dd6e-877c-0010-82c7-eda71af511fa.html

https://archive.sap.com/discussions/thread/255854

Re: abap program for optimization

retired_member — Fri, 06 Apr 2018 09:44:23 GMT

Maybe it is a good idea, to read the ABAP Programming Guidelines and to follow the rules:

https://help.sap.com/http.svc/rc/abapdocu_752_index_htm/7.52/en-US/index.htm?file=abenabap_pgl.htm

Re: abap program for optimization

Former Member — Thu, 19 Apr 2018 06:09:42 GMT

zabap.txt

Hi ABAP Gurus,

Please see my attached file.

Do you think my program is now optimized?

Thank you all sooooo much.

Juan

Re: abap program for optimization

matt — Thu, 19 Apr 2018 06:38:16 GMT

I refer you to my earlier answer:

No. It's not optimised.

It uses performs, which are obsolete ABAP constructs. You should re-write in an object oriented fashion ,

It's not well modularised. Rule of thumb, no modular unit should be more than about 20 statements. (Rule of thumb so there are exceptions).

You've not used HASHED or SORTED tables.

Looks like you've got nested database selects - you don't have any INNER JOINs.

Rather than ask if your code is optimised, why don't you research the tools that analyse the runtime and apply them. Find out where the bottlenecks are and look for ways to improve.

You don't appear to have followed the advice in the links in the answer you've marked as accepted (even though it got negative votes).