SFML - Dynamická alokace paměti – C / C++ – Fórum – Programujte.com

Zdravím, zasekl sem se u alokace paměti pro texturu. Mám funkci která vykresluje pole 10x10 o velikosti čtverce 50x50

a potřeboval bych pro třídní funkci alokovat paměť jelikož neznám přesný počet vykreslených textur. Funkce se dá přirovnat ke grafickému editoru. Když kliknu do pozice x : 7 a y 6 tak zde by se měla vykreslit má zvolená textura

class brush
{
public:
	.....
	void draw(sf::RenderWindow &window);

private:
	sf::Texture t_Grass1;
	sf::Sprite s_Grass1;
};

void brush::draw(sf::RenderWindow &window)
{
	s_Grass1.setTexture(t_Grass1);
	
}

Vše ostatní mám jen nevím jak alokovat paměť pro s_Grass1.

Předem díky za radu.

#1 Kenvelo
Však ty s_Grass1 nepoužíváš jako pointer, místa už máš dost, na stacku, stačí do něho přiřadit hodnotu, nebo co ti na tom nefunguje?

#2 ondrej39
Asi sem to špatně vysvětlil ta textura se má vykreslit na pozici tam kde kliknu myší a myslím, že bez alokace to nejde. Respektive pozice ukládám do pole ale je hloupost tvorit pole o 2000 prvcích když se textura vytvoří třeba jen 4x nebo bude potřeba větší pole. Jen prostě nevím jak to udělat..

#3 Kenvelo
A co ti nefunguje na tomto způsobu?

void brush::draw(sf::RenderWindow &window) {
    s_Grass1.setTexture(t_Grass1);

    auto x = static_cast<float>(sf::Mouse::getPosition(window).x);
    auto y = static_cast<float>(sf::Mouse::getPosition(window).y);

    s_Grass1.setPosition(sf::Vector2f(x, y));
    window.draw(s_Grass1)
}

#4 ondrej39
Mno nevím jak to funguje, textura se sice zobrazí na určitém místě ale když kliknu myší znovu tak se přepíše pozice a vykreslí se jinde. Potřebuji aby tam kde kliknu zůstala
 

void brush::draw(sf::RenderWindow &window)
{
		s_Grass1.setTexture(t_Grass1);

		if (sf::Mouse::isButtonPressed(sf::Mouse::Left))
		{
			auto x = static_cast<float>(sf::Mouse::getPosition(window).x);
			auto y = static_cast<float>(sf::Mouse::getPosition(window).y);
			s_Grass1.setPosition(sf::Vector2f(x, y));
		}

			
				
				window.draw(s_Grass1);

		
	
}

#5 Kenvelo
Nabízejí se ti dvě možnosti...

1) Chceš mít jednu instanci třídy brush a kdykoliv když zavoláš brush::draw chceš, aby se vykreslila další položka. V tomto případě budeš potřebovat přímo ve třídě brush kolekci sf::Sprite, třeba std::vector<sf::Sprite>, pokaždé, když zavoláš brush::draw, vytvoří se ti nový sf::Sprite, přidá do kolekce a zavolá se window.draw(nověPřidanýSpriteDoKolekce).

2) Chceš mít více instanci třdy brush, protože třída brush vždycky vykreslí jen jeden sf::Sprite. V takovém případě budeš potřebovat někde výš ve svém programu například std::vector<brush>, když bude chtít někdo kreslit, tak se vytvoří nový brush a na něm se zavolá brush::draw.

Osobně bych se přikláněl spíš k prvnímu z těch dvou řešení, vzhledem k tomu, že brush vždycky bude kreslit stejnou věc.

P.S. Vypadá to, že sf::Sprite nemá nadefinovaný move konstruktor ani assignment operátor. V tom případě by možná bylo lepší namísto std::vector<sf::Sprite> použít std::vector<std::unique_ptr<sf::Sprite>>, abys zbytečně nekopíroval celé objekty.

#6 ondrej39
První možnost nezní špatně jen teda jestli sem to pochopil dobře tak můžu vykreslovat jen s_Grass1 ale když budu mít druhou texturu s_Grass2 tak se přepíše přes tu první ? 

#7 Kenvelo
Nevím, jestli budu mít přesnou syntaxi správně, v C++ v poslední době zas tak moc nedělám, ale tady je nástřel řešení, jak nějak by to mohlo vypadat:

#include <vector>
#include <memory> // std::unique_ptr<T>

class brush
{
public:
    .....
    void draw(sf::RenderWindow &window);

private:
    sf::Texture t_Grass1;
    std::vector<std::unique_ptr<sf::Sprite>> sprites;
};

void brush::draw(sf::RenderWindow &window)
{
    auto grassSprite = std::unique_ptr<sf::Sprite>(new sf::Sprite(t_Grass1));

    if (sf::Mouse::isButtonPressed(sf::Mouse::Left)) {
        auto x = static_cast<float>(sf::Mouse::getPosition(window).x);
        auto y = static_cast<float>(sf::Mouse::getPosition(window).y);
        grassSprite->setPosition(sf::Vector2f(x, y));
    }

    sprites.push_back(std::move(grassSprite));
    window.draw(grassSprite);
}