да, вообще я ожидал, что если внутри makeRequest происходит ошибка, то я для getProjects отдаю все равно успешный ответ

да, отдаете

return ['test']

вам приходит ответ в виде массива со строкой test

да, я понял... я же сделал деструктуризацию...

потом вот эта строка const { data }

радостно вытягивает у него поле data

фэйл и мягкий js

спс

кря

https://ru.wikipedia.org/wiki/%D0%9C%D0%B5%D1%82%D0%BE%D0%B4_%D1%83%D1%82%D1%91%D0%BD%D0%BA%D0%B0

ну, тут дело в том, что я просто в упор не видел, что я сделал деструктуризацию

кря :)

я не пойму, как лучше обрабатывать ошибки? В самом экшене или же в функции? Просто если делать в экшене, то в будущем наплодится огромное количество try,catch на каждый экшен

ну скажем так

вот эта строчка слегка отстой

const { data } = await projectsApi.getProjects()

почему - потому что у вас "протекла" абстракция - штука которая получает данные знает о формате ответов аксиоса

это стоит прятать за апи, чтоб вам сразу возращался результат

и ошибки стоит обрабатывать в экшне, потому что апи не имеет права "глотать" ошибки

хм

но как это совместить? Я должен через getProjects() возращать типо data, но если ошибка то должен вернуть headers?

а зачем мы вообще глотаете ошибку

я тоже не знаю

просто, у меня getProjects выходит просто шорткат какой-то для аксиоса

любое апи и есть простая обертка над какой-нибудь библиотекой

транспортный слой не должен содержать бизнес логики

у нас так же

https://gist.github.com/anonymous/df3a35aee50bc7413c30d8b8a6942ef4

все апи просто обертка над аксиосом

получается, в этот раз мне утенок нужен, потому, что я не понимаю, что я хочу от своего апи.

я хочу получать данные, но если ошибка какая-та, то хочу ее обработать

ошибку надо обрабатывать в экшнах

Потому что апи - транспортный слой

обработка ошибок - бизнес логика

нельзя их смешивать

хм, понятно. Но как быть с const { data } = await projectsApi.getProjects()

эта штука должна отдавать данные или ошибку

я должен всегда знать, в каком формате приходят данные от этой штуки

или нет?

ошибку не глотать и ловить ее через catch

const actions = { async getProjects ({ commit }) { try { const { data } = await projectsApi.getProjects() commit(PROJECTS_LIST, data) } catch (err) { console.log(err) } } }

ну вот там в catch и делайте то что вам надо

как вам надо обрабатывать ошибки

а что насчет протечки абстракции?

я знаю, что мне приходят данные в дату

или я путаюсь?

да, это тоже бы пофиксить

возвращать .data сразу из getProjects()

вряд ли вам где-то заголовки нужны

т.е. я всегда возвращаю дату, но если ошибка какая-та, то уже разбираюсь с заголовками?

ну я не помню, возвращает ли аксиос свой класс ошибки

чтоб можно было через instanceof проверить

но в общем да

"ошибка" и "данные" это разные сущности

им более чем допустимо иметь разный формат

ясно, спасибо. Буду писать, думать, работать. Осваивать

ошибку надо обрабатывать в экшнах

о да, я этому научился, когда переписывал redis protocol parser на плюсах в этом году, еще тогда удивлялся, почему я раньше так не сделал -_-

вообще я очень рекомендую всем периодически переписывать свой старый код

по нескольким причинам: 1) явно виден прогресс 2) понятно, зачем нужно писать хорошо и/или зачем нужны коментраии 3) четко формулируется мысль, почему что-то плохо, а что-то хорошо

комментарии не нужны

за очень редким исключением

поэтому и написал "и/или", потому что либо говнокод и коменты, либо хороший и понятный код

у нас на все апи один коммент

"это апи"

// See http://repository.cmu.edu/cgi/viewcontent.cgi?article=3059&context=compsci why we need that

перед стремной строчкой

ERROR: S client not available

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

:D

у меня пока больше 5 слов не было

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

лол

лол

мое темное прошлое, именно поэтому дорога в рай мне закрыта

))))

я как Константин из одноименного фильма

кстати, хороший фильм

ога

фильм да, сериал...

фильм да, сериал...

а про сериал никто и не знает

я вот не знаю

я смотрел. Там какой-то бомж был ,а не актер

на данный момент мне только 4 сериала понравились

и ангел негр

мне гораздо больше (:

но я впринципе сериалы только в этом году начал смотреть, до этого - нини

да годный фильм

а что скала могёт без jvm?

я кст сериалы не смотрел никакие вообще

мой топчик: 1) боевое братство 2) поколение убийц 3) во плоти 4) миллиарды

порадок не соблюден

все более или менее равноценны

ну на 25% сойдет :)

4 класс вещь

4 класс вещь

ога, как раз смотрю потихонечку

ну на 25% сойдет :)

на чем сошлось?

миллиарды вообще впечатлил, нету убийств и всякого типичного для us сериалов

мой топчик: 1) боевое братство 2) поколение убийц 3) во плоти 4) миллиарды

ктстаи это все от HBO, кмк ребята умеют снимать

Я сериалы тоннами поглощаю

Billions конечно же

Billions конечно же

смотрел The boss?