Update app.py

app.py

@@ -2,11 +2,12 @@ import streamlit as st
 import requests
 import pandas as pd
 import time
+import urllib.parse
 
-# Open Library API base URL (using JSON endpoint for easy parsing)
+# Base API URL for Open Library
 BASE_API_URL = "https://openlibrary.org/search.json"
 
-# List of available subjects (feel free to modify)
+# List of subjects for selection
 SUBJECTS = [
     "Classic Literature", "Modern Literature", "Postmodern Literature", "Romanticism",
     "Realism", "Naturalism", "Existentialism", "Absurdism", "Gothic Fiction", "Science Fiction",
@@ -14,30 +15,21 @@ SUBJECTS = [
     "Adventure Novels", "Bildungsroman", "Epistolary Novels", "Satire", "Tragedy", "Comedy"
 ]
 
-# List of time facets available (as centuries)
-TIME_FACETS = [
-    "16th century", "17th century", "18th century", "19th century", "20th century", "21st century"
-]
-
-# Function to fetch books based on subjects and selected time facet
 def fetch_books(subjects, time_facet):
-    books = []
-    # Build the query string for subjects. If more than one subject is selected,
-    # they will be joined with a space (which becomes a '+' in the URL).
+    # Join the selected subjects into one search query
     query_subject = " ".join(subjects)
     
-    # Replace spaces with '+' for the URL
-    query_subject_url = query_subject.replace(" ", "+")
-    
-    # Format the time facet by replacing spaces with '+'
-    time_facet_url = "/" + time_facet.replace(" ", "+")
+    # URL-encode the query components
+    query_subject_encoded = urllib.parse.quote_plus(query_subject)
+    time_facet_encoded = urllib.parse.quote_plus(time_facet)
     
-    # Build the full URL with the mode and time_facet parameters.
-    query_url = f"{BASE_API_URL}?q={query_subject_url}&mode=everything&time_facet={time_facet_url}"
-    
-    st.write(f"🔗 Query URL: {query_url}")  # Optional: show the query URL for debugging
+    # Build the full URL with the mode and time_facet parameters
+    query_url = f"{BASE_API_URL}?q={query_subject_encoded}&mode=everything&time_facet={time_facet_encoded}"
+    st.write(f"🔗 Query URL: {query_url}")  # Optional: Display the URL for debugging purposes
     
     response = requests.get(query_url)
+    books = []
+    
     if response.status_code == 200:
         data = response.json()
         for book in data.get("docs", []):
@@ -47,27 +39,29 @@ def fetch_books(subjects, time_facet):
                 "first_publish_year": book.get("first_publish_year", "N/A")
             })
     else:
-        st.error("Failed to fetch data from Open Library.")
+        st.error("Error fetching data from Open Library.")
     
-    # Optional: small delay to be polite to the API server.
-    time.sleep(1)
+    time.sleep(1)  # A short delay to be respectful to the API server
     return books
 
 # Streamlit UI
 st.title("📚 Open Library Book Finder")
-st.write("Search books by subject and time facet.")
+st.write("Search for books by subject and a custom time facet.")
 
-# Subject selection: allow the user to choose one or more subjects.
+# Let the user select one or more subjects
 selected_subjects = st.multiselect("Select subject(s)", SUBJECTS, default=["Classic Literature"])
 
-# Time facet selection: choose a century.
-selected_time_facet = st.selectbox("Select time facet", TIME_FACETS)
+# Let the user enter a custom time facet
+time_facet = st.text_input("Enter a time facet (e.g., 20th century)")
 
 if st.button("Find Books"):
-    st.write("🔍 Searching books...")
-    books = fetch_books(selected_subjects, selected_time_facet)
-    if books:
-        df = pd.DataFrame(books)
-        st.write(df)
+    if not time_facet:
+        st.error("Please enter a time facet!")
     else:
-        st.write("❌ No books found for the selected criteria.")
+        st.write("🔍 Searching books...")
+        books = fetch_books(selected_subjects, time_facet)
+        if books:
+            df = pd.DataFrame(books)
+            st.write(df)
+        else:
+            st.write("❌ No books found for the selected criteria.")