-1

Я пытаюсь написать класс который хранил бы карту блоков. Написал пробный вариант. Правда оказалось что вариант совсем не работоспособный.


struct block{
    uint units{30};
    sf::Color color{128,128,128};
    int owner_id{-1};
    sf::ConvexShape cube;

    void updateCube(const int &size, const sf::Color &ccolor={128,128,128}){
        cube.setPointCount(4); //sigsegv
        cube.setPoint(0,sf::Vector2f(0,0));
        cube.setPoint(1,sf::Vector2f(0,size));
        cube.setPoint(2,sf::Vector2f(size,size));
        cube.setPoint(3,sf::Vector2f(size,0));
        color = ccolor;
    }
};




class blockmap{
    typedef std::vector<block> line;
    std::vector<line> map;

public:

    blockmap(std::size_t x, std::size_t y, const int size){
        map.resize(y);
        for(std::size_t i{0};i!=y;i++){
            map[i].resize(x);
            for(std::size_t d{0};d!=x;d++){
                map[i][x].updateCube(size);
                map[i][x].cube.setPosition(i*(size+4),x*(size+4));
            }
        }


    }

    std::vector<block>& operator[](const std::size_t index){
        return map[index];
    }

    void draw(sf::RenderWindow &window){
        for(std::size_t y{0};y!=map.size();y++)
            for(std::size_t x{0};x!=map[y].size();x++){
                window.draw(map[x][y].cube);
            }
    }

};
6
  • Очередной "найди мне ошибку без отладчика" тред. Вам бы собеседования проводить. Запустите под отладкой и посмотрите, на какой строчке падает. Ну или ждите человека, которому на столько нечем заняться, что он будет либо анализировать ваш код в уме, либо провернет манипуляцию с отладчиком вместо вас.
    – gbg
    23 июл 2021 в 13:56
  • Чисто из соображений читаемости, я бы заменил ваши циклы на ranged_for, потому что продираться через ваше std::size_t x{0} (ох уж эти модные статьи по C++) лично мне решительно трудно.
    – gbg
    23 июл 2021 в 13:58
  • @gbg дело в том что все что мне выдал отладчик это место где код умер. Больше полезной информации не было. 23 июл 2021 в 14:00
  • Вам-то выдал, а нам вы выдадите?
    – gbg
    23 июл 2021 в 14:01
  • @gbg i.imgur.com/Iazv12U.png 23 июл 2021 в 14:03

1 ответ 1

0

ох ох. внутренний цикл под d, а в индексе x. Логично, что выход за пределы

for(std::size_t d = 0;d<x;d++){
    map[i][d].updateCube(size);
    map[i][d].cube.setPosition(i*(size+4),x*(size+4));
}

и пишите нормально for. вот это d!=i когда то очень больно может ударить.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.