32. 커맨드 재전송 시 서버측 중복 실행 방지(dedup) 부재

// ============================================================================
// 시나리오 (서버-클라)
// ----------------------------------------------------------------------------
// 클라이언트는 서버에 커맨드(아이템 사용, 스킬 시전 등)를 보낸 뒤 ACK 를
// 기다린다. 일정 시간 안에 ACK 를 받지 못하면 클라이언트는 같은 커맨드를
// 동일한 CommandId 로 재전송한다.
//
// 서버는 각 커맨드를 수신하는 즉시 실행하고 ACK 를 돌려준다. 서버가 보낸
// ACK 자체가 네트워크에서 유실될 수도 있다 — 이 경우 서버는 커맨드를 이미
// 정상 실행했는데도 클라이언트는 ACK 를 받지 못해 재전송한다.
//
// 요구사항:
//   - 같은 CommandId 를 가진 커맨드가 재전송되더라도, 서버는 그 효과를
//     정확히 한 번만 적용해야 한다.
//   - 재전송에 대해서도 서버는 ACK 를 응답해야 한다.
//
// 과제: (A)(B) 지점을 검토해 위 요구사항이 깨지는 경로를 찾고 수정하라.
// ============================================================================

#include <cstdint>
#include <string>

struct UseItemCommand
{
    int64_t  commandId;
    uint64_t playerId;
    int      itemId;
};

class Inventory
{
public:
    bool ConsumeItem(uint64_t playerId, int itemId)
    {
        // 아이템 소모 처리 (구현 생략, 성공 시 true)
        return true;
    }
};

class AckSender
{
public:
    void SendAck(uint64_t playerId, int64_t commandId)
    {
        // ACK 패킷 전송 (구현 생략)
    }
};

class CommandHandler
{
public:
    CommandHandler(Inventory* inventory, AckSender* ackSender)
        : inventory_(inventory), ackSender_(ackSender) {}

    // 네트워크 수신 스레드에서, 커맨드를 수신할 때마다 호출된다.
    // 같은 commandId 로 여러 번 호출될 수 있다(재전송).
    void OnUseItemCommand(const UseItemCommand& cmd)
    {
        // (A) 수신한 커맨드를 곧바로 실행한다 — 이전에 같은 commandId 를
        //     처리한 적이 있는지는 확인하지 않는다.
        bool consumed = inventory_->ConsumeItem(cmd.playerId, cmd.itemId);

        if (consumed)
        {
            ApplyItemEffect(cmd.playerId, cmd.itemId);
        }

        // (B) 실행 결과와 무관하게 ACK 는 항상 돌려준다.
        ackSender_->SendAck(cmd.playerId, cmd.commandId);
    }

private:
    void ApplyItemEffect(uint64_t playerId, int itemId)
    {
        // 아이템 효과 적용 (구현 생략)
    }

    Inventory* inventory_;
    AckSender* ackSender_;
};