Когда использовать статические классы в C# и почему они не всегда плохи?

Короткий ответ: используйте статические классы только для утилитарных функций, не имеющих состояния и зависимостей

Статические классы в C# — это мощный инструмент, но их легко использовать неправильно. Главное правило: статический класс должен быть набором чистых функций, которые не хранят состояние и не зависят от внешних сервисов.

Давайте разберём ваш пример с MathHelper. Это идеальный случай для статического класса. Почему?

• Методы не зависят от состояния объекта — результат определяется только входными параметрами
• Нет необходимости в наследовании или полиморфизме
• Нет внешних зависимостей (баз данных, файлов, веб-сервисов)
• Функции математически детерминированы (один и тот же ввод → один и тот же вывод)

Вот пример, где нельзя использовать статический класс:

// ❌ ПЛОХО: Статический класс с зависимостью от БД
public static class UserService
{
    private static string _connectionString = "Server=...";
    
    public static User GetUser(int id)
    {
        // Невозможно заменить БД на мок для тестов!
        using var connection = new SqlConnection(_connectionString);
        // ...
    }
}

Такой код невозможно протестировать изолированно. Правильный подход — интерфейс + DI:

// ✅ ХОРОШО: Интерфейс с реализацией
public interface IUserService
{
    User GetUser(int id);
}

public class UserService : IUserService
{
    private readonly IDbConnection _connection;
    
    public UserService(IDbConnection connection)
    {
        _connection = connection;
    }
    
    public User GetUser(int id)
    {
        // Легко подменить connection в тестах
    }
}

Когда статические классы — это правильно?

• Утилитарные функции: конвертеры, валидаторы, расширения (extension methods)
• Константы и настройки: public static class AppSettings с константами
• Фабричные методы: простые фабрики без состояния
• Методы расширения: они обязаны быть статическими

Статические классы vs Синглтон

Синглтон — это обычный класс, но с одним экземпляром. Разница в том, что синглтон может реализовывать интерфейсы и быть заменён в тестах. Статический класс — нет.

// Синглтон (можно тестировать через интерфейс)
public class Logger : ILogger
{
    private static readonly Lazy<Logger> _instance = new(() => new Logger());
    public static Logger Instance => _instance.Value;
    
    private Logger() { }
    public void Log(string message) { /* ... */ }
}

// Статический класс (нельзя подменить)
public static class Logger
{
    public static void Log(string message) { /* ... */ }
}

Итог: Используйте статические классы для операций, которые:

1. Не имеют состояния (или используют только readonly-поля)
2. Не зависят от внешних сервисов
3. Не требуют замены в тестах
4. Являются "чистыми" функциями

Во всех остальных случаях — предпочитайте обычные классы с интерфейсами и внедрением зависимостей.

public static class Program
{
    public static void Main(string[] args)
    {
        var host = CreateHostBuilder(args).Build();
        host.Run();
    }
    
    private static IHostBuilder CreateHostBuilder(string[] args) => 
        Host.CreateDefaultBuilder(args)
            .ConfigureServices(...);
}

public static class CountryCache
{
    private static readonly Dictionary<string, string> _countries = new()
    {
        ["US"] = "United States",
        ["RU"] = "Russia"
    };
    
    public static string GetName(string code) => 
        _countries.TryGetValue(code, out var name) ? name : "Unknown";
}