Merge branch 'recreate-movie-search-app'

* recreate-movie-search-app:
  Your turn instructions updated for new API.
  Final recorded version (take 2, with movie_service.talkpython.fm)
  base error handling in place.
  play around version done and pythonic.
  starter (again)
  ignore .envs

Author: mikeckennedy

.gitignore

@@ -65,3 +65,4 @@ target/
 .ipynb_checkpoints
 .DS_Store
 .idea
+.env

apps/10_movie_search/final/movie_client.py

@@ -0,0 +1,29 @@
+import collections
+import requests
+
+MovieResult = collections.namedtuple(
+    'MovieResult',
+    "imdb_code,title,duration,director,year,rating,imdb_score,keywords,genres")
+
+
+def find_movies(search_text):
+
+    if not search_text or not search_text.strip():
+        raise ValueError("Search text is required")
+
+    url = 'http://movie_service.talkpython.fm/api/search/{}'.format(search_text)
+
+    resp = requests.get(url)
+    resp.raise_for_status()
+
+    movie_data = resp.json()
+    movies_list = movie_data.get('hits')
+
+    movies = [
+        MovieResult(**md)
+        for md in movies_list
+    ]
+
+    movies.sort(key=lambda m: -m.year)
+
+    return movies

apps/10_movie_search/final/movie_svc.py

@@ -3,50 +3,47 @@
 
 MovieResult = collections.namedtuple(
     'MovieResult',
-    'Title,Poster,Type,imdbID,Year'
-)
+    "imdb_code,title,duration,director,year,rating,imdb_score,keywords,genres")
 
-search = 'capital'
-url = 'http://www.omdbapi.com/?s={}&y=&plot=short&r=json'.format(search)
+search = input("What movie do you want to search for? ")
+url = 'http://movie_service.talkpython.fm/api/search/{}'.format(search)
 
-r = requests.get(url)
-data = r.json()
+resp = requests.get(url)
+resp.raise_for_status()
 
-results = data['Search']
+movie_data = resp.json()
+movies_list = movie_data.get('hits')
 
 # movies = []
-# for result in results:
+# for md in movies_list:
 #     m = MovieResult(
-#         Title=result['Title'],
-#         Poster=result['Poster'],
-#         Type=result['Type'],
-#         imdbID=result['imdbID'],
-#         Year=result['Year']
+#         imdb_code=md.get('imdb_code'),
+#         title=md.get('title'),
+#         duration=md.get('duration'),
+#         director=md.get('director'),
+#         year=md.get('year', 0),
+#         rating=md.get('rating', 0),
+#         imdb_score=md.get('imdb_score', 0.0),
+#         keywords=md.get('keywords'),
+#         genres=md.get('genres')
 #     )
 #     movies.append(m)
 
-# def method_with_kws(pos1, **kwargs)
-# pass
+# def method(x, y, z, **kwargs):
+#     print("kwargs=", kwargs)
+#
+# method(7, 1, z=2, format=True, age=7)
 
 # movies = []
-# for result in results:
-#     m = MovieResult(**result)
+# for md in movies_list:
+#     m = MovieResult(**md)
 #     movies.append(m)
 
 movies = [
-    MovieResult(**m)
-    for m in results
+    MovieResult(**md)
+    for md in movies_list
 ]
 
-print(movies)
-
-
-
-
-
-
-
-
-
-
-
+print("Found {} movies for search {}".format(len(movies), search))
+for m in movies:
+    print("{} -- {}".format(m.year, m.title))

apps/10_movie_search/final/play.py

@@ -1,6 +1,7 @@
-from movie_client import MovieClient
+import movie_svc
 import requests.exceptions
 
+
 def main():
     print_header()
     search_event_loop()
@@ -18,37 +19,24 @@ def search_event_loop():
 
     while search != 'x':
         try:
-            search = input('Title search text (x to exit): ')
+            search = input('Movie search text (x to exit): ')
             if search != 'x':
-                client = MovieClient(search)
-
-                results = client.perform_search()
+                results = movie_svc.find_movies(search)
                 print("Found {} results.".format(len(results)))
                 for r in results:
                     print('{} -- {}'.format(
-                        r.Year, r.Title
+                        r.year, r.title
                     ))
+                print()
+        except ValueError:
+            print("Error: Search text is required")
         except requests.exceptions.ConnectionError:
-            print('ERROR: Cannot search, you network is down.')
-        except ValueError as ve:
-            print('ERROR: Your search string is invalid: {}'.format(ve))
+            print("Error: Your network is down.")
         except Exception as x:
-            print("ERROR: {}".format(x))
+            print("Unexpected error. Details: {}".format(x))
 
     print('exiting...')
 
 
 if __name__ == '__main__':
     main()
-
-
-
-
-
-
-
-
-
-
-
-

apps/10_movie_search/final/program.py

@@ -2,20 +2,29 @@
 
 ![image](app-10-screenshot.png)
 
-If you want to try this yourself, try to build the interactive app above. 
+Your turn. Try to build the interactive app above. 
 
-This application use the OMDb API to search live movie data for movies matching the title in your search text. It focuses on being reliable even when the network fails or the user enters bad information.
+This application use the Talk Python MovieDb API to search live movie data for movies 
+matching the title in your search text. It focuses on being reliable 
+even when the network fails or the user enters bad information.
 
 Key concepts introduced
 =================
 
 **The API**
 
-You can find the details of the JSON HTTP API at [www.omdbapi.com](http://www.omdbapi.com).
+You can find the details of the JSON HTTP API at [movie_service.talkpython.fm](http://movie_service.talkpython.fm/).
 
 **Try/Except Error Handling**
 
-    try:        method1()        method2()        method3()    except ConnectionError as ce:        # handle network error    except Exception as x:        # handle general error
+    try:
+        method1()
+        method2()
+        method3()
+    except ConnectionError as ce:
+        # handle network error
+    except Exception as x:
+        # handle general error
 
 **Raising your own errors and exceptions**