мы уже свой вопрос решаем ;)

Да я вижу:)

@itsjohncena , @XaveScor ссылку выше гляньте плиз, я не думаю что это плохой подход

@itsjohncena , @XaveScor ссылку выше гляньте плиз, я не думаю что это плохой подход

Да я чекнул, всё-таки лучше вынести эту логику в отдельный компонент/файл

Да я чекнул, всё-таки лучше вынести эту логику в отдельный компонент/файл

а причина? я про this прочел, но не понял

а причина? я про this прочел, но не понял

Ну если он разрастется и всё-таки надо будет вынести его, а у тебя данные не из пропсов а из this, придётся потратить время на переделку

Ну если он разрастется и всё-таки надо будет вынести его, а у тебя данные не из пропсов а из this, придётся потратить время на переделку

а, ну это ок. С другой стороны, преждевременная оптимизация получается. м?)

@itsjohncena , @XaveScor ссылку выше гляньте плиз, я не думаю что это плохой подход

Плохой, у тебя всё в куче. + всё время ререндерятся кнопки.

а, ну это ок. С другой стороны, преждевременная оптимизация получается. м?)

То, что делается на автомате и преждевременная оптимизация - это разные вещи. Не нужно оправдывать говнокод тем, что ты не занимаешься преждевременной оптимизацией.

Плохой, у тебя всё в куче. + всё время ререндерятся кнопки.

про кнопки - хорошо, про то что все в куче - не совсем согласен

То, что делается на автомате и преждевременная оптимизация - это разные вещи. Не нужно оправдывать говнокод тем, что ты не занимаешься преждевременной оптимизацией.

ученье-свет, в любом случае что-то полезное сейчас выясняется.

Как минимум isFetching нужно вынести в хок. Это вообще не должно быть частью компонентов. А из-за рендер методов ты не можешь нормально композицию сделать.

И это не единственные косяки. В твоём коде key - это индексы элементов массива. В коде абсолютно всё плохо.

Как минимум isFetching нужно вынести в хок. Это вообще не должно быть частью компонентов. А из-за рендер методов ты не можешь нормально композицию сделать.

ну смотри, кнопкам тогда придется передавать callback для клика по ним, это не лишнее усложнение? тут на месте все доступно, читаемость хорошая

И это не единственные косяки. В твоём коде key - это индексы элементов массива. В коде абсолютно всё плохо.

да хрен там) кей = индекс тут отлично

элементы не меняются местами, не городить же для них айдишники, фу

элементы не меняются местами, не городить же для них айдишники, фу

Для годов key может быть годами.

Для годов key может быть годами.

и индексом тоже может быть, вообще не будет никакой проблемы. Но про год - хорошо

коллеги доброе утро. подскажите как отпискатся от события onfocus?

коллеги доброе утро. подскажите как отпискатся от события onfocus?

Флаг в стейте.

И это не единственные косяки. В твоём коде key - это индексы элементов массива. В коде абсолютно всё плохо.

я согласен, что index для ключа не подходит, когда элементы не статичны

ну смотри, кнопкам тогда придется передавать callback для клика по ним, это не лишнее усложнение? тут на месте все доступно, читаемость хорошая

Не лишнее. Компонент должен знать и делать только то, что ему надо знать и делать.

я согласен, что index для ключа не подходит, когда элементы не статичны

В данном случае я бы год в ключ положил

Не лишнее. Компонент должен знать и делать только то, что ему надо знать и делать.

ок, тут в принципе аргументы хорошие.

а если Ванила window.onfocus or document.addEventListener('focus', func)

В данном случае я бы год в ключ положил

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

а если Ванила window.onfocus or document.addEventListener('focus', func)

remove event listener

а если Ванила window.onfocus or document.addEventListener('focus', func)

Ререндери по флагу в стейте нужен ли тебе onFocus или нет.

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

Можно делать нормально на автоматизме, а не ломать каждый раз голову почему человек тут эту хрень делает.

const year = +e.currentTarget.innerText

Еее

!!

Можно делать нормально на автоматизме, а не ломать каждый раз голову почему человек тут эту хрень делает.

да можно, если уровень автоматизма совпадает с тем, кто на этот код смотрит

да можно, если уровень автоматизма совпадает с тем, кто на этот код смотрит

Неплохое оправдание.

const year = +e.currentTarget.innerText

тут что не нравится? что аргумент не передается? могу из дата атрибута прочитать) или тоже есть хорошие аргументы против?

тут что не нравится? что аргумент не передается? могу из дата атрибута прочитать) или тоже есть хорошие аргументы против?

В том, что ты нормальный parseInt не используешь.

В том, что ты нормальный parseInt не используешь.

я знаю что в кнопке, зачем мне туда parseInt, или мы опять про автоматизм? тут в принципе, твоя правда.

А вообще определять кнопку по тексту в ней - это пиздец.

я знаю что в кнопке, зачем мне туда parseInt, или мы опять про автоматизм? тут в принципе, твоя правда.

Ты понимаешь, что ты увеличиваешь сложность кода? Тебе для работы всего приложения надо знать всё. Абсолютно всё.

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

Ты понимаешь, что ты увеличиваешь сложность кода? Тебе для работы всего приложения надо знать всё. Абсолютно всё.

согласен, все верно

согласен, все верно

И пока это 10 строчек - это не проблема. А когда это 100+ строк, то ты первый становишься на растрел.

не, я мысль понял

все в тему

И пока это 10 строчек - это не проблема. А когда это 100+ строк, то ты первый становишься на растрел.

как видишь, без ревью на автомате, написать не получилось, спасибо

а чем конкретно плохо нследование компонентов?

