如何重构验证方法？清理重复代码的实现方案咨询

兄弟，这种被重复代码淹没的痛苦我太懂了！把日志记录和异常抛出的逻辑抽成通用方法绝对是清理冗余的绝佳方案，我来给你拆解具体怎么落地。

把重复的日志打印+异常抛出逻辑集中到一个通用模块里，业务代码只需要传入关键参数（错误信息、异常类型、可选的异常原因）就能完成操作，彻底告别复制粘贴。

示例实现（Java版） #

假设你用SLF4J做日志框架，我们可以写一个静态工具类：

import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

public class ExceptionHandlerUtil {
    private static final Logger logger = LoggerFactory.getLogger(ExceptionHandlerUtil.class);

    // 基础版：记录错误日志并抛出指定异常
    public static <T extends Exception> void throwWithLog(String errorMsg, Class<T> exceptionType) throws T {
        // 第一步：打印错误日志
        logger.error(errorMsg);
        
        // 第二步：反射创建并抛出目标异常
        try {
            T exception = exceptionType.getConstructor(String.class).newInstance(errorMsg);
            throw exception;
        } catch (Exception e) {
            // 兜底处理：如果反射失败，抛出RuntimeException
            throw new RuntimeException("Failed to instantiate exception", e);
        }
    }

    // 增强版：支持传入原始异常作为触发原因
    public static <T extends Exception> void throwWithLog(String errorMsg, Class<T> exceptionType, Throwable cause) throws T {
        logger.error(errorMsg, cause);
        
        try {
            T exception = exceptionType.getConstructor(String.class, Throwable.class).newInstance(errorMsg, cause);
            throw exception;
        } catch (Exception e) {
            throw new RuntimeException("Failed to instantiate exception", e);
        }
    }
}

业务代码里怎么用？ #

原来的重复代码：

try {
    // 业务逻辑：比如读取配置文件
    loadConfig("config.properties");
} catch (IOException e) {
    logger.error("配置文件加载失败: {}", e.getMessage());
    throw new BusinessException("配置加载出错", e);
}

现在简化成：

try {
    loadConfig("config.properties");
} catch (IOException e) {
    ExceptionHandlerUtil.throwWithLog("配置文件加载失败", BusinessException.class, e);
}

示例实现（Python版） #

如果是Python项目，用装饰器或者通用函数都很方便：

import logging

logger = logging.getLogger(__name__)

def throw_with_log(error_msg: str, exception_cls, cause=None):
    # 记录日志（带异常栈信息）
    if cause:
        logger.error(error_msg, exc_info=cause)
    else:
        logger.error(error_msg)
    
    # 抛出异常并保留因果链
    if cause:
        raise exception_cls(error_msg) from cause
    else:
        raise exception_cls(error_msg)

业务代码简化示例： #

try:
    with open("data.csv", "r") as f:
        f.read()
except FileNotFoundError as e:
    throw_with_log("数据文件不存在", ValueError, e)

额外优化建议 #

• 可以扩展日志级别：比如新增warnWithLog方法，处理非致命错误场景
• 确保自定义异常类有对应的构造方法（带String或String+Throwable参数），避免反射/实例化失败
• 如果是Spring这类框架，也可以结合@ControllerAdvice做全局异常处理，但业务层的重复逻辑用工具类更灵活

内容的提问来源于stack exchange，提问作者Saad