[debutant] Probleme de recuparation de donnes dans une fonct

Battosaiii · 14/11/2005, 11h39

J'ai une fonction dans laquelle les valeurs de t[i][j] sont corrects :

Code :

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

1
2
3
4
5
6
7
8
9
 
void matrix_load(char *file_name,int t[8][8]) 
{
...
t[i][j]=a[j];
				//fprintf(fpfile2,"%d",d); // We write on the output file
				printf(" %d\n",t[i][j]);
...
}

Le printf affiche bien les valeur de a[j]

Dans le main en revanche auqnd j'utilise

Code :

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

1
2
3
4
5
6
7
8
9
matrix_load(file_name1,t);
 
for(i=1;i<=8;i++)
{		
	for(j=1;j<=8;j++)
	{
	printf(" %d\n",t[i][j]);
	}
}

il y a un probleme car dans t[i][j] affiche des valeurs differentes que j'obtenais dans la fonction. genre des valeurs 435253525 34253253.
ca peut venir dou ?

**KiLVaiDeN** · 14/11/2005, 11h42

Bonjour,

en C, quand tu passes autre chose qu'un pointeur, tu passes une "COPIE" du tableau original.

Donc dans ton cas, tu passes une copie de ton tableau, que tu modifies; mais à la sortie de ta fonction, le tableau "original" est inchangé.

Battosaiii · 14/11/2005, 11h47

pour pointer sur le tableau je fais :
void matrix_load(char *file_name,int*t[8][8])
et matrix_load(file_name1,&t);
ca marche pas : je me souviens plus comment on fait avec un tableau t pour pointer.

**KiLVaiDeN** · 14/11/2005, 11h48

Nevermind, je dis des bêtises, l'erreur est ailleurs je pense, j'ai tout d'un coup un énorme doute quand aux pointeurs de tableaux, et le fait que le nom du tableau est lui même un pointeur ??
AAAAAAAAAAAAAAH au secours

Excuse moi si l'information est fausse...

En fait ça me sautait aux yeux, mais maintenant après coup j'ai des gros doutes par rapport à ma réponse initiale.........

Battosaiii · 14/11/2005, 11h51

lol t'as pas une idee comment sortir la vraie valeur de la fonction avec un tableau ?

**KiLVaiDeN** · 14/11/2005, 11h56

Je viens de vérifier : ma réponse initiale est une ânerie.

En fait en C, quand tu passes le tableau à ta fonction comme ceci : ta_fonction(tontableau); tu passes bien l'addresse du tableau, et donc les modifications que tu effectues dans la fonction se font sur le tableau d'origine..

Maintenant, pourquoi est-ce que dans ton cas ça ne marche pas ? J'imagine que le problème se situe au niveau des dimensions du tableau, mais là je préfère laisser la place à de vrais experts en la matière qui te donneront la bonne solution, je devrais retourner lire quelques cours de C avant d'affirmer des bêtises

j'étais sûr de mon coup pourtant... manque de pratique

**Emmanuel Delahaye** · 14/11/2005, 12h00

Envoyé par Battosaiii

il y a un probleme car dans t[i][j] affiche des valeurs differentes que j'obtenais dans la fonction. genre des valeurs 435253525 34253253.
ca peut venir dou ?

Comportement indéfini (ici, débordement)

C'est pas <= 8, mais < 8.

**Trap D** · 14/11/2005, 12h14

Le problème est là :

Code :

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

for(j=1;j<=8;j++)

alors qu'en C on fait

Code :

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

for(j=0;j<8;j++)

ce qui fait toujours bien 8 valeurs mais numérotées de 0 à 7.

**KiLVaiDeN** · 14/11/2005, 12h55

Ah oui, en C il n'y a pas d'ArrayIndexOutOfBounds

Encore désolé pour la mauvaise information, j'ai honte ! Pour me rattraper, dans le cours d'informatique générale que je suis en train de faire, j'illustrerais cet exemple pour expliquer les résultats incohérents dus à un accès mémoire incorrect

En l'occurence, j'ai quand même une question qui me vient à l'esprit : le compilateur n'est-il pas assez intelligent, dans le cas d'une boucle for simple, de detecter que l'index dépassera la taille du tableau initiale ? En fait je pose cette question, mais je connais un peu la réponse, cependant j'imagine qu'il serait "possible" d'extraire les cas "simples" pour lesquels une vérification de ce type serait faisable, après est-ce que le compilateur GNU le fait par exemple ?

**Jack_serious** · 14/11/2005, 13h04

Le compilateur ne connait pas la taille du tableau.

C'est inherent au C il n'y a aucune verification de faite pour gerer les debordements.

Apres on pourrait concevoir qu'un compilo soit capable de reperer les structures du genre :

Code :

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

1
2
3
4
5
6
7
8
9
 
int    tab[TAB_SIZE];
 
...
 
for (i = 0; i <= TAB_SIZE; i++)
  {
    ....
  }

Mais au pire il ne pourrait que donner un avertissement, qui ne serait meme pas garanti d'etre justifie, vu qu'entre temps on aurait pu dans n'importe quelle fonction faire un realloc, et changer ainsi la taille du tableau.

