Discussion :

[Smashmaster]

Bonjour/Bonsoir à tous,

J'aimerai allouer dynamiquement une quarantaine de tableaux (de taille 576) :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
unsigned char* var1;
unsigned char* var2;
unsigned char* var3;
unsigned char* var4;
unsigned char* var5;
unsigned char* var6;
//...
unsigned char* var39;
unsigned char* var40;

étant donné que je suis un petit développeur assez fainéant, je ne voulais pas utiliser une quarantaine de fois la fonction malloc(), j'ai donc stocké les 40 variables dans un tableau :

Code :

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

1
2
 
unsigned char* var[] = {var1, var2, var3, var4, var5, ..., var40};

Puis j'ai alloué dynamiquement chaque élément du tableau :

Code :

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

1
2
3
4
5
 
for (int i = 0 ; i < 40 ; i++)
{
    var[i] = (unsigned char*) malloc (sizeof (unsigned char) * 576);
}

Le problème, c'est que cela ne marche pas, en effet lorsque j'essaye d'enregistrer des valeurs dans la variable var1, le programme plante systématiquement.

Code :

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

1
2
3
4
5
 
for (int i = 0 ; i < 10 ; i++)
{
    var1[i]=0; //ne fonctionne pas
}

J'ai donc deux questions :

• Pourquoi mon code ne marche pas?
• Comment puis-je corriger ce problème?

Le code en entier (j'ai supprimé quelques variables pour une meilleure lisibilité) :

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
 
int main(int argc, char* argv[]){
    unsigned char* var1;
    unsigned char* var2;
    unsigned char* var3;
    unsigned char* var4;
    unsigned char* var5;
    unsigned char* var6;
    unsigned char* var7;
    unsigned char* var8;
    unsigned char* var9;
    unsigned char* var10;
 
 
    unsigned char* var[] = {var1,var2,var3,var4,var5,var6,var7,var8,var9,var10};
 
    for (int i = 0 ; i < 10 ; i++)
    {
        var[i] = (unsigned char*) malloc (sizeof (unsigned char) * 576);
    }
 
    for (int i = 0 ; i < 10 ; i++)
    {
        var1[i]=0; //ne marche pas
    }
 
    return 0;
 
}

Je précise aussi qu'il n'y a pas d'erreur de compilation.

Voilà, merci et bonne soirée à vous tous !

[foetus]

Ton problème est trivial : tu as créé ("malloc"é si tu préfères) les 10 cases de ton tableau var, mais aucune variable varXX

[Smashmaster]

Mince, je n'avais pas vu

malloc() retourne une adresse qui écrase l'adresse de varX, je n'y avais pas pensé tout de suite.

Encore merci foetus, bonne soirée à vous tous !

Je marque le sujet comme résolu.

[Obsidian]

Bonjour,

malloc() retourne une adresse qui écrase l'adresse de varX, je n'y avais pas pensé tout de suite.

C'est un poil plus subtil encore : lorsque tu crées tes 40 variables, puis que tu alloues ton tableau en spécifiant leur nom, tu remplis ton tableau avec la valeur actuelle des variables en question, pas avec leur adresse propre. Il n'y a donc aucun lien entre ton tableau et les variables initiales une fois l'opération effectuée.

Le mieux consiste à déclarer directement un tableau de pointeurs de 40 entrées :

Code :

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

unsigned char * var[40];

… puis de les initialiser avec un malloc dans une boucle comme tu le fais :

Code :

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

for (i=0; i<40; i++) var[i] = malloc(sizeof (unsigned char) * 576);

… mais surtout, si toutes tes entrées ont la même taille, que celle-ci est relativement modeste et que tu ne libères jamais cette place avant la fin du programme ou du bloc ou elles sont déclarées, alors le plus simple consiste à déclarer directement un tableau à deux dimensions :

Code :

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

    unsigned char var [40][576]; /* Attention : pas de « * » ici */

Cela dit, cela donne ici un tableau de 22,5 Kio. C'est encore acceptable mais il faudra effectivement veiller à ne pas remplir la pile trop loin de cette façon.