rspec/rules/S2242/abap/rule.adoc
Fred Tingaud 16f6c0aecf
Inline adoc when include has no additional value (#1940)
Inline adoc files when they are included exactly once.

Also fix language tags because this inlining gives us better information
on what language the code is written in.
2023-05-25 14:18:12 +02:00

52 lines
1.2 KiB
Plaintext

== Why is this an issue?
``++SELECT INTO TABLE++`` is much more efficient than ``++SELECT ... ENDSELECT++``. ``++SELECT INTO TABLE++`` needs more memory to hold the result set, but in normal situations, this is not a concern. When memory is a concern, the result set can be divided into smaller sets.
=== Noncompliant code example
[source,abap]
----
SELECT * FROM T006 INTO X006_WA.
...
ENDSELECT.
----
=== Compliant solution
[source,abap]
----
SELECT * FROM T006 INTO TABLE X006.
LOOP AT X006 INTO X006_WA.
...
ENDLOOP.
----
ifdef::env-github,rspecator-view[]
'''
== Implementation Specification
(visible only on this page)
=== Message
Refactor this select to use "SELECT INTO TABLE int1 + LOOP AT int1".
'''
== Comments And Links
(visible only on this page)
=== on 2 Dec 2014, 14:39:21 Ann Campbell wrote:
\[~freddy.mallet] I'm confused by this description. ``++SELECT INTO TABLE++`` is more efficient but it needs more memory...?
=== on 4 Dec 2014, 09:41:22 Freddy Mallet wrote:
Indeed @Ann:
* In one case, we we're going to consume less memory but the execution time might be really bigger
* In the other case, we start by quickly loading all information in memory to then process it
endif::env-github,rspecator-view[]