Как проверяешь качество своей работы

Проверка качества работы: комплексный подход Java Developer

Качество кода — это не одна проверка, а многоуровневый процесс. За 10+ лет я выработал систему контроля, которая гарантирует надежность, производительность и поддерживаемость.

1. Статический анализ кода

Инструменты автоматизации

Не полагаюсь на человеческую память. Все проверяется автоматически:

// Плохо: выявлено SonarQube
public void processData(List<String> items) {
    for (int i = 0; i < items.size(); i++) { // используй enhanced for
        System.out.println(items.get(i));
    }
}

// Хорошо
public void processData(List<String> items) {
    items.forEach(System.out::println);
}

Мои инструменты:

• SonarQube/SonarLint — 300+ правил для Java (дублирование кода, сложность, безопасность)
• CheckStyle — стиль кода и соглашения
• SpotBugs — поиск потенциальных багов
• PMD — потенциальные проблемы программирования
• Archunit — проверка архитектуры (не нарушаются слои)
• Jacoco — покрытие кода тестами

2. Юнит-тесты и TDD

Тесты — это основа качества. Пишу тесты ДО кода:

// Сначала тест (RED)
@Test
void shouldCalculateTotalWithDiscount() {
    // Given
    Order order = new Order(List.of(
        new Item("Laptop", 1000),
        new Item("Mouse", 50)
    ));
    
    // When
    BigDecimal total = order.calculateTotal(0.1); // 10% скидка
    
    // Then
    assertEquals(new BigDecimal("945.00"), total);
}

// Затем минимальный код (GREEN)
public class Order {
    private List<Item> items;

    public BigDecimal calculateTotal(BigDecimal discountPercent) {
        BigDecimal subtotal = items.stream()
            .map(Item::getPrice)
            .reduce(BigDecimal.ZERO, BigDecimal::add);
        
        BigDecimal discount = subtotal.multiply(discountPercent);
        return subtotal.subtract(discount);
    }
}

Проверяю:

• Happy path (успешный сценарий)
• Edge cases (граничные случаи)
• Error handling (обработка ошибок)
• Null checks (null-безопасность)

@Test
void shouldThrowExceptionWhenOrderIsEmpty() {
    Order order = new Order(List.of());
    assertThrows(IllegalArgumentException.class, 
        () -> order.calculateTotal(BigDecimal.ZERO));
}

Метрики: Минимум 80-90% код-coverage (Jacoco), все критические пути покрыты.

3. Интеграционные тесты

Проверяю взаимодействие компонентов с реальной БД:

@SpringBootTest
class OrderServiceIntegrationTest {
    @Autowired
    private OrderService orderService;
    
    @Autowired
    private OrderRepository orderRepository;

    @Test
    void shouldPersistOrderToDatabase() {
        // Given
        Order order = new Order("order-123", BigDecimal.valueOf(100));

        // When
        Order saved = orderService.save(order);

        // Then
        Order found = orderRepository.findById(saved.getId()).orElseThrow();
        assertEquals("order-123", found.getNumber());
    }
}

4. Performance тестирование

// JMH для микробенчмарков
@BenchmarkMode(Mode.AverageTime)
public class OrderCalculationBenchmark {
    @Benchmark
    public BigDecimal calculateTotal() {
        return service.calculateTotal(items);
    }
}

5. Code Review (Self + Peer)

Самопроверка перед commit:

• Все тесты проходят
• Coverage > 80%
• Нет SonarQube issues
• Код отформатирован
• Архитектура не нарушена
• No hardcoded values
• Exception handling правильный

6. Логирование и Monitoring

private static final Logger logger = LoggerFactory.getLogger(OrderService.class);

public Order processOrder(String orderId) {
    logger.info("Processing order: {}", orderId);
    
    try {
        Order order = orderRepository.findById(orderId).orElseThrow();
        orderProcessor.process(order);
        return order;
    } catch (Exception e) {
        logger.error("Failed to process order: {}", orderId, e);
        throw new OrderProcessingException("Cannot process order", e);
    }
}

7. Security проверки

// Правильно: параметризованные запросы
String query = "SELECT * FROM users WHERE username = ?";
preparedStatement.setString(1, username);

// Spring Data безопасен по умолчанию
User findByUsername(String username);

Использую OWASP Dependency-Check и SonarQube Security rules.

8. CI/CD Pipeline

Автоматизирую все проверки:

• mvn clean compile
• mvn test
• mvn sonar:sonar
• mvn jacoco:report
• OWASP Dependency-Check

9. Метрики и KPI

• Defect Rate < 5 на 1000 строк
• Code Coverage >= 85%
• Cyclomatic Complexity <= 10 на метод
• Technical Debt < 5%
• Response Time < 200ms для 95%
• Error Rate < 0.1%

Качество — это система и привычка, а не случайное действие.