Sobes.tech
Middle+

Există un proces de suport pentru clienți. În cadrul acestuia, există un proces de atribuire a tichetelor de suport către un operator. Un operator nu poate avea mai mult de 5 tichete simultan. TABELUL Tickets ( Id int PK, -- alte coloane ) TABELUL Operators ( Id int PK, -- alte coloane ) -- mapare operator-tichet pentru tichetele active. -- La închiderea tichetei, maparea este eliminată. TABELUL OperatorsTicket ( TicketId int PK, OperatorId int NULL ) // în cadrul unei tranzacții public void TryAssignOperatorToTicket(int ticketId) { var avalibleOperator = _db.GetOperatorsWithTicketsCount(maxTickets: 4) .OrderBy(o => o.TicketsCount).FirstOrDefault(); if(avalibleOperator is not null) _db.AssignTicketToOperator(ticketId, avalibleOperator.Id); else _db.AssignTicketToOperator(ticketId, null); }

sobes.tech AI

Răspuns de la AI

В текущей реализации есть риск гонки при параллельном назначении тикетов операторам, особенно если несколько потоков одновременно пытаются назначить тикеты. Это может привести к превышению лимита в 5 тикетов на оператора.

Чтобы избежать этого, можно:

  1. Использовать блокировку на уровне базы данных — например, SELECT ... FOR UPDATE при выборке операторов с количеством тикетов.
  2. Внести ограничение на уровне базы данных, например, триггер или ограничение CHECK, чтобы не допустить более 5 активных тикетов на оператора.
  3. Внести дополнительное поле в таблицу OperatorsTicket, чтобы отслеживать статус тикета (активен/закрыт), и учитывать только активные.

Пример улучшенного подхода с использованием транзакции и блокировки:

public void TryAssignOperatorToTicket(int ticketId)
{
    using var transaction = _db.BeginTransaction();

    // Блокируем операторов с количеством тикетов меньше 5
    var availableOperators = _db.Query<Operator>(@"
        SELECT o.Id, COUNT(ot.TicketId) as TicketsCount
        FROM Operators o
        LEFT JOIN OperatorsTicket ot ON o.Id = ot.OperatorId
        GROUP BY o.Id
        HAVING COUNT(ot.TicketId) < 5
        ORDER BY TicketsCount
        FOR UPDATE
    ");

    var availableOperator = availableOperators.FirstOrDefault();

    if (availableOperator != null)
        _db.AssignTicketToOperator(ticketId, availableOperator.Id);
    else
        _db.AssignTicketToOperator(ticketId, null);

    transaction.Commit();
}

Такой подход гарантирует, что в момент выбора операторов они будут заблокированы для других транзакций, что предотвращает превышение лимита.