Found shorthand type coercions JS-0066
Anti-pattern
Minor
15 days ago
—
3 years old
use
Number(n) instead 3}
4
5export function isInteger(n: number): boolean {
6 return n === +n && n === (n | 0); 7}
8
9export function isFloatOrInt(n: number): boolean {use
Number(n) instead 1export function isFloat(n: number): boolean {
2 return n === +n && n !== (n | 0); 3}
4
5export function isInteger(n: number): boolean {use
Number(i) instead629 let sink = this._sinks[i];
630
631 if (sink.component.id == componentid) {
632 this._sinks.splice(+i, 1);633 ret = true;
634 }
635 }
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);