Size: a a a

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

2020 August 19

DK

Daniil Kuznetsov in React — русскоговорящее сообщество
Andriy Dudariev
Пссс, хотите немного плохого кода, который нужно отрефакторить?
😂
источник

AD

Andriy Dudariev in React — русскоговорящее сообщество
{
               ( props.data.transitions.some( tr => tr.button_id == '0' )
               ?
               props.data.transitions
               :
               [{ button_id:'0' }, ...props.data.transitions ] )
               .filter( tr =>
                   ( props.drawFrom && props.drawedData.messageId == props.data.id && props.drawedData.buttonId == tr.button_id && props.drawTo ))
               .map( tr => {
                   const fromX = props.coords.x + 270;
                   const fromY = tr.button_id === 0 ? props.coords.y + 17 : props.coords.y + 20 + (props.data.keyboard.findIndex( k => k[ 0 ].id == tr.button_id ) + 1) * 30 ;
                   return (
                       <div
                           style={{
                               position:'absolute',
                               left: fromX + 'px',
                               top: ( fromY - 11 ) + 'px',
                               zIndex:'6'
                           }}
                           className="toEndBlock"
                           onMouseUp={ e => props.mouseUp( 'end' , e )}
                       >
                           Завершити діалог
                       </div>
                   )
               })
           }

Я даже так сделаю))
источник

A

A A in React — русскоговорящее сообщество
Andriy Dudariev
{
               ( props.data.transitions.some( tr => tr.button_id == '0' )
               ?
               props.data.transitions
               :
               [{ button_id:'0' }, ...props.data.transitions ] )
               .filter( tr =>
                   ( props.drawFrom && props.drawedData.messageId == props.data.id && props.drawedData.buttonId == tr.button_id && props.drawTo ))
               .map( tr => {
                   const fromX = props.coords.x + 270;
                   const fromY = tr.button_id === 0 ? props.coords.y + 17 : props.coords.y + 20 + (props.data.keyboard.findIndex( k => k[ 0 ].id == tr.button_id ) + 1) * 30 ;
                   return (
                       <div
                           style={{
                               position:'absolute',
                               left: fromX + 'px',
                               top: ( fromY - 11 ) + 'px',
                               zIndex:'6'
                           }}
                           className="toEndBlock"
                           onMouseUp={ e => props.mouseUp( 'end' , e )}
                       >
                           Завершити діалог
                       </div>
                   )
               })
           }

Я даже так сделаю))
Читабельно вроде)
источник

🧨

🧨 ethorz in React — русскоговорящее сообщество
какой-то пиздец
источник

HH

Hbaf Hbaf in React — русскоговорящее сообщество
А что плохого в этом коде?
За исключением магических чисел и немного странного форматирования?
источник

V

VD in React — русскоговорящее сообщество
Привет подскажите что не так
источник

A

A A in React — русскоговорящее сообщество
VD
Привет подскажите что не так
в map( ()=>{
return надо тут
})
источник

AD

Andriy Dudariev in React — русскоговорящее сообщество
Hbaf Hbaf
А что плохого в этом коде?
За исключением магических чисел и немного странного форматирования?
лично мне тернарка, по которой идет фильтер и мап не нравится
источник

🧨

🧨 ethorz in React — русскоговорящее сообщество
Hbaf Hbaf
А что плохого в этом коде?
За исключением магических чисел и немного странного форматирования?
если тебе дадут пофиксить баг в этом куске кода, ты пофиксишь?
источник

HH

Hbaf Hbaf in React — русскоговорящее сообщество
VD
Привет подскажите что не так
Функция в мапе ничего не возвращает
источник

V

VD in React — русскоговорящее сообщество
A A
в map( ()=>{
return надо тут
})
ага протупил спс
источник

V

VD in React — русскоговорящее сообщество
Hbaf Hbaf
Функция в мапе ничего не возвращает
спс
источник

HH

Hbaf Hbaf in React — русскоговорящее сообщество
🧨 ethorz
если тебе дадут пофиксить баг в этом куске кода, ты пофиксишь?
Скорее всего да, он не оч структурированный не более
источник

🧨

🧨 ethorz in React — русскоговорящее сообщество
Hbaf Hbaf
Скорее всего да, он не оч структурированный не более
окей,  props.coords.x + 270, понятно почему 270? props.data.keyboard.findIndex( k => k[ 0 ].id == tr.button_id ) + 1 - а это?)
источник

AD

Andriy Dudariev in React — русскоговорящее сообщество
Hbaf Hbaf
Скорее всего да, он не оч структурированный не более
у меня еще есть 2 куска похожего))
и там похлеще))
источник

HH

Hbaf Hbaf in React — русскоговорящее сообщество
🧨 ethorz
окей,  props.coords.x + 270, понятно почему 270? props.data.keyboard.findIndex( k => k[ 0 ].id == tr.button_id ) + 1 - а это?)
Ну я и говорю, что магические числа и форматирование
источник

AD

Andriy Dudariev in React — русскоговорящее сообщество
🧨 ethorz
окей,  props.coords.x + 270, понятно почему 270? props.data.keyboard.findIndex( k => k[ 0 ].id == tr.button_id ) + 1 - а это?)
за цифры лучше забей, там к верстке привязано))
источник

🧨

🧨 ethorz in React — русскоговорящее сообщество
нужно чистый код читать, потому что это дичь
источник

BB

Bugs Bunny in React — русскоговорящее сообщество
const MAGIC_LAYOUT_OFFSET = 270
источник

A

A A in React — русскоговорящее сообщество
+
источник