54 lines
1.2 KiB
Plaintext
54 lines
1.2 KiB
Plaintext
== Why is this an issue?
|
|
|
|
When `if` is the only statement in the `else` block, it is better to use `else if` because it simplifies the code and makes it more readable.
|
|
|
|
When using nested `if` statements, it can be difficult to keep track of the logic and understand the flow of the code. Using `else if` makes the code more concise and easier to follow.
|
|
|
|
[source,javascript,diff-id=1,diff-type=noncompliant]
|
|
----
|
|
if (condition1) {
|
|
// ...
|
|
} else {
|
|
if (condition2) { // Noncompliant: 'if' statement is the only statement in the 'else' block
|
|
// ...
|
|
}
|
|
}
|
|
|
|
|
|
if (condition3) {
|
|
// ...
|
|
} else {
|
|
if (condition4) { // Noncompliant: 'if' statement is the only statement in the 'else' block
|
|
// ...
|
|
} else {
|
|
// ...
|
|
}
|
|
}
|
|
----
|
|
|
|
Fix your code by using `else if` if the nested `if` is the only statement in the `else` block.
|
|
|
|
[source,javascript,diff-id=1,diff-type=compliant]
|
|
----
|
|
if (condition1) {
|
|
// ...
|
|
} else if (condition2) {
|
|
// ...
|
|
}
|
|
|
|
|
|
if (condition3) {
|
|
// ...
|
|
} else if (condition4) {
|
|
// ...
|
|
} else {
|
|
// ...
|
|
}
|
|
----
|
|
|
|
|
|
== Resources
|
|
=== Documentation
|
|
|
|
* MDN web docs - link:++https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/if...else++[``++if...else++``]
|