SRP Analyzer

Analyze C# classes for Single Responsibility Principle violations

Hide framework services

C# Code Editor

537 chars

public class Processor { private readonly IOrderService _orderService; private readonly IEmailService _emailService; public Processor(IOrderService orderService, IEmailService emailService) { _orderService = orderService; _emailService = emailService; } public void ProcessOrder(Order order) { _orderService.Process(order); } public void SendConfirmationEmail(Order order) { _emailService.SendEmail(order.CustomerEmail, "Your order has been processed."); } }

public class Processor
{
    private readonly IOrderService _orderService;
    private readonly IEmailService _emailService;

    public Processor(IOrderService orderService, IEmailService emailService)
    {
        _orderService = orderService;
        _emailService = emailService;
    }

    public void ProcessOrder(Order order)
    {
        _orderService.Process(order);
    }

    public void SendConfirmationEmail(Order order)
    {
        _emailService.SendEmail(order.CustomerEmail, "Your order has been processed.");
    }
}

Potential SRP Violation Detected

This class appears to have multiple responsibilities. Each dependency is used in different methods, suggesting the class handles multiple concerns. Consider splitting into smaller, focused classes.

Responsibility Breakdown:

Uses IOrderService:

ProcessOrder

Uses IEmailService:

SendConfirmationEmail

SRP Analyzer

How it works

C# Code Editor

Potential SRP Violation Detected

Dependencies