Found unreachable code JS-0025
Performance
Major
a month ago
—
2 years old
Unreachable code
57 // <- default params should come last
58 response.status(code);
59 return response.end();
60 return true; // unreachable code 61}
62
63// unsafe permissionsUnreachable code
30
31 if (staticData.data) {
32 return staticData.data.toString();
33 return "static data not found";34 }
35 if (staticData) {
36 return staticData.data;
Description
Some code paths are unreachable because the return, throw, break, and continue statements unconditionally exit a block of code.
The code statements after the above keywords (which exit the code block) will not execute.
Bad Practice
function func() {
return true;
next_func();
}
function add(a, b) {
throw new Error("Oops!");
return a + b;
}
while(value) {
break;
do_action();
}
throw new Error("Oops!");
console.log("done");
function check() {
if (Math.random() < 0.5) {
return;
} else {
throw new Error();
}
console.log("done");
}
for (;;) {}
console.log("done");
Recommended
function func() {
return calc();
function calc() {
return 1;
}
}
switch (func) {
case 1:
var x = get_x();
use_x(x);
break;
}