Torna indietro   Hardware Upgrade Forum > Software > Programmazione

Recensione REDMI Note 17 Pro: il midrange con batteria da 8.340 mAh e ricarica veloce
Recensione REDMI Note 17 Pro: il midrange con batteria da 8.340 mAh e ricarica veloce
REDMI Note 17 Pro porta in fascia media una batteria da 8.340 mAh con ricarica HyperCharge a 67W, un display AMOLED da 6,83 pollici capace di picchi di luminosità molto elevati e una struttura certificata TÜV SÜD contro cadute e infiltrazioni d'acqua, il tutto racchiuso in una scocca da 223 grammi. Lo abbiamo provato per diversi giorni tra fotocamera, prestazioni, autonomia e prezzo sul mercato italiano
Insta360 Luna Ultra: la potenza del sensore da 1 pollice incontra la portabilità estrema
Insta360 Luna Ultra: la potenza del sensore da 1 pollice incontra la portabilità estrema
Insta360 Luna Ultra integra un sensore da 1 pollice 8K, ottiche Leica e triplo chip IA. Tra schermo OLED rimovibile, workflow I-Log a 10 bit e stabilizzazione a tre assi, analizziamo le doti tecniche di una gimbal camera pensata per i professionisti
Marvel's Wolverine, la recensione: Logan torna protagonista in un'avventura brutale e intensa
Marvel's Wolverine, la recensione: Logan torna protagonista in un'avventura brutale e intensa
Marvel's Wolverine porta Logan in un'avventura inedita, violenta e fortemente narrativa, costruita attorno alla sua natura di combattente e al difficile rapporto con il proprio passato. Insomniac Games punta su combattimenti spettacolari, progressione e personalizzazione, inserendo l'azione in un mondo segnato dalla persecuzione dei mutanti. Un viaggio intenso, che alterna mattanza, esplorazione e momenti sorprendentemente emotivi.
Tutti gli articoli Tutte le news

Vai al Forum
Rispondi
 
Strumenti
Old 14-07-2012, 23:30   #1
vendettaaaaa
Senior Member
 
L'Avatar di vendettaaaaa
 
Iscritto dal: Jan 2012
Messaggi: 1267
[C++] Memory leak nella mia classe matrice

Ciao, ci riprovo anche se nessuno si fila mai i miei problemi.
Ho un problema di memory leak nell'uso della mia classe per rappresentare matrici...e non lo trovo. Riporto solo parte del codice, quello che ritengo poter contenere la causa dell'errore, quindi la faccio abbastanza breve e leggibile.

Header:
Codice:
#ifndef GUARD_BBMATRIX_HPP
#define GUARD_BBMATRIX_HPP

#include <vector>
#include <memory>

class BbVector;

class BbMatrix
{
public:
	// Typedefs for the user
	typedef double* iterator;
	typedef const double* const_iterator;
	typedef std::size_t size_type;

	bool range_check(int i, int j) const;

private:
	double** myMatrix;
	size_type nRows;
	size_type nColumns;
	bool mySize;

	std::allocator<double> auxAlloc;
	std::allocator<double*> matrixAlloc;

	void create();
	void create(size_type r, size_type c, const double& val);
	void create(const BbMatrix& other);
	void uncreate();
	void delete_matrix();
	inline void delete_array(int i);

public:
	BbMatrix() { create(); }
	explicit BbMatrix(int r, int c, double val = 0.);
	BbMatrix(const BbMatrix& other) { create(other); }

	double& operator()(int i, int j) { range_check(i, j); return myMatrix[i][j]; }
	const double& operator()(int i, int j) const { range_check(i, j); return myMatrix[i][j]; }
	double* operator[](int k) { return myMatrix[k]; }
	const double* operator[](int k) const { return myMatrix[k]; }
};
#endif
Implementazione delle funzioni che gestiscono la memoria dinamicamente:
Codice:
#include "BbMatrix.hpp"
#include <algorithm>

using namespace std;

BbMatrix::BbMatrix(int r, int c, double val)
{
	if (r <= 0 || c <= 0)
		create();
	else
		create(r, c, val);
}

void BbMatrix::create()
{
	myMatrix = 0;
	nRows = 0;
	nColumns = 0;
	mySize = false;
}

void BbMatrix::create(size_type r, size_type c, const double& val)
{
	myMatrix = matrixAlloc.allocate(r + 1);
	
	for (int i = 1; i <= r; ++i) {
		myMatrix[i] = auxAlloc.allocate(c + 1);
		uninitialized_fill(myMatrix[i] + 1, myMatrix[i] + 1 + c, val);
	}

	nRows = r;
	nColumns = c;
	mySize = true;
/*
	myMatrix = new double*[r + 1];

	for (int i = 1; i <= r; ++i) {
		myMatrix[i] = new double[c + 1];
		for (int j = 1; j <= c; ++j)
			myMatrix[i][j] = val;
	}

	nRows = r;
	nColumns = c;
	mySize = true;
*/
}

void BbMatrix::delete_array(int i)
{
	iterator it = myMatrix[i] + nColumns + 1;

	while (it != myMatrix[i])
		auxAlloc.destroy(--it);

	auxAlloc.deallocate(myMatrix[i], nColumns + 1);
}

