Discussion :

[Anouschka]

Bonjour,

Je dois réaliser un programme qui envoie un fichier d'un PC à un autre par un protocole particulier. Pour cela, j'ai des fonctions écrites en C que je peux adapter.

En début de main, je demande divers paramètres nécessaires à ma fonction d'envoi, dont le nom du fichier à envoyer sous forme d'un const char* pour respecter l'argument correspondant de fopen (j'ai bien sûr inclus le fichier en question dans mon répertoire de travail) :

Code :

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

1
2
	cout << "nom du fichier a envoyer (sans l'extension) "  << endl;
        cin >> &fichier; //stockage dans l'adresse pointée par fichier

J'appelle ensuite ma fonction d'envoi. La lecture du fichier se fait dans celle-ci par:

Code :

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

infile = fopen(fichier, "rt");

Tout ce beau code se compile parfaitement, à l'exception du warning :

"[C++ Avertissement] sendrou.cpp(26): W8030 Un temporaire a été utilisé pour le paramètre 'p' dans l'appel à 'istream::operator >>(void * &)'"

Par contre, au moment de l'exécution, mon PC se bloque sur la ligne ci-dessus, et je reçois le message d'erreur suivant :

Le projet rou_v_car.exe a provoqué une classe d'exception EAcessViolation avec le message 'Violation d'accès à l'adresse 7C9112B4. Lecture de l'adresse 00000100.' Processus stoppé.

Quelqu'un peut-il m'aider à résoudre mon problème svp :
Merci d'avance

[Caine]

Bonjour, un exemple de code avec fopen:

Code :

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

1
2
3
4
5
6
7
 
FILE *FrameFile;
 
if (!(FrameFile = fopen("Frame.txt","r")))
    return EXIT_FAILURE;
...
fclose(FrameFile);

Remarque 1: Le premier argument de fopen est un char *, utilises tu des strings C++ ?

2> fopen renvoie un FILE *, il faut avant de continuer tester que le fichier a été alloué. Car en fonction des droits d'accès et d'un certains nombre de paramètres, le valeur de retour est NULL et le code qui suit provoque une violation de mémoire.

[Anouschka]

Remarque 1: Le premier argument de fopen est un char *, utilises tu des strings C++ ?

1-Le programme original utilisait bien un char*, mais l'aide de mon IDE me disait que le premier argument de fopen était un const char*, d'où ma modification. Quelle est la différence svp?

2-C'est quoi un "string"?

2> fopen renvoie un FILE *, il faut avant de continuer tester que le fichier a été alloué. Car en fonction des droits d'accès et d'un certains nombre de paramètres, le valeur de retour est NULL et le code qui suit provoque une violation de mémoire.

C'est prévu dans la suite de ma fonction d'envoi, et je ne le vois pas s'afficher :

Code :

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

1
2
3
4
5
if (infile == NULL)
		{
		printf("Error: Could not open the file for reading.\n");
                ...
                 }

Code :

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

1
2
3
4
5
6
FILE *FrameFile;
 
if (!(FrameFile = fopen("Frame.txt","r")))
    return EXIT_FAILURE;
...
fclose(FrameFile);

L'ajout des guillemets aux bornes de mon nom de fichier provoque le même warning à la compilation. A l'exécution, le programme s'exécute normalement jusqu'à ce que j'entre le nom du fichier à envoyer. Ensuite la fenêtre se ferme

[Dje14]

Salut !
Si tu veux vraiment faire du C++, utilise <fstream> pour les fichiers, <string> pour les chaines.
Et pour récuperer le nom de fichier, utilises plutôt getline(cin,nomFichier) pour éviter de louper les noms de fichiers contenant des espaces,etc.

Par contre pour ce qui est de la communication, je suppose que c'est du code spécifique au système (qui est..?)

[TheRedLed]

Code :

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

cin >> &fichier; //stockage dans l'adresse pointée par fichier

Ce serait pas

Code :

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

cin >> fichier;

plutôt ?

Ton compilo te mets une erreur à la ligne 26 c'est quoi à la ligne 26

[Anouschka]

Code :

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

cin >> &fichier; //stockage dans l'adresse pointée par fichier

Ce serait pas

Code :

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

cin >> fichier;

plutôt ?

