rspec/rules/S3626/javascript/rule.adoc

== Why is this an issue?

Jump statements, such as ``++return++``, ``++break++`` and ``++continue++`` let you change the default flow of program execution, but jump statements that direct the control flow to the original direction are just a waste of keystrokes.

=== Noncompliant code example

[source,javascript]
----
function redundantJump(x) {
  if (x == 1) {
    console.log("x == 1");
    return; // Noncompliant
  }
}
----

=== Compliant solution

[source,javascript]
----
function redundantJump(x) {
  if (x == 1) {
    console.log("x == 1");
  }
}
----

=== Exceptions

``++break++`` and ``++return++`` inside ``++switch++`` statement are ignored, because they are often used for consistency. ``++continue++`` with label is also ignored, because label is usually used for clarity. Also a jump statement being a single statement in a block is ignored.

ifdef::env-github,rspecator-view[]

'''
== Implementation Specification
(visible only on this page)

include::../message.adoc[]

include::../highlighting.adoc[]

'''
== Comments And Links
(visible only on this page)

include::../comments-and-links.adoc[]

endif::env-github,rspecator-view[]