Appels multiples du destructeur - Page 1

void fonction()
{
    Image img("test.png");
    img = Image("autre.png");
}

Texture::Texture(const Texture& t)
{
    this->width = t.width;
    this->height = t.height;
    ...
    this->isCopy = true;
}

Texture::Texture(const Texture& t)
{
    memcpy(this, &t, sizeof(*this));
    this->isCopy = true;
}

Image::Image(const Image& img) : Texture(img)
{
    memcpy((u8*)this + sizeof(Texture), &img, sizeof(*this) - sizeof(Texture));
    this->isCopy = true;
}

Brunni (./3) :
Au risque d'en oublier si j'étend la classe (). Là actu ce que je fais c'est une horreur du style:

Texture::Texture(const Texture& t)
{
    memcpy(this, &t, sizeof(*this));
    this->isCopy = true;
}

Texture::Texture(const Texture& t) :
    width (t.width),
    height (t.height),
    ...
{ 
}

class Texture
{
    public:
        Texture (const char* path)
        {
             this->refCount = malloc (sizeof (*this->refCount)); // compteur partagé du nombre de références sur cette textures
             *this->refCount = 1;

             this->imageData = new char [...];
             // charger l'image "path" dans this->image_data
        }

        Texture (const Texture& texture)
        {
            this->refCount = texture.refCount; // on récupère le compteur partagé pour compter une nouvelle référence
            ++*this->refCount;

            this->imageData = texture.imageData;
        }

        ~Texture ()
        {
            if (--*this->refCount == 0) // si le compteur arrive à 0, alors plus aucun objet ne référence cette texture,
            {
                delete [] this->imageData; // on peut donc l'effacer pour de bon
                free (this->refCount); // sans oublier de supprimer également le compteur
            }
        }

    private:
        char* imageData;
        int* refCount;
}

Zephyr (./2) :
C'est normal que 3 appels de destructeurs soient effectués, puisqu'il y a 3 créations d'objet :

- Le premier objet : img ("test.png")
- Le second, temporaire : Image ("autre.png")- Le troisième, créé par copie : img = ...

Sasume (./10) :
C'est "Foo foo = Foo();" qui est équivalent à "Foo foo();" (ce qui est différent de ce qui a été proposé dans le post initial que j'avais mal lu)

int APIENTRY WinMain(HINSTANCE hInstance,
                     HINSTANCE hPrevInstance,
                     LPTSTR    lpCmdLine,
                     int       nCmdShow)
{
	// Variables locales
	int ordre = 1, taille = 240;
	int positionX = 0, positionY = 0;
	char lecture;
	wingraph::Font verdana("Tahoma", 13, true);

	// Initialisation de la librairie
	wingraph::initLib();
	wingraph::initWindow("Fractales de Koch", 640, 480, true);
	turtle::init();

	while (!wingraph::closed)
	{
		const int vitesseDeplacement = 20;

		// Fonctions de la librairie pour la saisie (à faire à chaque frame)
		wingraph::keyboard.readInput();

		// Lit les caractères au clavier tant qu'il y en a
		while (wingraph::keyboard.availableChar())
		{
			lecture = wingraph::keyboard.readChar();
			if (lecture >= '1' && lecture <= '9')
				ordre = lecture - '0';
			else if (lecture == '0')
				ordre = 10;
		}

		// Teste les touches de direction
		if (wingraph::keyboard.held(VK_UP))
			positionY -= vitesseDeplacement;
		if (wingraph::keyboard.held(VK_DOWN))
			positionY += vitesseDeplacement;
		if (wingraph::keyboard.held(VK_LEFT))
			positionX -= vitesseDeplacement;
		if (wingraph::keyboard.held(VK_RIGHT))
			positionX += vitesseDeplacement;

		// Et celles de zoom
		if (wingraph::keyboard.held('A'))
			// Si la taille devient trop petite, on ne peut plus la récupérer...
			taille = max(taille * 9 / 10, 10);
		if (wingraph::keyboard.held('S'))
			taille = taille * 11 / 10;

		// Et le milieu
		if (wingraph::keyboard.held('M'))
			positionX = positionY = 0;

		if (!wingraph::skipFrame)
		{
			wingraph::beginDrawing();
			// Un petit dégradé de fond
			wingraph::drawFillRect(
				wingraph::Vertex(0, 0, wingraph::Color(128, 128, 255)),
				wingraph::Vertex(wingraph::screenWidth, 0, wingraph::Color(128, 128, 255)),
				wingraph::Vertex(0, wingraph::screenHeight, wingraph::Color(255, 255, 255)),
				wingraph::Vertex(wingraph::screenWidth, wingraph::screenHeight, wingraph::Color(255, 255, 255))
			);
			// Dessin de la fractale
			curColor = wingraph::Color(0, 0, 0);
			turtle::setPosition(taille / 2 - positionX, taille / 3 - positionY);
			koch(taille, ordre);
			// Affiche l'ordre par-dessus
			std:: ostringstream titre;
			titre << "Ordre: " << ordre;
			wingraph::setColor(wingraph::Color(0, 0, 0));
			verdana.drawText(0, 0, titre.str().c_str());
			// Ainsi que des informations supplémentaires
			verdana.drawText(0, 13, "0-9: définir l'ordre");
			verdana.drawText(0, 26, "a/s: modifier la taille");
			verdana.drawText(0, 39, "m: revenir au milieu");
			wingraph::endDrawing();
		}

		// On fait tourner à 10 fps pour ne pas trop surcharger le GPU =)
		wingraph::syncFrame(10, 8);
	}
}

wingraph::Font verdana("Tahoma", 13, true);

Pollux (./23) :

wingraph::Font verdana("Tahoma", 13, true);

Kevin Kofler (./25) :
Quelle idée aussi d'utiliser du Win32 directement (WinMain, VK_UP etc.), ce n'est pas du tout portable, ton truc! Pourquoi pas utiliser quelque chose de portable et en C++ propre plutôt que de faire ton propre wrapper C++ de l'API Win32?

Zephyr (./22) :
Ah bah ça, ça s'appelle la conception et c'est difficile de donner une réponse simple à ton problème ; pour moi c'est quelque chose de bien plus complexe que d'écrire du code par exemple

Zephyr (./22) :
Entre image.draw (surface) et surface.draw (image) j'aurais tendance à préférer la 2eme solution en général (pour plein de raisons que je peux prendre le temps d'expliquer mais pas ce soir ^^)

class MainForm : public QMainWindow
{
public:
	MainForm(QWidget *parent = 0);

private:
	virtual void resizeEvent(QResizeEvent *e);
	virtual void paintEvent(QPaintEvent *e);
	virtual void mouseMoveEvent(QMouseEvent * e);
	virtual void mousePressEvent(QMouseEvent * e);
	virtual void mouseReleaseEvent(QMouseEvent *e);
	virtual void wheelEvent(QWheelEvent *e);
	virtual void keyPressEvent(QKeyEvent *e);
..........
};

int main(int argc, char* argv[])
{
    QApplication app(argc, argv);
    MainForm mf(cf);
    mf.resize(640,480);
    mf.show();
    return app.exec();
}

void MainForm::paintEvent(QPaintEvent *e)
{
	QPainter painter(this);
	painter.setFont(QFont("Arial", 8));
	painter.setPen(Qt::blue);
	painter.drawText(rect(), Qt::AlignLeft | Qt::AlignTop, "plop");
	e->accept(); //et encore ça marche sans
}

void MainForm::keyPressEvent(QKeyEvent *e)
{
	if(e->key()==Qt::Key_Space) {
	/* blabla*/
	e->accept();
}