Not enough space

Version imprimable

10/06/2013, 01h11
kays86

Not enough space

Bonsoir à tous,
J'ai un tableau qui me retourne un Null et um message m'affiche « Not enough space code de l'erreur 12 ». En se basant sur ce lien http://www.developpez.net/forums/d13...e-1-dimension/ , j'ai ajouté l'instruction ci-dessous à la fin de mon code pour voir si je dépasses la taille admise ou pas.
« printf ("_HEAP_MAXREQ = %llx\n",(unsigned long long int)_HEAP_MAXREQ); ».
Malheureusemnt? j'ai eu ces deux warning :
«unknown conversion type character 'l' in format»
«too many arguments for format »
Comment je peux le régler sachant que je code en C sous windows7 64 bit?
Merci d'avance
10/06/2013, 02h06
Obsidian

Bonjour,

Un tableau ne peut pas « te retourner » un NULL. Ce n'est pas une fonction.

Peut-on voir ton code et, en particulier, la manière dont est alloué ton tableau ?

voici mon code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
        pile= malloc (n*sizeof (long int));
        if (pile==NULL)
        {
            printf ("pile Code de l'erreur : %d\n", errno);
            perror("Erreur");
        }
        else
        {
 
            fonction (n,copie1,sum,fichier);
            if(pile != NULL)
            {
                solution++;
            }
 
            free (pile);
            pile=NULL;
        }

J'ai le message suivant pile code de l'erreur 12 : not enough space

10/06/2013, 12h29
Bktero

Et que vaut n ? :roll:

La macro que tu présentes est définie dans malloc.h et semble spécifique à Windows : http://msdn.microsoft.com/fr-fr/libr...(v=vs.80).aspx Je n'ai perso pas les erreurs de compilation que tu décris. Attention quand même à cette valeur : elle n'est pas dynamique. Cela veut dire que c'est la taille maximale potentielle que tu peux demander. Cela ne veut pas dire qu'à un instant t, tu puisses demander tout ça car la quantité de mémoire disponible est sûrement inférieure.

n= 60
j'ai les allocations suivante avant pile :
Tableau = malloc (60 * sizeof(job));job est de type struct, elle contient 4 champs de type long int
Code:

1 2 ordre = malloc (57 * sizeof(long int)); copie1 = malloc (60 * sizeof(job));
ens et combi dans une procedure qui génèrent toutes les combinaisons possibles de combi parmi ens (par exemple je veux 27C11 ) :
ens = malloc (27*sizeof (long int)); combi= malloc (11*sizeof (long int));puis à chaque combinaison généré je réserve pile je fais des calcules et je réserve choix. (nbre entre 0 et 60 )
Code:

1 2 3 4 5 6 7 8 9 10 nbre=0; choix= malloc (nbre *sizeof (long int)); for (i=0;i<n;i++) { if (copie1[i].d>=sum) nbre++; } choix= realloc(choix, sizeof(long int) *nbre);
et je désalloue choix puis pile
pile= malloc (60*sizeof (long int));et a la fin de mon programme je désalloue ens, combi, copie1, tableau et ordre

J'ai fais un code test:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 #include <stdio.h> #include <stdlib.h> #include <malloc.h> int main() { printf("Hello world!\n"); printf ("_HEAP_MAXREQ = %llx\n",(unsigned long long int)_HEAP_MAXREQ); return 0; }
et j'ai les même erreurs de compilation (je code avec code blocks).

10/06/2013, 19h51
Bktero

Tu compiles sans modifier les options de compilation par défaut ? Si oui, l'option par défaut est sûrement un mode C89/90, c'est-à-dire soit -ansi soit -std=gnu90. Or, le type long long int est venu avec le C99 si ma mémoire est bonne. Rajoute donc une option -std=c99 dans les options du compilateur pour voir si cela corrige le problème.

Pour ce qui est du problème mémoire, il faut que tu essayes de créer un code minimum qui reproduise le problème, en réduisant ton code actuel.

j'ai coché l'option : In C mode, ISO C90 et mon code marche sans erreurs de compilation ;). Merci Bktero pour votre aide :ccool:

_HEAP_MAXREQ = FFFFFFe0 ça veut dire quoi ?

sans cette procédure mon code marche sans avoir le message not enough space
static long int *pile=NULL;

Code:

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
48
49
50
51
52
53
54
55
56
57
58
 
void fonction (long int n, job *copie1, long int sum,FILE *fichier)
{
 
    long int i,k,j,nbre,max,ind;
        long int *choix=NULL;
        k=n-1;
        while (k>=0)
        {
            nbre=0;
            choix= malloc (nbre *sizeof (long int));
            j=0;
            for (i=0;i<n;i++)
            {
                if (copie1[i].d>=sum)
                {
                    nbre++;
                    choix= realloc(choix, sizeof(long int) *nbre);
                    if (choix==NULL)
                    {
                        printf (" choix Code de l'erreur : %d\n", errno);
                        perror("Erreur");
                    }
                    else
                    {
                       choix[j]=i+1;
                       j++;
                    }
                }
 
            }
 
            if (nbre==0)
            {
                pile=NULL;
                return;
            }
            else
            {
                max = choix[0]-1;
                for (i=0;i<nbre;i++)
                {
                    ind=choix[i]-1;
                    if ( copie1[ind].p>copie1[max].p)
                    {
                        max= ind;
                    }
                }
 
                pile[k]= max+1;
                sum=sum-copie1[max].p;
                copie1[max].d=-1;
                k--;
                free (choix);
                choix=NULL;
            }
 
}

11/06/2013, 08h13
Bktero
Mouais c'est bizarre qu'en activant le mode C90 cela compile, alors que long long int a bien été introduit en C99. Passons.

Citation:

_HEAP_MAXREQ = FFFFFFe0 ça veut dire quoi ?

Je présume que ça veut dire que si tu demandes plus, malloc() échouera forcément.

Citation:

sans cette procédure mon code marche sans avoir le message not enough space

Quelle procédure ?

Le code que tu nous montres n'est pas satisfaisant pour t'aider. Il faut nous donner un main qui initialise les paramètres et qui appelle la fonction qui plante (au moins, qui plante chez toi). Sans ça, on ne pourra rien faire.

Plus généralement sur le code, on peut dire qu'il est incompréhensible. Des points d'amélioration :
- Une fonction ne devrait jamais s'appeler fonction() car on ne sait pas ce qu'elle fait
- Tu ne mets pas de commentaire, tu devrais.
- Tu mets un if à l'intérieur d'un for et si le if est vrai, alors realloc() direct. Pourquoi as-tu besoin de réallouer ?
- Tes variables sont mal nommées. On se doute d'une variable de type long int est un nombre, nomme la autrement que nbre. copie1 mais il n'y a pas de copie0 et surtout il n'y a pas d'autres structure du même type alors elle ne copie rien. Bref, il est très important de mettre des noms de variables qui ont un sens et décrivent ce que font vraiment les variables.
- Evite d'utiliser les variables globales autant que possible.
- Tu vérifies que realloc() n'échoue pas mais tu ne vérifies pas le retour de malloc().