Issue 3703 - Add "coin flip" feature to abpbot

decisionbot.py

+"""
+   Decisionbot - A simple IRC bot to help make "coin flip" decisions.
+
+   botname: x or y?
+   => x
+
+   botname: a or b or c?
+   => b
+"""
+import random

[Sebastian Noack, 2016/02/25 20:47:53]

Nit: Please add an empty line between the docstring and imports.

[kzar, 2016/02/25 20:52:40]

Done.

+import re
+
+from irclib import nm_to_n
+
+
+class Decisionbot():
+  def __init__(self, config, queue):
+    self.queue = queue
+
+    nickname = config.get("main", "nickname")
+    self.question_regexp = re.compile("^%s:?(.+\s+or\s+.+)\?+\s*$" %

[Sebastian Noack, 2016/02/25 20:47:53]

I just realized that you don't use r"" strings. But for some reason is "\s" ==
"\\s" == r"\s". However, I think it's still better to use r"".

[kzar, 2016/02/25 20:52:40]

Done.

+                                      re.escape(nickname), re.IGNORECASE)
+    self.question_delim_regexp = re.compile("\s+or\s+", re.IGNORECASE)
+
+  def on_pubmsg(self, connection, event):
+    channel = event.target()
+    message = event.arguments()[0]
+    sender = nm_to_n(event.source())
+
+    match = self.question_regexp.search(message)
+    if (match):
+      choices = self.question_delim_regexp.split(match.group(1).strip("? \t"))
+      if len(choices) > 1:
+        self.say_public(channel, "%s: %s" % (sender, random.choice(choices)))
+
+  def say_public(self, channel, text):
+    self.queue.send(text, channel)