Discussion :

[kryptong]

Bonjour;
J'ai fais une liste linéaire chaînée de véhicules ; puis je dois écrire un code pour pouvoir supprimer un véhicule bien déterminé ; sauf que je rencontre DEUX problèmes :
1/ Si ma liste contient UN seul véhicule lorsque je clique sur supprimer la console se plante
2/Si ma liste contient plusieurs véhicules , lorsque je veux supprimer le dernier véhicule que j'ai saisi dans la liste le premier se supprime aussi; ci dessous mon code merci de vouloir bien m'aider

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
case 2:
cout << "nVeuillez saisir l'identifiant du vehicule a supprimer:n";
cin >> idt;
 
 
if (l->GetIdentifiant() == idt)
    l = l->lien;		// l c'est ma liste
q = l;
if (q == NULL)
    cout << "liste vide";
 
else
    while ((q->lien != NULL) & (q->GetIdentifiant() != idt)) {
	p = q;
	q = q->lien;
    }
 
if (nombre_vehi == 1)
    l->lien = NULL;
else
    p->lien = q->lien;
 
_getch();
break;

[panthere noire]

salut
Quand sa plante sa dit quoi ?

[LittleWhite]

Bonjour,

Lorsque cela plante, je conseille l'utilisation du débogueur (souvent intégrer dans l'IDE)

[kryptong]

Bonjour,
Bah c'est la console qui se plante donc ca dis rien ca bloque et pour continuer je dois fermer visual studio et reouvrir

[LittleWhite]

Euh ...

Vous êtes sur qu'il ne suffit pas d'appuyer sur le bouton stop ? (un carré bleu)

[Sunsawe]

Bonjour,
Bah c'est la console qui se plante donc ca dis rien ca bloque et pour continuer je dois fermer visual studio et reouvrir

Boucle sans fin?

[Sunsawe]

Salut,

Je pense que pour qu'on puisse te répondre, il faut que tu prennes en compte le fait... qu'on ne peut absolument pas savoir ce que tu as en tête. Du coup, lorsque que tu donnes un bout de code, (ce qui est nécessaire) assure toi qu'il soit le plus complet possible.

Ici, tu fournis juste un "case".
On ne peut pas savoir si tu en as 65 autre avant ou après et l'erreur pourrait se trouver en amont tout comme en aval de ton code. Il y a même dedans une floppée de variables dont on ne sait rien et dont le nom n'est pas explicite.
Même si dans ton code tu utilises des noms de variable courts (ce que je ne te conseille pas vu le risque de s'y perdre après avec le nombre croissant de variables), renomme les de manière explicite dans ton post. Par exemple:
"pointeur_sur_ma_liste" plutot que "l", ça semble exagéré, mais au moins c'est clair pour le lecteur.
Ou au moins, ajoute des commentaires explicites.

Maintenant, j'essai de comprendre ton code et avant de pouvoir chercher une erreur possible, une chose m'interpelle: je n'y comprend rien. Je m'explique:

if (l->GetIdentifiant() == idt)
l = l->lien; // l c'est ma liste

si l est un pointeur sur la liste, je m'attends à ce que "l->GetIdentifiant()" renvoie l'identifiant de la liste. Donc si je demande un identifiant de véhicule à l'utilisateur, pas étonnant que ça ne corresponde jamais. à moins que:
- le vehicule soit représenté comme une liste (d'autres éléments par exemple)
- l est un pointeur sur un vehicule dans la liste et pas sur la liste elle-même
- la liste garde un repère sur le véhicule courant (comme un descriptor) et renvoie son identifiant
ect
Et chacune de ces interprétations a des implications pour la compréhension du reste du code...

Finallement, j'essaie de comprendre ce que ton code fait et voilà ce que j'en tire:

- Demande identifiant de véhicule iv à l'utilisateur
- Si idenfiant de liste correspond à iv, avance le pointeur sur la liste sur le lien de la liste (plusieurs listes???)
- q = l
- Si q est null affiche "liste vide".
Sinon tant que le lien de q n'est pas vide et que son identifiant ne correspond pas à iv sauvegarde q dans p et avance q sur son lien.
- Si il y a un véhicule le lien de la liste est null
Sinon le lien de p est égal au lien de q.
(J'interprète le lien de l'avant dernière liste est égale au lien de la dernière liste et donc est null)

Au final, j'ai pris le temps d'écrire autant pour te dire qu'on ne comprend pas ce que tu cherches à faire avec ce code, qu'il ne semble pas correspondre à la problématique que tu exposes et donc je ne vois pas comment t'aider.