Ich habe folgende Function geschrieben:

long GetAllUsers(CString **ppstrName, int **ppnGroup)
{
//do something..

*ppstrName = new CString[nRecordCount];
*ppnGroup = new int[nRecordCount];

//do something..
}

Funktion Aufruf:

CString *pstrName = NULL;
int *pnGroup = NULL;

GetAllUsers(&pstrName, &pnGroup);

delete [] pstrName;
delete [] pnGroup;

Ist das sauber Programmiert?

Und noch ein anderer Fall bitte:

class CUser
{
public:
int *m_pnGroup;
CString *m_pstrName;
void GetIDRight();
}

CUser::CUser(CWnd* pParent)
{
m_pnGroup = NULL;
m_pstrName = NULL;
}

void CUser::GetIDRight()
{
GetAllUsers(&m_pstrName, &m_pnGroup);
}

CUser::~CUser(void)
{
if(m_pnGroup != NULL)
{
delete []m_pnGroup;
m_pnGroup = NULL;
}
if(m_pstrName)
{
delete m_pstrName;
m_pstrName = NULL;
}
}

danke für jede Hilfe

Im zweitens Fall gibt es auf jeden Fall zwei Punkte, die überflüssig sind: (a) "delete NULL;" ist im Standard vorgesehen und macht gar nichts (damit entfallen die if(...!=NULL)-Tests) und (b) nach dem Ende des Dtors sind die Member sowieso weg, also brauchst du sie auch nicht mehr auf NULL setzen.
(und du solltest auch hier auf Pointer verzichten und die Daten direkt in deine Klasse packen.

PS: sfds

(a) "delete NULL;" ist im Standard vorgesehen und macht gar nichts (damit entfallen die if(...!=NULL)-Tests)

Das macht diesen Test weder sinnlos noch überflüssig. Zunächst einmal trifft das ganze nur auf die Deallokationsfunktionen der Standardbibliothek zu, eine eigene Überladung darf sich durchaus anders verhalten. Zudem präzisiert der Standard nicht näher, was 'has no effect' eigentlich bedeutet. Auf jeden Fall ist dieses if logisch sinnvoll - denn wo nichts ist, kann auch nichts gelöscht werden; auch wird durch ein delete NULL die natürliche Symmetrie zwischen (erfolgreicher) Allokation und Deallokation aufgehoben. Und nicht zuletzt führt ein solches if in der Regel zu besserer Peformance (wenn der Fall NULL einigermaßen häufig ist), evident z.B. bei auto_ptr - eine Implementation, die hier auf den Test verzichtete, würde erheblich Performance verschenken (denn der Test auf 0 und der Rest drumherum kann ja nach jeder Zuweisung wegoptimiert werden).