я следующий код,Написание кратким множественным еще/если заявление
@checkedin=[]
@accepted=[]
@rejected=[]
result.each do |parse_order|
orderId = parse_order['orderID']
if parse_order['status'] == -1
@rejected << orderId
elsif parse_order['status'] == 1
@accepted << [orderId, parse_order['createdAt']]
elsif parse_order['status'] == 2
@checkedin << [orderId, parse_order['createdAt']]
elsif parse_order['status'] == 3
next
end
end
есть лучший способ его кратким. Спасибо.
В этом случае массивы инициализируются на уровне класса? – harshit
Нет, вам придется приписывать все «себе». Я думаю, что решение Пита синтаксически лучше всего из всех размещенных решений –
Я очень предпочитаю то, что у вас есть наверху. Вы сказали, что изменили его на удобочитаемость, но я думаю, что это гораздо менее читаемо. Это в основном потому, что у нас есть многолетний опыт чтения по горизонтали (вверху), и у него неплохо получается. Чтение по вертикали (внизу) просто неэффективно. Кроме того, чем компактнее код (в пределах, конечно), тем больше глаз может занять сразу. Когда я смотрю на ваш код сверху, я знаю, что вы делаете почти мгновенно. На дне я должен охотиться, прежде чем понять общую картину. При написании кода «скорость чтения» является одним из моих самых важных критериев. –