Что может привести к повреждению значения, которое устанавливается только при инициализации? - PullRequest
Новая
       32

Что может привести к повреждению значения, которое устанавливается только при инициализации?

0 голосов
/ 18 декабря 2018

У меня есть следующий код 

    typedef struct{
    uint8_t carOpening:2;
    uint8_t fCarElement:4;
}lockState_t;

    enum Doors
{
    DRIVER_DOOR = 0,
    PASSENGER_DOOR,
    ...
    NUMBER_OF_ELEMENTS
};

class DoorState
{
public:
    DoorState(Doors door, uint8_t initialState)
    {
        fState.fCarElement = static_cast<uint8_t>(door);
        if(initialState == SIG_VAL_DOOR_CLOSED)
        {
            fState.carOpening = static_cast<uint8_t>(DOOR_CLOSED);
        }
        else if(initialState == SIG_VAL_DOOR_OPENED)
        {
            fState.carOpening = static_cast<uint8_t>(DOOR_OPENED);
        }
        else
        {
            fState.carOpening = static_cast<uint8_t>(DOOR_UNKNOWN);
        }
    };
    void checkSignal(uint8_t signal);
    enum tState { DOOR_UNKNOWN = 0, DOOR_CLOSED, DOOR_OPENED };
private:
    static const uint8_t SIG_VAL_DOOR_CLOSED = 0x00;
    static const uint8_t SIG_VAL_DOOR_OPENED = 0x01;

    lockState_t fState;
};

Затем у меня есть несколько экземпляров этого класса, например: 

DoorState fDriverDoorState;
DoorState fPassengerDoorState;

Вот как я инициализирую экземпляры: 

, fDriverDoorState(DRIVER_DOOR
, DS::instance().get<uint8_t>(DS::ComStatZvKlappen_StDswDrd_ID))
, fPassengerDoorState(PASSENGER_DOOR, DS::instance().get<uint8_t>(DS::ComStatZvKlappen_StDswPsd_ID))

Затем я проверяю, изменилось ли состояние двери, и публикую сообщение, если оно имеет: 

tState doorState = DOOR_UNKNOWN;
if (SIG_VAL_DOOR_CLOSED == signal)
{
    doorState = DOOR_CLOSED;
}
else if (SIG_VAL_DOOR_OPENED == signal)
{
    doorState = DOOR_OPENED;
}

if(doorState != static_cast<tState>(fState.carOpening))
{
    fState.carOpening = static_cast<uint8_t>(doorState);
    if(DOOR_UNKNOWN != doorState)
    {
        LooperEventBroker::publish(::csm::events::CsmEvents::VEHICLE_LOCK_STATE_EVENT(), &fState);
        Logger::info(VEHICLE, "Door %s event published for %s",
                     doorState == DOOR_OPENED ? "openend" : "closed", opening2string(fState.fCarElement));
    }
}

По большей части оно работает отлично.Странно то, что иногда я вижу, как этот журнал публикуется: «Событие закрытой двери опубликовано для INVALID DOOR», что не должно происходить, поскольку ни один экземпляр не инициализируется с помощью INVALID DOOR, а fCarElement, который хранит тип двери, никогда не изменяется.Что означает, что каким-то образом fCarElement будет поврежден.Вопрос в том, где или почему?

Как и было запрошено, здесь реализована функция open2string: 

static const char* opening2string(uint8_t opening)
{
    switch (opening)
    {
        case DRIVER_DOOR:
            return "DRIVER DOOR";
        case PASSENGER_DOOR:
            return "PASSENGER DOOR";
        ...
        default:
            return "INVALID DOOR";
    }
}

С уважением, Габриэль

...