Found shorthand type coercions JS-0066
Anti-pattern
Minor
8 months ago
—
2 years old
use
Boolean(monitor.isDragging()) instead22 post: post,
23 },
24 collect: (monitor) => ({
25 isDragging: !!monitor.isDragging(),26 }),
27 }))
28 useEffect(() => {use
Boolean(monitor.canDrop()) instead 71 },
72 collect: (monitor) => ({
73 isOver: !!monitor.isOver(),
74 canDrop: !!monitor.canDrop(), 75 }),
76 }))
77 const { data, isLoading, error } = useGetProductDayPostCountQuery(productId)use
Boolean(monitor.isOver()) instead 70 })
71 },
72 collect: (monitor) => ({
73 isOver: !!monitor.isOver(), 74 canDrop: !!monitor.canDrop(),
75 }),
76 }))use
Boolean(monitor.canDrop()) instead 55 },
56 collect: (monitor) => ({
57 isOver: !!monitor.isOver(),
58 canDrop: !!monitor.canDrop(), 59 }),
60 }))
61 const [DD, DoneDrop] = useDrop(() => ({use
Boolean(monitor.isOver()) instead 54 })
55 },
56 collect: (monitor) => ({
57 isOver: !!monitor.isOver(), 58 canDrop: !!monitor.canDrop(),
59 }),
60 }))
Description
Prefer using explicit casts by calling Number, Boolean, or String over using operators like +, !! or "" +.
This is considered best practice as it improves readability.
Bad Practice
const b = !!foo;
// The `+` operator does not change the value of its operand
// unless it's already a number.
let n = +foo;
n = 1 * foo;
const s = "" + foo;
Recommended
const b = Boolean(foo);
const n = Number(foo);
const s = String(foo);