To be discussed: Insert TODO-statement when CLEAR of EXPORTING-Parameter is missing?

[vonglan]

I think this might prevent some bugs.
We would not want ABAPCleaner to directly insert the CLEAR, because this would change the logic.

https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#clear-or-overwrite-exporting-reference-parameters
"Generally, it is a good idea to clear the parameter as a first thing in the method after type and data declarations."

Disadvantages:

• The TODO-comment is ugly if it is left in the code
• For "false positives" (where the Exporting-Parameter is intentionally and correctly not CLEARed), ABAPCleaner would need some pseudocomment
• You might argue that this should be an ATC check (or even retrofitted into ABAP syntax ;-)) . I suppose it is NOT an ATC check because of the high percentage of false positives (which are difficult to detect by ATC).

What is your opinion?

Best regards,
Edo