Found unused objects JS-R1002
Anti-pattern
3 months ago
—
3 months old
Avoid instantiating unused object 'new Tooltip.default(el)'.
34 if (typeof window !== "undefined" && tooltip && !disabled) {
35 import("bootstrap/js/dist/tooltip").then((Tooltip) => {
36 const el = document.querySelector(`#${id}`);
37 if (el) new Tooltip.default(el);38 });
39 }
40 }, [tooltip, disabled]);
Description
Class constructors being invoked and then dropped immediately is an anti-pattern and should be avoided.
Objects instantiated with new should always be useful in some way (passed as a function argument, stored in a variable for later use, etc).
If the object is instantiated only to invoke some side effect in the constructor, then the side effect should be refactored out into it's own function.
Bad Practice
new Foo()
Recommended
const myFoo = new Foo()