style(Navigation): implement switch cases for code reduction - I91

Signed-off-by: irmerk <[email protected]>

src/Navigation/actions.js

@@ -1,17 +1,18 @@
-export const isClause = input => input === 'clause';
-export const isHeadingOne = input => input === 'heading_one';
-export const isHeadingTwo = input => input === 'heading_two';
-export const isHeadingThree = input => input === 'heading_three';
-
 const truncateHeading = (heading, length) => ((heading.length > length)
   ? `${heading.substring(0, length)}...`
   : heading);
 
-
 export const truncateHeader = ({ type, text }) => {
-  if (isClause(type)) return truncateHeading(text, 20);
-  if (isHeadingOne(type)) return truncateHeading(text, 22);
-  if (isHeadingTwo(type)) return truncateHeading(text, 18);
-  if (isHeadingThree(type)) return truncateHeading(text, 14);
-  return 'Error!';
+  switch (type) {
+    case 'clause':
+      return truncateHeading(text, 20);
+    case 'heading_one':
+      return truncateHeading(text, 22);
+    case 'heading_two':
+      return truncateHeading(text, 18);
+    case 'heading_three':
+      return truncateHeading(text, 14);
+    default:
+      return 'Error!';
+  }
 };

src/Navigation/generators.js

@@ -2,7 +2,7 @@
 import React from 'react';
 
 /* Actions */
-import * as ACT from './actions';
+import { truncateHeader } from './actions';
 
 /* Styling */
 import * as SC from './styles';
@@ -18,50 +18,49 @@ export const headerGenerator = (props) => {
 
   return headers.map((header) => {
     const { type, key } = header;
-    if (ACT.isClause(type)) {
-      return (
+    switch (type) {
+      case 'clause':
+        return (
           <SC.HeaderClause
               key={key}
               onClick={() => navigateHeader(key, type)}
               {...props.styleProps}
           >
-              {ACT.truncateHeader(header)}
+              {truncateHeader(header)}
           </ SC.HeaderClause>
-      );
-    }
-    if (ACT.isHeadingOne(type)) {
-      return (
+        );
+      case 'heading_one':
+        return (
           <SC.HeaderOne
               key={key}
               onClick={() => navigateHeader(key, type)}
               {...props.styleProps}
           >
-              {ACT.truncateHeader(header)}
+              {truncateHeader(header)}
           </ SC.HeaderOne>
-      );
-    }
-    if (ACT.isHeadingTwo(type)) {
-      return (
+        );
+      case 'heading_two':
+        return (
           <SC.HeaderTwo
               key={key}
               onClick={() => navigateHeader(key, type)}
               {...props.styleProps}
           >
-              {ACT.truncateHeader(header)}
+              {truncateHeader(header)}
           </ SC.HeaderTwo>
-      );
-    }
-    if (ACT.isHeadingThree(type)) {
-      return (
+        );
+      case 'heading_three':
+        return (
           <SC.HeaderThree
               key={key}
               onClick={() => navigateHeader(key, type)}
               {...props.styleProps}
           >
-              {ACT.truncateHeader(header)}
+              {truncateHeader(header)}
           </ SC.HeaderThree>
-      );
+        );
+      default:
+        return 'Error!';
     }
-    return 'Error!';
   });
 };