Почему не доступен объект CircleShape?

Question

[Евгений Якушов]

При компиляции пишет: Вызвано исключение по адресу 0x0FE581E6 (sfml-graphics-d-2.dll) в infectionGraph.exe: 0xC0000005: нарушение прав доступа при чтении по адресу 0xCDCDCDD5.
И ссылается на строку: init->circle.setRadius(size);
Если закомментить, то след ошибка на строке: window.draw(root->circle);
Я так понимаю, объект недоступен почему-то

#include <SFML/Graphics.hpp>
#include <stdint.h>

using namespace sf;

#define AMOUNTNODES 10
#define AMOUNTLINKS 5
#define SIZEX 600
#define SIZEY 600

RenderWindow window(VideoMode(SIZEX, SIZEY), "Infection Graph", Style::Close);

struct Node {
	CircleShape circle;
	uint16_t position[2];
	Node * links[AMOUNTLINKS];
	bool healthy = true;
};

class opNode
{
	private:
		uint8_t size;
		const Color & healthy = Color(150, 220, 55, 255);
		const Color & infected = Color(220, 113, 55, 255);
		Node * root = nullptr;
		void initRoot();
		void create(Node * node);
	public:
		opNode(uint8_t size);
		void draw();
};

opNode::opNode(uint8_t size)
{
	this->size = size;
	create(root);
}

void opNode::initRoot()
{
	Node * init = (struct Node*)malloc(sizeof(struct Node));
	//init->circle.setRadius(size);
	init->position[0] = rand() % SIZEX + size / 2 + 1;
	init->position[1] = rand() % SIZEY + size / 2 + 1;
	for (uint16_t i = 0; i < AMOUNTLINKS; i++) init->links[i];
	init->healthy = true;
	root = init;
}

void opNode::create(Node * node)
{
	if (!root) initRoot();
	//for (uint32_t i = 0; i < AMOUNTNODES; i++)
	//{

	//}
}

void opNode::draw()
{
	root->circle.setFillColor(root->healthy ? healthy : infected);
	root->circle.setPosition(root->position[0], root->position[1]);
	window.draw(root->circle);
}

int main()
{
	opNode pluguInc = opNode(20);

	while (window.isOpen())
	{
		Event event;
		while (window.pollEvent(event))
		{
			if (event.type == Event::Closed)
				window.close();
		}

		window.clear(Color(240, 240, 240, 1));

		pluguInc.draw();
		
		window.display();
	}

	return 0;
}

Comments

[Евгений Шатунов]

Евгений Якушов ,

Node * init = (struct Node*)malloc(sizeof(struct Node));

Опиши словами все что делает эта строка.

[Евгений Якушов]

Евгений Шатунов,
мы создаем переменную типа структуры, которой присваиваем указатель на память размером структуры, преобразованный на указатель структуры. Вроде так

[Евгений Шатунов]

Евгений Якушов, ну да, вроде так. Примерно так. А в твоих словах есть что-нибудь о конструкторе Node?
Какими данными будет заполнена память по адресу init после вот этой строчки с инициализацией?

[Евгений Якушов]

Евгений Шатунов,

//init->circle.setRadius(size);
  init->position[0] = rand() % SIZEX + size / 2 + 1;
  init->position[1] = rand() % SIZEY + size / 2 + 1;
  for (uint16_t i = 0; i < AMOUNTLINKS; i++) init->links[i];
  init->healthy = true;

[Евгений Шатунов]

Евгений Якушов, смотри. Мы пишем не на C, мы пишем на C++, в котором есть понятие экземпляра типа.
По факту в своем коде ты сейчас допускаешь Undefined behavior, т.к. начинаешь пользоваться экземпляром до начала времени его жизни. Выделенная память только в C может означать начало времени жизни объекта.
Время жизни экземпляра начинается после выполнения конструктора его типа и завершается после выполнения деструктора его типа.
У тебя в коде нет ни обращения к конструктору Node, ни обращения к его деструктору. Это значит, что для компилятора экземпляра init не существует, но память ты сказал выделить и компилятор просто подчинился.
В рантайме при этом ни init у тебя не существует, ни init->circle, соответственно. Поэтому и ошибки доступа к памяти. Это - последствия внесения UB в свой код.

В C++ не принято пользоваться malloc/free, здесь принято пользоваться семейством операторов new и delete.

[Евгений Якушов]

Евгений Шатунов, спасибо за информацию, сейчас попробую