From 4e19ef9b25e7461be654c6e5bc085addbefc459e Mon Sep 17 00:00:00 2001 From: Carlos Antonio da Silva Date: Tue, 3 Dec 2013 19:42:49 -0200 Subject: Refactor to avoid earlier returns --- .../lib/active_support/number_helper/number_converter.rb | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) (limited to 'activesupport/lib') diff --git a/activesupport/lib/active_support/number_helper/number_converter.rb b/activesupport/lib/active_support/number_helper/number_converter.rb index 537bf1e5b6..471c686997 100644 --- a/activesupport/lib/active_support/number_helper/number_converter.rb +++ b/activesupport/lib/active_support/number_helper/number_converter.rb @@ -119,14 +119,18 @@ module ActiveSupport end def initialize(number, options) - @number = number - @opts = options.symbolize_keys + @number = number + @opts = options.symbolize_keys end def execute - return unless @number - return @number if validate_float? && !valid_float? - convert + if !@number + nil + elsif validate_float? && !valid_float? + @number + else + convert + end end private -- cgit v1.2.3