void BbMatrix::delete_matrix()
{
	double** it = myMatrix + nRows + 1;
	int i = nRows;

	while (it != myMatrix) {
		delete_array(i--);
		matrixAlloc.destroy(--it);
	}

	matrixAlloc.deallocate(myMatrix, nRows + 1);
/*
	for (int i = 0; i <= nRows; ++i)
		delete[] myMatrix[i];

	delete[] myMatrix;
*/
}

void BbMatrix::uncreate()
{
	if (myMatrix)
		delete_matrix();

	myMatrix = 0;
	nRows = 0;
	nColumns = 0;
	mySize = false;
}
Mi sono accorto del memory leak così facendo:
Codice:
int main(int argc, char* argv[])
{
	for (int i = 1; i <= 5000000; ++i)
		BbMatrix A(3,3);

	return 0;
}
Cosa sbaglio? Sia usando un allocator che usando new e delete il memory leak rimane. Ho creato anche una classe BbVector definita in modo simile (alloco, usando allocator<double>, n + 1 double, dove n è la dimensione del vettore; anche uncreate è identica, utilizza alloc.destroy() e alloc.deallocate()) e lì il problema non c'è.

Help
vendettaaaaa è offline   Rispondi citando il messaggio o parte di esso
Old 15-07-2012, 08:26   #2
WarDuck
Senior Member
 
L'Avatar di WarDuck
 
Iscritto dal: May 2001
Messaggi: 13043
Non vedo il distruttore dell'oggetto

Inoltre non c'è bisogno di passare i tipi base per riferimento, salvo se vuoi fare side-effect su una variabile passata da fuori (ma non credo sia il tuo caso).
WarDuck è offline   Rispondi citando il messaggio o parte di esso
Old 15-07-2012, 09:25   #3
vendettaaaaa
Senior Member
 
L'Avatar di vendettaaaaa
 
Iscritto dal: Jan 2012
Messaggi: 1267
Quote:
Originariamente inviato da WarDuck Guarda i messaggi
Non vedo il distruttore dell'oggetto

Inoltre non c'è bisogno di passare i tipi base per riferimento, salvo se vuoi fare side-effect su una variabile passata da fuori (ma non credo sia il tuo caso).
Il distruttore chiama uncreate(), ho dimenticato di scriverlo qui, ma c'è.
E passo const double& val per rendere ovvio che val non venga modificato. Non penso si possa considerare "cattiva prassi", giusto?.

Riguardo agli allocatori, dici che li uso correttamente?

EDIT: scusate...devo averlo cancellato, o forse l'ho messo per la classe Vector ma non per la Matrice, non so...oddio, la volta che ricevo una risposta ho cannato io...sigh

Ultima modifica di vendettaaaaa : 15-07-2012 alle 09:38.
vendettaaaaa è offline   Rispondi citando il messaggio o parte di esso
 Rispondi


Recensione REDMI Note 17 Pro: il midrange con batteria da 8.340 mAh e ricarica veloce Recensione REDMI Note 17 Pro: il midrange con ba...
Insta360 Luna Ultra: la potenza del sensore da 1 pollice incontra la portabilità estrema Insta360 Luna Ultra: la potenza del sensore da 1...
Marvel's Wolverine, la recensione: Logan torna protagonista in un'avventura brutale e intensa Marvel's Wolverine, la recensione: Logan torna p...
DJI Romo 2: tante novità lo rendono un robot completo DJI Romo 2: tante novità lo rendono un ro...
Sony Bravia 9 II: il True RGB alla prova, dove l'LCD sfida l'OLED Sony Bravia 9 II: il True RGB alla prova, dove l...
Home Assistant su uno schermo touch econ...
L'IA entra nei negozi Unieuro con Google...
"What Holds Us", la nuova most...
La Formula 1 divide sui motori, ma il su...
Un computer a DNA esegue moltiplicazioni...
Ibride cinesi, l'Europa fissa il tetto a...
iPhone Duo potrebbe dar vita a una gamma...
Denza Z9S: la berlina elettrica con il r...
Valve abilita i driver e il supporto all...
Terafab, il progetto di Musk finisce in ...
Il meglio di Amazon in 42 articoli: sono...
ECOVACS WINBOT W2 PRO OMNI a 349€: ecco ...
Samsung Gallery Cloud Sync introduce il ...
Dyson V8 Cyclone a 249€: scopa elettrica...
Rifiuti elettronici, l'altra faccia dell...
Chromium
GPU-Z
OCCT
LibreOffice Portable
Opera One Portable
Opera One 106
CCleaner Portable
CCleaner Standard
Cpu-Z
Driver NVIDIA GeForce 546.65 WHQL
SmartFTP
Trillian
Google Chrome Portable
Google Chrome 120
VirtualBox
Tutti gli articoli Tutte le news Tutti i download

Strumenti

Regole
Non Puoi aprire nuove discussioni
Non Puoi rispondere ai messaggi
Non Puoi allegare file
Non Puoi modificare i tuoi messaggi

Il codice vB è On
Le Faccine sono On
Il codice [IMG] è On
Il codice HTML è Off
Vai al Forum


Tutti gli orari sono GMT +1. Ora sono le: 12:40.


Powered by vBulletin® Version 3.6.4
Copyright ©2000 - 2026, Jelsoft Enterprises Ltd.
Served by www3v