@rubylang
Michael06.09.2016
12:34:21
12:34:21
Googleojab06.09.2016
12:35:55
12:35:55
ну если поля сравнивает - то проходите по полям при помощи all?
таки да, сделать у @shipment методы .shipment_origin/.shipment_destination и можно будет по массиву ключей пройтисьчто-то типа
%(shipment_origin shipment_destination weight_code volume_code shcs).each do |k|
return false if @info[k] != @shipment.send(k)
end
true
Michael06.09.2016
12:37:16
12:37:16
ojab06.09.2016
12:37:21
12:37:21
почему?
Igor06.09.2016
12:37:33
12:37:33
@info.merge!(
Michael06.09.2016
12:37:33
12:37:33
хотя бы последних три сделать одной строкой
Sergey06.09.2016
12:38:18
12:38:18
Приведите их к одному типу с помощью какого-нибудь http://dry-rb.org/gems/dry-types/
GoogleIgor06.09.2016
12:39:06
12:39:06
:(
I06.09.2016
12:39:41
12:39:41
%w(weight_code volume_code shcs).all? { |t| @info[k] == @shipment.public_send(k) } &&
@info[:shipment_origin] == @shipment.origin.iata &&
@info[:shipment_destination] == @shipment.destination.iata
ojab06.09.2016
12:39:57
12:39:57
Добрый день, подскажите пожалуйста, как мне сделать этот метод более короче
def check_permit
@info[:shipment_origin] == @shipment.origin.iata
@info[:shipment_destination] == @shipment.destination.iata
@info[:weight_code] == @shipment.weight_code
@info[:volume_code] == @shipment.volume_code
@info[:shcs] == @shipment.shcs
end
он возвращает true если все поля info равны полям shipment
и возвращает false если хотябы одно поле не равно
А этот метод вообще где находится? В контроллере?Michael06.09.2016
12:40:23
12:40:23
в классе
ojab06.09.2016
12:42:54
12:42:54
я бы вообще сделал что-то типа https://gist.github.com/ojab/4f2e19e99d713700fb55c7714566da07
почему к shipment'y нельзя дополнительные методы добавить?
Sergey06.09.2016
12:43:58
12:43:58
Потому что это говнокод
ojab06.09.2016
12:44:31
12:44:31
смелое утверждение, с учётом того что код ты не видел
Michael06.09.2016
12:46:59
12:46:59
Sergey06.09.2016
12:47:32
12:47:32
Когда понадобится сравнить еще с чем-нибудь, где другие названия, то еще методы будете создавать?
ojab06.09.2016
12:47:58
12:47:58
а ты можешь показать, как эта задача решается с помощью dry-rb?
@info — хеш, @shipment и origin — модельки с минимально возможными полями, shipment belongs_to origin
Michael06.09.2016
12:50:20
12:50:20
сделал так и мне более чем подходит такой вариант
def check_permit
return false if @info[:shipment_origin] != @shipment.origin.iata
return false if @info[:shipment_destination] != @shipment.destination.iata
%i[weight_code volume_code shcs].each { |k| return false if @info[k] != @shipment.send(k) }
true
end
ojab06.09.2016
12:50:48
12:50:48
сделал так и мне более чем подходит такой вариант
def check_permit
return false if @info[:shipment_origin] != @shipment.origin.iata
return false if @info[:shipment_destination] != @shipment.destination.iata
%i[weight_code volume_code shcs].each { |k| return false if @info[k] != @shipment.send(k) }
true
end
@shipment и origin же модельки и это рельсы?Michael06.09.2016
12:51:03
12:51:03
да
Michael06.09.2016
12:51:29
12:51:29
))
Aldar06.09.2016
12:51:44
12:51:44
рельсы, они везде
GoogleSergey06.09.2016
12:52:01
12:52:01
w/e
Michael06.09.2016
12:52:44
12:52:44
таки да, сделать у @shipment методы .shipment_origin/.shipment_destination и можно будет по массиву ключей пройтись
тут мне кажется, не вариант, так как это единоразовая задача, и подгонять модель под хеш, мне не кажется правильным
Mikhail06.09.2016
12:52:57
12:52:57
нормально сделал
работает и ладно)
тока название check_permit какое то не логичное
не стоит так называть методы
Michael06.09.2016
12:54:32
12:54:32
тока название check_permit какое то не логичное
да я обзывал все как хотел, потом в методе многое менял, а названия остались еще старые))
сейчас у меня рефакторинг, логически назвать методы, тоже в него войдут)))ojab еще раз спасибо
Aldar06.09.2016
12:57:36
12:57:36
придумывать названия - самое сложное в программировании
Michael06.09.2016
13:02:28
13:02:28
Mikhail06.09.2016
13:05:27
13:05:27
???
Michael06.09.2016
13:06:02
13:06:02
trickster06.09.2016
13:06:18
13:06:18
придумываешь на русском закидываешь в переводчик === профит
Mikhail06.09.2016
13:06:48
13:06:48
всяко лучше check_permit
Aldar06.09.2016
13:09:45
13:09:45
а то будет как в яндексе class Stroka
GoogleMichael06.09.2016
13:11:57
13:11:57
а то будет как в яндексе class Stroka
ну правильно, у яндекса же нет своего нормального яндекс переводчика, а в оффисе наверное заблочили все ресурсы гуглаAldar06.09.2016
13:12:21
13:12:21
не, это про них бородатая история
Aldar06.09.2016
13:12:31
13:12:31
сейчас то уже давно нет этого, наверное)
Mikhail06.09.2016
15:36:28
15:36:28
Кто нибудь знает проект с 5 рельсой и новыми турболинками? Бейскемп не надо
Vitaliy06.09.2016
15:40:45
15:40:45
Из крупных? Мне кажется - нет таких
Крупные проекты инертны. Больше времени уходит на апгрейд стека
AdminERROR: S client not available
Mikhail06.09.2016
15:41:28
15:41:28
Можно не крупные
ojab06.09.2016
15:41:47
15:41:47
а зачем?
Mikhail06.09.2016
15:41:56
15:41:56
хочу посмотреть
Vitaliy06.09.2016
15:42:14
15:42:14
у меня мелочь пока в виде https://github.com/gambala/gambala
Mikhail06.09.2016
15:42:26
15:42:26
да не
Vitaliy06.09.2016
15:42:27
15:42:27
то есть совсем микро-проект
Mikhail06.09.2016
15:42:31
15:42:31
мне потыкать хочется
https://ostsn.ru/
вот мне нравится как отдает
а турболинками по идее отдавать будет быстрее
Vitaliy06.09.2016
15:43:15
15:43:15
быстро, да, и турболинков в нем нет
GoogleVitaliy06.09.2016
15:43:32
15:43:32
йес, визуально быстрее
Vitaliy06.09.2016
15:43:43
15:43:43
потыкать мой тоже можно http://gambala.pro
побродить там по разделам, поклацать назад вперед по истории
Mikhail06.09.2016
15:44:40
15:44:40
да у меня проект есть и по более примера. Думал может ктото уже запилил что нибудь еще)
@gambala https://github.com/hashrocket/decent_exposure посмотри вот этот гем. Мне кажется тебе его нехватает
s06.09.2016
16:23:56
16:23:56
это еще одни вариации на тему inherited_resources ?
s06.09.2016
16:24:50
16:24:50
if thing.save
redirect_to thing_path(thing)
else
render :new
end
зачем все это когда есть responders?
и has_scope или scopie
Mikhail06.09.2016
16:26:10
16:26:10
это вскупе с респондерами
s06.09.2016
16:26:53
16:26:53
ну вот скоупы для контроллера + респондерс = то же самое, только без магии
Mikhail06.09.2016
16:27:03
16:27:03
Ясн-пнт
s06.09.2016
16:27:05
16:27:05
или я чего-то не понял
Mikhail06.09.2016
16:27:57
16:27:57
Я не большой любитель драить такое, но гем классный
Nikolay06.09.2016
16:43:04
16:43:04
а я чо, молодец. случайно грохнул наработки за три дня. зато повод офигенно порефакторить то шо есть %)
и выкинуть наконец тонны говнологики которая отнесена в валидации и бд %)
вместо накоебливания в где-попало типа вьюшек
Открыть в Telegram