violation d'acces avec fclose et delete

violation d'acces avec fclose et delete - C++ - Programmation

Marsh Posté le 22-03-2005 à 15:07:30    

Bonjour a tous,
 
je travaille sous C++ builder 5.2, lorsque je compile mon code tout se passe bien, mais lorsque je le lance j'ai des violations d'acces et je ne comprend pas à quoi ça correspond comme problème: si mon code n'est pas bon ou si j'ai un problème de mémoire ou autre...
je vous met la partie du code qui me fait les violations d'accés, j'ai marqué en commentaire où elles se manifestent le plus souvent, mais c'est aléatoire :

Code :
  1. void MaClasse::MaFonction ()
  2. {
  4.  TWindow     *Window = NULL;
  5. TWindow     *WindowTest = NULL;
  6. char far * str = NULL;
  7. int maxCount, maxCountTest, verif;
  9. char filename[20];
  10. char recherche [40];
  11. int flag;
  12. FILE *fp = NULL;
  14. str = new char far;
  15. fp = new FILE;
  16. Window = new TWindow(this);
  17. WindowTest = new TWindow(this);
  19. flag = 0;
  21. //recherche du type de fichier
  22. WindowTest = GetWindowPtr(GetActiveWindow());
  23. maxCountTest = WindowTest->GetWindowTextLength();
  24. WindowTest->GetWindowText(str,maxCountTest);
  25. strcpy(recherche, str);
  27. delete WindowTest; // violation d'acces ici
  29. for (int i=0; i<40; i++)
  30. {
  31.  if (recherche[i]=='[')
  32.  {
  33.   if(recherche[i+1]=='L')
  34.   {
  35.    flag = 0;
  36.    break;
  37.   }
  38.   else if (recherche[i+1]=='C')
  39.   {
  40.    flag = 1;
  41.    break;
  42.   }
  43.   else
  44.   {
  45.    flag = 2;
  46.    break;
  47.   }
  49.  }
  50. }
  52. //traitement
  53. if (flag == 1)
  54. {
  55.  Window = GetActiveMDIChild()->GetClientWindow();
  56.  maxCount = Window->GetWindowTextLength();
  57.  strcpy(filename,"C:\\test\\test.txt" );
  59.  if ((fp = fopen(filename, "w+" )) == NULL)
  60.  {
  61.   fprintf(stderr, "Erreur à l'ouverture du fichier %s en mode w+.\n\n", filename);
  62.   exit(1);
  63.  }
  65.  verif = (Window->GetWindowText(str,maxCount));
  66.  if(verif!=0)
  67.  {
  68.   fprintf(fp, "%s", str);
  69.  }
  70.  else
  71.  {
  72.   fprintf(fp, "il n'y a pas de texte" );
  73.  }
  75.  fclose(fp);//violation d'acces ici
  76.  delete fp;//violation d'acces a tous les "delete"
  77.  delete str;
  78.  delete Window;
  79. }
  80. else
  81. {
  82. suite.....
  83. }
  84. }


 
Quelqu'un a t'il le courage de me donner un coup de main?? ;)
 
Merci


Message édité par barucca le 22-03-2005 à 18:48:51
Reply

Marsh Posté le 22-03-2005 à 15:07:30   

Reply

Marsh Posté le 24-03-2005 à 20:27:13    

1. Pour accéder à un fichier tu n'as pas à allouer un FILE, ni à le delete après. D'autant plus que FILE c'est du C et pas du C++, tout comme tes fprintf, etc...;
2. Pourquoi allouer WindowTest pour après le remplacer par un pointeur vers la fenêtre active et surtout essayer de supprimer cette fenêtre ?
 
Tu fais des new et des delete qui n'ont absolument rien à faire là ! Je ne saurais que trop te recommander de prendre le temps de lire un cours de C++ au plus vite.

Reply

Marsh Posté le 24-03-2005 à 20:54:47    

pourquoi tu fais un delete ?
 
pourquoi tu utilises les FILE* ? c'est du C !
pourquoi tu utilises les char* ? c'est du C !
 
et en plus comme t'y comprends rien, ben ton programme fait n'importe quoi !

Reply

Marsh Posté le 24-03-2005 à 20:55:07    

#include <fstream>
#include <string>
 
using namespace std;

Reply

Marsh Posté le 24-03-2005 à 22:02:50    

Barucca, soit tu utilises les fstream, soit à la rigueur tu te crées une classe fichier dans laquelle tu fais un fclose dans le destructeur (en prenant soin de pas lever d'exception).

Reply

Marsh Posté le 29-03-2005 à 15:14:19    

merci à ceux qui ont répondu gentiment.  
J'ai fini par trouver où été l'erreur principale, j'avais plusieurs erreurs, bien sur, dont des tableau avec de mauvaises tailles donc j'empiété sur le parametre suivant dans la pile. Bien qu'étant du C les FILE etc... peuvent fonctionner en C++.

Reply

Marsh Posté le 29-03-2005 à 15:47:25    

ça peut, mais c'est dégeux, contraire à l'esprit du C++ et terriblement casse gueule.

Reply

Sujets relatifs:

Leave a Replay

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