Detected empty functions JS-0321
Anti-pattern
Minor
6 months ago
—
3 years old
Unexpected empty arrow function
20let scale: number;
21const PATCH_SIZE = 32;
22const PADDING = 2;
23let resolver = () => {}; 24const ready = new Promise<void>(r => {
25 resolver = r;
26});
Description
Having empty functions hurts readability, and is considered a code-smell. There's almost always a way to avoid using them. If you must use one, consider adding a comment to inform the reader of its purpose.
Bad Practice
getUser('SwaGaLisTiQuE', () => {})
Recommended
getUser('SwaGaLisTiQuE', () => {
// empty because <reason>
})