В общем для общественного спокойствия тема закрывается

В общем для общественного спокойствия тема закрывается

спасибо

:с

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

<ItemsArray /> -> {ItemsArray}

если уж так писать, то <this.ItemsArray />

если уж так писать, то <this.ItemsArray />

бля

если уж так писать, то <this.ItemsArray />

Адище.

если уж так писать, то <this.ItemsArray />

компоненты с большой буквы ес че)

бля

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

Это будет не слишком абстрактно, ведь я этот список буду использовать только в этом компоненте?

компоненты с большой буквы ес че)

а ты проверь, ес чо)

спойлер: работает

поищи в интернете бесплатные курсы по реакт там дадут неплохую вводную

компоненты с большой буквы ес че)

если есть точка, тоже прокатить должно (а точнее expression)

Это будет не слишком абстрактно, ведь я этот список буду использовать только в этом компоненте?

Так не нужно его шарить, вынеси по месту, локально.

но лучше, наверное, всё-таки {this.renderItemsArray()} какой-нибудь, если не хочется в отдельный компонент выносить

Это будет не слишком абстрактно, ведь я этот список буду использовать только в этом компоненте?

это ты styled-components ещё не видел)

но лучше, наверное, всё-таки {this.renderItemsArray()} какой-нибудь, если не хочется в отдельный компонент выносить

Расскажи, зачем так делать?

Пропсы лень кидать?

но лучше, наверное, всё-таки {this.renderItemsArray()} какой-нибудь, если не хочется в отдельный компонент выносить

да. я что-то обкурился. и ul туда унести

а вообще map то можно и внутри render сделать

Расскажи, зачем так делать?

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

Всем спасибо за ответы, еще поищу стайл гайды какие-то

по-моему, на эту тему стайлгайдов нету

документацию просто посмотри и примеры там

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

Так ведь это... Реакт он же как раз про то, чтоб разделять всё на компоненты :)

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

поэтому надо делать системную свалку компонентов)

@AtomicDesign и вот это всё

Как ты это говно тестировать будешь, если вдруг понадобится?

Так ведь это... Реакт он же как раз про то, чтоб разделять всё на компоненты :)

Не следует множить сущности без необходимости (с) Дэн Абрамов

Не следует множить сущности без необходимости (с) Дэн Абрамов

То же мне авторитет кек :)) Необходимость кто определяет?

Так ведь это... Реакт он же как раз про то, чтоб разделять всё на компоненты :)

это, конечно, круто, но не когда это “серый прямоугольник поверх столбца” так у меня и столбцы рисуются такими рендер-функциями, и прямоугольники поверх них а без родителя-контейнера это вообще не компоненты

То же мне авторитет кек :)) Необходимость кто определяет?

Денчик боженька, извинись

То же мне авторитет кек :)) Необходимость кто определяет?

А кто для тебя авторитет?

А кто для тебя авторитет?

Фаулер. Мартин чутка мб.

То же мне авторитет кек :)) Необходимость кто определяет?

пишущий, конечно имхо, если не можешь показать компонент в сторибуке и он что-то сам рендерит, то этот компонент не стоит того, чтобы быть венесеным в отдельный файл а несколько компонентов держать в одном файле — такое)

(особенно когда тебе эти компоненты надо оборачивать каким-нибудь `withStyle`)

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

Фаулер. Мартин чутка мб.

Не следует множить сущности без необходимости (с) Мартин Фаулер

Не следует множить сущности без необходимости (с) Мартин Фаулер

?

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

если это просто стейтлесс-компонент, то сойдет а если уже со всякими обертками, то пизда

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

+

Не следует множить сущности без необходимости (с) Мартин Фаулер

Это ваще кажется сказал чувак, который жил чуть пораньше :))

если это просто стейтлесс-компонент, то сойдет а если уже со всякими обертками, то пизда

я у себя разделил это по типу компонентов

это, конечно, круто, но не когда это “серый прямоугольник поверх столбца” так у меня и столбцы рисуются такими рендер-функциями, и прямоугольники поверх них а без родителя-контейнера это вообще не компоненты

