소스 검색

refactor: extract hardcoded values to const

Sasha Aryutin 3 년 전
부모
커밋
599f35a185

+ 7 - 3
Extensions/UserScript/Return Youtube Dislike.user.js

@@ -19,6 +19,10 @@
 // @grant        GM_addStyle
 // @run-at       document-end
 // ==/UserScript==
+const LIKED_STATE = "LIKED_STATE";
+const DISLIKED_STATE = "DISLIKED_STATE";
+const NEUTRAL_STATE = "NEUTRAL_STATE";
+
 function cLog(text, subtext = '') {
   subtext = subtext.trim() === '' ? '' : `(${subtext})`;
   console.log(`[Return YouTube Dislikes] ${text} ${subtext}`);
@@ -60,12 +64,12 @@ function isVideoNotDisliked() {
 
 function getState() {
   if (isVideoLiked()) {
-    return "liked";
+    return LIKED_STATE;
   }
   if (isVideoDisliked()) {
-    return "disliked";
+    return DISLIKED_STATE;
   }
-  return "neutral";
+  return NEUTRAL_STATE;
 }
 
 function setLikes(likesCount) {

+ 14 - 10
Extensions/chrome/return-youtube-dislike.script.js

@@ -1,7 +1,11 @@
+const LIKED_STATE = "LIKED_STATE";
+const DISLIKED_STATE = "DISLIKED_STATE";
+const NEUTRAL_STATE = "NEUTRAL_STATE";
+
 (function (extensionId) {
   let storedData = {
     dislikes: 0,
-    previousState: 'neutral'
+    previousState: NEUTRAL_STATE
   };
 
   function cLog(message, writer) {
@@ -54,12 +58,12 @@
 
   function getState() {
     if (isVideoLiked()) {
-      return {current: "liked", previous: storedData.previousState};
+      return {current: LIKED_STATE, previous: storedData.previousState};
     }
     if (isVideoDisliked()) {
-      return {current: "disliked", previous: storedData.previousState};
+      return {current: DISLIKED_STATE, previous: storedData.previousState};
     }
-    return {current: "neutral", previous: storedData.previousState};
+    return {current: NEUTRAL_STATE, previous: storedData.previousState};
   }
 
   //---   Sets The Likes And Dislikes Values   ---//
@@ -122,10 +126,10 @@
 
   function likeClicked() {
     console.log(storedData.previousState)
-    if (storedData.previousState == 'disliked') {
+    if (storedData.previousState == DISLIKED_STATE) {
       storedData.dislikes--;
       setDislikes(numberFormat(storedData.dislikes));
-      storedData.previousState = 'liked';
+      storedData.previousState = LIKED_STATE;
     }
   }
 
@@ -134,14 +138,14 @@
 
     console.log("Dislike State:",getState());
 
-    if (state == 'disliked') {
+    if (state == DISLIKED_STATE) {
       storedData.dislikes++;
       setDislikes(numberFormat(storedData.dislikes));
-      storedData.previousState = 'disliked';
-    } else if (state == 'neutral') {
+      storedData.previousState = DISLIKED_STATE;
+    } else if (state == NEUTRAL_STATE) {
       storedData.dislikes--;
       setDislikes(numberFormat(storedData.dislikes));
-      storedData.previousState = 'neutral';
+      storedData.previousState = NEUTRAL_STATE;
     }
 
     // setState();

+ 12 - 8
Extensions/firefox/return-youtube-dislike.script.js

@@ -1,7 +1,11 @@
+const LIKED_STATE = "LIKED_STATE";
+const DISLIKED_STATE = "DISLIKED_STATE";
+const NEUTRAL_STATE = "NEUTRAL_STATE";
+
 if (!storedData) {
   var storedData = {
     dislikes: 0,
-    previousState: 'neutral'
+    previousState: NEUTRAL_STATE
   };
 }
 
@@ -52,12 +56,12 @@ function isVideoNotDisliked() {
 
 function getState() {
   if (isVideoLiked()) {
-    return {current: "liked", previous: storedData.previousState};
+    return {current: LIKED_STATE, previous: storedData.previousState};
   }
   if (isVideoDisliked()) {
-    return {current: "disliked", previous: storedData.previousState};
+    return {current: DISLIKED_STATE, previous: storedData.previousState};
   }
-  return {current: "neutral", previous: storedData.previousState};
+  return {current: NEUTRAL_STATE, previous: storedData.previousState};
 }
 
 //---   Sets The Likes And Dislikes Values   ---//
@@ -120,14 +124,14 @@ function likeClicked() {}
 
 function dislikeClicked() {
   let state = getState().current;
-  if (state == "disliked") {
+  if (state == DISLIKED_STATE) {
     storedData.dislikes++;
     setDislikes(numberFormat(storedData.dislikes));
-    storedData.previousState = 'disliked';
-  } else if (state == "neutral") {
+    storedData.previousState = DISLIKED_STATE;
+  } else if (state == NEUTRAL_STATE) {
     storedData.dislikes--;
     setDislikes(numberFormat(storedData.dislikes));
-    storedData.previousState = 'neutral';
+    storedData.previousState = NEUTRAL_STATE;
   }
 
   // setState();