Discussion :

[dekalima]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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;
};

[fregolo52]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Ca sent le beau plantage, non ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

1ere fois que je vois ça !!!

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

[Emmanuel Deloget]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

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.

[dekalima]

Merci pour vos impressions à vous deux