Что происходит с string при передаче ссылки строки в структуру, почему может крашится?

Question

[ITeration_Test]

Не передавая данные в структуру - работает. Иначе, нет.

typedef struct{
      std::string url;
    }OBJECT;

OBJECT object;

void init_obj(OBJECT* obj,  std::string _url)
{
	obj->url = _url;
}
init_obj(&object, "0.png"); //Ссылка на текстуру.
// SFML
sf::Texture texture;
texture.loadFromFile(obj->url);  //  void draw(OBJECT* obj)

Comments

[Adamos]

Вообще-то оно и компилироваться не должно. откуда в последней строчке взялся obj?

[Dmitrii]

Во-первых, текстуры лучше выделять в динамической памяти. Большое количество структур на стеке/data вы просто не сможете сохранить, да и не нужно оно вам.

[Василий Банников]

Вообще-то оно и компилироваться не должно. откуда в последней строчке взялся obj?

Так вот типа чуть выше, чем init_obj он объявлен.

[Василий Банников]

У тебя на последней строке обращение к полю url идёт через стрелочку - тоесть как-будто obj - это указатель на структуру.
Хотя у тебя вот выше переменная с таким именем лежит.

[Василий Банников]

Напиши хоть с какой ошибкой крашится

[Wexter]

Василий Банников,

void init_obj(OBJECT* obj,  std::string _url)
{ <--
  obj->url = _url;
} <--
init_obj(&object, "0.png"); //Ссылка на текстуру.
// SFML
sf::Texture texture;
texture.loadFromFile(obj->url);  //  void draw(OBJECT* obj)

[Wataru]

ITeration_Test, Со string у вас тут никах проблем нет. Более того, никакой "ссылки на строку" у вас там нет. Строка передается по значению.

Проблема в каком-то другом месте, которое вы в вопросе не привели. Судя по использованию указателей, скорее всего у вас там проблема работы с памятью и вы не выделяете объект где-то и программа падает, как только вы пытаетесь записать туда строку.

И вообще, инициализация сделана ужасно криво. Если это C++, то заведите конструктор ну или метод в структуре. Или вообще используйте obj.url = "0.png";.

[ITeration_Test]

Dmitrii, Доброе утро. Текстуры выделяются в динамической памяти.

//
  OBJECT* GetNewBrick()
    {
      ++brickLength;
      brick = (OBJECT*)realloc(brick, sizeof(*brick) * brickLength); ///
      return brick + brickLength - 1;
    }
///
      for (int y = 0; y < mapHeight; ++y)
        for (int x = 0; x < mapWidth; ++x)
          {
            if (map[y * mapWidth + x] == 35) // "#" == 35 --> от wstring
              init_obj(GetNewBrick(),  "ground.png"); // brick + n,  url;
          }

[ITeration_Test]

Василий Банников, Выше я инициализировал переданный параметр , пример: &player.

// SFML
sf::Texture texture;
texture.loadFromFile(obj->url);  //  void draw(OBJECT* obj)

Как бы я передаю ссылку &player --> от которого я и беру эти данные (приведенные выше).

[ITeration_Test]

Василий Банников, Сам код компилируется, но, графическое приложение просто зависает и всё. Где-то данные в памяти теряются на моменте со структурой.

void Engine::draw_sprite(OBJECT* obj,  std::string _url)
{
	sf::Texture texture;
	texture.loadFromFile(_url); // передавая на прямую - всё работает.
}

[Dmitrii]

ITeration_Test, так распечатайте содержимое
_url и obj->url и сравните.

Answer 1

[Wataru]

brick = (OBJECT*)realloc(brick, sizeof(*brick) * brickLength);

Вот этот код вызывает ошибку. Вы выделяете стурктуру OBJECT через realloc. Но она не POD (plain old data) - там поле url - объект std::string. А объекты нельзя вот так выделять. Надо чтобы обязательно конструктор отработал. Вообще, конечно, есть способ извратиться и вызвать конструктор руками, но это костыль.

Вы тут намешали вещи из C++ (объекты) и вещи из C (malloc) - и это все вместе не работает.

Правильное решение будет создавть объект через new:
brick = new Object[brickLength];

Не забудьте только в конце отчистить это все через delete[].

А еще лучше, используйте std::vector<Object>.

Comments

[ITeration_Test]

Учту. Спасибо.