Discussion :

[john_Low]

Bonjour a tous,
Je voudrais faire un programme en langage C qui permet de créer un tableau dynamique en utilisant malloc et realloc
Je voudrais arrêter la boucle do while pour la création du tableau lorsque je tape ok et enfin afficher le contenu du tableau.
Je n'ai pas d'erreur dans les résultats de compilation mais j'ai besoin d'aide car mon programme ne fonctionne pas lorsque je l'exécute

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
 
#include <stdio.h>
#include <stdlib.h>
int main(int argc, char* argv[])
{
 
char *T;
int i=0,j=0;
 
T=(char*)malloc(1*sizeof(char));
do {
 
i++;
T=(char*)realloc(T,i*sizeof(char));
printf("type a number :");
scanf("%s",&T[i-1]);
} while(strcmp(T[i-1],"ok")!=0);
// display the content of the table*
for(j=0;j<i-1;j++)
{
printf("%s ",T[j]);
}
return 0;}

Merci d'avance

[Matt_Houston]

Quel est le type de T[i-1] ? De &T[i-1] ? De "ok" ?

De plus :

• compile avec le respect de la norme et la plupart des avertissements disponibles : -std=c11 -pedantic -Wall -Wextra ;
• prends connaissance des erreurs annoncées par ces avertissements, puis corrige-les. Toutes.
• Ne cast pas la valeur de retour des fonctions d'allocation ;
• consulte la valeur de retour des fonctions d'allocation, et traite les potentiels échecs ;
• ce n'est pas comme cela qu'on utilise realloc correctement : je te laisse rechercher des exemples sur le forum car c'est un sujet qui revient tous les mois ;
• ce n'est pas comme cela qu'on utilise scanf correctement : tu as ici un comportement indéterminé dès lors que l'utilisateur tape autre chose qu'un retour chariot.

[john_Low]

Tout d'abord merci pour votre réponse
T[i-1] ? De &T[i-1] sont de type int : entier et ok c'est de type char
j'ai constaté que je dois recuperer la chaine dans une autre variable et que si c'est un nombre je la stocke dans mon tableau, sinon si c'est « OK »,je quitte.

je n'ai pas pu la transformé ca en code pourais tu m'aider pour completer mon code

voila ce que j'ai pu faire :

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
29
30
 
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char* argv[])
 
{
int *T;
char *chaine
int i=0,j=0;
T=(int*)malloc(1*sizeof(int));
 
do {
 
i++;
T=(int*)realloc(T,i*sizeof(int));
printf("type a number :");
scanf("%d",&T[i-1]);
 
} while(aide??);
 
// display the content of the table*
 
for(j=0;j<i-1;j++)
{
printf("%d ",T[j]);
}
return 0;
 
}

[Matt_Houston]

Tu n'as pas répondu aux questions, ni appliqué les recommandations précédentes.

[john_Low]

Merci J'ai trouvé enfin tout seul la solution voici le code final :

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
29
30
31
32
33
34
35
36
37
#include <stdio.h>
#include <stdlib.h>
int main(int argc, char* argv[])
{
 
int *T;
int i=0,j=0;
int variable;
char answer[3]; 
T=NULL;
 
do {
 
printf("type a number :\n");
fgets(answer,sizeof(answer),stdin);
variable = atoi(answer);
if(variable!=0)
{
i++;
T=realloc(T,i*sizeof(int));
T[i-1]=variable;
}
else
{
printf("erreur vous devez entrer ok pour terminer\n");	
}	  	   
} while(strcmp(answer,"ok")!=1);
 
// display the content of the table*
 
for(j=0;j<i;j++)
{
printf("%d ",T[j]);
}
 
return 0;
}

[Bousk]

Je suis pas convaincu par ce test

Code :

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

while(strcmp(answer,"ok")!=1);

2 string sont identiques si strcmp == 0
https://en.cppreference.com/w/c/string/byte/strcmp

[john_Low]

oui vous avez raison voici le code final :

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
29
30
31
32
33
34
35
36
37
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
int main(void)
{
    int *T;
    int len=0,cpt=0;
    int number;
    char answer[1000];
    T=NULL;
    do
    {
        printf("type a number :\n");
        fgets(answer,sizeof(answer),stdin);
        if(1 == sscanf(answer, "%d", &number))
        {
            len++;
            T=realloc(T,len*sizeof(int));
            T[len-1]=number;
        }
 
        else if (strncmp(answer,"ok",2)!=0)
        {
            printf("erreur vous devez entrer ok pour terminer\n");
        }
    }
    while (strncmp(answer,"ok",2)!=0);
 
    // display the content of the table*
 
    for(cpt=0; cpt<len; cpt++)
    {
        printf("%d ",T[cpt]);
    }
 
    return 0;
}

il me reste un petit souci c'est que :
je ne veux pas qu'il alloue une case dans le tableau dans le cas ou le nombre est très grand car dans le cas de capture ecran il a mis la valeur 1593315709 a la place du grand nombre

j'ai essayé de faire comme ca mais ca n 'a pas marché :

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <limits.h>
int main(void)
{
    int *T;
    int len=0,cpt=0;
    int number;
    char answer[1000];
    T=NULL;
    do
    {
        printf("type a number :\n");
        fgets(answer,sizeof(answer),stdin);
        if(1 == sscanf(answer, "%d", &number))
        {
            if((number>INT_MAX)||(number<INT_MIN))
            {
                printf("le nombre que vous avez entrer est très grand");
            }
            else if((number<INT_MAX)&&(number>INT_MIN))
            {
                len++;
                T=realloc(T,len*sizeof(int));
                T[len-1]=number;
            }
 
 
        }
 
        else if (strncmp(answer,"ok",2)!=0)
        {
            printf("erreur vous devez entrer ok pour terminer\n");
        }
    }
    while (strncmp(answer,"ok",2)!=0);
 
    // display the content of the table*
 
    for(cpt=0; cpt<len; cpt++)
    {
        printf("%d ",T[cpt]);
    }
 
    return 0;
}

[Sve@r]

il me reste un petit souci c'est que :
je ne veux pas qu'il alloue une case dans le tableau dans le cas ou le nombre est très grand car dans le cas de capture ecran il a mis la valeur 1593315709 a la place du grand nombre

Bonjour

C'est un souci de troncature de scanf(). Tu lui indiques "%d" donc il s'attend à devoir stocker un signed int. Donc ensuite, si ta saisie est plus grosse que la plage acceptable ben il tronque.

Commence par établir tes règles, puis utilises des types adéquats puis ensuite un format approprié aux types.

Accessoirement ton strncmp() fonctionne mais acceptera alors toute chaine commençant par "ok". Si cette construction est là juste à cause du '\n' de la saisie alors ce n'est pas ce qu'il y a de mieux pour le gérer.
De plus tu doubles ton test (lourdeur).

Code c :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
char *c
while (1) {
	printf("type a number :\n");
	fgets(answer,sizeof(answer),stdin);
	if ((c=strchr(answer, '\n')) != NULL) *c='\0';
	if (sscanf(answer, "%d", &number) == 1) {
		len++;
		T=realloc(T,len*sizeof(int));
		T[len-1]=number;
	}
 	elif (strcmp(answer,"ok") == 0) break;
	else printf("erreur vous devez entrer ok pour terminer\n");
}