rspec/rules/S109/abap/rule.adoc
Rudy Regazzoni bc15ffe77e
Modify S109: Migrate To LayC (#3235)
## Review

A dedicated reviewer checked the rule description successfully for:

- [x] logical errors and incorrect information
- [x] information gaps and missing content
- [x] text style and tone
- [x] PR summary and labels follow [the
guidelines](https://github.com/SonarSource/rspec/#to-modify-an-existing-rule)
2023-10-11 13:58:56 +02:00

41 lines
687 B
Plaintext

include::../description.adoc[]
include::../how-to-fix-it.adoc[]
=== Code examples
==== Noncompliant code example
[source,abap,diff-id=1,diff-type=noncompliant]
----
IF sy-subrc EQ 42. " Noncompliant - 5 is a magic number
screen-request = 'OK'.
ENDIF.
----
==== Compliant solution
[source,abap,diff-id=1,diff-type=compliant]
----
answer = 42.
IF sy-subrc EQ answer. " Compliant
screen-request = 'OK'.
ENDIF.
----
ifdef::env-github,rspecator-view[]
'''
== Implementation Specification
(visible only on this page)
include::../message.adoc[]
'''
== Comments And Links
(visible only on this page)
include::../comments-and-links.adoc[]
endif::env-github,rspecator-view[]