Что не так тут с оператором void?

Question

[sergeyviktorovich]

Promises must be handled appropriately or explicitly marked as ignored with the `void` operator.
    167 |   methods: {
    168 |     openUnit(it: string):void {
  > 169 |       this.$router.push(`/unit/${it}`)
        |       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    170 |     }
    171 |   },
    172 |   async beforeMount():Promise<void> {

Comments

[Василий Банников]

Rerurk, так это вроде компилятор и говорит. Ну или eslint

[Rerurk]

Василий Банников, ой туплю... Надо спать

Answer 1

[Aetae]

ESlint правило кторое не любит "висящих в воздухе" Promise.
Оно хочет чтоб ты сделал что-то из этого:

methods: {
  openUnit(it: string):void {
    void this.$router.push(`/unit/${it}`);
  }
}

methods: {
  openUnit(it: string) {
    return this.$router.push(`/unit/${it}`);
  }
}

methods: {
  async openUnit(it: string) {
    await this.$router.push(`/unit/${it}`);
  }
}

Типа если Promise явно не обработан, возможно ты ошибся и пошёл выполнять код, не дождавшись его разрешения.

Answer 2

[Василий Банников]

Он хочет чтобы ты написал вот так:
void this.$router.push(`/unit/${it}`);

Comments

[sergeyviktorovich]

Василий Банников, или так Promise<void>? не могу понять зачем перед this.$router писать void ,что возвращает функция пишут же после круглих скобок?
openUnit(it: string)::Promise {
this.$router.push(`/unit/${it}`)
}

[Aetae]

sergeyviktorovich, void перед промис - это не тип, а оператор javascript. Нужен он исключительно для eslint, чтоб явно указать, что возвращаемый промис мы вкидываем нафиг - т.е. не ждём.

Не путай ошибки компилятора и ошибки eslint. Первый указывает на реальные ошибки, из-за которых код не будет работать. Второй указывает на потенциальные ошибки, когда формально код работать будет, но, вероятно, не так, как ты хотел. В данном случае срабатывание ложное, но тут уж ничего не поделать.