Size: a a a

React: русскоязычное сообщество

2019 December 05

A

Aleksandr in React: русскоязычное сообщество
Gleb
ругаясь в консоли и просит задать стартвое значение
Он не на стейт ругается а на Инпут
источник

A

ALDDDDOVI in React: русскоязычное сообщество
ребята хелп  ,  вообщем этот компонент рендерится с помощью map, я делаю чек на класс ,   но почему то один элемент только получает активный классс , хотя все элементы проходят иттерацию , может в компоненте выше в пропсы передавать класс ?
источник

P

Pavel in React: русскоязычное сообщество
ALDDDDOVI
Гайзы кхмм , вообщем есть такой метод , которая возвращает класс  элемента , return className . У меня есть условие , которые проверяет если value === activeValue допустим. Окей , он возвращает всегда только один active  класс , хотя активных дней  > 1
используй в коде методы по назначению
find - это поиск по массиву
источник

A

ALDDDDOVI in React: русскоязычное сообщество
Pavel
используй в коде методы по назначению
find - это поиск по массиву
ну я ищу потому что в массиве . Логично не ? скорее всего стейт мутирую
источник

A

ALDDDDOVI in React: русскоязычное сообщество
короче
источник

AK

Andrey Kuznetsov in React: русскоязычное сообщество
Я бы руки ноги оторвал за жесть такую
источник

A

ALDDDDOVI in React: русскоязычное сообщество
Andrey Kuznetsov
Я бы руки ноги оторвал за жесть такую
омг , это просто набросок , рефакторинг чуть позже
источник

P

Pavel in React: русскоязычное сообщество
ALDDDDOVI
омг , это просто набросок , рефакторинг чуть позже
У тебя ппц в каждой строчке скрина. 🙈
источник

G

Gleb in React: русскоязычное сообщество
ALDDDDOVI
ребята хелп  ,  вообщем этот компонент рендерится с помощью map, я делаю чек на класс ,   но почему то один элемент только получает активный классс , хотя все элементы проходят иттерацию , может в компоненте выше в пропсы передавать класс ?
воще да, идеальная иллюстрация того как не стоит использовать find
источник

A

ALDDDDOVI in React: русскоязычное сообщество
Pavel
У тебя ппц в каждой строчке скрина. 🙈
аргумент павел
источник

G

Gleb in React: русскоязычное сообщество
ALDDDDOVI
аргумент павел
а правда, почему find? а не forEach?
find же тут будет сильно медленней
источник

G

Gleb in React: русскоязычное сообщество
так как ничего не ищется вроде
источник

AP

Aleksandr Panchuk in React: русскоязычное сообщество
Зачем вообще второй цикл
источник

P

Pavel in React: русскоязычное сообщество
ALDDDDOVI
аргумент павел
setClass- метод ничего не устанавливает, Class - зарезервировано JS-oм - стоит избегать таких названий.
getClasses или как-то так.
Используй деструктуризацию. Метод у тебя ничего не принимает.
Это по первой строчке.
Дальше надо?)
источник

A

ALDDDDOVI in React: русскоязычное сообщество
Pavel
setClass- метод ничего не устанавливает, Class - зарезервировано JS-oм - стоит избегать таких названий.
getClasses или как-то так.
Используй деструктуризацию. Метод у тебя ничего не принимает.
Это по первой строчке.
Дальше надо?)
давай
источник

A

ALDDDDOVI in React: русскоязычное сообщество
Pavel
setClass- метод ничего не устанавливает, Class - зарезервировано JS-oм - стоит избегать таких названий.
getClasses или как-то так.
Используй деструктуризацию. Метод у тебя ничего не принимает.
Это по первой строчке.
Дальше надо?)
сделай меня лучше
источник

AK

Andrey Kuznetsov in React: русскоязычное сообщество
Это код ревью.. :) но там и логика битая..
источник

P

Pavel in React: русскоязычное сообщество
ALDDDDOVI
сделай меня лучше
источник

P

Pavel in React: русскоязычное сообщество
+- примерно так
источник

P

Pavel in React: русскоязычное сообщество
хотя reduce тут тоже не по теме ))
источник