Correction de mon code...

Correction de mon code... - C++ - Programmation

Marsh Posté le 16-05-2003 à 00:46:48    

Salut,
 
alors histoire de vérifier tout de suite que j'ai pas fait de grosses boulettes, je voudrais soumettre mon code à vos yeux aguerris...
 
Je tiens à préciser que ce code fonctionne bien tel quel. Ce que je veux juste savoir, c'est s'il ne fonctionnerait pas malgré quelques (?) erreurs dont je n'aurais pas conscience...
 
Tout commence avec la déclaration des structures:

Code :
  1. struct ContenuUneStatUnXML
  2. {
  3.   AnsiString NomStat;
  4.   vector<int> VecUneStatUnXML; //Contient chaque Frame
  5. };
  7. struct ContenuTtesStatUnXML
  8. {
  9.   AnsiString PathXML;
  10.   vector<ContenuUneStatUnXML> VecTtesStatUnXML;
  11. };


 
Ensuite voilà seulement les fonctions concernées:

Code :
  1. void __fastcall TListeMatchesForm::CreerClipClick(TObject *Sender)
  2. {
  3. bool AUnParent = false;
  4. vector<int> VecSelectedIndex;
  5. vector<int> VecSelectedStats;
  7. VecSelectedIndex.clear();
  8. VecSelectedStats.clear();
  10. if (ListeMatches->Selected->Parent)
  11.   AUnParent = true;
  13. //On remplit VecSelectedIndex avec l'index de chacun des fichiers sélectionnés
  14. if (AUnParent)
  15.   VecSelectedIndex.push_back(ListeMatches->Selected->Parent->Index);
  16. else
  17.   for (unsigned int i=0; i < ListeMatches->SelectionCount; i++)
  18.   VecSelectedIndex.push_back(int(ListeMatches->Selections[i]->Index));
  20. //On remplit VecSelectedStats avec l'index de chacune des stats choisies
  21. for (int i=0; i < ListeStats->Items->Count; i++)
  22.   if (ListeStats->Selected[i])
  23.     VecSelectedStats.push_back(i);
  25. if (MakeClip(VecSelectedIndex,VecSelectedStats))
  26.   ShowMessage("Le clip est réalisé" );
  27. }


 

Code :
  1. bool __fastcall TListeMatchesForm::MakeClip(
  2.     vector<int> &VecSelectedIndex,
  3.     vector<int> &VecSelectedStats)
  4. {
  5. NomSelectionne = "";
  6. //Si on a choisi une équipe, EquipeChoisie = true, sinon false
  7. EquipeChoisie = true;
  9. if (EquipeChoisie = (ListeJoueurs->ItemIndex == -1))
  10.   NomSelectionne = ListeEquipes->Items->Strings[ListeEquipes->ItemIndex];
  11. else
  12.   NomSelectionne = ListeJoueurs->Items->Strings[ListeJoueurs->ItemIndex];
  14. //On va créer VecTtesStatsTsXML qui contiendra toutes les données
  15. vector<ContenuTtesStatUnXML> VecTtesStatsTsXML;
  16. VecTtesStatsTsXML.clear();
  18. for (unsigned int i=0; i < VecSelectedIndex.size(); i++)
  19. {
  20.   int IndexDsContenu = VecSelectedIndex[i];
  22.   //On enregistre le path du XML qui va être analysé
  23.   AnsiString PathXML = VecListeMatches[IndexDsContenu].PathFichierStats;
  25.   //On va chercher dans les fichiers de stats choisis les données
  26.   //pour le clip vidéo
  27.   if (!RemplirVecTSTX(VecTtesStatsTsXML,VecSelectedStats,PathXML))
  28.     return false;
  29. }
  31. return true;
  32. }


 

