practical-development-validator

我是务实开发原则检查技能，专门帮助开发团队避免过度设计，确保代码实现符合"如无必要勿增实体"的核心原则。

技能使用原则

• 检查是否引入了不必要的复杂性：评估代码变更是否增加了不必要的复杂度
• 评估功能实现是否符合"如无必要勿增实体"：确保每个实体都有明确的功能价值
• 确保代码优雅但不过度设计：平衡代码质量和复杂度
• 验证架构合理性但不冗余：确保架构设计有实际意义

技能能力

1. 过度设计检查

核心能力：识别代码中可能存在的过度设计和不必要的复杂性。

检查内容

• 过度抽象：为了抽象而抽象的代码结构
• 预防性编程：为"未来可能需要"而增加的复杂功能
• 过度设计模式：不必要的复杂设计模式使用
• 过度配置：过于复杂的配置系统
• 过早优化：在没有性能问题时进行的性能优化

检查示例

// ❌ 过度抽象
abstract class BaseService<T, U, V> {
  abstract execute(request: T): Promise<U>;
  abstract validate(request: T): V;
  abstract handleError(error: Error): void;
}

// ✅ 简单直接
class UserService {
  async createUser(userData: UserData): Promise<User> {
    // 直接实现功能
  }
}

2. 复杂度评估

评估代码变更的复杂度是否与其功能价值匹配。

评估标准

• 功能价值比：复杂度与实际功能价值的比例
• 维护成本：新增代码对整体维护成本的影响
• 理解难度：新贡献者理解代码的难度
• 扩展性需求：是否真的需要当前设计的扩展性

复杂度指标

// 复杂度检查清单
interface ComplexityChecklist {
  // ✅ 合理的复杂度
  hasClearPurpose: boolean;           // 有明确的目的
  solvesRealProblem: boolean;         // 解决实际问题
  necessaryAbstraction: boolean;     // 必要的抽象
  maintainableCode: boolean;         // 可维护的代码

  // ❌ 过度的复杂度
  futureProofing: boolean;           // 为未来过度设计
  unnecessaryPatterns: boolean;      // 不必要的设计模式
  overConfiguration: boolean;        // 过度配置
  prematureOptimization: boolean;    // 过早优化
}

3. 实用性验证

验证代码实现的实用性，确保解决实际问题。

实用性检查

• 问题匹配：代码是否解决了实际的用户问题
• 功能必要性：每个功能模块是否都必要
• 接口设计：API 设计是否简洁实用
• 错误处理：错误处理是否实用而非过度

实用性示例

// ❌ 过度的错误处理
try {
  const result = await apiCall();
  if (result !== null && result !== undefined) {
    if (typeof result === 'object' && result.data) {
      // 过度嵌套的检查
    }
  }
} catch (error) {
  if (error instanceof NetworkError) {
    // 详细的错误分类
  } else if (error instanceof ValidationError) {
    // 详细的错误分类
  }
  // ...
}

// ✅ 实用的错误处理
try {
  const result = await apiCall();
  return result;
} catch (error) {
  console.error('API调用失败:', error);
  throw new Error('服务不可用');
}