
Tim
28.08.2018
17:48:41

Darth
28.08.2018
17:48:58

Tim
28.08.2018
17:49:00
Заодно другие поревьювят как я ревьювю
Вот так надо, да

Google

Tim
28.08.2018
17:49:55
Ток зачем @radius после объявления класса?
И я чето в последнее время стал чересчур большим любителем кэширования значения функции
Метода

Darth
28.08.2018
17:51:01

Crypt
28.08.2018
17:51:13

Tim
28.08.2018
17:51:24
Типа
def perimeter
@perimeter ||= Math::PI * @radius
end
Не нужен

Crypt
28.08.2018
17:51:47
мы ведь можем узнать радиус круга, Егор??? :D :D :D запретный геттер

Tim
28.08.2018
17:53:05
Ну типа запрет геттеров имеет смысл в том что когда ты юзаешь геттеры, то ты определяешь поведение и логику связанную с классом не в классе а где-то там где используешь
И это нарушение инкапсуляции
И типа зона ответственности у классов становится жидкой

Crypt
28.08.2018
17:53:43
ну это понятно, но он так обобщил, что прям любой гет/сет - уже выбивается из нормы

Tim
28.08.2018
17:53:59
Ну он радикальный оопист

Google

Tim
28.08.2018
17:54:26
Он вообщем специально утрирует и радикализирует

Crypt
28.08.2018
17:54:29
а почему ридер не обязательно указывать?

Zamira
28.08.2018
17:54:33
Slack у кого нибудь работает сейчас?

Tim
28.08.2018
17:54:42
Чтобы идеи разлетались сильнее (и потенциально вызывали бугурт)
Если будет совсем прям надо (но скорее всего нет), то потом укажешь

rekero
28.08.2018
17:55:22

Crypt
28.08.2018
17:55:23

Tim
28.08.2018
17:55:25
В задаче подсчета периметра он не нужен

Crypt
28.08.2018
17:55:46
нууууу, так то и класс не нужен)

Darth
28.08.2018
17:55:53

Tim
28.08.2018
17:56:15
Имхо не надо так делать
Конструктор только для ассайна
Значений

?
28.08.2018
17:57:17
Тема на грувбокс похожа, но немного другая. Какая?

Crypt
28.08.2018
17:58:02

Darth
28.08.2018
17:59:39

?
28.08.2018
18:00:15

Google

?
28.08.2018
18:00:19
У меня послабее
Вот и не узнал

Darth
28.08.2018
18:00:58
У меня что дома что на работе экраны не очень и темой спасаюсь

Tim
28.08.2018
18:04:37

Crypt
28.08.2018
18:04:38
Почему нет?
походу есть шанс, что для не для всех объектов будет периметр вычисляться. Поэтому не тратим время на это в конструкторе

Darth
28.08.2018
18:06:53
Ну ок

Tim
28.08.2018
18:29:24
Ну и вообще нету смысла делать конструктор пухлым
И кастомизированности мешает и тд

Crypt
28.08.2018
18:31:00
тут еще attr_reader :perimeter вроде как лишний, есть же метод уже

Darth
28.08.2018
18:31:58
С ридером не будет повторных вычислений

Crypt
28.08.2018
18:33:09

Darth
28.08.2018
18:34:38
А теперь согласен )

Dmitry
28.08.2018
18:47:57

Максим
28.08.2018
19:26:28
Сходил на митап было оч круто
Кто не пошел можете жалеть что не пошли

Tim
28.08.2018
19:27:24
Че за митинг

Google

Tim
28.08.2018
19:27:27
Аа

Максим
28.08.2018
19:27:44
Сорян да
Оч много оч умных людей

Admin
ERROR: S client not available

Максим
28.08.2018
19:28:00
Руби умирает короче

Tim
28.08.2018
19:28:06
Ну буду в Москве, тоже буду на всякие митапчики ходить
Модные

Максим
28.08.2018
19:28:15
Прям вдохновился ща домой приду буду тоже убивать руби
А ещё с зп сразу куплю билет на руби раша

Nikolay
28.08.2018
19:31:45
300 билетов на RubyRussia уже проданы, осталось 200; торопитесь! (с)

Максим
28.08.2018
19:32:32
Ебать!

Crypt
28.08.2018
19:33:53

Максим
28.08.2018
19:34:17
Очень долго
С таким количеством людей на митапе умирать так ему ещё очень долго и очень мучительно

Zamira
28.08.2018
19:41:43
Руби не умирает, от него всегда плохо пахнет)
В смерть рельсов поверю, но не в смерть руби

Максим
28.08.2018
19:42:27
Ну яж с сарказмом ребят ну...

Zamira
28.08.2018
19:42:40
А я тоже)

Максим
28.08.2018
19:42:43
Ну типа много умных интересных людей говорит о том что язык живее всех ..

Zamira
28.08.2018
19:44:47
Эх... когда же у меня будет возможность постоянно летать на митапы

Alex
28.08.2018
19:47:50

Google

Alex
28.08.2018
19:47:56
удаленка + нормальный оклад и в бой )
но не постоянно конечно

Zamira
28.08.2018
19:48:18
Каждый месяц летать все равно не получится
А тут смотрю в Питере пару раз за последнее время что-то мутили
Тут сейчас про москву говорят
Кто-то про конфу в Вене писал сегодня
Кроме денег, на это времени не хватит)

Dmitry
28.08.2018
22:59:16

kolas
29.08.2018
04:05:32
опять скилловые рубистки понабежали ?

Alexander
29.08.2018
04:24:10

Felix
29.08.2018
04:31:20

Zamira
29.08.2018
05:41:25
Я его никогда не читала как "раби". Боже упаси