Правильно ли я использую try-catch?

Question

[Артем Андреев]

Mеня смущает, как я использую try-catch в моем коде, но все работает как нужно. Хотелось бы услышать от вас критику или примеры. Спасибо!

async function getRequest(url) {
    try {
      let response = await fetch(url);
      if (!response.ok) {
        throw new Error(errorFunc(response.status));
      }
      return await response.json();
    } catch (err) {
      console.log(err);
    }
  }

  async function getJson(url) {
    try {
      let json = await getRequest(url);
      okFunc(json);
    }
    catch (err) {
      console.log(err);
    }
  }


  let input = document.getElementById('choiseCityName');

  const okFunc = (json) => {
    try {
      if (json) {
        input.placeholder = 'Please enter a your city name.';
        input.classList.remove('placeholderred');
        input.classList.add('placeholderOk');
        input.focus();
        return weather.showWeatherData(json);
      }
    } catch (err) {
      console.log(err);
    }
  }

  const errorFunc = (err) => {
    if (err == '404')  {
     input.value = '';
     input.classList.add('placeholderred');
     input.placeholder = "Sorry we can not find your city";
     return err;
   }
   throw new Error(err);
  }

Answer 1

[Stockholm Syndrome]

у fetch есть своя обработка ошибок

async function getRequest(url) {
  let response = await fetch(url);
  return await response.json().catch((err) => new Error(errorFunc(response.status)));
}

async function getJson(url) {
  let json = await getRequest(url).catch((err) => console.log(err));
  okFunc(json);
}

Comments

[Артем Андреев]

Спасибо большое! А можете прокомментировать то, как я вывожу в инпут ошибку? Правильный ли подход?

[Дима Гашко]

Артем Андреев, placeholder - это пример заполнения формы. Например, для поля "ФИО" placeholder может быть равным: "Андреев Андрей Андреевич".
Оно уж точно не предназначено для ошибок.

[Артем Андреев]

Дима Гашко, да, согласен. Не думал об этом. Но там просто выводится красным "Sorry we can not find your city". Так не делают?

[Дима Гашко]

Артем Андреев, Некоторые верстальщики мирового уровня вообще не против использования placeholder-ов.

Но в любом случае это уж точно совсем не красиво, не семантично да и вообще такой способ в принципе ужасный. Если я сделал опечатку в названии я не хочу потом все занного вводить временами совсем не хочу.

[Артем Андреев]

Дима Гашко, понял что вы имеете ввиду. Сделаю по другому. Просто я пишу сам от себя (на портфолио так сказать) функционал сам придумываю)