En effet Avec cette modification, tout se compile bien sans aucun warning, mais l'exécution plante dès que j'ai entré le nom de mon fichier avec le même message d'erreur que dans mon premier post.

[Caine]

Il nous faut le code après le if (infile == NULL) ...sinon, on ne pourra pas t'aider.

Question, ce if n'est pas exécuté, tu ne vois pas le message d'erreur s'afficher ?

[Anouschka]

Non, le message n'apparaît pas, donc je suppose qu'il n'est pas exécuté.

Le code c'est :

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
	if (infile == NULL)
		{
		printf("Error: Could not open the file for reading.\n");
		free(packet);
		return;
		}
	bytesread = 1; // initialisation de bytesread
 
	while ((!feof(infile)) && (bytesread < filesize))
		{
		/* Attempt to read in a chunk of bytes */
		count = fread(packet + bytesread + 2, sizeof(unsigned char),
		filesize - bytesread, infile);
 
		if (ferror(infile))
			{
			printf("Error: Could not read from file.\n");
			free(packet);
			return;
			}
 
		bytesread += count;
		}
 
	fclose(infile);

Je pense que ça devrait vous suffire.

[Caine]

Effectivement,

tu es sûre de cette portion du code:

Code :

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

1
2
3
 
count = fread(packet + bytesread + 2, sizeof(unsigned char), 
      filesize - bytesread, infile);

Notamment, que packet est alloué et avec la taille correcte, que l'expression packet + bytesread + 2 reste dans l'espace mémoire du tableau packet ?

C'est la seule source d'erreur possible dans ce code.

[Laurent Gomila]

eof est toujours mal utilisé, en l'occurence cela ne doit jamais être utilisé comme condition de fin de lecture pour une boucle. Voir la FAQ à ce sujet, ou la FAQ C++ lite si ça n'a pas encore été ajouté (me rappelle plus).

De toute façon, et comme ça t'a déjà été dit, en C++ on utilise :

- std::string pour les chaînes
- std::getline pour saisir des chaînes avec espaces
- std::*fstream pour les fichiers

C'est bien plus simple et sécurisé.

[fd]

tu la déclare comment ta var fichier ?
char *fichier ?

[Laurent Gomila]

EDIT : tiens, il n'y a plus la réponse qui disait "oui exactement".


J'ai l'impression que je vais me répéter, mais bon...

Si ton code est le suivant

Code :

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

1
2
char* fichier;
cin >> fichier;

Alors il est normal qu'il plante : il n'y a aucune mémoire allouée pour ta chaîne.

Si ton code est le suivant

Code :

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

1
2
char fichier[256]; // ou char* fichier = new char[256]
cin >> fichier;

Alors ça a des chances de fonctionner mais tu imposes une limite inutile à ton programme, qui finira un jour où l'autre par te causer du tord.

En C++ on utilise std::string pour les chaînes

Code :

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

1
2
3
4
5
6
7
8
#include <string>
 
std::string fichier;
cin >> fichier;
 
// Ou bien, si on veut pouvoir saisir des espaces dans le nom de fichier
 
std::getline(cin, fichier);

J'espère que tout ça est bien clair maintenant.

[Anouschka]

tu la déclare comment ta var fichier ?
char *fichier ?

Exactement.

Notamment, que packet est alloué et avec la taille correcte, que l'expression packet + bytesread + 2 reste dans l'espace mémoire du tableau packet ?

Il y a en effet quelque chose que je trouve étrange dans cette ligne : packet est un tableau de char et bytesread un unsigned long. Ca semble quand même bizarre qu'ils aient voulu ajouter des variables de types différents

Ceci dit, j'ai trouvé un autre moyen de transmettre mes fichiers (qui marche, celui-là), donc la réponse à ce sujet urge moins dorénavant. Mais j'aimerais quand même connaître la solution à mon problème, car je dois utiliser du C ou du C++ pour la suite de mon stage.
En tout cas merci beaucoup à tous vous êtes vraiment super

[bouazza92]

Salut tous , j'avais moi aussi cette erreur et voici la solution :
Si tu utilise une variable std::string , tu dois passer à la fonction fopen cet argument : fichier.c_str() et pas fichier !
ça plante car il y'avait une erreur d'accés à la mémoire donc à une variable , bah c'est la seul solution que je connais .

@+ Bouazza