Refactored printing into separate method

Siddharta Govindaraj · Siddharta Govindaraj · commit 9e9ba2e6725c · 2015-03-23T18:31:34.000+05:30
diff --git a/stock_alerter/legacy.py b/stock_alerter/legacy.py
@@ -12,15 +12,15 @@ def __init__(self, autorun=True, exchange=None):
             self.exchange = exchange
         rule_1 = PriceRule("GOOG", lambda stock: stock.price > 10)
         rule_2 = PriceRule("AAPL", lambda stock: stock.price > 5)
-        self.exchange["GOOG"].updated.connect(
-            lambda stock: print(stock.symbol, stock.price) \
-                          if rule_1.matches(self.exchange) else None)
-        self.exchange["AAPL"].updated.connect(
-            lambda stock: print(stock.symbol, stock.price) \
-                          if rule_2.matches(self.exchange) else None)
+        self.exchange["GOOG"].updated.connect(lambda stock: self.print_action(stock, rule_1))
+        self.exchange["AAPL"].updated.connect(lambda stock: self.print_action(stock, rule_2))
         if autorun:
             self.run()
 
+    def print_action(self, stock, rule):
+        print(stock.symbol, stock.price) \
+            if rule.matches(self.exchange) else None
+
     def do_updates(self, updates):
         for symbol, timestamp, price in updates:
             stock = self.exchange[symbol]
