дальше не смотрел

уже параша

запросы смешаны с логикой

ну так там нужно условие..

Это все не будет работать в принципе

Потому что ничего функция не возвращает

Это все не будет работать в принципе

кстати работает.. приведи пример как бы переписать?

Добавит return

Добавит return

куда?

Return Promise.all...

Return Promise.all...

а... ну оно не надо там..

я ее вызываю прость await

И это не работает

а результат мне ее во вне не нужен

Точнее это не правильно

Await не нужен если функция не вернёт промис твой

Разберись что такое async await

Серьезно

Это всего лишь сахар

Await не нужен если функция не вернёт промис твой

согласен.. можно ее и так вызывать

тк не возвращает promise да и вообще ничего

Вот именно

Она undefined возвращает

да это баг... а как в целом все компактнее сделать?

Вынести запросы оттуда

Сделай типо сервиса

Который будет апи для запросов

Один запрос - один метод

И уже в твоей функции дергай именно методы эти

Вынести запросы оттуда

может просто в отдельные actions

Можно и в actions

Но неудобно будет переиспользовать

Если нужно будет ещё где то запрос сделать

Такой же

то дергать этот action

Не семантичный)

Логика работы с бэком отдельно

Бизнес логика стора отдельно

чуть чуть излишне думаю.

Это просто рекомендуемый паттерн

что ты предлагаешь вынести в func

тока axios

Бизнес логика стора отдельно

ну просто вижу что пока лишнее

Пока да

Но если рассчитывается что потом приложение будет расти то обязательно надо

расти если то не в этом)

Работы по времени много не увеличит

мне счас нужно чисто аккуратно..

Так что почему бы сразу не разделить сущности

переписать успею..

думаю так.

Дело твоё )

Мое дело предложить

promise.all дергать в компоненте

запросы в отдельные actions

Дело твоё )

но грубых ошибк кроме НЕ return нет?

в отдельный js файл только axios.get без содержимого then

так красиво считаешь?

и будет типа getServers.then() в store?

а его уже в all ?

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

но на него никакой реакции нету, даже если запихнуть его в computed

так вот, нормально ли юзать в данном случае created, или реально сделать красивое решение с mapActions?

но на него никакой реакции нету, даже если запихнуть его в computed

Куда ты там что запихиваешь? Map actions тебе ссылку на экшн стора создаст. В created дергай этот экшен

ERROR: S client not available

в данном случае данные подгружаются и без computed и mapActions

в данном случае данные подгружаются и без computed и mapActions

Ты обращаешься напрямую к стору. А map actions тебе метод компонента создаст. this.getProducts()

кажется понял, спасибо

может дать кто совет, если у меня форма состоит из 5 шагов(spa) я использую роутер для каждого шага, как мне быть с данными которые нужно потом отправить на сервер? мне все во vuex сохранять и в последнем шаге отправлять?

может дать кто совет, если у меня форма состоит из 5 шагов(spa) я использую роутер для каждого шага, как мне быть с данными которые нужно потом отправить на сервер? мне все во vuex сохранять и в последнем шаге отправлять?

да

может дать кто совет, если у меня форма состоит из 5 шагов(spa) я использую роутер для каждого шага, как мне быть с данными которые нужно потом отправить на сервер? мне все во vuex сохранять и в последнем шаге отправлять?

Я бы на всякий случай в сторадж ещё писал

получается обхожу нативную отправку формы

Именно

Но так ты можешь всегда восстановить данные формы на каждом шагу

Даже если страницу обновить

А потом пойти дальше и например если пользователь закрыл страницу и закончил работу с формой - выслать напоминание на почту и дать продолжить заполнять данные с любого шага

спасибо

Чекните как переписал

async getDefaultsTest({ commit, getters, dispatch }) { console.log("getDefaults") try { await Promise.all([ new Promise((resolve, reject) => { if (getters["settings/app_settings"]) { resolve() } else { console.log("need settings_get") return this.$axios .$get("settings_get") .then(result => { console.log("settings get ok") commit("settings/SET_APP_SETTINGS", result.appSettings) resolve() }) .catch(err => reject(err)) } }), new Promise((resolve, reject) => { if (getters["configure/servers"]) { resolve() } else { console.log("need servers_get") return this.$axios .$get("servers_get") .then(result => { console.log("servers get ok") commit("configure/SET_SERVERS", result.servers) resolve() }) .catch(err => reject(err)) } }) ]) console.log("promise all end") } catch (err) { const error_msg = err.response && err.response.data ? err.response.data : err throw new Error(Request error. ${error_msg}) } }

async getSettings({ commit, getters }) { console.log("getSettings") if (!getters["settings/app_settings"]) { const { appSettings } = await this.$axios.$get("settings_get") console.log("settings get ok") commit("settings/SET_APP_SETTINGS", appSettings) } }, async getServers({ commit, getters }) { console.log("getServers") if (!getters["configure/servers"]) { const { servers } = await this.$axios.$get("servers_get") console.log("servers get ok") commit("configure/SET_SERVERS", servers) } } ...... async fetch({ route, app, store }) { try { await Promise.all([store.dispatch("getSettings"), store.dispatch("getServers")]) } catch (err) { const error_msg = err.response && err.response.data ? err.response.data : err throw new Error(Request error. ${error_msg}) }

и не надо then() catch() и new Promise

@AlexShepard89 теперь норм, кроме того что сам axios вынести?

Мусор какой-то

что теперь можно сократить?

чуток еще подправил

ну а как правильно если нужно заполнять store несколькими паралельными запросами?

Пиши код в каком нибудь сервисе позволяющем расшарить его

Мне с телефона адово сложно читать )

А вообще найди на гитхабе просто проекты и посмотри на полную картинку

Оптимизировать одну функцию вне контекста задачи как то бесполезно )

Если бы ты работал в команде то согласовал бы стиль кода с ней

А для себя хреначь и делай выводы сам что удобно а что нет

Толку если я к примеру скажу как делать но ты не найдёшь собственные выводы о пользе

я смотрел.. тут все таки своя задача и не совсем тривиальная.. fetch этот часть картинки..