Javascript-форум (https://javascript.ru/forum/)
-   Events/DOM/Window (https://javascript.ru/forum/events/)
-   -   Учебный проект - проблема с карточками (https://javascript.ru/forum/events/84855-uchebnyjj-proekt-problema-s-kartochkami.html)

Alina Sarbu 17.01.2023 12:09

Учебный проект - проблема с карточками
 
Всех приветствую! Не уверена, что мой вопрос 100% относится к данному разделу, но пока решила написать сюда, так как уже который день бьюсь с кодом.
Проблема касается закрытия карточек. Например, когда я пишу adam (или что-то еще) в поиске, нажимаю на Search- открывается карточка с пользователем adam (ну или с тем пользователем, которого ввела). Затем я пишу еще что-то в поиске, опять нажимаю Search и ожидаю, что первая карточка закроется. Но нет- вторая карточка просто открывается под первой, а первая никуда не девается. Есть только предположения, что например можно не добавлять wrapper каждый раз в mainEl, а просто сделать контейнером для карточек. Но так как я совсем новичок, я не могу это сделать технически. Вариант полного очищения, как я поняла не подходит в случае, так как в mainEl есть другие элементы.
Подскажите пожалуйста в каком месте кода и что именно мне надо поменять, чтоб добиться нужного поведения. Самой кажется, что надо что-то менять в функции if (response.ok), но никак не получается сделать то, что надо. А вот сам код:

const mainEl = document.querySelector(".main");
const wrapper = document.createElement("div");

const formEl = document.createElement("form");
formEl.addEventListener("submit", async (e) => {
  e.preventDefault();
  const inputsValue = Object.fromEntries(new FormData(e.target));
  const response = await fetch(`
https://api.github.com/users/${inputsValue.name}`);

  if (response.ok) {
    const data = await response.json();
    wrapper.appendChild(createProfileEl(data));
    mainEl.appendChild(wrapper);

    inputEl.value = "";
  } else {
    alert("The user is not found");
  }
});

const inputEl = document.createElement("input");
inputEl.classList.add("search-input");
inputEl.setAttribute("name", "name");

const searchButtonEl = document.createElement("button");
searchButtonEl.classList.add("search-button");
searchButtonEl.setAttribute("type", "submit");
searchButtonEl.innerHTML = "Search";

formEl.appendChild(inputEl);
formEl.appendChild(searchButtonEl);
mainEl.appendChild(formEl);

function createProfileEl(profileData) {
  const element = document.createElement("div");
  element.classList.add("profile");

  element.innerHTML = `
  <img class="search-image" src=${profileData.avatar_url}></img> 
  <p class="search-text"><span>Name: </span>${profileData.name || "No info"}</p>
    <p class="search-text"><span>City: </span>${
      profileData.location || "No info"
    }</p>
    <p class="search-text"><span>About: </span>${
      profileData.bio || "No info"
    }</p>
  `;

  element.appendChild(createDeleteBtnEl());
  return element;
}

function createDeleteBtnEl() {
  const element = document.createElement("button");
  element.classList.add("close-button");
  element.innerText = "Close";
  element.addEventListener("click", (e) => {
    wrapper.innerHTML = "";
  });
  return element;
}

Скрины: https://drive.google.com/drive/u/0/f...4cFy4ScxfBsvbg

voraa 17.01.2023 12:26

Попробуйте после строки 12
wraper.remove();
wraper.textContent = '';


Зы Вместо appendChild можно просто append
https://developer.mozilla.org/ru/doc...Element/append

рони 17.01.2023 12:36

Alina Sarbu,
if (response.ok) {
    const data = await response.json();
*!*
    wrapper.innerHTML = "";
*/!*
    wrapper.appendChild(createProfileEl(data));
    mainEl.appendChild(wrapper);
    inputEl.value = "";
  }

IamAnton 18.01.2023 09:59

Alina Sarbu,
Я бы если я новичок, писал комментарии в коде. Быстрей становится понятно и тебе и другим.

IamAnton 18.01.2023 10:18

Alina Sarbu,
Я бы не стал создавать элемент wrapper во второй строке, а сделал бы это после if (response.ok)....

Alina Sarbu 18.01.2023 11:36

Цитата:

Сообщение от voraa (Сообщение 550037)
Попробуйте после строки 12
wraper.remove();
wraper.textContent = '';


Зы Вместо appendChild можно просто append
https://developer.mozilla.org/ru/doc...Element/append

Огромное вам спасибо за помощь! Применила оба совета, как с wrapper'ом так и сократила appendChild, написала просто append и теперь все работает как надо.

Alina Sarbu 18.01.2023 11:41

Цитата:

Сообщение от рони (Сообщение 550038)
Alina Sarbu,
if (response.ok) {
    const data = await response.json();
*!*
    wrapper.innerHTML = "";
*/!*
    wrapper.appendChild(createProfileEl(data));
    mainEl.appendChild(wrapper);
    inputEl.value = "";
  }

И вам огромное спасибо за ответ, тоже работает как надо.

Alina Sarbu 18.01.2023 11:45

Цитата:

Сообщение от IamAnton (Сообщение 550057)
Alina Sarbu,
Я бы не стал создавать элемент wrapper во второй строке, а сделал бы это после if (response.ok)....

Насчет комментов да, вы правы, надо будет хоть какие-то вставлять в последующих проектах учебных. А вот насчет wrapper, написала его после if (response.ok) и в таком случае у меня ломается кнопка Close, т.е. когда хочу закрыть карточку Close просто не реагирует. Потому решила оставить как и было - объявить переменную с wrapper сверху :)

voraa 18.01.2023 12:00

Вообще трудно понять что такое wraper, и когда он должен появляться? Если он всегда должен быть на странице, то почему его не разместить сразу, а не после считывания информации о карточке? И зачем его тогда снова присоединять к main?
Если он должен быть только вместе с карточкой, то почему он не убирается при закрытии карточки, а остается пустым на странице?

voraa 18.01.2023 12:08

Цитата:

Сообщение от Alina Sarbu
и в таком случае у меня ломается кнопка Close, т.е. когда хочу закрыть карточку Close просто не реагирует.

Сделать так
function createDeleteBtnEl(element) {
  const button = document.createElement("button");
  button.classList.add("close-button");
  button.innerText = "Close";
  button.addEventListener("click", (e) => {
    element.remove()
  });
  element.append(button);
}

А стр 50
element.appendChild(createDeleteBtnEl());

Переписать так
createDeleteBtnEl(element);


Часовой пояс GMT +3, время: 04:00.