Vieux codes C++

Version imprimable

Vieux codes C++

Bonjour,
Ce sont des vieux codes fait entre 1994 et 1997.
Comme je suis plutôt débutant, j'aimerais avoir vos opinions sur l'utilité du code, ce qu'il fait, la signification de ObjSys par exemple...
Avis aux masters...
thx

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
DKKString::DKKString(PSTRZ szFromString) : DKKObject(DKKOBJECT(DKKString))
{
    DKKObject::Initialized();
	nSize=0; pszString=NULL;
    // PD19990107 - PROMODES
    if (szFromString)
    {
    	nLen=strlen(szFromString);
        Size(nLen+1);
        strcpy(pszString, szFromString);
    }
    else
    {
        nLen=0; Size(nLen+1);
    }
    bMakeWords=false;	// PD20010112
    bPersistent=false;  // PD20011014
}

Code:

1
2
3
4
5
6
7
8
DKKObject::DKKObject(PSTRZ pszNewType, INT nSize)
{
    objSys.EnterCritical();
    nDKKObjects++;
    objSys.LeaveCritical();
	memset((PCHAR)this, ZERO, nSize);
	pszObjType=pszNewType;
};

27/01/2011, 11h37
fregolo52
Soit il manque des lignes de code pertinantes, soit y'a un souci !!!
Code:

1 2 3 nSize=0; pszString=NULL; .... strcpy(pszString, szFromString);
Ca sent le beau plantage, non ?

Code:

memset((PCHAR)this, ZERO, nSize);

1ere fois que je vois ça !!!

objSys doit être déclaré quelque part dans un .h.
27/01/2011, 12h21
Emmanuel Deloget
Citation:
Envoyé par fregolo52

Soit il manque des lignes de code pertinantes, soit y'a un souci !!!

Code:

1 2 3 nSize=0; pszString=NULL; .... strcpy(pszString, szFromString);

Ca sent le beau plantage, non ?
Entre temps, il y a un appel à la méthode Size(), qui doit certainement allouer la mémoire.

Citation:

Envoyé par fregolo52

Code:

memset((PCHAR)this, ZERO, nSize);

1ere fois que je vois ça !!!

Ca, effectivement, c'est une véritable horreur. D'autant plus que le fait qu'on utilise l'héitage tendrait à montrer qu'il y a, quelque part, une notion d'objet polymorphique avec probablement une table de fonction virtuelle. Donc ce code va nécessairement faire planter le programme, car la table des fonctions virtuelles va pointer sur NULL au lieu des méthodes véritables.

Citation:

Envoyé par fregolo52

objSys doit être déclaré quelque part dans un .h.

Je dirais même plus : il s'agit d'un objet dont le but est de proposer une interface avec le système d'exploitation (ici, l'utilisation de sections critiques pour rendre le code "atomique").

Pour répondre à la question première, je dirais : non. Le standard fournit une classe std::string qui propose tous les services dont tu as besoin pour gérer des chaines de caractère. Inutile de venir l'ennuyer avec du code prédatant le standard qui est visiblement perclus de bugs.
27/01/2011, 12h33
dekalima

Merci pour vos impressions à vous deux