Discussion :

[Azghar]

Bonjour,
je bloque sur un petit programme composé de deux fonctions.
j'ai d'abord crée un nouveau type item_t grâce au type def;
dans le main je défini mes différentes variables et pointeurs
la deuxième fonction permet de modifier les valeurs associé à mon nouveau type.
Mais malheureusement ça ne marche pas j'ai cette erreur:
error: passing argument 1 of ‘item_fill’ from incompatible pointer type [-Wincompatible-pointer-types]
item_fill(&objet1, name1, nbr1);

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
typedef struct {char name[30] ;  
	int quantite;} item_t;
 
int item_fill(item_t* obj, char a[], int quantite);
 
int inventory_display(item_t* tab);
 
int inventory_add(item_t* inventory, item_t item);
 
 
int main()
{
	int nbr1=1;
	char nom1;
	char* name1 = NULL;
	item_t* objet1 = NULL;
	name1=&nom1;
 
	objet1 = (item_t*) malloc(30*sizeof(item_t));
	scanf("%s",nom1);
	/*scanf("%i",&nbr1);*/
 
	item_fill(&objet1, name1, nbr1);
	inventory_display(objet1);
	printf("%i '%s' \n",objet1->quantite,objet1->name);
	return 0;
}
 
int item_fill(item_t* obj, char* a, int quantite)
{
	strcpy(obj->name,a);
	obj->quantite=quantite;
	return 0;
}

[Azghar]

oups j'ai oublié un truc ;virez le 3eme prototype pour tester le programme, ça concerne la suite de mon programme qui ne nous intéresse pas ici

[Sve@r]

Mais malheureusement ça ne marche pas j'ai cette erreur:
error: passing argument 1 of ‘item_fill’ from incompatible pointer type [-Wincompatible-pointer-types]
item_fill(&objet1, name1, nbr1);

code :

Code :

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

1
2
3
4
5
int main()
{
	item_t* objet1 = NULL;
	item_fill(&objet1, name1, nbr1);
}

Bonjour

Regarde le code suivant

Code c :

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

1
2
3
4
5
6
7
8
9
10
void fct(int *pt)
{
     ...
}
 
int main()
{
     int var;
     fct(&var);
}

var est de type "int". Son adresse "&var" étant donc l'adresse d'un int, sera alors stockée dans une variable dédiée aux stockages des adresses d'int, donc dans un "int étoile".

Dans ton code, quel est le type de "objet1" ? Et donc quel type tu invoques quand tu appelles l'adresse "&objet1" ???

Aide

Code c :

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

1
2
3
4
5
6
7
8
9
10
11
int main()
{
     int var;
     int *pt=&var
 
     // En utilisant "var"
     fct(&var);
 
    // En utilisant "pt"
     fct(...);
}

Que dois-je mettre dans les parenthèses (en utilisant "pt") pour que ma fonction "fct" reçoive ce qu'il faut ?

oups j'ai oublié un truc ;virez le 3eme prototype pour tester le programme, ça concerne la suite de mon programme qui ne nous intéresse pas ici

Déjà on s'en fout, un prototype en plus ne gêne en rien (c'est plutôt un prototype en moins qui peut gêner) mais sinon t'as vu que t'avais un bouton "modifier" ???

[Azghar]

merci pour ta réponse,
objet1 est de type item_t, le type que j'ai défini en haut si j'ai bien compris.
dans les parenthèses de ton code , je mettrai : (*pt)
J'ai bon ?

[Sve@r]

item_t* objet1

objet1 est de type item_t, le type que j'ai défini en haut si j'ai bien compris.

Non, sinon il y aurait écrit item_t objet1 or ce n'est pas ce que je vois dans ton premier code...

dans les parenthèses de ton code , je mettrai : (*pt)
J'ai bon ?

Non mais c'est ma faute, j'ai écrit ça à la façon "raccourcie" dont on a l'habitude mais qui doit impérativement se lire d'une certaine façon...

Je réécris mon second code de façon plus détaillée...

Code c :

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

1
2
3
4
5
6
7
8
9
10
11
12
int main()
{
     int var;
     int * pt;
     pt=&var;
 
     // En utilisant "var"
     fct(&var);
 
    // En utilisant "pt"
     fct(...);
}

[Azghar]

ah oui, objet1 est un pointeur de item_t
dans ta fonction il faut mettre pt entre parenthèses alors
j'ai modifié &objet1 par objet1 dans mon code à la ligne 28
effectivement ça marche mais l'attribut name (je sais pas si on dit comme ça) de l'objet1 que je passe a la fonction fill n'est pas modifié on dirait puisque quand je vérifie le nom de l'objet1 il me renvoit ' ' alors que j'ai tappé clef dans la console

[Sve@r]

ah oui, objet1 est un pointeur de item_t

Exact. objet1 est un "item_t étoile" (ce qui n'est absolument pas du tout pareil qu'un "item_t")

dans ta fonction il faut mettre pt entre parenthèses alors

Bien entendu. D'ailleurs puisqu'il est écrit que "pt" égale (égalité donc "pareil que") "&var", ça veut dire que partout où il est écrit "&var" (comme en ligne 8) on peut remplacer par "pt". Donc "pt" est de type "int étoile", si je passe "pt" à ma fonction elle reçoit alors bien un "int étoile" ce qui est conforme à ce qu'elle attend.

j'ai modifié &objet1 par objet1 dans mon code à la ligne 28

Exactement. C'est le même raisonnement qui s'applique. Puisque "objet1" est de type "item_t étoile", et que la fonction attend un "item_t étoile", c'est "objet1" qu'il faut passer et non "&objet1" (qui, lui, sera vu comme étant un "item_t étoile étoile" !!!). D'ailleurs je ne comprends même pas que tu aies fait l'erreur vu que le second paramètre est un "char étoile" et que tu lui passes "name1" (et non "&name1") !!!

Code c :

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

1
2
char nom1;
scanf("%s",nom1);

effectivement ça marche mais l'attribut name (je sais pas si on dit comme ça) de l'objet1 que je passe a la fonction fill n'est pas modifié on dirait puisque quand je vérifie le nom de l'objet1 il me renvoit ' ' alors que j'ai tappé clef dans la console

Hum, quelle est la définition d'un type "char" ??? Est-ce bien le bon type adéquat pour stocker une chaine ???