styleguides
styleguides copied to clipboard
SAP
This repository provides SAP style guides for coding and coding-related topics.
there is https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-US/abenabap_pgl.htm and other places in the ABAP help, which defines guidelines/recommendations Personally I think it would be nice to have everything in one place However, there is also...
With the [ABAP RESTful Application Programming Model (RAP)](https://help.sap.com/viewer/923180ddb98240829d935862025004d6/Cloud/en-US/289477a81eec4d4e84c0302fb6835035.html) were the ABAP language enriched by Entity Manipulation Language (EML) statements to access and interact with Business Objects (BO). Here are some...
> Omit the parameter name in single parameter calls > > ```ABAP > DATA(unique_list) = remove_duplicates( list ). > ``` > > instead of the needlessly longer > > ```ABAP...
in https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#check-vs-return the good example ```abap METHOD read_customizing. IF keys IS NOT INITIAL. " do whatever needs doing ENDIF. ENDMETHOD. ``` Might not be super good, especially it introduces nesting,...
In the `Comments` section, the styleguide states that: - `FIXME` points to errors that are too small or too much in-the-making for internal incidents. - `TODO`s are places where you...
In short, we should not [chain up-front variable declarations](https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#do-not-chain-up-front-declarations). But, how about `METHODS`, `CLASS-METHODS`, `TYPES`, and `CONSTANTS`? For instance: ```ABAP METHODS: get_name, get_age, set_name, set_age. ``` ```ABAP CONSTANTS: min_name_size TYPE...
Clean Code recommends ordering methods by importance and call sequence. I find this helpful and do it where it makes sense. It would be nice to incorporate this into the...
Updated as per comments in the previous pull request.
From an SAP developer: > For LOOPs, FIELD-SYMBOLS are the better choice and faster - unless the tables are not already references themselves. > There is also an ATC check...
