rspec/rules/S2197/java/rule.adoc

== Why is this an issue?

include::../description.adoc[]

=== Noncompliant code example

[source,java]
----
public boolean isOdd(int x) {
  return x % 2 == 1;  // Noncompliant; if x is an odd negative, x % 2 == -1
}
----


=== Compliant solution

[source,java]
----
public boolean isOdd(int x) {
  return x % 2 != 0;
}
----


== Resources

* https://wiki.sei.cmu.edu/confluence/x/pDdGBQ[CERT, NUM51-J.] - Do not assume that the remainder operator always returns a nonnegative result for integral operands
* https://wiki.sei.cmu.edu/confluence/x/_NUxBQ[CERT, INT10-C] - Do not assume a positive remainder when using the % operator

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[]
migrate rule descriptions to new education format 2023-05-03 11:06:20 +02:00			`== Why is this an issue?`

RULEAPI-665: Remove security standards from the irrelevant language-specific rules (#362) 2021-09-21 15:40:35 +02:00			`include::../description.adoc[]`

Clean rule at root In some cases, the `rule.adoc` at root of a rule is never included anywhere and thus is dead code. It's a maintenance cost by itself, but also it misses opportunities to inline code that seems used by two documents when in fact only one document is actually rendered. And this missed opportunity, in turn, stops us from applying the correct language tag on the code samples. 2023-10-16 16:34:38 +02:00			`=== Noncompliant code example`

			`[source,java]`
			`----`
			`public boolean isOdd(int x) {`
			`return x % 2 == 1; // Noncompliant; if x is an odd negative, x % 2 == -1`
			`}`
			`----`


			`=== Compliant solution`

			`[source,java]`
			`----`
			`public boolean isOdd(int x) {`
			`return x % 2 != 0;`
			`}`
			`----`
RULEAPI-665: Remove security standards from the irrelevant language-specific rules (#362) 2021-09-21 15:40:35 +02:00

migrate rule descriptions to new education format 2023-05-03 11:06:20 +02:00			`== Resources`
RULEAPI-665: Remove security standards from the irrelevant language-specific rules (#362) 2021-09-21 15:40:35 +02:00
			`* https://wiki.sei.cmu.edu/confluence/x/pDdGBQ[CERT, NUM51-J.] - Do not assume that the remainder operator always returns a nonnegative result for integral operands`
			`* https://wiki.sei.cmu.edu/confluence/x/_NUxBQ[CERT, INT10-C] - Do not assume a positive remainder when using the % operator`
Export comments and rspec-to-rspec links from jira 2021-06-02 20:44:38 +02:00
Fix the comment display: rule-id, timestamp, GH visibility, link direction 2021-06-03 09:05:38 +02:00			`ifdef::env-github,rspecator-view[]`
RULEAPI-666: Migrate the "List of parameters", "Highlighting" and "Message" fields from jira RSPEC (#346) 2021-09-20 15:38:42 +02:00
			`'''`
			`== Implementation Specification`
			`(visible only on this page)`

			`include::../message.adoc[]`

RULEAPI-576: add a horizontal rule between rule description and comments 2021-06-08 15:52:13 +02:00			`'''`
Export comments and rspec-to-rspec links from jira 2021-06-02 20:44:38 +02:00			`== Comments And Links`
			`(visible only on this page)`

			`include::../comments-and-links.adoc[]`
Make sure that includes are always surrounded by empty lines (#2270) When an include is not surrounded by empty lines, its content is inlined on the same line as the adjacent content. That can lead to broken tags and other display issues. This PR fixes all such includes and introduces a validation step that forbids introducing the same problem again. 2023-06-22 10:38:01 +02:00
Fix the comment display: rule-id, timestamp, GH visibility, link direction 2021-06-03 09:05:38 +02:00			`endif::env-github,rspecator-view[]`