Merge: Nitdoc: more cleaning and uniformize method names
[nit.git] / contrib / nitrpg / src / statistics.nit
index 65d5a69..7b493c6 100644 (file)
@@ -22,23 +22,31 @@ module statistics
 
 import game
 import github::hooks
+import counter
 
-redef class Game
+redef class GameEntity
 
-       # Statistics for this game instance.
-       var stats = new GameStats
+       # Statistics for this entity.
+       fun stats: GameStats is abstract
 
-       redef fun from_json(json) do
-               super
-               if json.has_key("statistics") then
-                       stats.from_json(json["statistics"].as(JsonObject))
-               end
+       # Load statistics for this `MEntity` if any.
+       fun load_statistics: nullable GameStats do
+               var key = self.key / "statistics"
+               if not game.store.has_key(key) then return null
+               var json = game.store.load_object(key)
+               return new GameStats.from_json(game, json)
        end
+end
 
-       redef fun to_json do
-               var obj = super
-               obj["statistics"] = stats.to_json
-               return obj
+redef class Game
+
+       redef var stats is lazy do
+               return load_statistics or else new GameStats(game)
+       end
+
+       redef fun save do
+               super
+               stats.save_in(self)
        end
 
        redef fun pretty do
@@ -48,55 +56,61 @@ redef class Game
                res.append stats.pretty
                return res.write_to_string
        end
+end
+
+redef class Player
+
+       redef var stats is lazy do
+               return load_statistics or else new GameStats(game)
+       end
 
-       redef fun clear do
+       redef fun save do
                super
-               stats.clear
+               stats.save_in(self)
+       end
+
+       redef fun pretty do
+               var res = new FlatBuffer
+               res.append super
+               res.append "# stats:\n"
+               res.append stats.pretty
+               return res.write_to_string
        end
 end
 
 # Game statistics structure that can be saved as a `GameEntity`.
 class GameStats
        super GameEntity
+       super Counter[String]
 
-       redef var key = "statistics"
-
-       # Used internally to stats values.
-       private var stats = new HashMap[String, Int]
+       redef var game
 
-       init do clear
+       redef var key = "statistics"
 
        # Load `self` from saved data
-       private fun from_json(json: JsonObject) do
-               for k, v in json do stats[k] = v.as(Int)
+       init from_json(game: Game, json: JsonObject) do
+               self.game = game
+               for k, v in json do self[k] = v.as(Int)
        end
 
        redef fun to_json do
                var obj = new JsonObject
-               for k, v in stats do obj[k] = v
+               for k, v in self do obj[k] = v
                return obj
        end
 
-       # Retrieves the current value of `key` statistic entry.
-       fun [](key: String): Int do return stats[key]
-
-       # Increments the value of `key` statistic entry by 1.
-       fun incr(key: String) do stats[key] += 1
-
        # Decrements the value of `key` statistic entry by 1.
-       fun decr(key: String) do stats[key] -= 1
-
-       # Reset game stats.
-       fun clear do
-               stats["issues"] = 0
-               stats["issues_open"] = 0
-               stats["pulls"] = 0
-               stats["pulls_open"] = 0
+       fun dec(key: String) do
+               if not has_key(key) then
+                       self[key] = 0
+               else
+                       self[key] -= 1
+               end
        end
 
        redef fun pretty do
                var res = new FlatBuffer
-               for k, v in stats do
+               for k, v in self do
                        res.append "# {v} {k}\n"
                end
                return res.write_to_string
@@ -107,11 +121,7 @@ end
 class StatisticsReactor
        super GameReactor
 
-       redef fun react_event(game, e) do
-               super # log events
-               e.react_stats_event(game)
-               game.save
-       end
+       redef fun react_event(game, e) do e.react_stats_event(game)
 end
 
 redef class GithubEvent
@@ -126,13 +136,24 @@ redef class IssuesEvent
 
        # Count opened and closed issues.
        redef fun react_stats_event(game) do
+               var player = issue.user.player(game)
                if action == "opened" then
-                       game.stats.incr("issues")
-                       game.stats.incr("issues_open")
+                       game.stats.inc("issues")
+                       game.stats.inc("issues_open")
+                       game.save
+                       player.stats.inc("issues")
+                       player.stats.inc("issues_open")
+                       player.save
                else if action == "reopened" then
-                       game.stats.incr("issues_open")
+                       game.stats.inc("issues_open")
+                       game.save
+                       player.stats.inc("issues_open")
+                       player.save
                else if action == "closed" then
-                       game.stats.decr("issues_open")
+                       game.stats.dec("issues_open")
+                       game.save
+                       player.stats.dec("issues_open")
+                       player.save
                end
        end
 end
@@ -141,13 +162,52 @@ redef class PullRequestEvent
 
        # Count opened and closed pull requests.
        redef fun react_stats_event(game) do
+               var player = pull.user.player(game)
                if action == "opened" then
-                       game.stats.incr("pulls")
-                       game.stats.incr("pulls_open")
+                       game.stats.inc("pulls")
+                       game.stats.inc("pulls_open")
+                       game.save
+                       player.stats.inc("pulls")
+                       player.stats.inc("pulls_open")
+                       player.save
                else if action == "reopened" then
-                       game.stats.incr("pulls_open")
+                       game.stats.inc("pulls_open")
+                       game.save
+                       player.stats.inc("pulls_open")
+                       player.save
                else if action == "closed" then
-                       game.stats.decr("pulls_open")
+                       game.stats.dec("pulls_open")
+                       player.stats.dec("pulls_open")
+                       if pull.merged then
+                               game.stats["commits"] += pull.commits
+                               player.stats["commits"] += pull.commits
+                       end
+                       game.save
+                       player.save
                end
        end
 end
+
+redef class IssueCommentEvent
+
+       # Count posted comments
+       redef fun react_stats_event(game) do
+               if action == "created" then
+                       var player = comment.user.player(game)
+                       game.stats.inc("comments")
+                       player.stats.inc("comments")
+                       # FIXME use a more precise way to locate reviews
+                       if comment.has_ok_review then
+                               game.stats.inc("reviews")
+                               player.stats.inc("reviews")
+                       end
+                       game.save
+                       player.save
+               end
+       end
+end
+
+redef class IssueComment
+       # Does this comment contain a "+1"?
+       fun has_ok_review: Bool do return body.has("\\+1\\b".to_re)
+end