"Align parameters and components": Options do not behave as expected

[ConjuringCoffee]

"Align parameters and components": Options do not behave as expected

[ConjuringCoffee]

Yes, I think this makes it much clearer. Thanks!

[jmgrassau]

Hi ConjuringCoffee,

type B ("for single-line parentheses") refers to table rows that contain multiple components, but still fit into one single code line, e.g.:

    mts_long_table_name  = VALUE #( ( item_key = '20220030000101'  event_date = '20220301'  total_quantity = '30'  quantity_unit = 'TAG' )
                                    ( item_key = '20220040000101'  event_date = '20220401'  total_quantity = '30'  quantity_unit = 'TAG' )
                                    ( item_key = '20220050000101'  event_date = '20220501'  total_quantity = '30'  quantity_unit = 'TAG' ) ) .

You could as well call this "tabular style". This could occur quite frequently in test code, and I think this tabular style is very nice to read. Therefore, by default, ABAP cleaner "grants" some extra line width for such cases (160 chars, as opposed to 120 chars for "normal" cases), thus trying to preserve tabular style, unless it significantly exceeds line length.

The alternative would be much harder to read:

    mts_long_table_name  = VALUE #( ( item_key       = '20220030000101'
                                      event_date     = '20220301'
                                      total_quantity = '30'
                                      quantity_unit  = 'TAG' )
                                    ( item_key       = '20220040000101'
                                      event_date     = '20220401'
                                      total_quantity = '30'
                                      quantity_unit  = 'TAG' )
                                    ( item_key       = '20220050000101'
                                      event_date     = '20220501'
                                      total_quantity = '30'
                                      quantity_unit  = 'TAG' ) ) .

Regarding your second question: The indentation is considered, but the closing parentheses and the period are not. That's why the "tipping point" is 3 chars later than you would have expected it. The reason for this is that in a case like …

    lo_any_instance->any_method( iv_any_parameter = VALUE #( ( total_quantity = '30'
                                                               quantity_unit  = 'TAG' )
                                                             ( total_quantity = '30'
                                                               quantity_unit  = 'TAG' )
                                                             ( total_quantity = '30'
                                                               quantity_unit  = 'TAG' ) ) ).

… the last row would otherwise have a severe "disadvantage", because it is followed by three closing parentheses. If this led to different alignment for the last row, it wouldn't look good.

That's why the line length setting isn't "exact science" here.

Kind regards,
Jörg-Michael

P.S.: At some point, I would also like to add an option (or even a cleanup rule) that creates tabular style in cases that are short enough. Currently, it is just preserved if it already exists in the code.

[ConjuringCoffee]

Thank you for the quick and thorough explanations! Do you think some of this could be better explained on the UI or somehwere in the documentation?

[jmgrassau]

Hi ConjuringCoffee,

yes, that makes sense… do you think "tabular style" would be better than "single-line parentheses"? I guess so …

Usually, I also try to set up the default example in a way that shows the effect of each option, so changing an option should usually also have an effect on the default examples (and that probably explains things better than a text would).

Kind regards,
Jörg-Michael

[jmgrassau]

Hi ConjuringCoffee,

I now renamed the option for "max. line length B" a bit …

… and added a comment in the examples, referring to the option:

Hope that helps to make "A" and "B" more understandable – along with the possibility to change the value and observe what happens to the example!

Kind regards,
Jörg-Michael