
Максим
09.08.2018
10:26:44
типа нужен тебе свой мини ransack взял написал консёрн генерирующий методы поиска и сортировки в зависимости от имён колонок или связей сущности заинклюдил в нужные сущности и пользуйся
ден я бы посоветовал тебе в первую очередь не недооценивать его, он вполне реально может знает о чём говорить, а тебе это поможет в итоге не упасть в грязь лицом
не у меня там проще было
у меня был в апликейшн контроллере ещё метод

Google

Максим
09.08.2018
10:28:01
который принимал запросы определённого вида
и слал определённый сигнал в модель
т.е. всё было полностью автоматизировано
и кода там было строк на 50
вроде три метода
один на генерацию сортировок
другой на генерацию поиска по своим полям
и ещё на генерацию по связным полям
вроде так
не помню уже

Crypt
09.08.2018
10:29:25
Вот генереннные методы так себе, их rdoc не подхватывает

Максим
09.08.2018
10:29:26
и в апликейшн контроллере 1 метод
ну мне тогда впринципе было интересно просто попробовать метапрогарммирование
я понимаю проблемы котоыре оно с собой привносит

Tim
09.08.2018
10:29:56
я про математику

Google

Максим
09.08.2018
10:29:58
но там всё вроде хорошо вышло

Tim
09.08.2018
10:30:10
ну собственно по этому и возвращает NaN

Dan
09.08.2018
10:31:08
Я думал он это все из мира рейлс тянет, типа писать логику в моделях, а теперь я вижу что это не так


Tim
09.08.2018
10:33:59
Ребят, такая ситуация. Начал писать на рейлс, так как на проекте на нем api. И мне сразу не понравилось огромное количество скрытой магии, которая есть в рейлс приложении. Магия собственно появляется из-за миксинов и наследования, ну вы сами знаете. И тут я узнал, что есть такая штука как сервис обжекты. И это как раз то, что мне не хватало. Я сам js разработчик и пишу на ember и там тоже есть сервисы. То есть во первых это для меня интуитивно, а во вторых это реально делает код более явным (не прям до конца как хотелось бы, но все же). Так вот я написал новую фичу, вынеся код из контроллеров в сервис и спросил ребят из команды как им это нравится и почему они не делали этого раньше. Парень, который написал большую часть кода приложения оказался против этого по следующим соображениям. Во-первых он считает что это нарушает текущую консистентность, а во-вторых это против рейлс-вэй, потому что все должно быть магией, хотя ему тоже это не нравится. Насколько я понимаю, сервисами в рейлс сообщество пользуется давно, но для него это было что-то новое. В общем я считаю, что он просто очень консервативный и аргументы его меня не убеждают. Как вы считаете, кто прав? Сорри за лонг рид
походу он просто не шарит


Crypt
09.08.2018
10:34:02

Максим
09.08.2018
10:34:04

Tim
09.08.2018
10:34:08
нан не ошибка

Максим
09.08.2018
10:34:09
тут за такое сразу убивают

Dan
09.08.2018
10:34:59
Видимо тяжело мне придётся. Он ещё упёртый как баран(

Tim
09.08.2018
10:35:07

Dan
09.08.2018
10:35:23
Нужно потерпеть до конца года, когда он уйдёт)

Tim
09.08.2018
10:35:43
ну это понятно

Ilya
09.08.2018
10:35:45

Tim
09.08.2018
10:36:06
ну короче создаётся ощущение что чел вообще не шарит

Максим
09.08.2018
10:36:13
+
но бля
чё он 7 лет то делал

Dan
09.08.2018
10:36:24
Проблема в том, что он сейчас учит двух джунов как писать код. И мне придётся бороться с новым поколением любителей логики в моделях

Google

Tim
09.08.2018
10:36:25
хотя синьор помидор 7 лет опыта

Ilya
09.08.2018
10:36:37

Dan
09.08.2018
10:36:38
Да, это странно

Tim
09.08.2018
10:36:51
ну может он побочные технологии знает норм?
там всякую эластику, бдшки, авс и т.д.

Максим
09.08.2018
10:37:28

Dan
09.08.2018
10:37:44

Максим
09.08.2018
10:38:06

Tim
09.08.2018
10:38:10

Максим
09.08.2018
10:38:16
типа тебе после него во всём этом разбираться

ShadoWalkeR
09.08.2018
10:38:23

Максим
09.08.2018
10:38:25
придётся на каждый чих его нанимать

Dan
09.08.2018
10:38:26
динозавр?
Да, консервативный и упёртый динозавр)

Tim
09.08.2018
10:38:32
я не говорил

Максим
09.08.2018
10:38:38
типа ща он написал там баш
а послезавтра забыл что он делает
и всё

Tim
09.08.2018
10:38:45

Dan
09.08.2018
10:38:59

Google

Ilya
09.08.2018
10:39:04

Dan
09.08.2018
10:39:05

Tim
09.08.2018
10:39:27
ну короче он просто кодить не умеет и всё
нелепые отмазки про путь рельс сыпает

Dan
09.08.2018
10:40:20
а послезавтра забыл что он делает
Он почти все пишет императивно и потом любит строчка за строчкой разбирать что это делает. Вместо того чтобы инкапсулировать в метод с норм неймингом и читать на английском код. Гавнокодер похоже))

Tim
09.08.2018
10:40:30
а у вас там как ревью проходит? он должен аппрувить твой код?

Admin
ERROR: S client not available

Tim
09.08.2018
10:40:56
сука
и ещё наверное аргумент

Dan
09.08.2018
10:41:02
Да, все плохо

Tim
09.08.2018
10:41:04
"так читается лучше"
"всё явно и понятно"

Dan
09.08.2018
10:41:24
Только я ворвался и жарю их за код на клиенте))

Tim
09.08.2018
10:41:52
ну если тебе не нужен его аппрув, просто делай по-нормальному и всё

Dan
09.08.2018
10:42:07
Поставил хотя бы линтер и притер, чтобы хоть как-то фильтровать их

Tim
09.08.2018
10:42:07
его код рефактори мб, если он тесты пишет
он тесты хоть пишет?

Максим
09.08.2018
10:42:20
))))))

Google

Максим
09.08.2018
10:42:24
)))))))))
удивлюсь если да

Tim
09.08.2018
10:42:34
таки да
ну он такой синьор помидор
что ему тесты не нужны

Dan
09.08.2018
10:42:55

Crypt
09.08.2018
10:42:57
https://stackoverflow.com/questions/46457227/concerns-vs-services

Dan
09.08.2018
10:43:22

Tim
09.08.2018
10:43:34
прям так и говорит?

Максим
09.08.2018
10:43:36

Crypt
09.08.2018
10:43:43
2 разных паттерна, которые не исключают друг друга ... Вот

Tim
09.08.2018
10:43:57
запусти тест на покрытие
(100%, просто zero-assertion тесты)

Anton
09.08.2018
10:44:39

Tim
09.08.2018
10:44:46
ну вот ооп и надо пропогандировать чтобы таких дубней было как можно меньше

Anton
09.08.2018
10:45:02

Crypt
09.08.2018
10:45:10

Anton
09.08.2018
10:45:10
кек