Evitar chamadas ao `.each_pair` e ao `.each` no `@tmp` se ele estiver vazio?

[thiagofm33]

Pessoal, seria uma boa evitar chamadas ao .each_pair e ao .each no @tmp se ele estiver vazio?

active_service/lib/active_service/response.rb

Lines 5 to 10 in 90c9fdb

	def initialize(data = {})
	@tmp = data
	handle_errors_and_messages
	define_accessors
	end

Tava pensando que algo assim poderia ficar bacana.

    def initialize(data = {})
      @tmp = data

      handle_data if data.any?
    end

    def handle_data
      handle_errors_and_messages
      define_accessors
    end

Que acham, @petlove/ecommerce?

[ferreiraandy]

Acho interessante, mas qual a motivação, @thiagofm33?
Só, melhoria para não percorrer o código ou rolou algum erro ao tentar iterar algo nulo?

[thiagofm33]

Acho interessante, mas qual a motivação, @thiagofm33? Só, melhoria para não percorrer o código ou rolou algum erro ao tentar iterar algo nulo?

Só melhoria mesmo, @ferreiraandy. É que notei que o método response é chamado muitas vezes sem parâmetro algum por services que herdam do ActiveService::Base.

[ferreiraandy]

Acho interessante, mas qual a motivação, @thiagofm33? Só, melhoria para não percorrer o código ou rolou algum erro ao tentar iterar algo nulo?

Só melhoria mesmo, @ferreiraandy. É que notei que o método response é chamado muitas vezes sem parâmetro algum por services que herdam do ActiveService::Base.

Top então.