Не-не, рендер-функции это еще куда ни шло. Но рендер-методы — кака.

Но, если ты все равно выносишь функцию, почему бы легким движением руки не сделать из этой функции SFC?

Не-не, рендер-функции это еще куда ни шло. Но рендер-методы — кака.

если честно, не вижу с ними проблемы не плодят компоненты на ровном месте (потом дерево в девтулзах получается читабельнее) не приходится переживать из-за ререндера не надо пробрасывать стейт и пропсы да, не очень красиво, но, по-моему, сойдёт

Нормально делай нормально будет ©️альберт эйнштейн

Но, если ты все равно выносишь функцию, почему бы легким движением руки не сделать из этой функции SFC?

увеличение количества кода на ровном месте

ERROR: S client not available

увеличение количества кода на ровном месте

Кода не должно быть мало, код должен быть понятен и предсказуем :)

Кода не должно быть мало, код должен быть понятен и предсказуем :)

Имхо, создание компонентов, придумывание им длиннющих имён, прокидывание им кучи пропсов и оборачивание их во всякие pure понятности не придаёт)

Но мб я неправ

если честно, не вижу с ними проблемы не плодят компоненты на ровном месте (потом дерево в девтулзах получается читабельнее) не приходится переживать из-за ререндера не надо пробрасывать стейт и пропсы да, не очень красиво, но, по-моему, сойдёт

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

файл размером 8КБ

Не-не, рендер-функции это еще куда ни шло. Но рендер-методы — кака.

Поддерживаю Не используйте методы `render*`, возвращающие JSX - такой код сложнее дебажить, потому что параметры и контекст компонента размазываются и их связи становятся менее явными. Так же в React-девтулзах намного сложнее отследить откуда взялся JSX . Лучше выносите это в отдельные компоненты. При этом, скорее всего, какие-то props вам не придется перепрокидывать, а можно будет подключить отдельно только к необходимым микро-компонентам - это уменьшает связанность, а значит сложность кода.

Поддерживаю Не используйте методы `render*`, возвращающие JSX - такой код сложнее дебажить, потому что параметры и контекст компонента размазываются и их связи становятся менее явными. Так же в React-девтулзах намного сложнее отследить откуда взялся JSX . Лучше выносите это в отдельные компоненты. При этом, скорее всего, какие-то props вам не придется перепрокидывать, а можно будет подключить отдельно только к необходимым микро-компонентам - это уменьшает связанность, а значит сложность кода.

напишу об этом в канале

ради б-га, юзайте https://marketplace.visualstudio.com/items?itemName=streetsidesoftware.code-spell-checker хотя бы

напишу об этом в канале

?

Кстати, например, надо отрендерить кусок html, который больше нигде не переиспользуется. Нет же смысла разбивать этот хтмл по компонентам и функциям (именно html)

На 400 строк html, к примеру

может единоразово и можно

Кстати, например, надо отрендерить кусок html, который больше нигде не переиспользуется. Нет же смысла разбивать этот хтмл по компонентам и функциям (именно html)

вынести в файл, а в компоненте импортнуть как строчку и сделать dangerouslySetInnerHtml

чтобы намекнуть будущим поколениям что здесь 400 строк уныния

вынести в файл, а в компоненте импортнуть как строчку и сделать dangerouslySetInnerHtml

к р а с и в о

к р а с и в о

н е в ы е б

вынести в файл, а в компоненте импортнуть как строчку и сделать dangerouslySetInnerHtml

вот так больше нравится, мне да

вынести в файл, а в компоненте импортнуть как строчку и сделать dangerouslySetInnerHtml

А если среди них надо ещё местами поставлять онклики которые через пропсы передаются и вывести информацию из пропсы?

Например, страница товара: очень много верстки и несколько интерактивных элементов

вынести в файл, а в компоненте импортнуть как строчку и сделать dangerouslySetInnerHtml

no please