(Quoi que... la j'ai un doute. Avec une definition de ce type peut-on reallouer le tableau ??? Il me semble que oui mais...)

En tout cas je n'ai jamais vu un compilateur qui le fasse. Apres je suis loin d'avoir tout vu.

**KiLVaiDeN** · 14/11/2005, 13h22

Merci pour la réponse, tu as tout à fait raison je pense.

En fait plus que le compilo, j'imaginerais bien un IDE capable de donner un warning : il aurait en mémoire la taille initiale du tableau pour chaque bloc de programme, quand la taille est dynamique il mettrait une inconnue, mais dans le cas présent la taille est fixe, donc l'IDE _SAIT_ que quand tu fais un for i<=8 tu balayes une plage qui dépasse les indexs autorisés du tableau, mais là je me lance dans des suppositions métaphysiques

**Emmanuel Delahaye** · 14/11/2005, 13h23

Envoyé par Jack_serious

Code :

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

int    tab[TAB_SIZE];

(Quoi que... la j'ai un doute. Avec une definition de ce type peut-on reallouer le tableau ??? Il me semble que oui mais...)

Non. On ne peut réallouer que ce qui a été alloué par *alloc().

**Jack_serious** · 14/11/2005, 13h27

Envoyé par Emmanuel Delahaye

Envoyé par Jack_serious

Code :

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

int    tab[TAB_SIZE];

(Quoi que... la j'ai un doute. Avec une definition de ce type peut-on reallouer le tableau ??? Il me semble que oui mais...)

Non. on ne peut réalloier que ce qui a été alloué par *alloc().

Mouais... Je me doutais qu'il y avait un truc comme ca...

Merci pour cette precision !

**Emmanuel Delahaye** · 14/11/2005, 13h28

Envoyé par KiLVaiDeN

En fait plus que le compilo, j'imaginerais bien un IDE capable de donner un warning<...>

Il y a un projet d'IDE en cours avec des gens du forum (demande à C-soldier). Si te sens le courage de mettre en pratique tes idées...

Battosaiii · 14/11/2005, 13h29

J'ai modifie mon prograqmme en fct de vos remarques mais jai tjrs la meme erreur . Voici dans le main :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
matrix_load(file_name1,t);
 
 
/** display t **/ 
for(i=0;i<8;i++)
{		
	for(j=0;j<8;j++)
	{
	printf(" %d\n",t[i][j]);
	}
}

la fonction :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void matrix_load(char *file_name,int t[7][7]) 
{
...
 
	for(j=0;j<8;j++)
				{
				t[i][j]=a[j];
				//fprintf(fpfile2,"%d",d); // We write on the output file
				printf(" %d\n",t[i][j]);
				}
...
}

J'ai la meme erreur ... CA M'ENERVE !!![/code]

**Emmanuel Delahaye** · 14/11/2005, 13h31

Envoyé par Battosaiii

J'ai modifie mon prograqmme en fct de vos remarques mais jai tjrs la meme erreur . Voici dans le main :

Code :

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

void matrix_load(char *file_name,int t[7][7])

On avait pas dit 8 au lieu de 7 ?

8, c'est ne nombre d'élement. L'indice correspondand varie de 0 à 7...

**KiLVaiDeN** · 14/11/2005, 13h34

Peut-être que ton problème est lié à une variable unsigned ?
Si tu essaies de mettre de l'unsigned dans du signed, tu risques d'avoir des dépassements de taille, et donc des valeurs incohérentes.

Ensuite, mets des trace de deboggage, pour savoir à tout moment dans tes boucles quelles sont les opérations que tu fais, par exemple un simple printf de tes valeurs i et j, ou mieux de l'assignation complète, ça te permettra de voir ce que tu fais vraiment sur ton tableau.

**Trap D** · 14/11/2005, 14h57

Envoyé par Emmanuel Delahaye

Envoyé par Battosaiii

J'ai modifie mon prograqmme en fct de vos remarques mais jai tjrs la meme erreur . Voici dans le main :

Code :

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

void matrix_load(char *file_name,int t[7][7])

On avait pas dit 8 au lieu de 7 ?

8, c'est ne nombre d'élement. L'indice correspondand varie de 0 à 7...

Oui mais si c'est int t[7][7], il faut mettre de 0 à 6 ...

**semaj_james** · 14/11/2005, 16h17

Code :

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

void matrix_load(char *file_name,int t[7][7])

int t[7][7] ne passe t il pas que la valeur de t[7][7] et pas le tableau complet ?? ne faudrait il pas faire:

Code :

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

1
2
3
void matrix_load(char *file_name,int *t[7][7]) 
ou
void matrix_load(char *file_name,int t)

**Jack_serious** · 14/11/2005, 16h30

Envoyé par semaj_james

Code :

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

void matrix_load(char *file_name,int t[7][7])

int t[7][7] ne passe t il pas que la valeur de t[7][7] et pas le tableau complet ?? ne faudrait il pas faire:

Code :

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

void matrix_load(char *file_name,int *t[7][7])

Ici t serait un poiteur sur un tableau a double entree, donc un pointeur sur un pointeur de pointeurs d'entiers, ou plus simplement (hahaha) un int ***

Code :

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

1
2
3
4
 
 
ou
void matrix_load(char *file_name,int t)

Ici en revanche t est un entier tout mimi tout simple.

Rien a voir je suis au regret de le dire.

La premiere ecriture aurait un sens si on voulait modifier le tableau lui meme (le reallouer en somme) ce qui serait dur vu comme il a ete defini.

La deuxieme... euuuuh...

je vois pas.

[debutant] Probleme de recuparation de donnes dans une fonct

C

Discussions similaires

Partager

Partager