refactoring - separated finding offers from printing them

Author: Shadi

crawl

@@ -10,4 +10,4 @@ back = sys.argv[4]
 flex = 0
 if len(sys.argv) == 6:
     flex = int(sys.argv[5])
-offersFinder.get_offers(orig_city=orig_city, dest_city=dest_city, leave=leave, back=back, forward_days=flex)
+offersFinder.find_offers(orig_city=orig_city, dest_city=dest_city, leave=leave, back=back, forward_days=flex)

expediacrawler/offersFinder.py

@@ -3,6 +3,14 @@
 from expediacrawler.dateCalculator import calculate_new_dates
 
 
+def find_offers(orig_city, dest_city, leave, back, forward_days):
+    all_offers = get_offers(orig_city, dest_city, leave, back, forward_days)
+    for day in all_offers:
+        print(day, ":")
+        for offer in all_offers[day]:
+            print(offer)
+
+
 def get_offers(orig_city, dest_city, leave, back, forward_days):
     all_offers = {}
     day_offer = get_day_offers(orig_city, dest_city=dest_city, leave=leave, back=back)
@@ -12,11 +20,7 @@ def get_offers(orig_city, dest_city, leave, back, forward_days):
         out_str, return_str = calculate_new_dates(leave, back, i)
         day_offer = get_day_offers(orig_city, dest_city=dest_city, leave=out_str, back=return_str)
         all_offers[out_str + '-' + return_str] = day_offer[0:5]
-
-    for day in all_offers:
-        print(day, ":")
-        for offer in all_offers[day]:
-            print(offer)
+    return all_offers
 
 
 def get_day_offers(orig_city, dest_city, leave, back):