Как улучшить код?

Question

[lavezzi1]

Подскажите пожалуйста как лучше можно переписать этот кусок?

async createItem() {
        let record;
        let success = false;
        this.isLoading= true;
        await this.$fetch(`/api/items/${this.params.id}/new/`, {
          method: 'POST',
          body: JSON.stringify(this.createItemBody),
        })
          .then(response => response.json())
          .then((response) => {
            item = response;
            success = true;
          })
          .catch((err) => {
            this.isLoading= false;
            window.console.log(err);
          });
        if (success) {
          await this.dispatch('fetchSecondItem', id);
          this.items.push(item);
          this.dispatch('notification', {
            type: 'success',
            message: 'Item was created.',
          });
          this.resetForm();
        }
        this.isLoading= false;
      },

Comments

[Negwereth]

Пингани меня через пару часов, напишу с пояснениями

[lavezzi1]

ping https://toster.ru/@Negwereth

[lavezzi1]

Negwereth, Привет?

[Negwereth]

lavezzi1, пардон. Тебе уже ответили так, как я собирался ) Разве что без комментариев )

[lavezzi1]

Negwereth, Ок. То есть try/catch и все, остальное все нормально?

[Negwereth]

lavezzi1, Да, именно так это и должно быть

Answer 1

[Ваге]

я не понимаю что вы делаете наверное работайте на redux или flux.вы можете использовать try/catch и await вместо then.может что то вроде этого .что поделать с let record.вы даже не использовали.из чего вы взяли item

.then((response) => {
            item = response;
            success = true;
          })

async createItem() {
    this.isLoading = true;

    try {
      let res = await this.$fetch(`/api/items/${this.params.id}/new/`, {
        method: "POST",
        body: JSON.stringify(this.createItemBody)
      });
      let json = await res.json();

      await this.dispatch("fetchSecondItem", id);
      this.items.push(json);
      this.dispatch("notification", {
        type: "success",
        message: "Item was created."
      });
      this.isLoading = true;
      this.resetForm();
    } catch (err) {
      this.isLoading = false;
      console.log(err);
    }
  }