Discussion :

[olivier1209]

Bonjour à tous je débute le C, et j'ai un peu de mal avec ces satanés pointeurs
Mon programme est simple : je déclare un tableau tab[10], dans lequel je place 10 nombres aléatoires.

j'obtiens ceci en message d'erreur : warning: return makes pointer from integer without a cast et je sais pas trop d'où ça peut venir

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>
#include <time.h>
 
int *Creer_Stock ()
{
    srand (time (NULL));
 
    int i,poidsEngin,*stock;
    stock = malloc(sizeof(int)*10);
 
    if (stock == NULL) exit (0);
    else
    {
        for (i = 0 ; i < 10 ; i++)
        {
            poidsEngin =  ( rand() % 7 ) + 4 ;
            stock [i] = poidsEngin ;
        }
        return *stock;
    }
}
 
 
int main()
{
 
int *tab = Creer_Stock();
 
for (i = 0 ; i < 10 ; i++)
        printf("%d ",tab[i]);
 
    return 0;
}

Voila en espérant avoir été clair

[Invité(e)]

Bonjour,

stock est un pointeur sur entier,
* stock est un entier

Code :

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

return stock;

devrait suffire.

PS : manque aussi la déclaration de i dans la fonction main.

[olivier1209]

Ouais c'est bon merci beaucoup
marre d'être aussi tarte

[tonton fred]

Salut,

Code :

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

return *stock;

Stock est de type int*
Creer_Stock doit retourner une variable de type int*
*Stock est de type int

[olivier1209]

merci pour vos réponses,
je continue mon programme et je vous tiens au courant

[olivier1209]

c'est encore moi ^^
cette fois j'aimerais créer une fonction qui travaille avec les @ de "poids" et "taille" de mon tableau crée au préalable. C'est la taille qui me pose problème (le poids semble fonctionner)

Voila 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
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
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include <strings.h>
 
 
int *Creer_Stock ()
{
    int i,poidsEngin,*stock;
    srand (time (NULL));
    stock = malloc(sizeof(int)*10);
 
    if (stock == NULL) exit (0);
    else
    {
        for (i = 0 ; i < 10 ; i++)
        {
            poidsEngin =  ( rand() % 7 ) + 4 ;
            stock [i] = poidsEngin ;
        }
        return stock;
    }
}
 
void Gerer_Stock( int *tab , int *taille , int *poids )
{
    int i;
 
    *taille = sizeof(tab);
 
    for (i=0 ; i<*taille ; i++)
    {
        *poids = *poids + tab[i] ;
    }
 
}
 
 
    int main()
{
 
    int i,taille=0,poids=0;
    int *tab = Creer_Stock();
 
    Gerer_Stock (tab,&taille,&poids);
 
    for (i = 0 ; i < 10 ; i++)
        printf("%d ",tab[i]);
 
    printf("taille %d poids %d ",taille,poids);
 
    return 0;
}

[Médinoc]

Code :

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

 *taille = sizeof(tab);

Mauvais. Tout ce que cette ligne fait, c'est *taille = sizeof(int*), qui est toujours égal à 4 sur un système x86 32 bits (ou 8 sur un système 64 bits).

[olivier1209]

Mauvais. Tout ce que cette ligne fait, c'est *taille = sizeof(int*), qui est toujours égal à 4 sur un système x86 32 bits (ou 8 sur un système 64 bits).

Ouais j'y ai pensé. Mon tableau étant une chaine , je devrais pouvoir utiliser stren(tab). Seulement le compilo n'aime pas

|warning: passing arg 1 of `strlen' from incompatible pointer type|

[Médinoc]

Si ton tableau est une chaîne de caractères, pourquoi son type est-il int* ?

[olivier1209]

Si ton tableau est une chaîne de caractères, pourquoi son type est-il int* ?

Ben avec une chaine d'entiers , la fonction strlen() devrait marcher, me tromp-je ?

[Emmanuel Delahaye]

Ben avec une chaine d'entiers , la fonction strlen() devrait marcher, me tromp-je ?

Oui, tu te trompes. strlen() ne fonctionne qu'avec les chaines. Or, en C, une chaine est portée par un tableau de char.

[olivier1209]

Voilà j'ai terminé
je met mon code ça peut servir à quelqun d'autre
Dites moi si vous voyez des énormités dans les commentaires lol

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
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include <strings.h>
 
 
int *Creer_Stock ()  // creer un stock de 10 engins
{
    int i,poidsEngin,*stock;
    srand (time (NULL)); // prend le timestamp pour l'initialisation et s'en sert de "graine" pour le random
    stock = malloc(sizeof(int)*10); // allocation de mémoire
 
 
    if (stock == NULL) exit (0); // si l'allocation se fait sans prob, on continue
    else
    {
        for (i = 0 ; i < 10 ; i++) // on parcours les 10 cases du tableau
        {
            poidsEngin =  ( rand() % 7 ) + 4 ; // poids entre 4 et 10
            stock [i] = poidsEngin ; // remplissage du tableau
        }
 
        return stock; // renvoie le tableau
    }
}
 
void Gerer_Stock( int *tab ,int *taille , int *poids ) // pour connaitre le poids d'un stock
{
    int i=0;
 
    for (i=0 ; i<*taille ; i++) // tant qu'on ne dépasse pas la tailleMAX
    {
        *poids = *poids + tab[i] ; // on somme tous les poids du tableau
    }
 
// on ne renvoie rien car on travaille sur l'adresse des valeurs directement
}
 
 
int *Creer_Stock_Max (int *taille) // creer un Stock sans dépasser 150 en poids total
{
 
    srand (time (NULL));
    int i=0,poidsEngin,*stock,poidsTotal=0;
 
    stock = malloc(sizeof(int)*50); // allocation suffisemment grande
 
    if (stock == NULL) exit (0);
    else
    {
        while (poidsTotal <= 140)
        {
            poidsEngin =  ( rand() % 7 ) + 4 ;
            stock [i] = poidsEngin ;
            poidsTotal += poidsEngin;
            i++ ;
        }
 
 
        *taille = i ;  // i est la taille du tableau
 
        return stock;
    }
}
 
 
 
int main()
{
 
    int i,taille=0,poids=0;
    int *tab;   // déclaration du tableau
 
    tab = Creer_Stock_Max(&taille); // appel de la fonction pour creer un stock
 
    Gerer_Stock (tab,&taille,&poids); // on rentre en paramêtre les adresse de taille , et de poids
 
 
    for (i=0 ; i<taille ; i++)
    {
        printf ("%d ",tab[i]);  // affiche stock total
    }
 
    printf ("\n\nNombre d'engin : %d\nPoids total du stock : %d\n",taille,poids);
 
 
 
    return 0;
}

PS : petite question annexe : je ne sais pas trop comment libérer la mémoire allouée à mon tableau. free(stock) devrait se mettre dans la fonction Creer_Stock_Max (int *taille). Seulement où je ne sais pas

[orfix]

PS : petite question annexe : je ne sais pas trop comment libérer la mémoire allouée à mon tableau. free(stock) devrait se mettre dans la fonction Creer_Stock_Max (int *taille). Seulement où je ne sais pas

Tu libères dès que tu en as plus besoin, dans ton code ça pourrait être juste après la boucle :

Code :

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

1
2
3
4
5
6
7
 
 	for (i=0 ; i<taille ; i++)
	{
		printf ("%d ",tab[i]);  // affiche stock total
	}
	free(tab);
       tab = NULL;

[olivier1209]

oki merci