short dump in ATC after READ TABLE replacement about abap-cleaner HOT 5 CLOSED

Hi FD1899,

thanks for opening this. Hm, could you give an example of the (original and changed) READ TABLE / ASSIGN statement that triggers this issue? If the changed code compiles without errors, I wonder whether this could be an issue in CL_CI_TEST_SELECT_BIN_SEARCH itself (i.e. a syntax of the ASSIGN statement which is not correctly considered by this check)?

From what I can see in the documentation, the possible syntax of ASSIGN has not changed too much since ABAP 7.50 (except for the addition ELSE UNASSIGN and assignment of dynamic components { struc-(comp) } | { dref->(comp_name) } | { COMPONENT comp OF STRUCTURE struc }).

If I understand the check CL_CI_TEST_SELECT_BIN_SEARCH correctly, it searches for READ ... BINARY SEARCH statements which wrongly rely on the result of SELECT statements to be sorted; and doing that, it analyzes other statements on the way, including in this case ASSIGN statements. The CL_CI_TEST_SELECT_BIN_SEARCH->ANALYZE_PROC code seems similar up to ABAP 7.52, then changes with ABAP 7.53 (where ANALYZE_PROC is not redefined anymore by CL_CI_TEST_SELECT_BIN_SEARCH, and the inherited implementation does not have a specific section on ASSIGN statements).

Kind regards,
Jörg-Michael

from abap-cleaner.

Hi FD1899,

I just learned that the check CL_CI_TEST_SELECT_BIN_SEARCH is obsolete, including in ABAP 7.50. Instead, you can use CL_CI_TEST_NO_ORDER_BY ("Robust Programming -> Search problematic statements for result of SELECT/OPEN CURSOR without ORDER BY").

Kind regards,
Jörg-Michael

from abap-cleaner.

Hi Jörg-Michael,

that's interesting. Can you give me the source for that information? So the reason for the dump is, that the ATC-check CL_CI_TEST_SELECT_BIN_SEARCH doesn't know the new syntax for reading internal tables? Unfortunately this test is still activated in our default check variant. So I have to talk the responsible person in my company.

Kind regards!

from abap-cleaner.

Hi,

the source is me - I'm the product owner for the ABAP Test Cockpit and the Code Inspector at SAP.

Note that CL_CI_TEST_SELECT_BIN_SEARCH is assigned to the category of internal tests in SAP_BASIS release 7.50 (the folder in transaction SCI it's in is named "Internal Tests"). Checks in this category should not be used by customers to begin with (and you must have done some modifcation or workaround to even include a check from this category in your check variant).

The officially supported, non-internal successor of CL_CI_TEST_SELECT_BIN_SEARCH is CL_CI_TEST_NO_ORDER_BY. This newer check is superior in every respect.

from abap-cleaner.

Hi,

thank you very much. So we can close that issue here.

Kind Regards,
FD

from abap-cleaner.