Move clearInputs functionality to view

amejid · Jul 3, 2022 · 68210f1 · 68210f1
1 parent 75b15d0
commit 68210f1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/index.js b/index.js
@@ -9,7 +9,6 @@ const addBook = () => {
   const [title, author] = bookAddView.getFormValues();
   if (!title || !author) return;
   model.addBook(title, author);
-  bookAddView.clearInputs();
   bookView.render(model.state.books);
 };
 

diff --git a/modules/views/bookAddView.js b/modules/views/bookAddView.js
@@ -5,13 +5,16 @@ class BookAddView {
 
   #btnAddEl = document.querySelector('.btn-submit');
 
-  clearInputs() {
+  #clearInputs() {
     this.#titleEl.value = '';
     this.#authorEl.value = '';
   }
 
   getFormValues() {
-    return [this.#titleEl.value.trim(), this.#authorEl.value.trim()];
+    const info = [this.#titleEl.value.trim(), this.#authorEl.value.trim()];
+
+    this.#clearInputs();
+    return info;
   }
 
   addHandlerAdder(handler) {