Permalink
Browse files

more complexity simplification

  • Loading branch information...
1 parent 2826c2b commit 7196b85ab33db58c3c1cea6ef805d02a0bec4e3c @michaelfeathers michaelfeathers committed May 14, 2011
Showing with 32 additions and 2 deletions.
  1. +1 −2 codehistory.rb
  2. +12 −0 investigations
  3. +19 −0 plans
View
@@ -15,8 +15,7 @@ def build_collection symbol, class_object
end
def complexity_of collection
- return 0.0 if collection.empty?
- collection.map(&:complexity).reduce(:+)
+ collection.map(&:complexity).reduce(0.0, :+)
end
end
View
@@ -0,0 +1,12 @@
+
+- Do large commits (in terms of the delta of complexity) occur
+ with equal frequency across the day?
+- Is commit size correlated with weighted complexity?
+- Is commit size correlated with the time since last commit?
+- Are large deltas in a complex method's life line correlated
+ with large changes?
+- Can we identify extract method?
+
+
+For the Corey session: 'what happened here?
+
View
19 plans
@@ -0,0 +1,19 @@
+
+
+- Have commits on Repository return a new Repository
+ which has only the CodeEvent for that commit
+- Have functions on repository which give a slice of commits
+ by sha or date
+- Add a declared_methods collection on Repository for all of
+ the methods in the repository. Will be very easy if
+ CodeEvent gets full_method_name method. Then it is
+ just a group_by on the original events
+- Make sure that group_by preserves order. If unable to,
+ embedded date sort checks across the code that can be
+ toggled.
+- Consider adding a changed_methods method which looks
+ at the current and last commit and gives the complexity
+ delta -- needed now as we have code events for methods
+ just because they are in a file. May decide that
+ it is better to just handle this with external code.
+

0 comments on commit 7196b85

Please sign in to comment.