Нужен ли return при ES6 синтаксисе?

Question

[Ninja Mate]

Нужна помощь в понимании es6
В стрелочных функциях есть возможность не писать return

export const receive = () => ({
    type: actions.SET
})

Выглядит очень красиво, но как сделать на более сложных примерах, где до return еще нужно произвести какие то действия

export const onSubmit = (data, token) => {

    axios.defaults.headers.common['Authorization'] = token;

    return dispatch => (
        axios.post(apiUrl + '/pss', data)
            .then(res => {

                if (res.data !== null) dispatch(onSubmitSuccess())
                else return { type: psActions.ON_SUBMIT_ERROR }

            })
    )
    
}

Буду рад подробному разъяснению

Answer 1

[Алексей Шашенков]

А в чем проблема. 1 строка - return не нужен. Много, напишите явно return. Или тяжело 6 клавиш нажать? Вы код для себя пишите или для тех кто потом будет с бубном танцевать?

Comments

[Ninja Mate]

нужны примеры моих проектов на es6 для собеседования, у меня все только на 5, пытаюсь переписать максимально красиво

[Алексей Шашенков]

Ninja Mate, Страсти масти. Ну тогда точно явно нужен return
А еще jsdoc не помешает. Чтобы ясно было, что и для чего написано.

[Роман Александрович]

Ninja Mate, думаю что рвение правильное. рано или поздно все равно придётся переходить. но если вы будете в тупую везде весь проект переписывать на es6 к хорошему это не приведет а вызовит только лишние вопросы.

[alvvi]

Ninja Mate, даа, ведь на работе важна будет именно красота, а не читаемость вашего кода...

[Ninja Mate]

Роман Александрович, конечно не буду, это просто пример для работодателя. очень всем es6 нравится

[Алексей Шашенков]

Ninja Mate, Ну es6(es2015) на этом не закончился. Я думаю деструктуризация, spread, классы, новые методы на массивах и объектах, строках. Так что, если включать на полную, то сразу на async/await переписывайте

[Ninja Mate]

Алексей Шашенков, "jsdoc" я вот это прилепил https://github.com/styleguidist/react-styleguidist

[Алексей Шашенков]

Ninja Mate, Ну если поселится только в react то пойдет. Но не все строится на реакте. Зачастую логика выносится из представления, по возможности. Ну это от приложения зависит. Если, чисто web-приложение для представления данных сервера, пойдет. А если на клиенте есть логика, то кто его. Откройте свой код который писали 2 месяца назад и почитайте.

[Ninja Mate]

Алексей Шашенков, все на rest api, а логика на клиенте совсем минимальная. "2 месяца назад" переписываю кусочек проекта который пилил почти год по 40 часов в неделю. "то сразу на async/await переписывайте" - совсем забыл, нужно сделать.

Answer 2

[Роман Александрович]

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

if (res.data !== null) dispatch(onSubmitSucsses())
                else ()=>({ type: psActions.ON_SUBMIT_ERROR })

Answer 3

[Антон]

Как уже сказали, на 1 строке можно не писать return.
Ну, или когда эта 1 строка ничего не должна возвращать, то тогда тоже оборачивать в {}
И вообще, можно прогонять свой код через eslint с конфигом от airbnb, там уже все подобные случаи предусмотрели.

Comments

[Ninja Mate]

И вообще, можно прогонять свой код через eslint с конфигом от airbnb

за мной следят) я как раз этим занимаюсь