M
Только оно и не работало ))
Size: a a a
2020 December 04
M
удалил )))
+
ES
Как-то отдали одну фичу на аутсорс типу из Индии
Я тебе скажу так, это писал человек который работает на рельсе со 2 версии, CTO на своем проекте, сегодян 2020 год, этот проект приносит деньги.
ES
но это же не про DRY
Почему, вот я вижу
@item = ShopifyAPI::CustomCollection.find(params[:id])
@url = "https://#{@shop.domain}/collections/#{@item.handle}"
@hide_summernote = true
АД
Чтобы бы ты с ним сделал?
ну если оно работает и покрыто тестами - ничего
M
Ну смотри, если он проект этот писал сам для себя
M
То может и ок
M
В остальных случаях - ну хз, хз )
DF
Я тебе скажу так, это писал человек который работает на рельсе со 2 версии, CTO на своем проекте, сегодян 2020 год, этот проект приносит деньги.
даже если ты 20 лет за рулем, ты все еще можешь стать хорошим водителем.
ES
даже если ты 20 лет за рулем, ты все еще можешь стать хорошим водителем.
🙂
АД
Почему, вот я вижу
@item = ShopifyAPI::CustomCollection.find(params[:id])
@url = "https://#{@shop.domain}/collections/#{@item.handle}"
@hide_summernote = true
а что тут не DRY?
VS
Можно создать класс базовый класс и наследников от него для каждого item_type
VS
и там будет по паре строчек
ES
ну если оно работает и покрыто тестами - ничего
Да, вроде так и есть.
M
Ну да, или если не хочется сущностей плодить - сделать хотя бы конфиги хешами и проитерировать. Не понятно, насколько широко оно в системе используется и сколько придется менять на саб-классы
АД
Да, вроде так и есть.
у меня конечно есть жгучее желание это все переписать, но мама говорит что так не надо делать
АД
БЕСПЛАТНО
ES
У него очень интересная позиция.
Он делит разрабов на две категории, которые рефакторят все и которые исправляют ошибку.
Он делит разрабов на две категории, которые рефакторят все и которые исправляют ошибку.
SS
иногда, когда очень болит, можно )