а чем конкретно плохо нследование компонентов?

В доке есть composition vs inheritance

при запросе к бд, просто считайте ее оттуда

точняк - просто fetch по умолчанию в заголовках куки не держит и вот я загнался

ребят вообще не получается сделать так что бы при нажатии на див чтото происходило

помогите пожалуста

в реакте дуб дубом.. разбираюсь конечно, читаю учусь. но времени категориески мало.

ребят вообще не получается сделать так что бы при нажатии на див чтото происходило

залей код на codepen или codesandbox

я могу на гит ссылку дать

на кодепен выносить из приложения слишком долго.

we haven’t found any use cases where we would recommend creating component inheritance hierarchies.

зашибись объяснили

Есть кто знает как происходит download pdf файла на реакте! Напишите в личку, пожалуйста!

Как и в обычном html? <a href="path to file" download>file</a>

Есть кто знает как происходит download pdf файла на реакте! Напишите в личку, пожалуйста!

как и везде

здравствуйте, никто не сталкивался с такой проблемой, есть реакт компонент, на вход подаются данные, возвращенный html добавляется к div с id определенным, этот див в отдельной вкладке, каждый раз при открытии этой вкладки, делается запрос за данными, и форма должна перезаписываться, но дело в том, что каждый раз создается новый инстанс реакта, и подвешивает свои обработчики, поэтому при 8-9 инстансах уже обычный клик пораждет 8-9 events

что пораждает задержку...

div элемент при уходе с вкладки, удаляется, как и весь вложенный html

но инстанс реакта все равно остается, вижу его в react dev tools

как и везде

У тебя самые дерьмовые ответы, прости пожалуйста за правду, ты можешь просто промолчать!

У тебя самые дерьмовые ответы, прости пожалуйста за правду, ты можешь просто промолчать!

:D

но инстанс реакта все равно остается, вижу его в react dev tools

ну видимо ты где то замыкаешь его

Есть кто знает как происходит download pdf файла на реакте! Напишите в личку, пожалуйста!

вы PDF хотите генерить на клиенте ? или у вас есть ссылка на файл, просто файл не скачивается?

ну видимо ты где то замыкаешь его

замыкаю? объясните пожалуйста...

ERROR: S client not available

У тебя самые дерьмовые ответы, прости пожалуйста за правду, ты можешь просто промолчать!

+

Есть кто знает как происходит download pdf файла на реакте! Напишите в личку, пожалуйста!

гугл выдал первой строкой ответ на SO - https://github.com/MrRio/jsPDF (генерация на клиенте). Пробовали?

У тебя самые дерьмовые ответы, прости пожалуйста за правду, ты можешь просто промолчать!

!ро хамство

замыкаю? объясните пожалуйста...

У каждой функции есть свой лексический контекст, в котором лежат все переменные, созданные в ней. Если ты обращаешься к переменной внутри функции, то она ищет ее в своем лексическом контексте. Если переменной там не найдено - то тогда она начинает искать ее в родительском лексическом контексте, то есть в другой функции, где твоя функция и была объявлена. Если и там нет, то идет в лексический контекст уровнем выше и так далее. Закончится это тем, что она либо где-то найдет нужную переменную, либо лексические контексты кончатся. Собсна как-то так и работает замыкание

У каждой функции есть свой лексический контекст, в котором лежат все переменные, созданные в ней. Если ты обращаешься к переменной внутри функции, то она ищет ее в своем лексическом контексте. Если переменной там не найдено - то тогда она начинает искать ее в родительском лексическом контексте, то есть в другой функции, где твоя функция и была объявлена. Если и там нет, то идет в лексический контекст уровнем выше и так далее. Закончится это тем, что она либо где-то найдет нужную переменную, либо лексические контексты кончатся. Собсна как-то так и работает замыкание

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

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

http://learn.javascript.ru/functions-closures

хм, тогда проблема возможно не в реакте, а в редаксе...

http://learn.javascript.ru/functions-closures

угу угу)

Что-то вы пишете странное....замыкания... редакс...

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

http://learn.javascript.ru/functions-closures

только я не понимаю, ведь допустим div, как на него подвешиваюся обработчики вторично

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

хм, тогда проблема возможно не в реакте, а в редаксе...

Можно код?

У меня на странице есть много больших изображений. Как то можно дождаться пока они все загрузятся и только потом отображать?

Можно код?

хм, а что вам нужно? сам компонент?

У меня на странице есть много больших изображений. Как то можно дождаться пока они все загрузятся и только потом отображать?

Погугли loading, кажется, в реакт апишке

Погугли loading, кажется, в реакт апишке

react-loader? вы про lazy загрузку?

а

сори)

У меня на странице есть много больших изображений. Как то можно дождаться пока они все загрузятся и только потом отображать?

Если не найдешь, крикни, через часик помогу найти

хм, а что вам нужно? сам компонент?

Где ошибка возникает?

react-loader? вы про lazy загрузку?

Да Не помню точное название модуля

Можно код?

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

а так, обычный реакт контейнер

Погугли loading, кажется, в реакт апишке

Код примерно такой import image from './image.jpg' <img src={image} />

Есть обертка

Которая видит сетевую активность своих детей

И позволяет в зависимости от нее что-то делать

Код примерно такой import image from './image.jpg' <img src={image} />

onload вроде ж есть

onload вроде ж есть

Они не сразу на странице появляются, а по очереди, а надо всех сразу дождаться, чтоб мерцаний не было

То есть у image src динамически меняеися

Они не сразу на странице появляются, а по очереди, а надо всех сразу дождаться, чтоб мерцаний не было

Прелоадер поставить можно?