Discussion :

[Surikat14]

Bonjour !!

Je souhaiterai utiliser la fonction malloc pour créer un tableau de nombre. Truc simple en apparence mais les exemples des différents tutos que j'ai trouvé ne marche pas chez moi... peut-être parce que je n'ai pas très bien compris le principe de la chose...

Je définis un variable ii puis je veux créer un tableau de ii+1 cases.... Est-ce possible ?
Pour le moment j'ai fait :

Code :

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

1
2
int ii = 512;
double *Qmoy = malloc(ii *sizeof(double));

ce qui normalement me crée un tableau de ii = 512 cases mais ça ne marche pas
J'ai le message d'erreur suivant :

initializer element is not constant

Si quelqu'un pouvait m'aider...

[stephl]

Où sont définies ces variables? Au niveau global? Si oui, il n'est pas possible d'initialiser de cette manière en faisant appel à une fonction. Il faut faire l'appel à malloc() dans une de vos propres fonctions (au début du main() par exemple).

Exemple:

Code :

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

1
2
3
4
5
6
7
8
int ii = 512;
double *Qmoy;
 
int main(void)
 {
 Qmoy=malloc(ii *sizeof(double));
 ...
 }

[Surikat14]

Je me suis construit un ti exemple tout simple

Quand je fais :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
void main()
{
int i, tt = 1440.;
double *h;
double *h = malloc(tt *sizeof(double));
 
for (i=0; i<=tt; i++)
    {
    h[i] = PI;
    printf("%f", h[i]);
    }  
}

ça marche nickel

Comme j'aimerai utiliser tt et [I]h d'autres fois dans mon programme, j'ai tenté ça et là...

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
 
int tt = 1440.;
double *h;
 
void remplissage()
{
int i;
for (i=0; i<=tt; i++)
    {
    h[i] = PI;
    printf("%f", h[i]);
    }
}
 
void main()
{
double *h = malloc(tt *sizeof(double));
 
remplissage();
}

Merci de ton aide rapide stephl

[stephl]

tt est un entier, donc pourquoi utiliser une constante double (1140.) au lieu de la constante entière (1140) pour l'initialiser?

Dans votre parcours du tableau, vous allez un élément trop loin. Le for doit être:

Code :

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

1
2
3
4
for (i=0;i<tt;i++)
 {
 ...
 }

Enfin dans le main(), il ne faut pas redéfinir h mais juste l'initialiser:

Code :

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

1
2
3
4
5
6
7
int main(void)
{
...
h = malloc(tt *sizeof(double));
remplissage();
...
}

Il pourrait aussi être judicieux de vérifier le retour de malloc() au cas où l'allocation n'aurait pas fonctionné.

[PsychoH13]

Surikat, dans le code que tu as écris tu sors de la mémoire que tu as allouée ! Quand tu itères dans un tableau de n valeurs, il faut aller de 0 à n-1, donc il faut comme condition : i < n et non pas i <= n...

[DaZumba]

En plus des reponses precedentes:

• l'utilisation de variables globales doit etre justifiee, donc justifiable. Ce n'est pas le cas ici. Il faut apprendre a passer des arguments aux fonctions.
• tout segment memoire alloue par malloc() doit etre libere par free().

[Surikat14]

En plus des reponses precedentes:

• l'utilisation de variables globales doit etre justifiee, donc justifiable. Ce n'est pas le cas ici. Il faut apprendre a passer des arguments aux fonctions.

Il vaut mieux des variables locales et passer des arguments dans les fonctions alors ? Pourquoi est-ce mieux ?

• tout segment memoire alloue par malloc() doit etre libere par free().

Pour le free(), oui j'avais vu mais je ne l'ai aps mis dans mon exemple mais je ne l'oublierai pas !

[Surikat14]

Surikat, dans le code que tu as écris tu sors de la mémoire que tu as allouée ! Quand tu itères dans un tableau de n valeurs, il faut aller de 0 à n-1, donc il faut comme condition : i < n et non pas i <= n...

Ah oui exact, le problème de la case n° 0...