Библиотека шарписта | C#, F#, .NET, ASP.NET: “☝️ Не повторяйтесь Принцип DRY (Don't Repeat Yourself)

☝️ Не повторяйтесь

Принцип DRY (Don't Repeat Yourself) — один из самых известных в программировании. Но, как и любой принцип, его можно применять неправильно. Давайте разберёмся, когда следовать DRY, а когда от него отступить.

Признаки over-DRYing

• Функция с 5+ параметрами:

// Универсальная функция-монстр
public async Task<object> FetchDataAsync(
    string entityType, int? id, string filter,
    int pageSize, bool includeDeleted, params string[] includes) { }

// Специализированные методы
public async Task<List<User>> GetActiveUsersAsync(int page, int pageSize)
    => await _context.Users.Where(u => u.IsActive)
        .Skip(page * pageSize).Take(pageSize).ToListAsync();

• Generic Repository:

// Кажется умным, но бесполезен
public interface IRepository<T> where T : class
{
    Task<T> GetByIdAsync(int id);
    Task<IEnumerable<T>> GetAllAsync();
}

// Специфичный и полезный
public interface IUserRepository
{
    Task<User> GetByEmailAsync(string email);
    Task<User> GetWithOrdersAsync(int id);
}

Когда дублирование правильно

• Случайное совпадение:

// НЕ объединяйте! Разные бизнес-концепции
public string FormatUserName(User u) => $"{u.FirstName} {u.LastName}";
public string FormatProductName(Product p) => $"{p.Brand} {p.Model}";

• Разные контексты:

// UI: простая проверка
public bool ValidateEmail(string email) => email.Contains("@");

// API: строгая проверка
public bool ValidateEmail(string email) => 
    Regex.IsMatch(email, @"^[a-zA-Z0-9._%+-]+@[a-zA-Z0-9.-]+\.[a-zA-Z]{2,}$");

Повторите код три раза, прежде чем рефакторить:

Первый раз — пишите
Второй раз — терпите
Третий раз — рефакторьте

Дублирование в тестах — нормально:

[Fact]
public void CreateUser_ShouldSucceed()
{
    var user = new User { Name = "Alice", Age = 25 };
    Assert.Equal("Alice", _service.CreateUser(user).Name);
}

[Fact]
public void UpdateUser_ShouldSucceed()
{
    var user = new User { Name = "Alice", Age = 25 }; // OK
    // Читаемость важнее DRY
}

📍 Навигация: Вакансии • Задачи • Собесы

🐸Библиотека шарписта

#sharp_view