Pregunta

I have the following 7 lines of code, how can i made them shorter and less wordy?

max_group_size = 0
wrong_services.each do |service|
  group_size = service.iep_service.group_size
  if group_size > max_group_size then
    max_group_size = group_size
  end
end
¿Fue útil?

Solución

max_group_size = wrong_services.max_by{|service| service.iep_service.group_size}.iep_service.group_size

(Editted after comments)

Otros consejos

max_group_size = wrong_services.map {|service| service.iep_service.group_size }.max
Licenciado bajo: CC-BY-SA con atribución
No afiliado a StackOverflow
scroll top