Телеграмм чат группы react

в прямом, так можно нагородить кучу всего в рендере, вместо того чтобы выносить в отдельные переменные. Из той же области тогда по-твоему

return (
   <div>
     {
        isLoggedIn && isSubscribed && isModalVisibilityFeatureEnabled && gender === 'male' ? (
            <Foo />
         ) : (
            <Bar />
         )
      } 
   </div>
)

лучше чем
co

nst isSomeVariable = (
   isLoggedIn 
   && isSubscribed 
   && isModalVisibilityFeatureEnabled 
   && gender === 'male'
)

return (
   <div>
     {
         isSomeVariable ? <Foo /> : <Bar />      
     } 
   </div>
)

пушить глаза придется 🙂

13:35пожаловаться #1

V

в прямом, так можно нагородить кучу всего в рендере, вместо того чтобы выносить в отдельные переменные. Из той же области тогда по-твоему

return (
   <div>
     {
        isLoggedIn && isSubscribed && isModalVisibilityFeatureEnabled && gender === 'male' ? (
            <Foo />
         ) : (
            <Bar />
         )
      } 
   </div>
)

лучше чем
co

nst isSomeVariable = (
   isLoggedIn 
   && isSubscribed 
   && isModalVisibilityFeatureEnabled 
   && gender === 'male'
)

return (
   <div>
     {
         isSomeVariable ? <Foo /> : <Bar />      
     } 
   </div>
)

пушить глаза придется 🙂

Так ты привёл другой аргумент сейчас, ты вынес только условие а не сам рендер условный полностью)

13:37пожаловаться #2

PI

холивар да и толко, дело вкуса, но все же.

13:37пожаловаться #3

BB

в прямом, так можно нагородить кучу всего в рендере, вместо того чтобы выносить в отдельные переменные. Из той же области тогда по-твоему

return (
   <div>
     {
        isLoggedIn && isSubscribed && isModalVisibilityFeatureEnabled && gender === 'male' ? (
            <Foo />
         ) : (
            <Bar />
         )
      } 
   </div>
)

лучше чем
co

nst isSomeVariable = (
   isLoggedIn 
   && isSubscribed 
   && isModalVisibilityFeatureEnabled 
   && gender === 'male'
)

return (
   <div>
     {
         isSomeVariable ? <Foo /> : <Bar />      
     } 
   </div>
)

пушить глаза придется 🙂

это ты сам подсчет условия убрал выше. кондишен у тебя по прежнему в рендере

13:37пожаловаться #4

V

^ this

13:37пожаловаться #5

PI

Vetro

Так ты привёл другой аргумент сейчас, ты вынес только условие а не сам рендер условный полностью)

а в чем разница? "пушо глазами вверх-вниз" появляется? да / нет 🙂

13:37пожаловаться #6

V

Одно дело когда ты видишь сразу отрисовку условную и по контексту названия переменной в которой собрал условия можешь додумать

13:38пожаловаться #7

BB

а в чем разница? "пушо глазами вверх-вниз" появляется? да / нет 🙂

ну я вижу, что тут не какая-то лунная переменная с готовым JSX, которую еще надо найти, а именно условие

13:38пожаловаться #8

V

А другое когда ты вообще не знаешь что в итоге отрисуется

13:38пожаловаться #9

BB

+

13:38пожаловаться #10

PI

понятно, ребят 🙂 расходимся

Pavel in React — русскоговорящее сообщество

13:38пожаловаться #11

P

Bugs Bunny

пушо глазами вверх-вниз скакать не надо потом

у меня приложение будет огромное, там контента будет более 50 компонентов и еще есть важный нюанс, нужно что бы это драверы наслаивались один на другой. то есть открывая дравер А ты мог открыть дравер B и так далее

Александр Бакиматов... in React — русскоговорящее сообщество

13:38пожаловаться #12

АБ

ах, срачик за условный рендеринг😊🌚

13:38пожаловаться #13

V

понятно, ребят 🙂 расходимся

Тема в любом случае бесспорно холиварная и каждый останется на своём)

13:39пожаловаться #14

PI

Pavel

у меня приложение будет огромное, там контента будет более 50 компонентов и еще есть важный нюанс, нужно что бы это драверы наслаивались один на другой. то есть открывая дравер А ты мог открыть дравер B и так далее

так разноси логику по мелким компонентам

13:39пожаловаться #15

BB

Pavel

у меня приложение будет огромное, там контента будет более 50 компонентов и еще есть важный нюанс, нужно что бы это драверы наслаивались один на другой. то есть открывая дравер А ты мог открыть дравер B и так далее

ну вот если ты destroyOnClose воткнешь, дети Drawer-а у тебя не будут маунтиться до вызова

13:39пожаловаться #16

BB

а наслоение это хз. не сталкивался с таким

Шепс in React — русскоговорящее сообщество

13:39пожаловаться #17

Ш

Ребят, а какой слушатель лучше использовать и максимально правильно и лаконично решить такую задачку - есть квадрат, мне надо менять в нем цвет при двух условиях - зажатая ЛКМ и нахождение курсора внутри

Александр Бакиматов... in React — русскоговорящее сообщество

13:41пожаловаться #18

АБ

там не наслоение нужно, а: текущий дровер закрылся - открылся новый дровер, закрылся новый дровер - открылся третий. причем что сам дровер один замаунчен. нужно просто сделать обвязку для разруливания контента и флага видимости для того иили иного стейта

PARA ZITE in React — русскоговорящее сообщество

13:41пожаловаться #19

PZ

@neosheps Я бы при клике добавил eventlistener