ABAP CODE CLEANER

[former_member870718]

Can we compare ABAP code cleaner Tool with ADT. If Yes what are the Metrics & Measurement of this Tool.

Can we measure below things using ABAP code cleaner:-

Code complexity , Code duplication, Execution time, Memory usage, Modularity, Testability, Code coverage

[Marissa]

Hi change_890,

SAP S/4HANA Cloud ABAP Environment is for public cloud. SInce you also tagged SAP S/4HANA Cloud, extended edition, I removed tag SAP S/4HANA Cloud ABAP Environment.

Best regards,

Marissa

[former_member870718]

Hi Scott, we have installed this tool and we were able to run it on 60 parameters mentioned below. These all parameters were related to formatting only. But we are curious to know if this tool can help us measure Code complexity , Code duplication, Execution time, Memory usage, Modularity, Testability, Code coverage.

Standardize empty lines within methods
Separate methods and classes with empty linesStandardize empty lines in class definitionsStandardize spaces next to parenthesesClose brackets at line endRemove space before commas and periodPut spaces around"comment sign"Remove needless spacesUnchain into multiple statementsRearrange local declarationsDelete unused variablesSimplify a chain with one elementMake implicit type explicitUse FINAL for immutable variablesStandardize escaping of !parametersRemove empty class definition SECTIONsAdd missing parameters to ABAP DocRemove lang="en" from ABAP DocComment with",not with * (for text)Remove end-of commentsReplace obsolete pseudo comments with pragmasMove pragmas to correct positionCorrect frequent typosResolve equals sign chain into several commandsPrefer calculation assignment operators (+=, -= etc.)Prefer =, <>, <= etc. to EQ, NE, LE etc.Prefer IS NOT to NOT ISMove AND/OR etc. from line end to next line startRemove empty commandsShorten VALUE statementsRemove the self-reference me->Omit RECEIVINGOmit optional EXPORTINGConvert CHECK outside loop to IF NOT ... RETURNConvert CHECK in loop to IF NOT ... CONTINUEReplace long IF blocks at loop endReplace long IF blocks at method endReplace CALL METHOD with functional callReplace CREATE OBJECT with NEW constructorReplace RAISE ... TYPE with RAISE ... NEWReplace obsolete ADD ... TO etc. with += etc.Replace obsolete MOVE ... TO with =Replace TRANSLATE with string functionsUse assert_true and assert_falseUse assert_subrc instead of assert_equalsUse assert class instead of ASSERTConvert upper and lower caseIndent linesAlign ABAP DocAlign METHODS declarationsAlign METHODS ... FOR TESTINGAlign METHODS ... REDEFINITIONAlign ALIASES ... FOR ...Align declarationsAlign assignments to the same objectAlign keywords with second word of first lineAlign CLEAR:, FREE: and SORTAlign parameters and componentsAlign logical expressionsAlign conditional expressions

[Sandra_Rossi]

Link to ABAP Code Cleaner: SAP/abap-cleaner: ABAP cleaner applies 60+ cleanup rules to ABAP code at a single keystroke (github.... (taken from Thomas answer)

[ThFiedler]

Hi,

the ABAP Cleaner is a code formatter tool which can be integrated in ADT. The tool can be used to apply the guidelines of Clean ABAP. So it replaces some how the standard formatter aka. Pretty Printer. https://github.com/SAP/abap-cleaner

Regards,

Thomas.

[soumen_s]

That means we can't measure Code complexity , Code duplication, Execution time, Memory usage, Modularity, Testability, Code coverage using ABAP code cleaner is this correct?.

Thanks,

Soumen

[Sandra_Rossi]

change_890 soumen_s How many people are asking the question? 😉 😄

[ThFiedler]

As I said: Purpose of ABAP Cleaner is code formatting in the ABAP Editor. There are other tools like ATC, trace, Clone finder for all the other use cases you have in mind.

Regards,

Thomas.

[BaerbelWinkler]

change_890

Hi Manu,

what you would like to check sounds more like a task for Code Pal described in this blog post from olga.dolinskaja :

How to enable clean code checks for ABAP

Code Pal is an additional (local) code inspector check variant which can be included in ATC-checks of the code.

Hope this helps!

Cheers

Bärbel