Discussion :

[Linu6]

Bonjour, je souhaite faire un programme qui lit le tableau t[100], depuis une fonction, qui reçoit comme argument un pointeur.

voici le code, mais je ne sais pas ce qui cloche là-dedans

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
void lire (char *t);
 
int main()
{
	char t[100];
 
	lire(t);
 
	printf("%s", t);
 
	return 0;
}
void lire (char *t){
 
    while(*t=getchar()!='\n'){
 
    ++t;
    }
    return;
}

[muad'dib]

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
void lire (char *t);
 
int main()
{
	char t[100];
 
	lire(t);
 
	printf("%s", t);
 
	return 0;
}
void lire (char *t){
 
    while(*t=getchar()!='\n'){
 
    ++t;
    }
    return;
}

• la méthode getchar() permet de réaliser la lecture d'un caractère sur l'entrée standard (stdin), ce qui ne correspond pas à ce que tu veux faire, qui est de lire le contenu du tableau
• ton tableau t n'est pas initialisé, les valeurs se trouvant à l'intérieur sont générée aléatoirement
• la méthode printf() écrit sur la sortie standard les caractères compris dans une chaine jusqu'à rencontrer le caractère de fin de chaine '\0'. Comme ton tableau n'est pas initialisé, le comportement est indéfini

Après avoir initialisé ton tableau à l'aide d'une boucle par exemple

Code :

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

1
2
3
int i;
 
for(i = 0; i < 100; i++) t[i] = i;

Tu pourras lire les éléments en reprogrammant la fonction lire() de cette manière

Code :

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

1
2
3
4
5
6
void lire(char *t)
{
   int i;
 
   for(i = 0; i < 100; i++) putchar(t[i]); /* ou putchar(*(t + i)); */
}

[Linu6]

re

j'ai élaboré cette version, mais le probleme c'est tant que je n'ai pas entré les 100 caractères, le retour chariot ne met pas fin à la lecture.

Mais d'autre part, je ne peux faire une allocation dynamique car je ne connais pas le nombre exacte de caractère à lire.

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
void lire (char *t);
 
int main()
{
	char t[100]={0};
	lire(t);
	printf("%s", t);
	return 0;
}
void lire (char *t){
    int i;
    for(i=0; i<100; i++){
    *t=getchar();
    ++t;
}
    return;
}

[Montag]

Salut,

Il y a plusieurs choses qui clochent dans ton programme :
tout d'abord :

cette ligne ne convient pas :

Code :

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

1
2
 
while(*t=getchar()!='\n')

il faut la remplacer par :

Code :

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

1
2
 
while((*t=getchar())!='\n')

Normalement ton compilateur devrait te le signaler si ce n'est pas le cas regarde ce lien :
http://emmanuel-delahaye.developpez....tm#cfg_compilo

De plus tu ne controles pas la taille de la saisie et donc tu peux avoir un dépassement de ton buffer et écraser les données dans la pile ce qui devrait provoquer un crash de ton programme.

Enfin il serait bon, une fois sorti de ta boucle while de rajouter le caractère '\0' à la fin de ta chaîne.

[EDIT]Je n'avais pas vu ton nouveau message mes remarques ne s'appliquent donc qu'à ton premier message[/EDIT]

[Linu6]

mais mon probleme c'est que je sais que la taille ne peut dépasser les 100 caractères, mais si je défini la taille par

#define DIM 100

et ensuite je pose comme condition:

i<DIM-1

alors le compilateur attend de moi au moins 99 caractères.

[Montag]

Voilà un code qui doit faire ce que tu souhaites :

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
26
27
28
#include <stdio.h>
 
#define NELEM 100
 
void lire (char *);
 
int main(void)
{
	char t[NELEM];
	lire(t);
	printf("%s\n", t);
	return 0;
}
 
void lire (char *t)
{ 
    if(t!=NULL)
    {
    	unsigned int counter=0;
    	while(counter<NELEM-1 && (*t=getchar())!='\n')
    	{
    		++counter;
    		++t;
    	}
    	*t='\0';
    }
    return;
}