设计原则：SOLID、DRY 与 YAGNI

这一课你将能做到

1. 用判题平台真实业务讲清 SOLID 五条原则各自解决什么问题
2. 识别代码里的"重复"和"过度设计"，知道 DRY 与 YAGNI 的边界
3. 给一个违反 SOLID 的类做最小重构，让它符合 OCP 和 DIP

为什么要学

如果说整洁代码是"一行代码怎么写"，那设计原则就是"类和类怎么协作"。判题平台里 SubmissionService、JudgeEngine、NotificationSender、PracticeRecordRepository 之间的依赖关系、扩展点、复用方式，全都由这些原则决定。

面试几乎必考 SOLID，但光背定义没用——面试官会追问"举一个你项目里的例子"。这一课就给你备好例子。

先建立图像

S 单一职责   一个类只被一个角色驱动修改
O 开闭       对扩展开放，对修改关闭
L 里氏替换   子类必须能替换父类
I 接口隔离   别强迫实现方实现用不到的方法
D 依赖倒置   依赖抽象，不依赖具体

DRY    重复逻辑只写一次
YAGNI  用不到的别提前写

跟着做

步骤 1：违反 SRP 的"上帝类"

// 反例：SubmissionService 干了太多事
public class SubmissionService {
    public void submit(String code, long userId, long lessonId) { /* ... */ }
    public void judge(long submissionId) { /* ... */ }
    public void sendEmail(long userId) { /* ... */ }
    public void exportReport(LocalDate from, LocalDate to) { /* ... */ }
    public void updateVipExpire(long userId) { /* ... */ }
}

这个类会被"判题工程师、运营、报表开发、会员产品"四种角色修改，任何改动都可能影响别人。

重构：按角色拆成 SubmissionService、JudgeService、NotificationService、ReportService、MembershipService。每个类只对一个角色负责。

步骤 2：用 OCP 设计判题器

新增"Python 判题"时不改老代码：

public interface JudgeEngine {
    boolean supports(String language);
    JudgeResult judge(Submission submission);
}

@Component
public class JavaJudgeEngine implements JudgeEngine {
    public boolean supports(String language) { return "java".equals(language); }
    public JudgeResult judge(Submission s) { /* 编译、跑用例、对比输出 */ }
}

@Component
public class PythonJudgeEngine implements JudgeEngine {
    public boolean supports(String language) { return "python".equals(language); }
    public JudgeResult judge(Submission s) { /* ... */ }
}

@Service
public class JudgeDispatcher {
    private final List<JudgeEngine> engines;   // Spring 自动注入所有实现
    public JudgeDispatcher(List<JudgeEngine> engines) { this.engines = engines; }

    public JudgeResult dispatch(Submission submission) {
        return engines.stream()
                .filter(e -> e.supports(submission.getLanguage()))
                .findFirst()
                .orElseThrow(() -> new IllegalStateException("no engine for " + submission.getLanguage()))
                .judge(submission);
    }
}