JS-0050 · Avoid use of `==` and `!=`

Avoid use of == and != JS-0050

Anti-pattern

Minor

2 years ago — 2 years old

Expected '===' and instead saw '=='

174 */
175const replaceMarquee = (match, direction, behavior, loop, scrollamount, scrolldelay, body) => {
176  // 내용이 빈 mq 태그는 무의미하므로 리턴
177  if (typeof body == "undefined") return "";178
179  // 빈 값 확인
180  if (typeof direction == "undefined") direction = "";

Expected '===' and instead saw '=='

inject/00-init/3-functions.js

181  if (typeof behavior == "undefined") behavior = "";
182  if (typeof loop == "undefined") loop = "";
183  if (typeof scrollamount == "undefined") scrollamount = "";
184  if (typeof scrolldelay == "undefined") scrolldelay = "";185
186  let scrollamount_value = scrollamount.replace(/[^0-9]/g, "");
187  // scrollamount 값을 50 이하로 제한함(50이 넘으면 50으로 강제 하향조정)

Expected '===' and instead saw '=='

inject/00-init/3-functions.js

180  if (typeof direction == "undefined") direction = "";
181  if (typeof behavior == "undefined") behavior = "";
182  if (typeof loop == "undefined") loop = "";
183  if (typeof scrollamount == "undefined") scrollamount = "";184  if (typeof scrolldelay == "undefined") scrolldelay = "";
185
186  let scrollamount_value = scrollamount.replace(/[^0-9]/g, "");

Expected '===' and instead saw '=='

inject/00-init/3-functions.js

179  // 빈 값 확인
180  if (typeof direction == "undefined") direction = "";
181  if (typeof behavior == "undefined") behavior = "";
182  if (typeof loop == "undefined") loop = "";183  if (typeof scrollamount == "undefined") scrollamount = "";
184  if (typeof scrolldelay == "undefined") scrolldelay = "";
185

Expected '===' and instead saw '=='

inject/00-init/3-functions.js

178
179  // 빈 값 확인
180  if (typeof direction == "undefined") direction = "";
181  if (typeof behavior == "undefined") behavior = "";182  if (typeof loop == "undefined") loop = "";
183  if (typeof scrollamount == "undefined") scrollamount = "";
184  if (typeof scrolldelay == "undefined") scrolldelay = "";

Description

It is considered good practice to use the type-safe equality operators === and !== instead of their regular counterparts == and !=.

The strict equality operators (=== and !==) use the strict equality comparison algorithm to compare two operands.

If the operands are of different types, return false.
If both operands are objects, return true only if they refer to the same object.
If both operands are null or both operands are undefined, return true.
If either operand is NaN, return false.
Otherwise, compare the two operand's values:
- Numbers must have the same numeric values. +0 and -0 are considered to be the same value.
- Strings must have the same characters in the same order.
- Booleans must be both true or both false.

The most notable difference between this operator and the equality (==) operator is that if the operands are of different types, the == operator attempts to convert them to the same type before comparing.

Bad Practice

a == b
foo == true
bananas != 1
value == undefined
typeof foo == 'undefined'
'hello' != 'world'
0 == 0
true == true
foo == null

drowsy-probius / twitch-icon-selector

Bad Practice

Recommended