Overflow in implicit constant conversion: Veut dire?

Fermé

Conversion

Linquisiteur - 26 avril 2013 à 21:33
Linquisiteur - 29 avril 2013 à 22:11

Bonjour, j'écris une fonction qui lit du "faux hexa" dans un fichier texte. Le 1er hexa étant la taille. Mais j'ai quelque problème lors de la compilation:

Je vous 2 possibles erreurs, ma ligne:
if(c>='A' || c<='F') c=('F'-'lettre'+15);
Mais je ne vois pas comment faire autrement.
et mon fwrite (qui est le 1er que je fait donc surement une erreur)
fwrite(lettre, 1 * sizeof(int), 1, fichier_Bin);

lab.c:45: warning: overflow in implicit constant conversion
lab.c:69: warning: passing arg 1 of 'fwrite' makes pointer from integer without a cast
lab.c:87: error: parse error at end of input

#include <stdio.h>
#include <stdlib.h>
#include <sys/wait.h>
#include <unistd.h>
#include <sys/types.h>
#include <string.h>
#include <fcntl.h>

#define fichierTexteHexa "hexa2.txt"
#define fichierBinaire "fichier.dat"
#define fichieTexte "fichier.txt"

/**
* Pré:
* @param fichierBin contier l'adresse d'un fichier binaire
* @param fichierHex contient un ficher texte contenant de l'hexadecimal
* Post:
* @param fichierBin contiendra l'hexa traitée
* @param fichierHex est inchangé
**/
void HexaToBin(char * fichierBin, char * fichierHex){
	
	FILE * fichier_Bin;
		//on ouvre le fichier binaire pour y écrire si il y a quelque chose on écrase les info
		if ((fichier_Bin = fopen(fichierBin,"wb"))==NULL) {
        perror("Write "); 
        exit(1);
		}
	
	FILE * fichier_Hex;
		
		//on ouvre le fichierHex pour lire les info dedans
		if ((fichier_Hex = fopen(fichierHex,"r"))==NULL) {
        perror("Read "); 
        exit(1);
		}
		int taille=0,i=0,j=0;
	    char c;
		
		/*temps que on n'as pas atteint la fin du fichier on lit*/
		while(c!=EOF){ 

		c=fgetc(fichier_Hex);  //on lit le charactère où se trouve le pointeur dans le fichier
			
			if(c>='A' || c<='F') c=('F'-'lettre'+15); //on transforme les lettres en chiffres
			
			if(i<2){ //les 2 premiers char donne la taille de la ligne
			taille=taille+c;
			i++;
			}
			if(i==2){
				int test_pair=taille%2; //on vérifie si la ligne contient un nombre paire ou impair de char
					if(test_pair==1) taille--;
			}
			
			if(i==2){
			int lettre;
			int l;
				while(j<taille){
				 lettre=fgetc(fichier_Hex); 
					if (l<2){
					lettre=lettre+c;
					l++;
					if(l==1) lettre=lettre*16;
					}
				    
					if (l==2){
					//fwrite(fichier_Bin,"%d",lettre);
					fwrite(lettre, 1 * sizeof(int), 1, fichier_Bin);  
					l=0;
					lettre=0;
					}
				 j++;
				 i=0;
				}
			}
		
        } 

int main (void){

	printf("Travail n°4, en C\n");
	
	HexaToBin(fichierBinaire,fichierTexteHexa);

return 0;
}

J'ai poster l'intégralité du code au cas où mais je suppose que seul les 2 lignes cité contiennent une erreur. J'ai lu de la docu sure le fwrite mais je suis pas sur de ma ligne

A voir également:

Warning: overflow in implicit constant conversion [-woverflow]
Aux in - Forum Audio
Dc in - Forum Audio
Deco in paris avis ✓ - Forum Consommation & Internet
In da place - Forum Mail
Light in the box problème ✓ - Forum Consommation & Internet

3 réponses

Réponse 1 / 3

fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 835
27 avril 2013 à 10:25

if(c>='A' || c<='F') c=('F'-'lettre'+15);
C'est quoi lettre ?
On ne peut mettre qu'un caractère entre apostrophe. Cela correspond au code ascii de la lettre. Donc, 'lettre' est faux. Si tu parles de la variable lettre, il faut enlever les apostrophes. Par contre, ta variable lettre est définie plus bas dans le code, donc ça risque de ne pas plaire au compilateur.

fwrite(lettre, 1 * sizeof(int), 1, fichier_Bin);
A toutes fins utiles : http://www.linux-kheops.com/doc/man/manfr/man-ascii-0.9/man3/fwrite.3.txt.html
Tu vois que le prototype de fwrite est : size_t fwrite (const void *ptr, size_t size, size_t nmemb,
FILE *stream);
Ta variable lettre est de type int => Erreur. Il faut un pointeur pour que le programme aille lire dedans.

Cdlt,

Réponse 2 / 3

Linquisiteur
27 avril 2013 à 12:22

J'ai modifier mon code car oui je m'étais trompé dans certaine partie de mon algorithme.
Mais je ne comprend pas comment écrire l'adresse d'un int? j'ai essayer *lettre comme un tableau mais sans succès. Ce que je recherche avec mon:

if(lettre>='A' || lettre<='F') lettre=('F'-lettre+15);

est de convertir l'hexadécimal en un simple chiffre que j'additionnerai avec le second chiffre hexadécimal. Par exemple je reçois 6 et bien pas besoin de reconvertir mais ci je reçois A et bien ça me ferra code ascii de F - code ascii de A = -5+15 = 10.
Ma nouvelle erreur:

lab.c: In function 'HexaToBin':
lab.c:71: error: invalid type argument of 'unary *'
lab.c:89: error: parse error at end of input

#include <stdio.h>
#include <stdlib.h>
#include <sys/wait.h>
#include <unistd.h>
#include <sys/types.h>
#include <string.h>
#include <fcntl.h>

#define fichierTexteHexa "hexa2.txt"
#define fichierBinaire "fichier.dat"
#define fichieTexte "fichier.txt"

/**
* Pré:
* @param fichierBin contier l'adresse d'un fichier binaire
* @param fichierHex contient un ficher texte contenant de l'hexadecimal
* Post:
* @param fichierBin contiendra l'hexa traitée
* @param fichierHex est inchangé
**/
void HexaToBin(char * fichierBin, char * fichierHex){
	
	FILE * fichier_Bin;
		//on ouvre le fichier binaire pour y écrire si il y a quelque chose on écrase les info
		if ((fichier_Bin = fopen(fichierBin,"wb"))==NULL) {
        perror("Write "); 
        exit(1);
		}
	
	FILE * fichier_Hex;
		
		//on ouvre le fichierHex pour lire les info dedans
		if ((fichier_Hex = fopen(fichierHex,"r"))==NULL) {
        perror("Read "); 
        exit(1);
		}
		int taille=0,i=0,j=0;
	    int c;
		
		/*temps que on n'as pas atteint la fin du fichier on lit*/
		while(c!=EOF){ 

		c=fgetc(fichier_Hex);  //on lit le charactère où se trouve le pointeur dans le fichier
			
			if(c>='A' || c<='F') c=('F'-c+15); //on transforme les lettres en chiffres
			
			if(i<2){ //les 2 premiers char donne la taille de la ligne
			taille=taille+c;
			i++;
			}
			if(i==2){
				int test_pair=taille%2; //on vérifie si la ligne contient un nombre paire ou impair de char
					if(test_pair==1) taille--;
			}
			
			if(i==2){
			int lettre;
			int l;
			int temp;
				while(j<taille){
				 lettre=fgetc(fichier_Hex);
				 if(lettre>='A' || lettre<='F') lettre=('F'-lettre+15);
					if (l<2){
					l++;
					if(l==1) temp=lettre*16;
					}
				    
					if (l==2){
					lettre=lettre+temp;
					//fwrite(fichier_Bin,"%d",lettre);
					fwrite(*lettre, 1 * sizeof(int), 1, fichier_Bin);  
					l=0;
					lettre=0;
					}
				 j++;
				 i=0;
				}
			}
		
        } 

int main (void){

	printf("Travail n°4, en C\n");
	
	HexaToBin(fichierBinaire,fichierTexteHexa);

return 0;
}

fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 835
27 avril 2013 à 12:45

if(lettre>='A' || lettre<='F') lettre=('F'-lettre+15);
Ce n'est pas un OU qu'il faut mettre mais un ET. Sinon, la condition sera toujours vraie. Sinon, je verrai plutôt : lettre=lettre-'A'+10; Sinon, ça ne marchera pas.

Je verrai plutôt : fwrite(&lettre, 1 * sizeof(int), 1, fichier_Bin); C'est l'esperluette & qui renvoie l'adresse d'une variable. Mais pourquoi, ne pas lire avec fscanf ? Ca sera plus simple.

Linquisiteur
27 avril 2013 à 14:35

merci ça compile :)
Effectivement j'ai encore fait quelque erreur dans l'algo. Par contre mon fichier binaire est incompréhensible (normal en même temps) comment savoir si j'ai bon ou mauvais ?

J'avais vu un de mes profs avec un programme qui permettait de lire les fichier binaire mais je ne me souviens plus du nom (ou sinon si il existe une commande unix)

Bon je vais continuer mon devoir merci ^^

fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 835
27 avril 2013 à 14:40

De quel fichier binaire parles-tu ? Du fichier généré lors de la compilation de ton programme (celui que tu exécutes) ou de fichier.dat (qui correspond à fichierBinaire) ?

Linquisiteur
27 avril 2013 à 14:45

du fichier.dat rempli dans la fonction

fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 835
27 avril 2013 à 15:02

Comment savoir si tu as bon ? En faisant un programme pour le lire avec fread() :-).
Sinon peut-être avec KHexEdit ou xxd. Si cela ne marche pas, je te conseille de demander sur le forum Linux (car ce n'est plus du C).

Réponse 3 / 3

Linquisiteur
27 avril 2013 à 21:41

J'ai corriger encore de nombreuse erreur mais voila que je boucle sans arrêt alors j'ai ajouter une modification pour finir mais toujours rien dans mon fichier.txt
Pourtant ca devrait enregistrer mais j'ai un problème avec la lecture de taille rien n'y fait j'ai des problèmes partout:

taille 1/taille 2/ligne, 119J vaut 2taille 5/ligne, 102J vaut 2ligne, 238J vaut 4ligne, 34J vaut 6taille 0/ligne, 102J vaut 2ligne, 85J vaut 4ligne, 119J vaut 6taille 8/ligne, 102J vaut 2ligne, 85J vaut 4ligne, 102J vaut 6ligne, 221J vaut 8ligne, 119J vaut 10ligne, 0J vaut 12ligne, 102J vaut 14taille 12/ligne, 102J vaut 2ligne, 85J vaut 4ligne, 34J vaut 6ligne, 0J vaut 8ligne, 119J vaut 10ligne, 51J vaut 12ligne, 102J vaut 14ligne, 153J vaut 16ligne, 102J vaut 18ligne, 221J vaut 20ligne, 119J vaut 22ligne, 0J vaut 24ligne, 102J vaut 26taille 12/ligne, 102J vaut 2ligne, 85J vaut 4ligne, 34J vaut 6ligne, 238J vaut 8ligne, -833J vaut 10ligne, -833J vaut 12ligne, -833J vaut 14ligne, -833J vaut 16ligne, -833J vaut 18ligne, -833J vaut 20ligne, -833J vaut 22ligne, -833J vaut 24ligne, -833J vaut 26ligne, -833J vaut 28ligne, -833J vaut 30ligne, -833J vaut 32ligne, -833J vaut 34ligne, -833J vaut 36ligne, -833J vaut 38taille -49/b

#include <stdio.h>
#include <stdlib.h>
#include <sys/wait.h>
#include <unistd.h>
#include <sys/types.h>
#include <string.h>
#include <fcntl.h>

#define fichierTexteHexa "hexa2.txt"
#define fichierBinaire "fichier.dat"
#define fichierTexte "fichier.txt"

/**
* Pré:
* @param fichierBin contier l'adresse d'un fichier binaire
* @param fichierHex contient un ficher texte contenant de l'hexadecimal
* Post:
* @param fichierBin contiendra l'hexa traitée
* @param fichierHex est inchangé
**/
void HexaToBin(char * fichierBin, char * fichierHex){
	
	FILE * fichier_Bin;
		//on ouvre le fichier binaire pour y écrire si il y a quelque chose on écrase les info
		if ((fichier_Bin = fopen(fichierBin,"wb"))==NULL) {
        perror("Write "); 
        exit(1);
		}
	
	FILE * fichier_Hex;
		
		//on ouvre le fichierHex pour lire les info dedans
		if ((fichier_Hex = fopen(fichierHex,"r"))==NULL) {
        perror("Read "); 
        exit(1);
		}
		int taille=0,i=1,j=0;
	    int c;
		
		/*temps que on n'as pas atteint la fin du fichier on lit*/
		while(c!=EOF){ 

		c=fgetc(fichier_Hex);  //on lit le charactère où se trouve le pointeur dans le fichier
		
			if(c>='A' && c<='F') c=c-'A'+10; //on transforme les lettres en chiffres
			else c=c-'0';
			if(i=1){ //les 2 premiers char donne la taille de la ligne
			taille*16;
			i++;
			}
			if(i==2){
					taille=taille+c;
					int test_pair=taille%2; //on vérifie si la ligne contient un nombre paire ou impair de char
					if(test_pair==1) taille--;
					printf("taille %d\n",c);
			if (c==-49) c=EOF;
			
			int lettre;
			int l=1;
			int temp;
				while(j<taille){
				 lettre=fgetc(fichier_Hex);
				 j++;
				 if(lettre>='A' && lettre<='F') lettre=lettre-'A'+10;
				 
				 else lettre=lettre-'0';
					
					if(l==1) {
					temp=lettre*16;
					l++;
					}
				    
					if (l==2){
					lettre=lettre+temp;
					fwrite(&lettre, 1 * sizeof(int), 1, fichier_Bin);
					printf("ligne, %d",lettre);
					l=1;
					lettre=0;
					}
				 j++;
				 printf("J vaut %d",j);
				 i=0;
				}
			j=0;
			}

        } 
	fclose(fichier_Hex);
	fclose(fichier_Bin);
	}

/**
* Pré:
* @param fichierBin contier l'adresse d'un fichier binaire remplie
* @param fichierTxt contient un ficher texte vide où remplie (mais sera écraser);
* Post:
* @param fichierBin est inchangé
* @param fichierTxt contiendra du texte
**/
void BinToTexte(char * fichierBin, char * fichierTxt){
	
	FILE * fichier_Bin;
		//on ouvre le fichier binaire pour lire le contenu
		if ((fichier_Bin = fopen(fichierBin,"rb"))==NULL) {
        perror("Read "); 
        exit(1);
		}
	
	FILE * fichier_Txt;
		
		//on ouvre le fichierTexte pour y écrire
		if ((fichier_Txt = fopen(fichierTxt,"w"))==NULL) {
        perror("Write "); 
        exit(1);
		}	
	
	int lettre;
	
		while (fread(&lettre,sizeof(int),1,fichier_Bin)){
         //fwrite(&lettre, 1 * sizeof(int), 1, fichier_Txt);  
		//fprintf(fichier_Txt,%c,lettre);
		fread(&lettre, 1 * sizeof(int), 1, fichier_Txt);
		}
		
	fclose(fichier_Txt);
	fclose(fichier_Bin);
	}
	
	
int main (void){

	printf("Travail n°4, en C\n");
	
	/* conversion faux hexadecimal en binaire */
	HexaToBin(fichierBinaire,fichierTexteHexa);
	/* Conversion binaire en texte lisible */
	BinToTexte(fichierBinaire,fichierTexte);
	
return 0;
}

Linquisiteur
27 avril 2013 à 23:18

vu que je peux pas modifier mon dernier post (pas trouver pourquoi) j'ai décider de refaire tout ma procédure a zéro car a force de modifier comme un cochon je m'y retrouvais plus.

J'ai réussi a écrire dans le .dat des plus bizarre les uns que les autres mais je n'ai aucune idée si cela est normal. Par contre rien ne s'écrit dans mon .txt et sa m'empèche de vérifier si la 1er procédure est just ou fausse. Si vous avez une idée je suis tout ouïe

#include <stdio.h>
#include <stdlib.h>
#include <sys/wait.h>
#include <unistd.h>
#include <sys/types.h>
#include <string.h>
#include <fcntl.h>

#define fichierTexteHexa "hexa1.txt"
#define fichierBinaire "fichier.dat"
#define fichierTexte "fichier.txt"

void HexaToBin(char * fichierBin, char * fichierHex){
	
	FILE * fichier_Bin;
		//on ouvre le fichier binaire pour y écrire si il y a quelque chose on écrase les info
		if ((fichier_Bin = fopen(fichierBin,"wb"))==NULL) {
        perror("Write "); 
        exit(1);
		}
	
	FILE * fichier_Hex;
		
		//on ouvre le fichierHex pour lire les info dedans
		if ((fichier_Hex = fopen(fichierHex,"r"))==NULL) {
        perror("Read "); 
        exit(1);
		}
		
		
		int temp=0, c;
		c=fgetc(fichier_Hex); //on vérifie en 1er si le fichier est vide 
		
		if(c!=EOF){ //si il ne l'est pas on commence a calculer la taille
			if(c>='A' && c<='F') c=c-'A'+10; //on transforme les lettres en chiffres
			else c=c-'0';
			temp=c*16;
		}
		
		int taille=0, i=1; 
		char sortir='n';
		/*temps que on n'as pas atteint la fin du fichier on lit*/
		while(c!=EOF && sortir=='n'){ 

			while(i<2){ //On regarde la taille de la ligne
				c=fgetc(fichier_Hex);  //on lit le charactère où se trouve le pointeur dans le fichier
				i++;
			
				if(c==EOF) sortir='o';
			
				if(c>='A' && c<='F') c=c-'A'+10; //on transforme les lettres en chiffres
				else c=c-'0';
			
				if (i==1) temp=c*16;
				else {
					taille=temp+c;
					int test_pair=taille%2; //on vérifie si la ligne contient un nombre paire ou impair de char
					if(test_pair==1) taille--;
				}
			}
		i=0;
		
			while(i<taille && sortir=='n'){
			
				c=fgetc(fichier_Hex);  //on lit le charactère où se trouve le pointeur dans le fichier
				i++;
			
				if(c>='A' && c<='F') c=c-'A'+10; //on transforme les lettres en chiffres
				else c=c-'0';
			
				if (i==1) temp=c*16;
				else {
				temp=temp+c;
				fwrite(&temp, 1 * sizeof(int), 1, fichier_Bin);
				}
			}
		i=0;
			
		}
	fclose(fichier_Hex);
	fclose(fichier_Bin);
	}

/**
* Pré:
* @param fichierBin contier l'adresse d'un fichier binaire remplie
* @param fichierTxt contient un ficher texte vide où remplie (mais sera écraser);
* Post:
* @param fichierBin est inchangé
* @param fichierTxt contiendra du texte
**/
void BinToTexte(char * fichierBin, char * fichierTxt){
	
	FILE * fichier_Bin;
		//on ouvre le fichier binaire pour lire le contenu
		if ((fichier_Bin = fopen(fichierBin,"rb"))==NULL) {
        perror("Read "); 
        exit(1);
		}
	
	FILE * fichier_Txt;
		
		//on ouvre le fichierTexte pour y écrire
		if ((fichier_Txt = fopen(fichierTxt,"w"))==NULL) {
        perror("Write "); 
        exit(1);
		}	
	
	int lettre;
	
		while (fread(&lettre,sizeof(int),1,fichier_Bin)){
		fread(&lettre, 1 * sizeof(int), 1, fichier_Txt);
		}
		
	fclose(fichier_Txt);
	fclose(fichier_Bin);
	}
	
	
int main (void){

	printf("Travail n°4, en C\n");
	
	/* conversion faux hexadecimal en binaire */
	HexaToBin(fichierBinaire,fichierTexteHexa);
	/* Conversion binaire en texte lisible */
	BinToTexte(fichierBinaire,fichierTexte);
	
return 0;
}

fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 835
28 avril 2013 à 00:04

Pourquoi faire appel à des headers non standards ? Sachant que tu ne les utilises pas (genre : #include <sys/wait.h>, mais c'est pas la seule).
Ensuite, mets plutôt exit(EXIT_FAILURE) que exit(1);
Attention, dans les if : if(i=1) => if(i==1) vérifie s'il n'y a pas d'autres erreurs de ce style.

Linquisiteur
28 avril 2013 à 01:42

pour les bibli on ne les à pas étudier je sais l'utilité de quelque une d'entre elle mais pas de tous. J'ai supprimer celle que tu ma dis mais pour les autres je ne sais pas.
j'ai aussi modifier les exit(1); mais pourquoi faut il plutot utiliser EXIT_FAILURE ?
Sinon pas moyen de trouver le if qui contient une erreur.

Linquisiteur
28 avril 2013 à 16:26

J'ai trouver le if en question (je regardais la mauvaise sauvegarde :( )
Sa à l'air de marcher mais je boucle a la lecture je lis -1 en boucle dans le fichier.. Donc je n'atteint jamais la fin

fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 835
Modifié par fiddy le 28/04/2013 à 16:53

J'ai supprimer celle que tu ma dis mais pour les autres je ne sais pas.
Tu peux supprimer également :
#include <unistd.h>
#include <sys/types.h>
#include <fcntl.h>

mais pourquoi faut il plutot utiliser EXIT_FAILURE ?
EXIT_FAILURE correspond à la valeur attendue pour indiquer que cela s'est mal déroulé. Dans certains cas (la plupart), cela vaut 1, mais pas systématiquement.

Pour le reste, je sens que tu vas devoir reposter ton code avec les modifications que tu as faites et sur la bonne version.

Courage, tu es sur la bonne voie ;-).

Discussions similaires

erreur "Notice: Array to string conversion in..."

Erreur file_get_contents(): Filename cannot be empty PHP

mysql_num_rows() expects parameter 1 to be resource

Erreur : Warning: PDOStatement::execute(): SQLSTATE[HY093]

Invalid parameter number : parameter was not defined...

Warning: mysqli_select_db() expects parameter 1 to be my..

Discussions similaires

Newsletters