Found shorthand type coercions JS-0066
Anti-pattern
Minor
8 months ago
—
3 years old
use
Boolean(ctx.session.authorized) instead36 if (await blacklist.check(ctx.ip)) {
37 const scanResult = await waf.scan(ctx)
38 if (!scanResult) {
39 if (!!ctx.session.authorized || !config.pow) {40 if (config.rate_limit) {
41 await Object.assign(
42 ctx.session,
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);