refactor: early return으로 변경

Author: bgyoons

src/components/TodoForm.ts

@@ -17,17 +17,17 @@ export default class TodoForm {
       <button>추가</button>
     `;
 
-    if (!this.isInit) {
-      this.$form.addEventListener("submit", (e) => {
-        e.preventDefault();
-        const $todo = this.$form.querySelector<HTMLInputElement>("input[name=todo]");
-        if (!$todo) return;
-        const text = $todo.value;
-        if (text.length > 1) {
-          $todo.value = "";
-          this.onSubmit(text);
-        } else alert("두 글자 이상 입력해주세요");
-      });
-    }
+    if (this.isInit) return;
+
+    this.$form.addEventListener("submit", (e) => {
+      e.preventDefault();
+      const $todo = this.$form.querySelector<HTMLInputElement>("input[name=todo]");
+      if (!$todo) return;
+      const text = $todo.value;
+      if (text.length > 1) {
+        $todo.value = "";
+        this.onSubmit(text);
+      } else alert("두 글자 이상 입력해주세요");
+    });
   };
 }

src/components/TodoList.ts

@@ -22,24 +22,24 @@ export default class TodoList {
       const target = e.target as HTMLLIElement;
       const $li = target.closest("li");
 
-      if ($li) {
-        const newState = [...this.state];
-        if (typeof $li.dataset.index !== "string") return;
-        const index = +$li.dataset.index;
+      if (!$li) return;
 
-        if (target.className === "deleteBtn") {
-          newState.splice(index, 1);
-          this.setState(newState);
-        } else if (target.className.includes("todoList")) {
-          const isCompleted = target.className.includes("completed");
-          if (isCompleted) target.classList.remove("completed");
-          else target.classList.add("completed");
-          newState[index] = {
-            ...newState[index],
-            isCompleted: !isCompleted,
-          };
-          this.setState(newState);
-        }
+      const newState = [...this.state];
+      if (typeof $li.dataset.index !== "string") return;
+      const index = +$li.dataset.index;
+
+      if (target.className === "deleteBtn") {
+        newState.splice(index, 1);
+        this.setState(newState);
+      } else if (target.className.includes("todoList")) {
+        const isCompleted = target.className.includes("completed");
+        if (isCompleted) target.classList.remove("completed");
+        else target.classList.add("completed");
+        newState[index] = {
+          ...newState[index],
+          isCompleted: !isCompleted,
+        };
+        this.setState(newState);
       }
     });
   }