Test Management System (TMS)
從 Legacy Code 到實驗室核心基礎設施

錯誤的寫法：高耦合、缺乏抽象、Raw SQL 拼接與無事務控制（維護極難）

# ❌ 舊 TMS legacy_views.py 中的代碼片段
import pymysql

def upload_test_report(request):
    # 直接在請求處理器中硬編碼處理業務邏輯
    project_id = request.POST.get('project_id')
    tester_name = request.POST.get('tester')
    raw_data = request.POST.get('data') # 格式例如 "test_case_1,PASS;test_case_2,FAIL"

    # 1. 直接連接資料庫，缺乏連線池管理
    conn = pymysql.connect(host='10.120.x.x', user='admin', password='password', db='tms_db')
    cursor = conn.cursor()

    # 2. 拼接 SQL，存在 SQL 注入風險，且無任何錯誤處理
    # 如果 raw_data 解析到一半失敗，前面寫入的資料會卡在髒狀態
    try:
        sql = f"INSERT INTO test_records (project_id, tester) VALUES ('{project_id}', '{tester_name}')"
        cursor.execute(sql)
        record_id = cursor.lastrowid
        
        # 3. 複雜的字串解析與高耦合插入
        for item in raw_data.split(';'):
            case_name, result = item.split(',')
            # 如果 result 格式不正確，會在這裡崩潰，但前面的 record_id 已經寫入
            cursor.execute(f"INSERT INTO case_results (record_id, name, status) VALUES ({record_id}, '{case_name}', '{result}')")
        
        conn.commit()
        return {"status": "success"}
    except Exception as e:
        # 錯誤拋出不完整，且沒有對 conn 進行 rollback，可能造成 DB Lock
        return {"status": "error", "message": str(e)}
    finally:
        cursor.close()
        conn.close()

改良後寫法：Repository-Service 模式，加入事務保護與強型別驗證

# 🚀 重構後的 modern_services.py
from django.db import transaction
from typing import Dict, List, Tuple

class TestRecordRepository:
    """負責資料庫存取的持久層 (Decoupled Data Layer)"""
    @staticmethod
    def create_record(project_id: int, tester: str) -> int:
        # 使用 Django ORM 或參數化 SQL，由底層連線池代管
        record = TestRecord.objects.create(project_id=project_id, tester=tester)
        return record.id

    @staticmethod
    def bulk_insert_results(record_id: int, results: List[Tuple[str, str]]):
        # 批量寫入以優化效能
        objs = [CaseResult(record_id=record_id, name=name, status=status) for name, status in results]
        CaseResult.objects.bulk_create(objs)

class TestReportService:
    """業務邏輯層 (Business Logic Layer)"""
    def __init__(self, repo=TestRecordRepository()):
        self.repo = repo

    def parse_raw_data(self, raw_data: str) -> List[Tuple[str, str]]:
        """將原始資料解析為結構化 Tuple，包含格式驗證 (Validate first)"""
        parsed = []
        if not raw_data:
            raise ValueError("數據不可為空")
            
        for index, item in enumerate(raw_data.split(';')):
            parts = item.split(',')
            if len(parts) != 2:
                raise ValueError(f"第 {index+1} 筆數據格式錯誤: '{item}'")
            name, result = parts[0].strip(), parts[1].strip()
            if result not in ['PASS', 'FAIL', 'BLOCK']:
                raise ValueError(f"無效的測試結果狀態: '{result}'")
            parsed.append((name, result))
        return parsed

    def process_report_upload(self, project_id: int, tester: str, raw_data: str) -> Dict[str, str]:
        # 使用 transaction.atomic 確保 ACID 事務完整性
        try:
            results_to_insert = self.parse_raw_data(raw_data)
            
            with transaction.atomic():
                record_id = self.repo.create_record(project_id, tester)
                self.repo.bulk_insert_results(record_id, results_to_insert)
                
            return {"status": "success", "record_id": str(record_id)}
        except ValueError as ve:
            # 捕獲型別或格式錯誤，返回清晰說明，不拋出 stack trace 洩密
            return {"status": "error", "message": f"驗證失敗: {str(ve)}"}
        except Exception as e:
            # 記錄 log 供監控系統使用
            logger.error(f"上傳測試報告時發生未知系統錯誤: {e}", exc_info=True)
            return {"status": "error", "message": "系統內部錯誤，請聯繫管理員"}