Code :
  1. bool __fastcall TListeMatchesForm::RemplirVecTSTX(
  2.     vector<ContenuTtesStatUnXML> & VecTtesStatsTsXML,
  3.     vector<int> & VecSelectedStats,
  4.     AnsiString PathXML)
  5. {
  6. //On crée une nouvelle structure qui contiendra le nom de la stat et les
  7. //frames correspondantes pour un fichier
  8. ContenuUneStatUnXML CUSUX;
  10. //On crée une nouvelle structure qui contiendra le path du fichier stat
  11. //et une structure CUSUX pour chaque stat choisie
  12. ContenuTtesStatUnXML CTSUX;
  14. //On place le path du XML des stats pour pouvoir le distinguer des autes
  15. //CTSUX
  16. CTSUX.PathXML = PathXML;
  18. XMLStats->FileName = PathXML;
  19. XMLStats->Active = true;
  21. if (!XMLStats->Active)
  22.   return false;
  24. _di_IXMLNode PtrContenu = XMLStats->DocumentElement;
  26. //On scanne dans l'ordre:
  28. // * les équipes
  29. int NbreJoueurs = PtrStaff->ChildNodes->GetCount();
  30. int NbreStatsChoisies = VecSelectedStats.size();
  31. int NbreStats = PtrStatGlob->ChildNodes->GetCount();
  33. for (int i=0; i < 2; i++)
  34. {
  35.   _di_IXMLNode PtrEquipe = PtrContenu->ChildNodes->GetNode(i);
  36.   AnsiString NomEquipe = PtrEquipe->GetAttribute("Equipe" );
  38.   _di_IXMLNode PtrStaff = PtrEquipe->ChildNodes->GetNode(0);
  40.   //On remet à zéro la liste des frames
  41.   CUSUX.VecUneStatUnXML.clear();
  43.   // * les joueurs
  44.   for (int j=0; j < NbreJoueurs; j++)
  45.   {
  46.     _di_IXMLNode PtrJoueur = PtrStaff->ChildNodes->GetNode(j);
  47.     AnsiString NumNomJoueur = PtrJoueur->GetAttribute("Nom" );
  49.     char * NomJoueur = NumNomJoueur.c_str();
  50.     NomJoueur = strchr(NomJoueur,' ') + 1;
  52.     //Si on a choisi un joueur, on arrête la boucle si on n'a
  53.     //pas choisi une équipe et si le nom scanné correspond
  54.     //au joueur choisi
  55.     if (EquipeChoisie || (AnsiString(NomJoueur) == NomSelectionne))
  56.     {
  57.     //On remet à zéro la liste des stats déjà scannées
  58.     CTSUX.VecTtesStatUnXML.clear();
  60.     // * les categories de stats
  61.     for (unsigned int k=0; k < NbreStatsChoisies; k++)
  62.     {
  63.       _di_IXMLNode PtrStatGlob = PtrJoueur->ChildNodes->GetNode(VecSelectedStats[k]);
  65.       //On ajoute à CUSUX le nom de la stat
  66.       //pour la distinguer
  67.       CUSUX.NomStat = pListeStats[VecSelectedStats[k]];
  69.       // * les stats individuellement
  70.       for (int l=0; l < NbreStats; l++)
  71.       {
  72.         _di_IXMLNode PtrStat = PtrStatGlob->ChildNodes->GetNode(l);
  74.         int FrameStat = PtrStat->GetNodeValue();
  76.         //On ajoute au vecteur de CUSUX la
  77.         //Frame actuelle
  78.         CUSUX.VecUneStatUnXML.push_back(FrameStat);
  79.       }
  81.       //On ajoute au vecteur de CTSUX la catégorie
  82.       //de stat scannée juste avant
  83.       CTSUX.VecTtesStatUnXML.push_back(CUSUX);
  84.     }
  85.   }
  86. }
  88. //Si on a choisi une équipe, on arrête la boucle quand le nom
  89. //scanné correspond au nom choisi
  90. if (EquipeChoisie && (NomEquipe == NomSelectionne))
  91.   break;
  92. }
  94. //On ajoute au vecteur qui contient toutes les données le résultat du scan
  95. //de ce fichier XML de stats...
  96. VecTtesStatsTsXML.push_back(CTSUX);
  98. return true;
  99. }


 
Si vous pouviez me dire quoi, notamment au niveau des passages par référence, notamment pour la dernière fonction RemplirVecTSTX:
Je déclare CUSUX et CTSUX dans cette fonction... hors, après l'appel à cette fonction dans MakeClip, je dois récupérer tout ce qui a été enregistré dans ces structures...
 
Que vaut-il mieux que je fasse:
Comme maintenant, et après l'appel à RemplirVecTSTX redéclarer CUSUX et CTSUX? J'ai essayé, ça fonctionne, mais je ne sais pas si c'est la manière de procéder...
 
Ou bien déclarer CUSUX et CTSUX avant d'appeller RemplirVecTSTX, et passer ces deux structures par référence?
 
J'attends vos avis...
 
Merci d'avance!!


---------------
Another .Net Blog
Reply

Marsh Posté le 16-05-2003 à 00:46:48   

Reply

Marsh Posté le 16-05-2003 à 09:50:39    

j'ai rien compris: t'es sur que ton code nous est utile? c'est pas plutot un problème de conception?

Reply

Marsh Posté le 16-05-2003 à 19:03:08    

Effectivement ce code ne vous servira pas à grand chose...
 
Il sert à parcourir un fichier XML pour récupérer des données qui y sont contenues.
 
Ce que je voulais juste savoir, c'est si mes références dans mes paramètres de fonctions sont correctes, et pour quel choix je devrai opter concernant ma dernière question:
 
déclarer les structures avant l'appel à RemplirVecTXTS et utiliser des références à ces structures dans cette fonction
 
ou bien les déclarer dans la fonction pour remplir les vecteurs de ces structures, et les déclarer une nouvelle fois dans la fonction appellante pour pouvoir exploiter le contenu de ces vecteurs...
 
Voilà tout...


---------------
Another .Net Blog
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed