Update history_manager.py

history_manager.py

@@ -29,54 +29,94 @@ class UserHistoryManager:
             print(traceback.format_exc())
             
 
-    def save_history(self, user_preferences: dict = None, results: list = None, search_type: str = "criteria", description: str = None) -> bool:
-        """
-        Save search history with proper timestamp handling.
+    # def save_history(self, user_preferences: dict = None, results: list = None, search_type: str = "criteria", description: str = None) -> bool:
+    #     """
+    #     Save search history with proper timestamp handling.
         
-        Args:
-            user_preferences: User's search preferences
-            results: List of breed recommendations
-            search_type: Type of search performed
-            description: Search description if applicable
+    #     Args:
+    #         user_preferences: User's search preferences
+    #         results: List of breed recommendations
+    #         search_type: Type of search performed
+    #         description: Search description if applicable
         
-        Returns:
-            bool: Success status of save operation
-        """
+    #     Returns:
+    #         bool: Success status of save operation
+    #     """
+    #     try:
+    #         # Initialize timezone and current time
+    #         taipei_tz = pytz.timezone('Asia/Taipei')
+    #         current_time = datetime.now(taipei_tz)
+            
+    #         # Create history entry with timestamp
+    #         history_entry = {
+    #             "timestamp": current_time.strftime("%Y-%m-%d %H:%M:%S"),
+    #             "search_type": search_type,
+    #             "results": results
+    #         }
+            
+    #         # Add preferences to history entry if provided
+    #         if user_preferences:
+    #             history_entry["preferences"] = user_preferences
+    
+    #         # Read existing history
+    #         with open(self.history_file, 'r', encoding='utf-8') as f:
+    #             history = json.load(f)
+    
+    #         # Add new entry and maintain history limit
+    #         history.append(history_entry)
+    #         if len(history) > 20:  # Keep last 20 entries
+    #             history = history[-20:]
+    
+    #         # Save updated history
+    #         with open(self.history_file, 'w', encoding='utf-8') as f:
+    #             json.dump(history, f, ensure_ascii=False, indent=2)
+    
+    #         return True
+    
+    #     except Exception as e:
+    #         print(f"Error saving history: {str(e)}")
+    #         return False
+
+    def save_history(self, user_preferences: dict = None, results: list = None, search_type: str = "criteria", description: str = None) -> bool:
         try:
-            # Initialize timezone and current time
+            # 確保 results 格式正確
+            if results:
+                for result in results:
+                    if not all(k in result for k in ('breed', 'overall_score')):
+                        print(f"Invalid result entry detected: {result}")
+                        raise ValueError("Each result must include 'breed' and 'overall_score'.")
+    
+            # 原始保存邏輯保持不變
             taipei_tz = pytz.timezone('Asia/Taipei')
             current_time = datetime.now(taipei_tz)
-            
-            # Create history entry with timestamp
+    
             history_entry = {
                 "timestamp": current_time.strftime("%Y-%m-%d %H:%M:%S"),
                 "search_type": search_type,
                 "results": results
             }
-            
-            # Add preferences to history entry if provided
             if user_preferences:
                 history_entry["preferences"] = user_preferences
     
-            # Read existing history
             with open(self.history_file, 'r', encoding='utf-8') as f:
                 history = json.load(f)
     
-            # Add new entry and maintain history limit
             history.append(history_entry)
-            if len(history) > 20:  # Keep last 20 entries
+            if len(history) > 20:  # 保留最近 20 條
                 history = history[-20:]
     
-            # Save updated history
             with open(self.history_file, 'w', encoding='utf-8') as f:
                 json.dump(history, f, ensure_ascii=False, indent=2)
     
+            print("History saved successfully:", history_entry)
             return True
     
         except Exception as e:
             print(f"Error saving history: {str(e)}")
+            print(traceback.format_exc())
             return False
 
+    
     def get_history(self) -> list:
         """獲取搜尋歷史"""
         try: