Discussion :

[mams1]

je veux realiser un programme qui initialise un tableau d'entier .La valeur de l'entier et la taille du tableau sont passés en paramétre.Mon programme est le suivant :

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
#include<stdio.h>
#include<stdlib.h>
 
int main (int argc, char** argv)
{
int i,j;
int* tab=(int*)calloc(argc,sizeof(int));
 
for (i=1;i<argc;i++)
{
tab[i]=argv[i]:
printf("%d\n",atoi(argv[i]));
}
return 0;
}

merci pour votre aide

[CedrX]

Moi j'aurais mis

Code :

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

1
2
 
tab[i-1]=atoi(argv[i])

Pour commencer à initialiser le tableau à l'élément 0.

Voici ce que serait mon code

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc,char *argv[]) {
        int i,j;
        int* tab=(int*)calloc(argc,sizeof(int));
        for (i=1;i<argc;i++) {
                tab[i-1]=atoi(argv[i]);
                printf("Valeur de l'indice %d:%d\n",i-1,tab[i-1]);
        }
        free(tab);
        return 0;
}

[Thierry Chappuis]

Moi, je verrais un code comme celui-là où on verifie que l'allocation a bien eu lieu et où on n'oublie pas de libérer la mémoire après utilisation:

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
#include<stdio.h>
#include<stdlib.h>
 
int main(int argc, char **argv)
{
    int i;
    int err = 0;
    size_t const TAILLE = argc - 1;
    int *tab = malloc(TAILLE * sizeof *tab);
    /* -tc- Pense a verifier que l'allocation est OK */
    if (tab != NULL)
    {
        for (i = 0; err == 0 && i < TAILLE; i++)
        {
            char *pend = NULL;
 
            tab[i] = strtol(argv[i+1], &pend, 0);
            if (*pend == '\0')
            {
                printf("%d\n", tab[i]);
            }
            else
            {
                /* -tc- erreur: eventuellement afficher un doc concernant l'usage
                   du prog */
                err = EXIT_FAILURE;
            }
        }
        /* -tc- pense a liberer les resources allouees dynamiquement */
        free(tab), tab = NULL;
    }
 
    return err;
}

Thierry

[mams1]

Moi j'aurais mis

Code :

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

1
2
 
tab[i-1]=atoi(argv[i])

Pour commencer à initialiser le tableau à l'élément 0.

Voici ce que serait mon code

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#include <stdio.h>
#include <stdlib.h>
 
int main(int argc,char *argv[]) {
        int i,j;
        int* tab=(int*)calloc(argc,sizeof(int));
        for (i=1;i<argc;i++) {
                tab[i-1]=atoi(argv[i]);
                printf("Valeur de l'indice %d:%d\n",i-1,tab[i-1]);
        }
        free(tab);
        return 0;
}

[matafan]

atoi() ne permet pas de détecter les erreurs (à part si pour ton programme, 0 n'est pas une valeur acceptable). Utilises plutôt strtol().