О
1. У тебя по коду строка dispatch сработает только в том случае если password = repeatPassword - это лишний аргумента для action
2. Никак - поэтому и уточнил
2. Никак - поэтому и уточнил
не лишний. там идет проверка на бэкенде тоже
Size: a a a
2020 October 19
DB
не лишний. там идет проверка на бэкенде тоже
тогда ок, но выглядит странно, обычно валидация repeatPassword чисто клиентская история (ну в моей практике)
D
Если это единичная форма, то ок. Если их парочка, есть смысл написать хелпер и описывать правила в отдельном объекте. Если их больше - то можно прибегнуть к готовым решениям, думаю.
EM
Взять форм-мененджер, react-final-form например 🤔
b
хочу без зависимостей
как вариант:
но я бы для валидации отдельное что-то использовал (тот же formik с yup)
let errorMessage = null;
let isValid = true;
// conditions
if (repeatPassword === '') { errorMessage = 'Enter password'; }
if (!errorMessage && repeatPassword !== password) { errorMessage = 'Passwords should be the same'; }
if (!errorMessage && ...) { errorMessage = '...'; }
// /conditions
if (!errorMessage) isValid = false;
if (!isValid) {
alert.show(errorMessage, {timeout: 5000, type: 'error'})
}
else {
dispatch(actionOnValidFormSubmitted(...));
}
но я бы для валидации отдельное что-то использовал (тот же formik с yup)
DB
как вариант:
но я бы для валидации отдельное что-то использовал (тот же formik с yup)
let errorMessage = null;
let isValid = true;
// conditions
if (repeatPassword === '') { errorMessage = 'Enter password'; }
if (!errorMessage && repeatPassword !== password) { errorMessage = 'Passwords should be the same'; }
if (!errorMessage && ...) { errorMessage = '...'; }
// /conditions
if (!errorMessage) isValid = false;
if (!isValid) {
alert.show(errorMessage, {timeout: 5000, type: 'error'})
}
else {
dispatch(actionOnValidFormSubmitted(...));
}
но я бы для валидации отдельное что-то использовал (тот же formik с yup)
как-то isValid выглядит лишним - нет?
b
да, isValid здесь не нужен )
b
let errorMessage = null;
// conditions
if (repeatPassword === '') { errorMessage = 'Enter password'; }
if (!errorMessage && repeatPassword !== password) { errorMessage = 'Passwords should be the same'; }
if (!errorMessage && ...) { errorMessage = '...'; }
// /conditions
if (!errorMessage) {
alert.show(errorMessage, {timeout: 5000, type: 'error'})
}
else {
dispatch(actionOnValidFormSubmitted(...));
}
О
let errorMessage = null;
// conditions
if (repeatPassword === '') { errorMessage = 'Enter password'; }
if (!errorMessage && repeatPassword !== password) { errorMessage = 'Passwords should be the same'; }
if (!errorMessage && ...) { errorMessage = '...'; }
// /conditions
if (!errorMessage) {
alert.show(errorMessage, {timeout: 5000, type: 'error'})
}
else {
dispatch(actionOnValidFormSubmitted(...));
}
спс)
b
urwelcome
b
p.s.
conditions можно ещё переписать на массив <функция валидации, сообщение>[], и errorMessage получать через reduce массива, чтобы избавиться от списка многих условий
conditions можно ещё переписать на массив <функция валидации, сообщение>[], и errorMessage получать через reduce массива, чтобы избавиться от списка многих условий
A
Dragon3DGraff DGN
Нашёл разные примеры в интернете и из них слепил. Это было тестовое задание на позицию джуна
понял, интересное ТЗ
DD
понял, интересное ТЗ
Пользуясь случаем благодарю за видео урок, он мне очень помог👍
A
Dragon3DGraff DGN
Пользуясь случаем благодарю за видео урок, он мне очень помог👍
у тебя круче получилось, кстати)
DD
у тебя круче получилось, кстати)
Работодатель разнёс его в пух и прах и на работу меня не взяли😁
EB
Тут замыкание работает ?
D
Замыкание чего?
D
this не отвалится потому что onPlanetLoaded это стрелка
EB
this не отвалится потому что onPlanetLoaded это стрелка
а если updatePlanet тоже стрелкой сделать, в чем отличие от текущей ситуации будет ? Я, видимо, плохо понимаю работу стрелочных функция, т.к. думал, что доступ из updatePlanet к onPlanetLoaded доступен при условии, если updatePlanet является стрелочной функцией
EB
this не отвалится потому что onPlanetLoaded это стрелка
Ну или через замыкание функции (если функция не может внутри себя найти на что ссылается, то выходит за свои границы наверх)
Сорри, если бред несу
Сорри, если бред несу