Vector не обновляется?

Question

[0xC0000005]

У меня есть ObjectsBuffer, который содержит std::vector<LevelObject> (objectsBuffer), а класс имеет функцию

updateObject(int index, LevelObject &updatedObject)

Исходный код функции:

void updateObject(int index, LevelObject &updatedObject) {
        this->getReferenceBuffer()[index] = updatedObject;
    }

getReferenceBuffer

std::vector<LevelObject> &getReferenceBuffer() {
        return this->objectsBuffer;
    }

Основной код:

LevelObject object = objectsBuffer.getWithIndex(selectedObjectIndex);

if(object.lobjecttype == LevelObjectType::RECTANGLE) {
    static char objectName[28];

    strcpy(objectName, object.lobjectname.c_str());

    static int objectWidth = object.objectWidth;
    static int objectHeight = object.objectHeight;

    static int objectX = object.objectX;
    static int objectY = object.objectY;

    ImGui::InputText("Object name.", objectName, IM_ARRAYSIZE(objectName));

    ImGui::InputInt("Object Width.", &objectWidth);
    ImGui::InputInt("Object Height.", &objectHeight);

    ImGui::InputInt("Object X.", &objectX);
    ImGui::InputInt("Object Y.", &objectY);

    if(ImGui::Button("Apply.")) {
        object.lobjectname = objectName;

        object.objectWidth = objectWidth;
        object.objectHeight = objectHeight;

        object.objectX = objectX;
        object.objectY = objectY;

        objectsBuffer.updateObject(selectedObjectIndex, object); // Сдесь пытаюсь обновить обьект.
    }
}

Из примера я получаю объект по индексу из вектора по индексу, изменяю его, а затем пытаюсь обновить объект. Все свойства, такие как objectWidth или другие, остаются прежними. Я не создаю буфер в цикле, я его нигде не перезаписываю (только обновляю его здесь и добавляю новые элементы). Как это исправить?

Comments

[mayton2019]

Вот сюда не заходит наверное.

if(ImGui::Button("Apply.")) {

Проверь.

[0xC0000005]

mayton2019, заходит, проверял и с дебагом и с std::cout, оказывается оно изменяет вектор но только на миллисекунду, а потом тот самый измененный object стает прошлой версией себя

[0xC0000005]

mayton2019, возможно я где то перезаписываю вектор, а если я читаю элементы с вектора это как то считается?

[mayton2019]

0xC0000005, ничего непонятно. Ну типа проблема решена?

[0xC0000005]

mayton2019, ну на половину возможно

[mayton2019]

А если ты здесь ссылку заменишь на копию объекта?

void updateObject(int index, LevelObject copyOfupdatedObject) {
        this->getReferenceBuffer()[index] = copyOfupdatedObject;
    }

[0xC0000005]

mayton2019, такая же ситуация
сделал чтобы выводило "apply" при нажатии на кнопку, и чтобы выводило object.objectX из вектора, результат: (250 это изначальные координаты, 300 - обновленные):

250
250
250
apply
300
250
250

Answer 1

[0xC0000005]

Решено: функциия

show(ObjectsBuffer objectsBuffer, int selectedObjectIndex)

принимала аргумент - буфер, и видимо я пытался получить не копию буфера, уже копии буфера, решением было передать ссылку на буфер:

show(std::vector<LevelObject> &buffer, int selectedObjectIndex)

, и вызов функции стал таким:

ObjectEditorUI::show(objectsBuffer.getReferenceBuffer(), selectedObjectIndex)

(getReferenceBuffer вверху). Похоже я оеально упустил слишком много деталей, всем спасибо

Comments

[mayton2019]

Как там один гитарист из 90х пел?

Следи за собой и будь осторожен! Следи за собой...

Answer 2

[Wataru]

(Исходя из обсуждения в комментариях)
Возможно, проблема с параллельным кодом. Какой-то код где-то изменяет вектор. Если не очевидно, где оно все перезаписывается, то помните, что добавление элементов вектор может вызвать увеличение внутреннего буфера и копирование всех элементов. Поэтому даже какой-то код, просто добавляющий новые элементы в вектор, может вызвать наблюдаемый эффект необъяснимой перезаписи объекта в векторе на предыдущее значение. Вообще, надо бы защищать вектор от параллельного доступа разными потоками всякими мютексами и критическими секциями. Ну нет, так поменяйте его хотя бы на list. тогда добавление новых элементов и изменение значений где-то не будут вызывать data race. Но тогда надо алгоритмы менять, ибо обращение по индексу - долгая операция.