lib/generators/stats.rb in metric_fu-1.3.0 vs lib/generators/stats.rb in metric_fu-1.4.0

@@ -6,22 +6,43 @@
       `rake stats > #{metric_directory + '/stats.txt'}`
     end
 
     def analyze
       output = File.open(metric_directory + '/stats.txt').read
-      output = output.split("\n")
-      output = output.find_all {|line| line[0].chr != "+" }
-      output = output.find_all {|line| line[0].chr != "(" }
-      output.shift
-      totals = output.pop
-      totals = totals.split("  ").find_all {|el| ! el.empty? }
+      lines = remove_noise(output)
+
       @stats = {}
+      
+      set_global_stats(lines.pop)
+      set_granular_stats(lines)
+
+      @stats
+    end
+
+    def to_h
+      {:stats => @stats}
+    end
+    
+    private
+    
+    def remove_noise(output)
+      lines = output.split("\n")
+      lines = lines.find_all {|line| line[0].chr != "+" }
+      lines = lines.find_all {|line| line[0].chr != "(" }
+      lines.shift
+      lines
+    end
+    
+    def set_global_stats(totals)
+      totals = totals.split("  ").find_all {|el| ! el.empty? }
       @stats[:codeLOC] = totals[0].match(/\d.*/)[0].to_i
       @stats[:testLOC] = totals[1].match(/\d.*/)[0].to_i
       @stats[:code_to_test_ratio] = totals[2].match(/1\:(\d.*)/)[1].to_f
-      
-      @stats[:lines] = output.map do |line|
+    end
+    
+    def set_granular_stats(lines)
+      @stats[:lines] = lines.map do |line|
         elements = line.split("|")
         elements.map! {|el| el.strip }
         elements = elements.find_all {|el| ! el.empty? }
         info_line = {}
         info_line[:name] = elements.shift
@@ -30,14 +51,9 @@
          :methods_per_class, :loc_per_method].each do |sym|
           info_line[sym] = elements.shift
         end
         info_line
       end
-      @stats
-    end
-
-    def to_h
-      {:stats => @stats}
     end
 
   end
 end