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

Avoid use of == and != JS-0050

Anti-pattern

Minor

3 months ago — a year old

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

 921      this.renderInstances.push(renderer);
 922
 923      let linesToRender = this.lines.slice(0, this.linesIndex - 1);
 924      if (linesToRender != null && linesToRender.length > 0) { 925         await renderer.render(linesToRender);
 926      }
 927      this.linesIndex = 0;

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

src/gcodeviewer.js

671      let value = this.clamp(this.simulationLastUpdate / this.simulationTimeToMove, 0, 1); 
672      Vector3.LerpToRef(this.simulationCurrentLine.start, this.simulationCurrentLine.end, value, this.simLine); 
673      this.updateToolPosition(this.simLine); 
674      if (this.simulationLastUpdate >= this.simulationTimeToMove || this.simulationTimeToMove == Infinity || this.simulationTimeToMove == 0) { 675         let update = this.simulationMultiplier > 10 ? this.simulationMultiplier : 1; 
676         let ratio = this.scene.getAnimationRatio() 
677         if (ratio > 3) { update += ~~ratio; }

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

src/gcodeviewer.js

671      let value = this.clamp(this.simulationLastUpdate / this.simulationTimeToMove, 0, 1); 
672      Vector3.LerpToRef(this.simulationCurrentLine.start, this.simulationCurrentLine.end, value, this.simLine); 
673      this.updateToolPosition(this.simLine); 
674      if (this.simulationLastUpdate >= this.simulationTimeToMove || this.simulationTimeToMove == Infinity || this.simulationTimeToMove == 0) { 675         let update = this.simulationMultiplier > 10 ? this.simulationMultiplier : 1; 
676         let ratio = this.scene.getAnimationRatio() 
677         if (ratio > 3) { update += ~~ratio; }

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

src/gcodeviewer.js

656         this.simulationStopped(); 
657      } 
658 
659      if (this.simulationCurrentLine != this.gcodeProcessor.renderedLines[this.gcodeProcessor.lastFilePositionIndex]) { 660         this.simulationCurrentLine = this.gcodeProcessor.renderedLines[this.gcodeProcessor.lastFilePositionIndex]; 
661         this.updateToolPosition(this.simulationCurrentLine.start); 
662         this.simulationLastUpdate = 0;

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

src/RenderModes/line.js

 25 
 26      for (var lineIdx = 0; lineIdx < lines.length; lineIdx++) { 
 27         let line = lines[lineIdx]; 
 28         if (line == null) break;  29         let tool = this.tools[line.tool]; 
 30 
 31         gcodeLineIndex[lineIdx] = line.gcodeFilePosition;

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

Sindarius / npm_gcodeviewer

Bad Practice

Recommended