Found empty block statements JS-0009
Anti-pattern
5 months ago
—
5 months old
Empty block statement.
130 var evt = createEvent("autosize:resized");
131 try {
132 ta.dispatchEvent(evt);
133 } catch (err) {134 }135 }
136 }
137 var pageResize = function pageResize2() {Empty block statement.
3261 const responseText = await response.text();
3262 const snapshot = PageSnapshot.fromHTMLString(responseText);
3263 this.snapshotCache.put(location2, snapshot);
3264 } catch (_) { 3265 } 3266 }
3267 };
3268 var Session = class {Empty block statement.
4125 disconnect() {
4126 try {
4127 this.remove();
4128 } catch (_a) { 4129 } 4130 }
4131 removeDuplicateTargetChildren() {
4132 this.duplicateChildren.forEach((c) => c.remove());Empty block statement.
8759 var i3 = o("autosize:resized");
8760 try {
8761 e4.dispatchEvent(i3);
8762 } catch (e5) { 8763 } 8764 }
8765 }
8766 }(e3);
Description
Empty block statements, while not technically errors, usually occur due to refactoring that wasn't completed. They can mislead the reader.
If you still want to keep an empty block, add a comment saying empty inside the block.
Bad Practice
if (someCheck) {}
while (someCheck) {}
try {
doSomething();
} catch(err) {
} finally {
}
Recommended
if (someCheck) {
// empty
}
while (someCheck) {
/* empty */
}
try {
doSomething();
} catch (err) {
// continue regardless of error
}
try {
doSomething();
} finally {
/* continue regardless of error */
}