
Oleg
10.01.2017
12:46:18
Сет интервал увы работает через интервал
Уже не бесконечный цикл
Точнее не так даже
Оное то бесконечно в идеале, но итерации по времени разнесенные

Google

Oleg
10.01.2017
12:47:00
Тут же - всё свободное время, ничего не простаивает за зря
Подсветка синтаксиса в редактора онлай так работает бывает
Чтобы не зависать раскрашивая тескст - раскрашивает кусочек, отдает управление, потом ещё кусочек и так пока весь текст не расскрасит
Вот пример из жизни и для фронтенда

Vitaliy
10.01.2017
13:14:52
Ребят, задам небольшой холиварный вопрос.
Где то читал и найти не могу.
Как правильней именовать метод?
onSomethingClick / onClickSomething

Михаил
10.01.2017
13:17:54
imho, если ты забыл метод, то в проекте легче найти onClickSomething, так как ты пройдешь по всем onClick и найдешь нужный

Oleg
10.01.2017
13:18:41
У всех онклик и прочих таких именований есть одна проблема
В целом конечно всё хорошо, но....
Имя метода говорит о том как он вызывается и почему
Но совершенно, вообще, никак не говорит о том а что он собственно то делает!

Oleg
10.01.2017
13:20:23
handleSomethingClick, а свойство onSomethingClick )

Oleg
10.01.2017
13:22:35
Моё либимое - doProcess

Vitaliy
10.01.2017
13:23:47

Google

Aleh
10.01.2017
13:24:42
а что может быть вместо Something?
почему не просто onClick?

Vitaliy
10.01.2017
13:25:26
и действия разные

Aleh
10.01.2017
13:25:36
у меня там обычно две строки, мапнуть что-то из ивента в структуру и прокинуть в слой с чем-то более умным

Vitaliy
10.01.2017
13:25:38
и не обязательно Click

Oleg
10.01.2017
13:25:39
Это был сарказм очевидно

Vitaliy
10.01.2017
13:25:46
Change / Blur

Oleg
10.01.2017
13:25:56
А суть вот в чем
Имя метода должно отражать то что в нем происходит
Оно может отражать что происходит также используя имя класса или области видимости
Если у нас класс-кнопка то очевидно что делает onClick
И то это уже не совсем то даже в таком виде
А вот что-то в стиле runPressAnimation уже интереснее
Сразу понятно - на клик у нас будет отрисована анимация нажатия

Alexander
10.01.2017
13:27:57
Я пулреквест сделал с фиксами eslint есть жлание смотреть? или сразу мержу?
https://bitbucket.sberned.ru/projects/PORT/repos/fpn_client/pull-requests/304/diff
Пинг

Oleg
10.01.2017
13:28:01
А метод onClick если был бы - ничего не понятно
Могут быть сложности с неймингом когда внутри 100500 разных действий
Но тогда можно давать общие названия

Google

Oleg
10.01.2017
13:28:54
И следить за тем чтобы в этом методе было только одно - вызов этих 100500 действий
И никакой бизнес-логики

Vitaliy
10.01.2017
13:28:59
ну когда проект большой, должна быть общая договоренность

Oleg
10.01.2017
13:29:04
Вот
Тем более
А в целом - рекомендую к прочтению книгу "Чистый код"
Там код весь на Java, но только как базовый код, суть одна - объяснить а как на самом деле можно было бы делать нейминг, располагать методы, рефакторить и ещё 100500 таких тем
Ну и здравый смысл конечно
Очевидно что runPressAnimation на порядок понятнее чем onClick

Oleg
10.01.2017
13:32:08
так тут нет нигде сильных аргументов кроме традиций. К примеру click в конце будет у всех у кого в бэкграунде Delphi

Vadim
10.01.2017
13:33:23
@kirenkov Иди читай умные книжки. В конце экшен... тебе уже и тут сказали ))

Oleg
10.01.2017
13:34:43
а если у тебя c# в бэкграунде, то будешь писать Somthing_Click

Konstantin
10.01.2017
13:35:03
так уже сто лет никто не пишет

Oleg
10.01.2017
13:35:47
А если ты девочка из 2007 то будет s0mThInG__cLiCk

Иван
10.01.2017
13:35:59
Еее

Oleg
10.01.2017
13:36:34
а если у тебя js, то пиши onclick )

Nikita
10.01.2017
14:29:21

Aleh
10.01.2017
14:34:01
onClick как обработчик клика в две строки самое норм, там будет что-то простое уровня doSmthWith(e.target.value)

Oleg
10.01.2017
14:40:05

Google

Oleg
10.01.2017
14:40:47
И не важно как оно записывается и где оно вызывается - важна суть
Почему бы не определять что будет делаться там где оно делается?

Aleh
10.01.2017
14:42:47
и вот надо мне на два разных ивента запускать эту анимацию, а структура у объектов ивента разная

Oleg
10.01.2017
14:43:26
А как это решит именование метода другими буквами?

Aleh
10.01.2017
14:43:36
тем что у меня есть два метода
onClick
который запускает runPressAnimation
преобразовав ивент в нужную структуру

Admin
ERROR: S client not available

Aleh
10.01.2017
14:44:22
и в onClick только логика обработки ивента клика

Oleg
10.01.2017
14:44:28
Почему бы не нормализовать эвент внутри runPressAnimation?

Aleh
10.01.2017
14:44:30
нет никакой логики чего-то еще, что-то еще отдельно
надо добавить ивент - надо менять метод
а хочется не менять работающий код, а добавлять новый

Oleg
10.01.2017
14:45:33
Не нужно менять если вынести нормализацию в отдельный метод
Я это по умолчанию имел в виду ?

Aleh
10.01.2017
14:46:25
у меня этот метод называется onClick(e: ClickEvent)
:)

Oleg
10.01.2017
14:46:50
Вот, а теперь попробуй догадайся что происходит внутри не читая исходники

Google

Aleh
10.01.2017
14:47:06
чьи исходники?

Oleg
10.01.2017
14:47:10
Метода

Aleh
10.01.2017
14:47:52
вот сейчас не уловил кейс, ты руками хочешь вызвать onClick?
зачем тебе знать, как работает onClick у компонента
в ситуации, когда ты не можешь открыть исходники
я понимаю про код, который ты хочешь вызвать
а это то, что вызывает внешняя инфраструктура, это по сути адаптер

Oleg
10.01.2017
14:49:02

Aleh
10.01.2017
14:49:36
ты его открыл и видишь все
ага, здесь анимация

Oleg
10.01.2017
14:49:48
Вот! В этом то и проблема
Чтобы понять - нужно код читать

Aleh
10.01.2017
14:50:00
в каком случае ты видишь метод компонента работы с внешним миром и не видишь его реализации?

Oleg
10.01.2017
14:50:07
Это как однобуквенные переменные
А что
Берешь и читаешь
Таааак... тут мы присвоили число
Значит q это число
Так, а что у нас в z....