refactor: use if one-liner where possible

src/display.js

@@ -66,31 +66,25 @@ export default class Display {
   /** Remove chart if it already exists */
   resetChart() {
     const chartStatus = Chart.getChart('chart') // eslint-disable-line
-    if (chartStatus != undefined) {
-      chartStatus.destroy()
-    }
+    if (chartStatus != undefined) chartStatus.destroy()
   }
 
   /**
    * Restart or reset the app.
    * If restart is triggered while timer isn't finished - mark as cancel.
-   * @param {boolean} reset - if the value is positive - change
+   * @param {boolean} isFullReset - if the value is positive - change
    * the api url, otherwise leave  the current one.
    */
-  restart(reset = false) {
-    if (this.timerElement.innerText !== '0s') {
+  restart(isFullReset = false) {
+    if (this.timerElement.innerText !== '0s')
       this.appElement.classList.add('cancel')
-    }
 
     this.resetDisplay()
     this.resetChart()
     this.resetStatsElements()
 
-    if (reset) {
-      this.create()
-    } else {
-      this.create(CURRENT_API_URL)
-    }
+    if (isFullReset) this.create()
+    else this.create(CURRENT_API_URL)
   }
 
   /**

src/key.js

@@ -79,13 +79,13 @@ export default class Key {
    *   - if the letter is incorrect - append it to word end.
    *     Oherwise ignore it.
    * - correct letter.
-   * @param {boolean} correct - true when match event.key.
+   * @param {boolean} isCorrect - true when match event.key.
    */
-  next(correct) {
-    const name = (correct === true) ? 'correct' : 'incorrect'
+  next(isCorrect) {
+    const name = (isCorrect === true) ? 'correct' : 'incorrect'
 
     if (!this.letterNode) {
-      if (!correct) this.appendLetter();
+      if (!isCorrect) this.appendLetter();
     } else {
       this.letterNode.classList.add(name)
       this.cursor.move(this.letterNode)

src/stats.js

@@ -26,11 +26,9 @@ export default class Stats {
     const match = JSON.parse(sessionStorage.getItem('stats'))
 
     let matches = retrieveLocalItem('matches')
-    if (matches) matches = matches.slice(-19)
 
-    if (!matches) {
-      matches = [match]
-    }
+    if (matches) matches = matches.slice(-19)
+    else matches = [match]
 
     matches.push(match)
 
@@ -74,9 +72,8 @@ export default class Stats {
     const corrWords = siblings.filter(sib => !sib.classList.contains('error'))
 
     let accuracy = 0
-    if (keystrokes > 0) {
+    if (keystrokes > 0)
       accuracy = toFixedWithoutZeros(keysCorrect / keystrokes * 100)
-    }
 
     const stats = {
       date: dateTimeNow(),

src/utils.js

@@ -26,9 +26,7 @@ export function dateTimeNow() {
  * completed touch-typing exercise.
  */
 export function retrieveLocalItem(key) {
-  if (localStorage.getItem(key)) {
-    return JSON.parse(localStorage.getItem(key))
-  }
+  if (localStorage.getItem(key)) return JSON.parse(localStorage.getItem(key))
 }
 
 /**
@@ -54,23 +52,16 @@ export async function getApiJson(url, element) {
   try {
     response = await fetch(url)
 
-    if (!response.ok) {
-      throw response
-    }
+    if (!response.ok) throw response
 
     const respJson = await response.json()
 
-    if (respJson.status == 404) {
-      throw respJson
-    } else {
-      return respJson
-    }
+    if (respJson.status == 404) throw respJson
+    else return respJson
+
   } catch (err) {
-    if (err.status) {
-      element.innerText = `${err.status}: connection failed.`
-    } else if (err.name) {
-      element.innerText = `Fails with ${err.name}.`
-    }
+    if (err.status) element.innerText = `${err.status}: connection failed.`
+    else if (err.name) element.innerText = `Fails with ${err.name}.`
 
     element.classList.add('network-error')