Update app.py

app.py

@@ -9,13 +9,10 @@ import os
 import matplotlib.font_manager as fm
 
 # ============================
-# 字型載入與設定 (修正重點)
+# 載入本地字型檔與設定
 # ============================
 
-# 指定字型檔案路徑
-font_path = "NotoSansTC-Regular.ttf"  # 確認此檔案在工作目錄中
-
-# 載入本地字型檔，並取得字型名稱
+font_path = "NotoSansTC-Regular.ttf"  # 確認此字型檔路徑正確
 if os.path.exists(font_path):
     fm.fontManager.addfont(font_path)
     font_prop = fm.FontProperties(fname=font_path)
@@ -24,19 +21,20 @@ else:
     st.error(f"找不到字型檔: {font_path}")
     CHINESE_FONT = "Noto Sans TC"  # fallback
 
-# 設定 Kaleido 全局參數
+# 配置 Kaleido 的預設參數與額外字型設定
 pio.kaleido.scope.default_font = CHINESE_FONT
 pio.kaleido.scope.default_format = "png"
 pio.kaleido.scope.default_width = 1200
 pio.kaleido.scope.default_height = 900
 pio.kaleido.scope.default_scale = 2
-
-# ★ 關鍵：告訴 Kaleido 額外字型的對應關係
 pio.kaleido.scope.extra_fonts = {CHINESE_FONT: font_path}
 
+# ============================
+# 系統字型檢查與安裝
+# ============================
 
 def setup_chinese_font():
-    """系統級字體檢查與安裝"""
+    """檢查系統中文字型是否可用，若無則嘗試安裝"""
     try:
         test_fig, ax = plt.subplots()
         ax.text(0.5, 0.5, "中文測試", fontfamily=CHINESE_FONT)
@@ -47,17 +45,19 @@ def setup_chinese_font():
         os.system('apt-get update && apt-get install -y --force-yes fonts-noto-cjk')
         st.experimental_rerun()
 
+# ============================
+# 資料讀取
+# ============================
+
 def load_data(uploaded_file):
-    """数据加载函数"""
+    """從 CSV 檔案中讀取資料"""
     try:
         df = pd.read_csv(uploaded_file, encoding='utf-8').dropna(how='all')
-        
         if '姓名' not in df.columns:
-            raise ValueError("CSV文件中缺少必要欄位：姓名")
-            
+            raise ValueError("CSV檔案中缺少必要欄位：姓名")
         numeric_columns = []
         potential_numeric = ['平均', '總分', '國文', '英文', '數學', '自科', 
-                           '社會', '地理', '歷史', '公民', '物理', '化學', '生物']
+                             '社會', '地理', '歷史', '公民', '物理', '化學', '生物']
         for col in potential_numeric:
             if col in df.columns:
                 try:
@@ -65,17 +65,19 @@ def load_data(uploaded_file):
                     numeric_columns.append(col)
                 except:
                     pass
-                    
         if not numeric_columns:
-            raise ValueError("CSV文件中未找到有效的數值欄位")
-            
+            raise ValueError("CSV檔案中未找到有效的數值欄位")
         return df, numeric_columns
     except Exception as e:
         st.error(f"數據加載錯誤：{str(e)}")
         return None, None
 
+# ============================
+# 生成雷達圖
+# ============================
+
 def create_radar_chart(df, selected_rows, selected_columns, student_name=""):
-    """生成雷達圖"""
+    """生成雷達圖 (包含圖例、極座標軸等設定)"""
     fig = go.Figure()
     line_styles = ['solid', 'dot', 'dash', 'longdash', 'dashdot']
     colors = ['#1F77B4', '#FF7F0E', '#2CA02C', '#D62728', '#9467BD']
@@ -131,8 +133,12 @@ def create_radar_chart(df, selected_rows, selected_columns, student_name=""):
     )
     return fig
 
+# ============================
+# 生成圖片
+# ============================
+
 def generate_radar_image(fig):
-    """生成圖片 (強制重置配置與更新字型設定)"""
+    """生成圖片 (重設 Kaleido 配置與更新字型設定)"""
     try:
         # 重設 kaleido 配置（含 extra_fonts）
         pio.kaleido.scope.default_font = CHINESE_FONT
@@ -181,6 +187,10 @@ def generate_radar_image(fig):
         st.error(f"圖片生成失敗: {str(e)}")
         return None
 
+# ============================
+# 主程式
+# ============================
+
 def main():
     st.title('學生成績雷達圖產生器')
     st.markdown("""
@@ -195,7 +205,7 @@ def main():
     </style>
     """, unsafe_allow_html=True)
 
-    setup_chinese_font()  # 檢查字型
+    setup_chinese_font()  # 檢查中文字型
 
     uploaded_file = st.file_uploader("上傳CSV檔案", type=['csv'])
     if uploaded_file is not None:
@@ -232,6 +242,7 @@ def main():
                 target_students = [s for s in all_students if s not in current_comparison]
                 for idx, student in enumerate(target_students):
                     try:
+                        # 動態組合比較組（比較基準加上當前學生）
                         fig = create_radar_chart(
                             df, 
                             current_comparison + [student],
@@ -243,14 +254,14 @@ def main():
                             image_bytes[student] = img
                     except Exception as e:
                         st.error(f"生成 {student} 圖表失敗：{str(e)}")
-                    progress_bar.progress((idx+1)/len(target_students))
+                    progress_bar.progress((idx + 1) / len(target_students))
                 if image_bytes:
-    st.write("### 圖表預覽")
-    cols = st.columns(3)
-    for i, (student, img) in enumerate(image_bytes.items()):
-        with cols[i % 3]:
-            st.image(img, use_container_width=True)
-            st.caption(f"{student} 比較圖")
+                    st.write("### 圖表預覽")
+                    cols = st.columns(3)
+                    for i, (student, img) in enumerate(image_bytes.items()):
+                        with cols[i % 3]:
+                            st.image(img, use_container_width=True)
+                            st.caption(f"{student} 比較圖")
                     zip_buffer = BytesIO()
                     with ZipFile(zip_buffer, "w") as zip_file:
                         for student, img in image_bytes.items():
@@ -265,4 +276,4 @@ def main():
                     )
 
 if __name__ == "__main__":
-    main()
+    main()