bucket list, prod by m.v.novikov88#1
Conversation
EvgenyOrekhov
left a comment
EvgenyOrekhov
left a comment
There was a problem hiding this comment.
Пояснения по оригинальным требованиям к тестовому заданию.
Use React with Hooks
Подразумевает, что все компоненты должны быть реализованы на Hooks, а не на классах.
Use styled-components
Подразумевает, что вся стилизация должна быть произведена с помощью styled-components, а не с помощью отдельных файлов со стилями.
| const validation = {}; | ||
|
|
||
| this.validations.map(rule => ( | ||
| validation[rule["field"]] = { isInvalid: false, message: '' } |
There was a problem hiding this comment.
Возможно было бы лучше использовать положительное слово, isValid, например. Так как дальше по коду производится проверка !isInvalid, а отрицание отрицания уже становится тяжеловато читать.
|
|
||
| function handleSortActivities(type) { | ||
| const sortedActivityArr = activitiesArr.sort((a, b) => { | ||
| if (a[type] < b[type]) return -1; |
There was a problem hiding this comment.
Фигурные скобки лучше никогда не опускать, даже если инструкция всего одна. Ужасные баги можно бы было предотвратить следуя этому правилу (см. "goto fail bug" от Apple в их библиотеке SSL).
| import { setMenuState } from "../../../actions/app"; | ||
|
|
||
| const mapState = (state) => ({ | ||
| _app: state.app |
There was a problem hiding this comment.
Слишком много идентификаторов с префиксом _ появилось. Лучше давать более осмысленные имена.
There was a problem hiding this comment.
да это есть, префикс я начал давать для actions(и для других некоторых переменных например как toast) т.к. функция(имя переменной) одна и та же(чтоб не было конфликтов имен). Тут да - надо что то придумать и придерживаться этого правила.
2 participants
No description provided.