Taak: je werd gevraagd om een code review te doen van een nieuwe functie, waarvan de essentie is dat je een tour kunt boeken (bestaande uit een vliegticket en een hotel). Functies voor het afzonderlijk boeken van het vliegticket en de tour bestaan al, er wordt verwacht dat deze taak een uniforme methode biedt die de hele tour boekt of geen van de componenten.

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

Основные моменты для ревью:

• Использовать транзакцию, которая охватывает оба бронирования.
• Обработать возможные ошибки каждого шага, чтобы при неудаче откатить изменения.
• Проверить, что вызовы к существующим методам бронирования авиабилета и отеля поддерживают откат или отмену.
• Логировать процесс для отладки и аудита.

Пример на C# с использованием транзакции (псевдокод):

using(var transaction = dbContext.Database.BeginTransaction())
{
    try
    {
        BookFlight(flightDetails);
        BookHotel(hotelDetails);
        transaction.Commit();
    }
    catch(Exception)
    {
        transaction.Rollback();
        throw;
    }
}

Если методы бронирования не поддерживают транзакции, нужно реализовать компенсационные действия (отмена бронирования), чтобы при ошибке откатить уже сделанные брони.