消除数据库插入冗余：MySQL双插入方法优化技术问询

嘿，这种重复代码确实挺闹心的——明明核心逻辑完全一样，就差个参数和对应的SQL，却要维护两套，后续改起来还容易漏。我之前也碰到过类似情况，给你几个实用的解决方案，帮你把冗余干掉：

1. 合并方法，用可选参数兼容旧调用 #

最直接的方式就是把两个方法合并成一个，让PaymentRequest成为可选参数（不同语言实现方式略有不同，比如Java用重载，Python用默认值）。这样既保留了原有代码的兼容性，又把重复逻辑收拢到一处：

代码示例（Java为例） #

// 统一的核心插入方法
public void insertOrder(Order order, PaymentRequest paymentRequest) {
    // 根据是否有PaymentRequest选择对应的SQL
    String sql = paymentRequest != null ? SqlQuery.INSERT_ORDER_WITH_IOC : SqlQuery.INSERT_ORDER;
    
    try (Connection conn = getConnection()) {
        PreparedStatement stmt = conn.prepareStatement(sql);
        // 设置所有订单相关的公共参数
        stmt.setString(1, order.getId());
        stmt.setBigDecimal(2, order.getAmount());
        
        // 处理PaymentRequest的差异部分
        if (paymentRequest != null) {
            stmt.setString(3, paymentRequest.getId());
            stmt.setTimestamp(4, paymentRequest.getCreateTime());
        }
        
        stmt.executeUpdate();
    } catch (SQLException e) {
        log.error("插入订单失败", e);
        throw new OrderInsertException(e);
    }
}

// 为了兼容原有不带PaymentRequest的调用，加个重载方法（也可以让调用方直接传null）
public void insertOrder(Order order) {
    insertOrder(order, null);
}

2. 统一SQL语句，动态补充字段 #

既然两条SQL几乎一模一样，不如把它们合并成一条可动态调整的SQL，避免维护两条重复的语句。比如在sql.yml里用模板语法定义一条基础SQL，然后在代码里根据参数渲染：

SQL示例（用Freemarker模板语法举例） #

INSERT_ORDER: |
  INSERT INTO orders (id, amount
    <#if paymentRequest??>, payment_request_id, payment_request_time</#if>
  ) VALUES (?, ?
    <#if paymentRequest??>, ?, ?</#if>
  )

如果不用模板引擎，也可以在代码里简单拼接字符串（注意要避免SQL注入，用PreparedStatement的话参数位置对应好就行）。这样不管有没有PaymentRequest，都用同一条SQL模板，不用维护两个几乎一样的SQL常量。

3. 提取公共逻辑到私有方法 #

如果暂时不想动方法结构，也可以把两个方法里完全一致的代码抽成一个私有工具方法，让两个公共方法都调用它，只在各自入口处理参数差异：

代码示例 #

// 原有不带PaymentRequest的方法，保持兼容
public void insertOrder(Order order) {
    insertOrderInternal(order, null);
}

// 新的带PaymentRequest的方法
public void insertOrderWithIOC(Order order, PaymentRequest paymentRequest) {
    insertOrderInternal(order, paymentRequest);
}

// 抽出来的核心私有方法，所有公共逻辑都在这
private void insertOrderInternal(Order order, PaymentRequest paymentRequest) {
    String sql = paymentRequest != null ? SqlQuery.INSERT_ORDER_WITH_IOC : SqlQuery.INSERT_ORDER;
    
    // 下面是原来两个方法里完全重复的执行逻辑
    try (Connection conn = getConnection()) {
        PreparedStatement stmt = conn.prepareStatement(sql);
        stmt.setString(1, order.getId());
        stmt.setBigDecimal(2, order.getAmount());
        
        if (paymentRequest != null) {
            stmt.setString(3, paymentRequest.getId());
            stmt.setTimestamp(4, paymentRequest.getCreateTime());
        }
        
        stmt.executeUpdate();
    } catch (SQLException e) {
        log.error("插入订单失败", e);
        throw new OrderInsertException(e);
    }
}

这样后续修改核心逻辑时，只需要改这一个私有方法，不用两处都改，避免了遗漏。

4. 用建造者模式应对未来扩展 #

如果以后可能还要加更多可选参数（比如物流信息、优惠券之类的），建造者模式会是更优雅的选择。你可以创建一个插入参数的Builder，让调用方按需设置，最后统一执行：

代码示例 #

public class OrderInsertBuilder {
    private Order order;
    private PaymentRequest paymentRequest;
    // 以后加新参数直接在这里加对应的属性和set方法就行

    public OrderInsertBuilder setOrder(Order order) {
        this.order = order;
        return this;
    }

    public OrderInsertBuilder setPaymentRequest(PaymentRequest paymentRequest) {
        this.paymentRequest = paymentRequest;
        return this;
    }

    public void execute() {
        // 统一的插入逻辑，和前面的示例类似
        String sql = paymentRequest != null ? SqlQuery.INSERT_ORDER_WITH_IOC : SqlQuery.INSERT_ORDER;
        
        try (Connection conn = getConnection()) {
            PreparedStatement stmt = conn.prepareStatement(sql);
            stmt.setString(1, order.getId());
            stmt.setBigDecimal(2, order.getAmount());
            
            if (paymentRequest != null) {
                stmt.setString(3, paymentRequest.getId());
                stmt.setTimestamp(4, paymentRequest.getCreateTime());
            }
            
            stmt.executeUpdate();
        } catch (SQLException e) {
            log.error("插入订单失败", e);
            throw new OrderInsertException(e);
        }
    }
}

// 调用方式也很灵活
// 原有调用（不带PaymentRequest）
new OrderInsertBuilder().setOrder(order).execute();
// 新调用（带PaymentRequest）
new OrderInsertBuilder().setOrder(order).setPaymentRequest(paymentRequest).execute();

这种方式扩展性极强，后续加新参数完全不用修改原有方法的签名，也不会产生新的冗余代码。

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