IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

Bibliothèque standard C Discussion :

Problème de réallocation et de Assert (?)


Sujet :

Bibliothèque standard C

  1. #1
    Membre éprouvé
    Avatar de beegees
    Homme Profil pro
    Développeur Web
    Inscrit en
    Mars 2004
    Messages
    3 610
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 48
    Localisation : Belgique

    Informations professionnelles :
    Activité : Développeur Web
    Secteur : Enseignement

    Informations forums :
    Inscription : Mars 2004
    Messages : 3 610
    Points : 1 277
    Points
    1 277
    Par défaut Problème de réallocation et de Assert (?)
    Bonjour tout le monde,

    Je suis bloqué sur une réallocation.

    Le nom n'est pas null et pas inférieure à 1 caractère donc il passe le premier if :

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    	if ( (Nom == NULL) || (strlen(Nom) < 1) ) return false;
    mais s'est à la réallocation qui plante :

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    void* Nouveau = realloc(Officier.Nom,(strlen(Nom) + 1)*sizeof(char));

    j'obtiens ce message d'erreur (voir attaché svp)

    J'ai d'autres info que voici (je n'y comprends rien) :

    /*
    * If this ASSERT fails, a bad pointer has been passed in. It may be
    * totally bogus, or it may have been allocated from another heap.
    * The pointer MUST come from the 'local' heap.
    */
    _ASSERTE(_CrtIsValidHeapPointer(pUserData));
    J'espère que vous saurez encore m'aider sur ce coup là.

    Un super grand merci d'avance.

    beegees

  2. #2
    Expert éminent sénior
    Avatar de Médinoc
    Homme Profil pro
    Développeur informatique
    Inscrit en
    Septembre 2005
    Messages
    27 369
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 40
    Localisation : France

    Informations professionnelles :
    Activité : Développeur informatique
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Septembre 2005
    Messages : 27 369
    Points : 41 519
    Points
    41 519
    Par défaut
    Ben le message est pourtant clair: Tu passes un mauvais pointeur à realloc().
    Ça peut, par exemple, être un pointeur sur lequel tu as déjà fait un free(), ou un pointeur alloué dans une DLL, ou un pointeur dont tu as modifié la valeur (avec ptr++ par exemple), etc.
    SVP, pas de questions techniques par MP. Surtout si je ne vous ai jamais parlé avant.

    "Aw, come on, who would be so stupid as to insert a cast to make an error go away without actually fixing the error?"
    Apparently everyone.
    -- Raymond Chen.
    Traduction obligatoire: "Oh, voyons, qui serait assez stupide pour mettre un cast pour faire disparaitre un message d'erreur sans vraiment corriger l'erreur?" - Apparemment, tout le monde. -- Raymond Chen.

  3. #3
    Expert éminent sénior
    Avatar de diogene
    Homme Profil pro
    Enseignant Chercheur
    Inscrit en
    Juin 2005
    Messages
    5 761
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Essonne (Île de France)

    Informations professionnelles :
    Activité : Enseignant Chercheur
    Secteur : Enseignement

    Informations forums :
    Inscription : Juin 2005
    Messages : 5 761
    Points : 13 926
    Points
    13 926
    Par défaut
    La question est de savoir si Officier.Nom est à NULL (correct) ou à une adresse obtenue par allocation dynamique (correct).
    Sinon (pointeur non initialisé ?), le realloc plante.
    Publication : Concepts en C

    Mon avatar : Glenn Gould

    --------------------------------------------------------------------------
    Une réponse vous a été utile ? Remerciez son auteur en cliquant le pouce vert !

  4. #4
    Rédacteur/Modérateur


    Homme Profil pro
    Développeur .NET
    Inscrit en
    Février 2004
    Messages
    19 875
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 42
    Localisation : France, Paris (Île de France)

    Informations professionnelles :
    Activité : Développeur .NET
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Février 2004
    Messages : 19 875
    Points : 39 749
    Points
    39 749
    Par défaut
    Attention, si tu as déclaré Officier.Nom de façon statique, tu ne peux pas le réallouer... :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
     
    struct t_Officier
    {
        char Nom[30 + 1];
    }
    Là le realloc ne marche pas, parce que Officier.Nom est alloué sur la pile et non sur le tas. Ce qui peut provoquer l'écrasement des données situés plus haut (ou plus bas, je me rappelle jamais...) sur la pile. Comme tu utilises une version "debug" des librairies standard, il y a des "assertions" qui vérifient ce genre de chose, d'où l'erreur. Si tu compiles ton programme en mode release, tu n'auras pas ce message d'erreur, par contre le programme va sans doute planter.

    Il faut donc déclarer Nom comme un char* pour pouvoir le modifier et le réallouer :

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
     
    struct t_Officier
    {
        char * Nom;
    }
    Enfin bref, c'est juste pour le cas où tu aurais fait cette erreur...

  5. #5
    Expert éminent sénior
    Avatar de Emmanuel Delahaye
    Profil pro
    Retraité
    Inscrit en
    Décembre 2003
    Messages
    14 512
    Détails du profil
    Informations personnelles :
    Âge : 67
    Localisation : France, Paris (Île de France)

    Informations professionnelles :
    Activité : Retraité

    Informations forums :
    Inscription : Décembre 2003
    Messages : 14 512
    Points : 20 985
    Points
    20 985
    Par défaut
    Citation Envoyé par beegees Voir le message
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    void* Nouveau = realloc(Officier.Nom,(strlen(Nom) + 1)*sizeof(char));
    j'obtiens ce message d'erreur (voir attaché svp)
    Les réponses des autres t'ont, j'espère, fait prendre conscience que l'on ne peut pas répondre précisément si on ne connait ni le type ni la valeur des données passées en paramètre. Il faut donc, et c'est le bon sens, non ? poster du code complet, réduit au minimum montrant le défaut.

    C'est d'ailleurs une stratégie de débogage bien connue...
    Pas de Wi-Fi à la maison : CPL

  6. #6
    Inactif  

    Profil pro
    Inscrit en
    Décembre 2002
    Messages
    534
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Décembre 2002
    Messages : 534
    Points : 403
    Points
    403
    Par défaut
    Salut,

    beegees écrit :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
     
    if ( (Nom == NULL) || (strlen(Nom) < 1) ) return false;
    C'est vrai cela ?

  7. #7
    Membre éprouvé
    Avatar de beegees
    Homme Profil pro
    Développeur Web
    Inscrit en
    Mars 2004
    Messages
    3 610
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 48
    Localisation : Belgique

    Informations professionnelles :
    Activité : Développeur Web
    Secteur : Enseignement

    Informations forums :
    Inscription : Mars 2004
    Messages : 3 610
    Points : 1 277
    Points
    1 277
    Par défaut
    Bonjour tout le monde,

    Tout d'abord, un super grand sorry pour le temps de ma réponse mais je n'ai PAS reçu de confirmation de réponse, chose que j'obtiens toujours normalement, je ne savais donc pas que vous aviez répondu.

    Merci pour vos réponses, je vois que le forum est toujours autant actif avec des gens motivé et volontaire.

    Je venais justement ajouter un complément d'information quand j'ai vu vos réponses qui se rapproche de ce que j'allais indiquer.

    Je vous met en attaché, le printscreen du programme qui ne fonctionne pas (officier.Nom) et d'un autre programme qui fonctionne (Commune.Nom) pour commune.Nom j'ai 0x0000000 et pour Officier.Nom j'ai 0xcccccccccc

    Je pense que le premier est NULL le second pas initialisé, me tompe-je

    Voici une bonne partie de mon code, je cherche depuis hier après-midi.

    Un super GRAND pour votre dévouement.

    beegees

    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
    19
    20
    21
    22
    23
    24
    25
    26
    27
    28
    29
    30
    31
    32
    33
    34
    35
    36
    37
    38
    39
    40
    41
    42
    43
    44
    45
    46
    47
    48
    49
    50
    51
    52
    53
    54
    55
    56
    57
    58
    59
    60
    61
    62
    63
    64
    65
    66
    67
    68
    69
    70
    71
    72
    73
    74
    75
    76
    77
    78
    79
    80
    81
    82
    83
    84
    85
    86
    87
    88
    89
    90
    91
    92
    93
    94
    95
    96
    97
    98
    99
    100
    101
    102
    103
    104
    105
    106
    107
    108
    109
    110
    111
    112
    113
    114
    115
    116
    117
    118
    119
    120
    121
    122
    123
    124
    125
    126
    127
    128
    129
    130
    131
    132
    133
    134
    135
    136
    137
    138
    139
    140
    141
    142
    143
    144
    145
    146
    147
    148
    149
    150
    151
    152
    153
    154
    155
    156
    157
    158
    159
    160
    161
    162
    163
    164
    165
    166
    167
    168
    169
    170
    171
    172
    173
    174
    175
    176
    177
    178
    179
    180
    181
    182
    183
    184
    185
    186
    187
    188
    189
    190
    191
    192
    193
    194
    195
    196
    197
    198
    199
    200
    201
    202
    203
    204
    205
    206
    207
    208
    209
    210
    211
    212
    213
    214
    215
    216
    217
    218
    219
    220
    221
    222
    223
    224
    225
    226
    227
    228
    229
    230
    231
    232
    233
    234
    235
    236
    237
    238
    239
    240
    241
    242
    243
    244
    245
    246
    247
    248
    249
    250
    251
    252
    253
    254
    255
    256
    257
    258
    259
    260
    261
    262
    263
    264
    265
    266
    267
    268
    269
    270
    271
    272
    273
    274
    275
    276
    277
    278
    279
    280
    #include <BasicConsole.h>
    #include "orionDD.h"
    #include "xml.h"
    #include <search.h> // _lfind
     
    void Orion_Initialiser(TOrion& Orion)
    {
    	TabVaisseaux_Initialiser(Orion.Vaisseaux);  //1
    }
     
    void TabVaisseaux_Initialiser(TTabVaisseaux& Tab)  //2
    {
    	Tab.Nombre = 0;
    	Tab.Tableau = NULL;
    }
     
    bool Orion_ChargerXML(TOrion& Orion, const char* NomFichier) //3
    {
     
    	TXmlDocument	Doc; //4
    	const long		nE0 = 0;	// Indice de l'élément racine
    	long			nE1;		// 1° boucle : Indice d'un élément enfant de la racine et s'appelant <competences>
    								// 2° boucle : Indice d'un élément enfant de la racine et s'appelant <officiers>
    								// 3° boucle : Indice d'un élément enfant de la racine et s'appelant <vaisseaux>
     
    	long			nE2;		// 1° boucle : Indice d'un élément enfant d'un élément d'indice nE1 et s'appelant <entite>
    								// 2° boucle : Indice d'un élément enfant d'un élément d'indice nE1 et s'appelant <commune>
    	long			nE3;		// 1° boucle : Indice d'un élément enfant d'un élément d'indice nE2 et s'appelant <commune>
    								// 2° boucle : Indice d'un élément enfant d'un élément d'indice nE2 et s'appelant <zone>
    	long			nE4;		// 2° boucle : Indice d'un élément enfant d'un élément d'indice nE3 et s'appelant <Vaisseau>
    	long			nA;			// Indice d'un attribut d'un élément
    	bool			Ok = false; //fin du 4
     
    	XmlDocument_Initialiser(Doc);  //5
    	if ( (XmlDocument_Charger(Doc,NomFichier)) && (strcmp(Doc.Elements.Tableau[nE0].Nom,"orion") == 0) ) //6
    	{
    		Ok = true; //Fin de 6
    		// Première boucle : pour charger les vaisseaux
    		for (nE1=nE0+1; (nE1 < Doc.Elements.Nombre) && (Doc.Elements.Tableau[nE1].IndiceParent >= nE0); nE1++) //7
    		{
    			TXmlElement& E1 = Doc.Elements.Tableau[nE1]; //Fin de 7
    			//ici on met le nom que l'on veut on n'est pas obligé de suivre le document xml
    			if ( (E1.IndiceParent == nE0) && (strcmp(E1.Nom,"vaisseaux") == 0) ) //8 Changer E1.Nom, "    "
    			{
    				for (nE2=nE1+1; (nE2 < Doc.Elements.Nombre) && (Doc.Elements.Tableau[nE2].IndiceParent >= nE1); nE2++)
    					{
    						TXmlElement& E2 = Doc.Elements.Tableau[nE2]; //9
    						if ( (E2.IndiceParent == nE1) && (strcmp(E2.Nom,"vaisseau") == 0) )
    							{
    								char* NomVaisseau = NULL;
    								for (nA=0; (nA < Doc.Attributs.Nombre) && (Doc.Attributs.Tableau[nA].IndiceElement < nE2); nA++);
    								for (; (nA < Doc.Attributs.Nombre) && (Doc.Attributs.Tableau[nA].IndiceElement == nE2); nA++)
    									{
    										TXmlAttribut& A = Doc.Attributs.Tableau[nA];
    										if (strcmp(A.Nom,"nom") == 0) NomVaisseau = A.Valeur;
    									}
    									//je tente ici d'ajouter une competence dans le tableau des competences
    									//dans le .h je passe TTabChose	
    									if (TabVaisseaux_Ajouter(Orion.Vaisseaux,NomVaisseau))//STOP ICI
    									{
    										TVaisseau& Vaisseau = Orion.Vaisseaux.Tableau[TabVaisseaux_Rechercher(Orion.Vaisseaux,NomVaisseau)];
    										for (nE3=nE2+1; (nE3 < Doc.Elements.Nombre) && (Doc.Elements.Tableau[nE3].IndiceParent >= nE2); nE3++) //fin de 8
    										{
    											TXmlElement& E3 = Doc.Elements.Tableau[nE3]; //9
    											if ( (E3.IndiceParent == nE2) && (strcmp(E3.Nom,"poste") == 0) )
    											{
    												char* NomOfficier = NULL;
    												for (nA=0; (nA < Doc.Attributs.Nombre) && (Doc.Attributs.Tableau[nA].IndiceElement < nE3); nA++);
    												for (; (nA < Doc.Attributs.Nombre) && (Doc.Attributs.Tableau[nA].IndiceElement == nE3); nA++)
    												{
    													TXmlAttribut& A = Doc.Attributs.Tableau[nA];
    													if (strcmp(A.Nom,"officier") == 0) NomOfficier = A.Valeur;
    												}
    												TabOfficier_Ajouter(Vaisseau.Officiers,NomOfficier);
     
    											}
    										}
     
     
     
     
     
     
     
    									}	
     
     
    						}
    					}
     
    			}
     
     
     
    		}
     
     
    	}
     
     
     
    	return true;
    }
     
     
    bool TabVaisseaux_Ajouter(TTabVaisseaux& Tab, const char* Nom)
    {
    	if (TabVaisseaux_Rechercher(Tab,Nom) >= 0) return true;
    	TVaisseau VaisseauAAjouter;
    	Vaisseau_Initialiser(VaisseauAAjouter);
    	if (!Vaisseau_Definir(VaisseauAAjouter,Nom))
    	{
    		//Commune_Detruire(CommuneAAjouter);
    		//return false;
    	}
    	void* Nouveau = realloc(Tab.Tableau,(Tab.Nombre+1)*sizeof(TVaisseau));
    	//if (Nouveau == NULL)
    	//{
    	//	Commune_Detruire(CommuneAAjouter);
    	//	return false;
    	//}
     	Tab.Tableau = (TVaisseau*)Nouveau;
    	Tab.Tableau[Tab.Nombre] = VaisseauAAjouter;
    	Tab.Nombre++;
     
    	return true;
     
    }
     
    void Vaisseau_Initialiser(TVaisseau& Vaisseau)
    {
    	Vaisseau.Nom = NULL;
    	TabOfficiers_Initialiser(Vaisseau.Officiers);
     
    }
     
    //cette fonction fonctionne avec la fonction TabVaisseaux_Comparer
    //il ne trouve pas mon vaisseau que je viens d'enregistrer
    long TabVaisseaux_Rechercher(const TTabVaisseaux& Tab, const char* Nom)
    {
     
    	if (Tab.Nombre == 0) return -1;
    	TVaisseau	VaisseauRecherche;
    	VaisseauRecherche.Nom = (char*)Nom;
    	TVaisseau* VaisseauTrouve = (TVaisseau*)_lfind(&VaisseauRecherche,Tab.Tableau,(unsigned int*)&(Tab.Nombre),sizeof(TVaisseau),Vaisseau_Comparer);
    	if (VaisseauTrouve == NULL) return -1;
    	return VaisseauTrouve - Tab.Tableau;
     
    }
     
     
    int Vaisseau_Comparer(const void* p1, const void* p2)
    {
    	const TVaisseau* e1 = (const TVaisseau*)p1;
    	const TVaisseau* e2 = (const TVaisseau*)p2;
    	return strcmp(e1->Nom,e2->Nom);
    }
     
    //cette fonction fonctionne avec la fonction TabVaisseaux_Rechercher
    //le problème est ici, il ne renvoie pas zéro
    //int Vaisseau_Comparer(const void* p1, const void* p2)
    //{
    //	const TVaisseau* r1 = (const TVaisseau*)p1;
    //	const TVaisseau* r2 = (const TVaisseau*)p2;
    //	if (r1->Nom < r2->Nom)
    //		return -1;
    //	else if (r1->Nom > r2->Nom)
    //		return 1;
    //	else
    //		return 0;
    //}
     
    void Orion_Afficher(TOrion& Orion)
    {
    	if (Orion.Vaisseaux.Nombre > 0)
    	{
    		printf("Vaisseau :\n");
    		for (long nE=0; nE < Orion.Vaisseaux.Nombre; nE++)
    		{
    			TVaisseau& E = Orion.Vaisseaux.Tableau[nE];
    			printf("- Vaisseau n°%ld :\n  - nom : %s\n",nE+1,E.Nom);
    			if (E.Officiers.Nombre > 0)
    			{
    				printf("  - communes :\n");
    				for (long nO=0; nO < E.Officiers.Nombre; nO++)
    				{
    					TOfficier& C = E.Officiers.Tableau[nO];
    					printf("    - commune n°%ld :\n      - nom : %s\n",nO+1,C.Nom);
    				}
    			}
    		}
    	}
    }
     
    bool	Vaisseau_Definir(TVaisseau& Vaisseau, const char* Nom)
    {
     
    	if ( (Nom == NULL) || (strlen(Nom) < 1) ) return false;
    	void* Nouveau = realloc(Vaisseau.Nom,(strlen(Nom) + 1)*sizeof(char));
    	if (Nouveau == NULL) return false;
    	Vaisseau.Nom = (char*)Nouveau;
    	strcpy(Vaisseau.Nom,Nom);
    	return true;	
    }
     
    bool	TabOfficier_Ajouter(TTabOfficier& Tab, const char* Nom)
    {
     
    	if (TabOfficiers_Rechercher(Tab,Nom) >= 0) return true;
    	TOfficier OfficierAAjouter;
    	Officier_Initialiser(OfficierAAjouter);
    	if (!Officier_Definir(OfficierAAjouter,Nom))
    	{
    		//Commune_Detruire(CommuneAAjouter);
    		//return false;
    	}
    	void* Nouveau = realloc(Tab.Tableau,(Tab.Nombre+1)*sizeof(TOfficier));
    	//if (Nouveau == NULL)
    	//{
    	//	Commune_Detruire(CommuneAAjouter);
    	//	return false;
    	//}
     	Tab.Tableau = (TOfficier*)Nouveau;
    	Tab.Tableau[Tab.Nombre] = OfficierAAjouter;
    	Tab.Nombre++;
     
    	return true;
     
    }
     
    void	Officier_Initialiser(TOfficier Officier)
    {
    	Officier.Nom = NULL;
    	TabCompetences_Initialiser(Officier.Competences); //ICIIIIIIIIIIIIIIIIIIIIIIIIIIIIIII
     
    }
     
    void TabCompetences_Initialiser(TTabCompetences& Tab)
    {
    	Tab.Nombre = 0;
    	Tab.Tableau = NULL;
    }
     
    bool	Officier_Definir(TOfficier& Officier, const char* Nom)
    {
     
    	if ( (Nom == NULL) || (strlen(Nom) < 1) ) return false;
    	void* Nouveau = realloc(Officier.Nom,(strlen(Nom) + 1)*sizeof(char));
    	if (Nouveau == NULL) return false;
    	Officier.Nom = (char*)Nouveau;
    	strcpy(Officier.Nom,Nom);
    	return true;	
     
    }
     
    void	TabOfficiers_Initialiser(TTabOfficier& Tab)
    {
     
    	Tab.Nombre = 0;
    	Tab.Tableau = NULL;
     
    }
     
    long	TabOfficiers_Rechercher(const TTabOfficier& Tab, const char* Nom)
    {
     
    	if (Tab.Nombre == 0) return -1;
    	TOfficier	OfficierRecherche;
    	OfficierRecherche.Nom = (char*)Nom;
    	TOfficier* OfficierTrouve = (TOfficier*)_lfind(&OfficierRecherche,Tab.Tableau,(unsigned int*)&(Tab.Nombre),sizeof(TOfficier),Officier_Comparer);
    	if (OfficierTrouve == NULL) return -1;
    	return OfficierTrouve - Tab.Tableau;
    }
     
    int Officier_Comparer(const void* p1, const void* p2)
    {
    	const TOfficier* c1 = (const TOfficier*)p1;
    	const TOfficier* c2 = (const TOfficier*)p2;
    	return strcmp(c1->Nom,c2->Nom);
    }
    voici aussi le code de ma structure :

    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
    19
    20
    21
    22
    23
    24
    25
    26
    27
    28
    29
    30
    31
    32
    33
    34
    35
    36
    37
    38
    39
    40
    41
    42
    43
    44
    45
    46
    typedef struct{
    	char*			Grade;
    	long			Valeur;
    }TBonus;
     
    typedef struct{
     
    	long			Nombre;
    	TBonus*			Tableau;
    }TTabBonus;
     
     
    typedef struct{
    	char*			Nom;
    	TTabBonus		Bonus;
    }TCompetence;
     
     
    typedef struct{
    	long			Nombre;
    	TCompetence*	Tableau;
    }TTabCompetences;
     
    typedef struct{
    	char*	Nom;
    	TTabCompetences	Competences;
    }TOfficier;
     
    typedef struct{
    	long			Nombre;
    	TOfficier*		Tableau;
    }TTabOfficier;
     
    typedef struct{
    	char*			Nom;
    	TTabOfficier	Officiers;
    }TVaisseau;
     
    typedef struct{
    	long			Nombre;
    	TVaisseau*		Tableau;
    }TTabVaisseaux;
     
    typedef struct{
    	TTabVaisseaux	Vaisseaux;
    }TOrion;

  8. #8
    Membre éprouvé
    Avatar de beegees
    Homme Profil pro
    Développeur Web
    Inscrit en
    Mars 2004
    Messages
    3 610
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 48
    Localisation : Belgique

    Informations professionnelles :
    Activité : Développeur Web
    Secteur : Enseignement

    Informations forums :
    Inscription : Mars 2004
    Messages : 3 610
    Points : 1 277
    Points
    1 277
    Par défaut
    Citation Envoyé par dj.motte Voir le message
    Salut,

    beegees écrit :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
     
    if ( (Nom == NULL) || (strlen(Nom) < 1) ) return false;
    C'est vrai cela ?
    Salut DJ,

    Je confirme que s'est VRAI.

    @+ et merci pour tona aide so precious.

    beegees

  9. #9
    Expert éminent sénior
    Avatar de Emmanuel Delahaye
    Profil pro
    Retraité
    Inscrit en
    Décembre 2003
    Messages
    14 512
    Détails du profil
    Informations personnelles :
    Âge : 67
    Localisation : France, Paris (Île de France)

    Informations professionnelles :
    Activité : Retraité

    Informations forums :
    Inscription : Décembre 2003
    Messages : 14 512
    Points : 20 985
    Points
    20 985
    Par défaut
    Citation Envoyé par beegees Voir le message
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
     
    void Orion_Initialiser(TOrion& Orion)
    {
    Hum... Le forum C++, c'est à coté... Je déplace ?
    Pas de Wi-Fi à la maison : CPL

  10. #10
    Membre éprouvé
    Avatar de beegees
    Homme Profil pro
    Développeur Web
    Inscrit en
    Mars 2004
    Messages
    3 610
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 48
    Localisation : Belgique

    Informations professionnelles :
    Activité : Développeur Web
    Secteur : Enseignement

    Informations forums :
    Inscription : Mars 2004
    Messages : 3 610
    Points : 1 277
    Points
    1 277
    Par défaut
    Citation Envoyé par diogene Voir le message
    La question est de savoir si Officier.Nom est à NULL (correct) ou à une adresse obtenue par allocation dynamique (correct).
    Sinon (pointeur non initialisé ?), le realloc plante.
    Salut,

    Merci pour ta réponse.

    Je pense en effet que Officier.Nom n'est pas sur NULL mais sur 0xcccccccc

    Vois-tu où je dois l'initialiser ?

    Merci encore.

    beegees

  11. #11
    Membre éprouvé
    Avatar de beegees
    Homme Profil pro
    Développeur Web
    Inscrit en
    Mars 2004
    Messages
    3 610
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 48
    Localisation : Belgique

    Informations professionnelles :
    Activité : Développeur Web
    Secteur : Enseignement

    Informations forums :
    Inscription : Mars 2004
    Messages : 3 610
    Points : 1 277
    Points
    1 277
    Par défaut
    Citation Envoyé par Emmanuel Delahaye Voir le message
    Hum... Le forum C++, c'est à coté... Je déplace ?
    Bonjour Emmanuel,

    Je termine mon cours de C Mercredi prochain ensuite je passe au C++.

    Serait-il stp possible de laisser ce message sur le forum de C et ne pas le déplacer en C++.

    Merci d'avance.

    beegees

  12. #12
    Expert éminent sénior
    Avatar de Emmanuel Delahaye
    Profil pro
    Retraité
    Inscrit en
    Décembre 2003
    Messages
    14 512
    Détails du profil
    Informations personnelles :
    Âge : 67
    Localisation : France, Paris (Île de France)

    Informations professionnelles :
    Activité : Retraité

    Informations forums :
    Inscription : Décembre 2003
    Messages : 14 512
    Points : 20 985
    Points
    20 985
    Par défaut
    Citation Envoyé par beegees Voir le message
    Je pense en effet que Officier.Nom n'est pas sur NULL mais sur 0xcccccccc
    C'est une valeur donnée par défaut par Visual C++ pour piéger les pointeurs non initialisés par le programmeur... Hé hé, ça marche !
    Vois-tu où je dois l'initialiser ?
    C'est une plaisanterie ?

    Avant toute utilisation (lecture, test, passage à une fonction), bien sûr.

    Tu ne sais pas ce que fais ton code ? Tu codes au hasard ? Tu n'as jamais initialisé une variable ?
    Pas de Wi-Fi à la maison : CPL

  13. #13
    Membre éprouvé
    Avatar de beegees
    Homme Profil pro
    Développeur Web
    Inscrit en
    Mars 2004
    Messages
    3 610
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 48
    Localisation : Belgique

    Informations professionnelles :
    Activité : Développeur Web
    Secteur : Enseignement

    Informations forums :
    Inscription : Mars 2004
    Messages : 3 610
    Points : 1 277
    Points
    1 277
    Par défaut
    Citation Envoyé par Emmanuel Delahaye Voir le message
    C'est une valeur donnée par défaut par Visual C++ pour piéger les pointeurs non initialisés par le programmeur... Hé hé, ça marche !

    C'est une plaisanterie ?

    Avant toute utilisation (lecture, test, passage à une fonction), bien sûr.

    Tu ne sais pas ce que fais ton code ? Tu codes au hasard ? Tu n'as jamais initialisé une variable ?
    J'ai une fonction qui initialise Officier.Nom à NULL :

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    void	Officier_Initialiser(TOfficier Officier)
    {
    	Officier.Nom = NULL;
    	TabCompetences_Initialiser(Officier.Competences); 
     
    }
    Je ne comprends pas pourquoi il n'est pas initialisé.

    Je mets en attaché plus d'info.

    beegees

  14. #14
    Expert éminent sénior
    Avatar de Médinoc
    Homme Profil pro
    Développeur informatique
    Inscrit en
    Septembre 2005
    Messages
    27 369
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 40
    Localisation : France

    Informations professionnelles :
    Activité : Développeur informatique
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Septembre 2005
    Messages : 27 369
    Points : 41 519
    Points
    41 519
    Par défaut
    En effet, en mode Debug 0xCCCCCCCC indique des variables non-initialisées.
    SVP, pas de questions techniques par MP. Surtout si je ne vous ai jamais parlé avant.

    "Aw, come on, who would be so stupid as to insert a cast to make an error go away without actually fixing the error?"
    Apparently everyone.
    -- Raymond Chen.
    Traduction obligatoire: "Oh, voyons, qui serait assez stupide pour mettre un cast pour faire disparaitre un message d'erreur sans vraiment corriger l'erreur?" - Apparemment, tout le monde. -- Raymond Chen.

  15. #15
    Expert éminent sénior
    Avatar de diogene
    Homme Profil pro
    Enseignant Chercheur
    Inscrit en
    Juin 2005
    Messages
    5 761
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Essonne (Île de France)

    Informations professionnelles :
    Activité : Enseignant Chercheur
    Secteur : Enseignement

    Informations forums :
    Inscription : Juin 2005
    Messages : 5 761
    Points : 13 926
    Points
    13 926
    Par défaut
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    void	Officier_Initialiser(TOfficier Officier)
    {
    	Officier.Nom = NULL;
    Officier est ici une variable locale. Si tu veux modifier la structure d'origine, il faut passer son adresse :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    void	Officier_Initialiser(TOfficier *  Officier)
    {
    	Officier->Nom = NULL;
    Publication : Concepts en C

    Mon avatar : Glenn Gould

    --------------------------------------------------------------------------
    Une réponse vous a été utile ? Remerciez son auteur en cliquant le pouce vert !

  16. #16
    Membre éprouvé
    Avatar de beegees
    Homme Profil pro
    Développeur Web
    Inscrit en
    Mars 2004
    Messages
    3 610
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 48
    Localisation : Belgique

    Informations professionnelles :
    Activité : Développeur Web
    Secteur : Enseignement

    Informations forums :
    Inscription : Mars 2004
    Messages : 3 610
    Points : 1 277
    Points
    1 277
    Par défaut
    Citation Envoyé par diogene Voir le message
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    void	Officier_Initialiser(TOfficier Officier)
    {
    	Officier.Nom = NULL;
    Officier est ici une variable locale. Si tu veux modifier la structure d'origine, il faut passer son adresse :
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    void	Officier_Initialiser(TOfficier *  Officier)
    {
    	Officier->Nom = NULL;
    Merci pour ta réponse diogene mais depuis le début de l'année nous faisaons :

    Officier.Nom = NULL; et ça a toujours fonctionné.

    Penses-tu que ça soit une variable locale car j'envoi ceci à la fonction d'initialisation :

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    TabOfficier_Ajouter(Vaisseau.Officiers,NomOfficier);
    Vaisseau vaut :

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    TVaisseau& Vaisseau = Orion.Vaisseaux.Tableau[TabVaisseaux_Rechercher(Orion.Vaisseaux,NomVaisseau)];
    Quand je fais ce que tu dis, j'obtiens ces messages d'erreur :

    error C2819: type 'TOfficier' does not have an overloaded member 'operator ->'
    et

    error C2227: left of '->Nom' must point to class/struct/union
    Qu'en penses-tu ?

    Merci encore.

    beegees

  17. #17
    Expert éminent sénior
    Avatar de diogene
    Homme Profil pro
    Enseignant Chercheur
    Inscrit en
    Juin 2005
    Messages
    5 761
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Essonne (Île de France)

    Informations professionnelles :
    Activité : Enseignant Chercheur
    Secteur : Enseignement

    Informations forums :
    Inscription : Juin 2005
    Messages : 5 761
    Points : 13 926
    Points
    13 926
    Par défaut
    Quand je fais ce que tu dis, j'obtiens ces messages d'erreur :

    error C2819: type 'TOfficier' does not have an overloaded member 'operator ->'
    Je pense que tu écris en C++. IL vaudrait mieux déplacer ces messages dans le forum C++. Veux-tu que je le fasse ?
    Publication : Concepts en C

    Mon avatar : Glenn Gould

    --------------------------------------------------------------------------
    Une réponse vous a été utile ? Remerciez son auteur en cliquant le pouce vert !

  18. #18
    Expert éminent sénior
    Avatar de Médinoc
    Homme Profil pro
    Développeur informatique
    Inscrit en
    Septembre 2005
    Messages
    27 369
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 40
    Localisation : France

    Informations professionnelles :
    Activité : Développeur informatique
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Septembre 2005
    Messages : 27 369
    Points : 41 519
    Points
    41 519
    Par défaut
    En effet, c'est du C++, comme je l'ai déjà dit ainsi qu'Emmanuel.
    Ce code utilise à outrance la fonctionnalité de passage par référence spécifique au C++ :
    Code C++ : Sélectionner tout - Visualiser dans une fenêtre à part
    void Orion_Initialiser(TOrion& Orion)
    Comprends-tu, beegees ?
    Ce code n'est pas du C, mais du C++.
    SVP, pas de questions techniques par MP. Surtout si je ne vous ai jamais parlé avant.

    "Aw, come on, who would be so stupid as to insert a cast to make an error go away without actually fixing the error?"
    Apparently everyone.
    -- Raymond Chen.
    Traduction obligatoire: "Oh, voyons, qui serait assez stupide pour mettre un cast pour faire disparaitre un message d'erreur sans vraiment corriger l'erreur?" - Apparemment, tout le monde. -- Raymond Chen.

  19. #19
    Membre éprouvé
    Avatar de beegees
    Homme Profil pro
    Développeur Web
    Inscrit en
    Mars 2004
    Messages
    3 610
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 48
    Localisation : Belgique

    Informations professionnelles :
    Activité : Développeur Web
    Secteur : Enseignement

    Informations forums :
    Inscription : Mars 2004
    Messages : 3 610
    Points : 1 277
    Points
    1 277
    Par défaut
    Citation Envoyé par Médinoc Voir le message
    En effet, c'est du C++, comme je l'ai déjà dit ainsi qu'Emmanuel.
    Ce code utilise à outrance la fonctionnalité de passage par référence spécifique au C++ :
    Code C++ : Sélectionner tout - Visualiser dans une fenêtre à part
    void Orion_Initialiser(TOrion& Orion)
    Comprends-tu, beegees ?
    Ce code n'est pas du C, mais du C++.
    Citation Envoyé par diogene Voir le message
    Je pense que tu écris en C++. IL vaudrait mieux déplacer ces messages dans le forum C++. Veux-tu que je le fasse ?
    Franchement je n'en peux RIEN si le prof nous enseigne le C de cette manière là.

    realloc s'est du C++ ?

    je vous dis que je fais un cours de C depuis septembre jusque janvier, je termine cette semaine avec c et dans deux semaines je commence c++.

    Si vous ne trouvez pas la réponse, ce n'est pas grave je vais chercher par moi même et si je trouve pas je demanderai au prof ce soir.

    Merci pour tout.

    beegees

  20. #20
    Expert éminent sénior
    Avatar de diogene
    Homme Profil pro
    Enseignant Chercheur
    Inscrit en
    Juin 2005
    Messages
    5 761
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Essonne (Île de France)

    Informations professionnelles :
    Activité : Enseignant Chercheur
    Secteur : Enseignement

    Informations forums :
    Inscription : Juin 2005
    Messages : 5 761
    Points : 13 926
    Points
    13 926
    Par défaut
    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    void	Officier_Initialiser(TOfficier & Officier)
    {
    	Officier.Nom = NULL;
    Ceci étant, tu as loupé qq chose dans la suite des cours et le prof est déjà passé au C++. Ton prof, même avec la plus mauvaise foi du monde, ne peut prétendre que c'est du C (je n'ose imaginer son incompétence si il le prétendait )
    Publication : Concepts en C

    Mon avatar : Glenn Gould

    --------------------------------------------------------------------------
    Une réponse vous a été utile ? Remerciez son auteur en cliquant le pouce vert !

+ Répondre à la discussion
Cette discussion est résolue.
Page 1 sur 2 12 DernièreDernière

Discussions similaires

  1. Réponses: 7
    Dernier message: 09/12/2014, 22h24
  2. [rad 6]problème utilisation assert
    Par link256 dans le forum Environnement de Développement Intégré (EDI)
    Réponses: 1
    Dernier message: 30/07/2008, 15h17
  3. Problème de réallocation de la mémoire
    Par dot-_-net dans le forum C++
    Réponses: 16
    Dernier message: 18/05/2008, 23h18
  4. Problème avec Assert
    Par Seb981 dans le forum Prolog
    Réponses: 16
    Dernier message: 20/05/2007, 17h39

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo