# ? 不推薦：命名不規(guī)范，縮進混亂
def Calculate_Total(item_list):
  total=0
  for i in item_list:
      total+=i['price']
  return total

# ? 推薦：清晰的命名和格式
def calculate_total(items):
    """計算商品總價"""
    total = 0
    for item in items:
        total += item['price']
    return total

# ? 不推薦：含義模糊
def process(d):
    r = []
    for i in d:
        if i['s'] > 100:
            r.append(i)
    return r

# ? 推薦：意圖清晰
def filter_high_value_orders(orders):
    """篩選金額超過 100 的訂單"""
    high_value_orders = []
    for order in orders:
        if order['amount'] > 100:
            high_value_orders.append(order)
    return high_value_orders

# ? 危險：未處理空列表和除零
def calculate_average(numbers):
    return sum(numbers) / len(numbers)

# ? 安全：完善的邊界處理
def calculate_average(numbers):
    """計算平均值，處理邊界情況"""
    if not numbers:
        return 0.0
    
    if not all(isinstance(n, (int, float)) for n in numbers):
        raise ValueError("所有元素必須是數(shù)字")
    
    return sum(numbers) / len(numbers)

# ? 不推薦：捕獲過于寬泛
def read_config(file_path):
    try:
        with open(file_path) as f:
            return json.load(f)
    except:  # 捕獲所有異常
        return {}

# ? 推薦：精確捕獲，明確處理
def read_config(file_path):
    """讀取配置文件，返回字典"""
    try:
        with open(file_path, 'r', encoding='utf-8') as f:
            return json.load(f)
    except FileNotFoundError:
        logger.warning(f"配置文件不存在: {file_path}")
        return {}
    except json.JSONDecodeError as e:
        logger.error(f"配置文件格式錯誤: {e}")
        raise
    except Exception as e:
        logger.error(f"讀取配置文件失敗: {e}")
        raise

# ? 低效：O(n2) 復(fù)雜度
def find_duplicates(items):
    duplicates = []
    for i in range(len(items)):
        for j in range(i + 1, len(items)):
            if items[i] == items[j] and items[i] not in duplicates:
                duplicates.append(items[i])
    return duplicates

# ? 高效：O(n) 復(fù)雜度
def find_duplicates(items):
    """查找重復(fù)元素"""
    seen = set()
    duplicates = set()
    
    for item in items:
        if item in seen:
            duplicates.add(item)
        else:
            seen.add(item)
    
    return list(duplicates)

# ? 危險：可能導(dǎo)致文件句柄泄漏
def process_large_file(file_path):
    f = open(file_path)
    data = f.read()  # 如果這里拋異常，文件不會關(guān)閉
    return data.split('\n')

# ? 安全：自動管理資源
def process_large_file(file_path):
    """處理大文件，逐行讀取節(jié)省內(nèi)存"""
    with open(file_path, 'r', encoding='utf-8') as f:
        for line in f:  # 使用迭代器，不一次性加載全部
            yield line.strip()

# ? 危險：SQL 注入風(fēng)險
def get_user(username):
    query = f"SELECT * FROM users WHERE username = '{username}'"
    return db.execute(query)

# ? 安全：參數(shù)化查詢
def get_user(username):
    """安全地查詢用戶信息"""
    if not isinstance(username, str) or len(username) > 50:
        raise ValueError("無效的用戶名")
    
    query = "SELECT * FROM users WHERE username = %s"
    return db.execute(query, (username,))

# ? 不安全：缺少權(quán)限檢查
def delete_user(user_id):
    User.objects.filter(id=user_id).delete()

# ? 安全：完善的權(quán)限驗證
def delete_user(user_id, operator):
    """刪除用戶，需要管理員權(quán)限"""
    if not operator.is_admin:
        raise PermissionError("需要管理員權(quán)限")
    
    user = User.objects.filter(id=user_id).first()
    if not user:
        raise ValueError("用戶不存在")
    
    if user.is_system_user:
        raise ValueError("不能刪除系統(tǒng)用戶")
    
    user.delete()
    logger.info(f"管理員 {operator.username} 刪除了用戶 {user.username}")

# ? 職責(zé)混亂：一個類做太多事
class UserManager:
    def create_user(self, data):
        # 驗證數(shù)據(jù)
        if not data.get('email'):
            raise ValueError("郵箱必填")
        
        # 發(fā)送郵件
        send_email(data['email'], "歡迎注冊")
        
        # 保存數(shù)據(jù)庫
        user = User(**data)
        user.save()
        
        # 記錄日志
        logger.info(f"創(chuàng)建用戶: {data['email']}")

# ? 職責(zé)分離：每個類專注一件事
class UserValidator:
    @staticmethod
    def validate(data):
        if not data.get('email'):
            raise ValueError("郵箱必填")
        return True

class UserRepository:
    @staticmethod
    def create(data):
        user = User(**data)
        user.save()
        return user

class UserService:
    def __init__(self, validator, repository, notifier):
        self.validator = validator
        self.repository = repository
        self.notifier = notifier
    
    def create_user(self, data):
        self.validator.validate(data)
        user = self.repository.create(data)
        self.notifier.send_welcome_email(user.email)
        logger.info(f"創(chuàng)建用戶: {user.email}")
        return user

# ? 難以測試：硬編碼依賴
class OrderService:
    def process_order(self, order_id):
        order = Database().get_order(order_id)  # 硬編碼數(shù)據(jù)庫
        PaymentGateway().charge(order.amount)   # 硬編碼支付
        EmailService().send(order.user_email)   # 硬編碼郵件

# ? 易于測試：依賴注入
class OrderService:
    def __init__(self, db, payment, email):
        self.db = db
        self.payment = payment
        self.email = email
    
    def process_order(self, order_id):
        order = self.db.get_order(order_id)
        self.payment.charge(order.amount)
        self.email.send(order.user_email)

# 測試時可以注入 Mock 對象
def test_process_order():
    mock_db = Mock()
    mock_payment = Mock()
    mock_email = Mock()
    
    service = OrderService(mock_db, mock_payment, mock_email)
    service.process_order(123)
    
    mock_payment.charge.assert_called_once()

# ? 好的測試：清晰、獨立、覆蓋邊界
def test_calculate_discount():
    """測試折扣計算邏輯"""
    # 正常情況
    assert calculate_discount(100, 0.1) == 10.0
    
    # 邊界情況
    assert calculate_discount(0, 0.1) == 0.0
    assert calculate_discount(100, 0) == 0.0
    assert calculate_discount(100, 1.0) == 100.0
    
    # 異常情況
    with pytest.raises(ValueError):
        calculate_discount(-100, 0.1)
    
    with pytest.raises(ValueError):
        calculate_discount(100, 1.5)

def calculate_compound_interest(principal, rate, time, frequency=1):
    """
    計算復(fù)利終值
    
    Args:
        principal (float): 本金
        rate (float): 年利率（小數(shù)形式，如 0.05 表示 5%）
        time (int): 投資年限
        frequency (int): 每年復(fù)利次數(shù)，默認(rèn)為 1
    
    Returns:
        float: 復(fù)利終值
    
    Raises:
        ValueError: 當(dāng)參數(shù)為負(fù)數(shù)時
    
    Examples:
        >>> calculate_compound_interest(1000, 0.05, 10)
        1628.89
    """
    if principal < 0 or rate < 0 or time < 0:
        raise ValueError("參數(shù)不能為負(fù)數(shù)")
    
    return principal * (1 + rate / frequency) ** (frequency * time)