Middle+

Überprüfung: angewandter Code Die Klasse enthält die Logik zur Verarbeitung neuer Bestellungen. Sie extrahiert die Bestellungen und Positionen aus der Datenbank, führt dann einen HTTP-Aufruf an einen externen Microservice Stock durch, der die Produkte entsprechend den Bestellpositionen reserviert. Wenn alle Reservierungen erfolgreich sind, bestätigt der Microservice die Reservierung der Bestellung in Stock und markiert die Bestellung als verarbeitet. Der Microservice läuft in mehreren Instanzen, daher kann die Methode Process parallel ausgeführt werden. Um zu vermeiden, dass dieselben Zeilen verarbeitet werden, wird ein Mechanismus der pessimistischen Sperrung in PostgreSQL Select for update ... skip locked verwendet. Die Implementierung von IBus sendet Nachrichten an RabbitMq. ```csharp public sealed class PendingOrdersProcessor { public async Task PendingOrdersProcessor(ILogger logger, IBus bus, HttpStockService st { var env = Environment.GetEnvironmentVariable("ASPNETCORE_ENVIRONMENT"); var cs = _cs[env]; _db = new OrdersRepository(cs); _bus = bus; logger = logger; ``` ```csharp _stockService = stockService; } private readonly ILogger _logger; private readonly IOrdersRepository _db; private readonly IStockService _stockService; private readonly IBus _bus; private static readonly Dictionary<string, string> _cs = new { { "Development", "User ID=root;Password=myPassword;Host=localhost;Port=5432;Database=myDataBase" }, { "Production", "User ID=root;Password=myPassword;Host=prod-db;Port=5432;Database=" "Pooling=true;Min Pool Size=0;Max Pool Size=1;" } }; ```

sobes.tech KI

Antwort von AI

В представленном коде есть несколько проблем и возможностей для улучшения:

Инициализация зависимостей в конструкторе
- В коде конструктор PendingOrdersProcessor неявно инициализирует репозиторий и другие зависимости, используя переменную окружения. Лучше передавать все зависимости через параметры конструктора, чтобы повысить тестируемость и гибкость.
Потокобезопасность
- Поскольку метод Process может выполняться параллельно в нескольких экземплярах, важно убедиться, что доступ к общим ресурсам (например, к базе данных) корректно синхронизирован.
- Использование SELECT FOR UPDATE SKIP LOCKED — хорошая практика для пессимистичной блокировки и предотвращения обработки одних и тех же заказов.
Обработка ошибок и транзакции
- Важно оборачивать операции с базой и вызовы внешнего сервиса в транзакции, чтобы обеспечить атомарность.
- Если бронирование товаров не удалось, нужно корректно откатить изменения и не подтверждать заказ.
Логирование
- Логирование ошибок и важных событий поможет в диагностике и мониторинге.
Конфигурация
- Хранение строк подключения в словаре _cs с ключом окружения — неплохой подход, но стоит вынести конфигурацию в отдельный конфиг-файл или использовать менеджер конфигураций.
Пример улучшенного конструктора:

public sealed class PendingOrdersProcessor
{
    private readonly ILogger _logger;
    private readonly IOrdersRepository _db;
    private readonly IStockService _stockService;
    private readonly IBus _bus;

    public PendingOrdersProcessor(ILogger logger, IBus bus, IStockService stockService, IOrdersRepository ordersRepository)
    {
        _logger = logger ?? throw new ArgumentNullException(nameof(logger));
        _bus = bus ?? throw new ArgumentNullException(nameof(bus));
        _stockService = stockService ?? throw new ArgumentNullException(nameof(stockService));
        _db = ordersRepository ?? throw new ArgumentNullException(nameof(ordersRepository));
    }

    // Остальная логика обработки заказов
}

Таким образом, класс становится более модульным и удобным для тестирования.

Также стоит проверить, что пул подключений к БД настроен корректно (в примере для Production max pool size=1 — это может стать узким местом при параллельной обработке).

Zurück Als nächstes