Structures à utiliser pour un exercice

Version imprimable

Bonjour, je viens ici car j'ai une question et je ne comprends pas bien le fonctionnement.

J'ai un exercice à réaliser avec une structure
Code:

1 2 3 4 5 6 7 8 9 struct SwiStruct { int ref; int nb_ports; int constr; int speed; float energy; float prix; int qte; };
et voici les valeurs que je dois définir pour cette structure :
Code:

1 2 3 4 5 6 7 8 9 {34215637, 8, 001, 256, 1.8, 123.59, 12}, {34215641, 64, 001, 512, 4.2, 223.59, 45}, {34215117, 8, 010, 256, 2.1, 93.59, 116}, {34215118, 2, 001, 64, 0.8, 23.59, 2}, {34200007, 24, 001, 256, 1.8, 123.59, 212}, {34214637, 8, 012, 256, 1.8, 49.99, 342}, {34200017, 32, 100, 256, 2.58, 183.49, 89} , {34225637, 4, 101, 256, 1, 33.59, 62} , {14215637, 16, 101, 256, 1.4, 89.49, 592}
Comment faire pour définir ma structure avec toutes ces valeurs ?

Merci à vous.

Il faut faire 1 tableau :mrgreen:

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
59
60
61
#include <stdio.h>
#include <stdlib.h>
 
typedef struct s_SwiStruct {
  int ref;
  int nb_ports;
  int constr;
  int speed;
  float energy;
  float prix;
  int qte;
} t_SwiStruct;
 
 
#define IS_EMPTY_ELEMENT(ONE_ELEMENT) ((ONE_ELEMENT->ref == 0) && (ONE_ELEMENT->nb_ports == 0) && \
                          (ONE_ELEMENT->constr == 0) && (ONE_ELEMENT->speed == 0) && \
                          (ONE_ELEMENT->energy == 0.0) && (ONE_ELEMENT->prix == 0.0) && \
                          (ONE_ELEMENT->qte == 0))
 
 
void display(t_SwiStruct* one_element) {
    if (one_element != NULL) {
        printf("%d, %d, %d, %d, %f, %f, %d\n", one_element->ref, one_element->nb_ports,
            one_element->constr, one_element->speed, one_element->energy,
            one_element->prix, one_element->qte);
    }
}
 
 
/*****************************************************************************/
/***********************************  Main  **********************************/
/*****************************************************************************/
 
int main(int argc, char** argv)
{
    t_SwiStruct list_elements[] = {
        {34215637, 8, 001, 256, 1.8, 123.59, 12},
        {34215641, 64, 001, 512, 4.2, 223.59, 45},
        {34215117, 8, 010, 256, 2.1, 93.59, 116},
        {34215118, 2, 001, 64, 0.8, 23.59, 2},
        {34200007, 24, 001, 256, 1.8, 123.59, 212},
        {34214637, 8, 012, 256, 1.8, 49.99, 342},
        {34200017, 32, 100, 256, 2.58, 183.49, 89} ,
        {34225637, 4, 101, 256, 1, 33.59, 62} ,
        {14215637, 16, 101, 256, 1.4, 89.49, 592},
        {0, 0, 0, 0, 0.0, 0.0, 0} // sentinelle
    };
 
    t_SwiStruct* elt;
 
    elt = list_elements;
 
    while ( !IS_EMPTY_ELEMENT(elt) ) {
        display(elt);
 
        ++elt;
    }
 
 
    return EXIT_SUCCESS;
}

29/10/2020, 18h26
IdiCoda

Merci beaucoup ! Cela marche parfaitement avec ce que j'ai réalisé.

Un peu bête de ma part de ne pas y avoir pensé...

Si je peux abuser un petit peu, je dois afficher le tableau trié par ordre croissant selon le 5è champ (energy).
Même en lisant la doc, j'ai du mal à comprendre son fonctionnement. Serait-il possible de me l'expliquer ?

Encore merci de votre temps et vous m'ôtez une énorme épine du pied !
29/10/2020, 19h02
foetus
Attention à ma solution parce que c'est 1 tableau constant :mrgreen:

Je suis presque sûr qu'on ne peut pas ajouter ou supprimer d'éléments.
Et ensuite, on peut peut-être modifier les valeurs des éléments :koi:

Pour ton tri, c'est trivial :langue2: : tu prends n'importe quel tri (tri rapide, tri à bulles, tri sélectif, tri par insertion, ...) mais
- Les tests vont porter sur le membre enegy if (elt1->energy <= elt2->energy) par exemple
- lors d'échange de 2 éléments, il faut recopier tous les membres. Donc il faut avoir 1 fonction void swap_2_elements(t_SwiStruct*, t_SwiStruct*).
- Peut-être même avoir 1 petite bibliothèque pour faire des choses de base : copier, échanger, tester en fonction d'1 membre, ... voire même des structures spécifiques comme 1 tableau, 1 pile par exemple pour ajouter ou supprimer 1 élément, inverser des éléments, partager ou fusionner 2 structures, ...
29/10/2020, 21h27
dalfab

Bonjour,

Si l'exercice consiste à utiliser la fonction qsort(), il faut :
a) définir une fonction qui indiquera comment trier sur le champ energy qui pourrait s'appeler int compare_energy_of_SwiStruct(const void*, const void*).
b) appeler la fonction qsort() en lui passant les bons paramètres, et le dernier des 4 est justement la fonction compare_energy_of_SwiStruct.
30/10/2020, 17h47
Sve@r

Bonjour

Citation:

Envoyé par foetus

Je suis presque sûr qu'on ne peut pas ajouter ou supprimer d'éléments.

Exact.

Citation:

Envoyé par foetus

Et ensuite, on peut peut-être modifier les valeurs des éléments :koi:

Oui, on peut. Ton tableau c'est exactement comme écrire int tab[]={1, 2, 3}; tab[0]=5.

Citation:

Envoyé par foetus

Peut-être même avoir 1 petite bibliothèque pour faire des choses de base : copier, échanger, tester en fonction d'1 membre, ... voire même des structures spécifiques comme 1 tableau, 1 pile par exemple pour ajouter ou supprimer 1 élément, inverser des éléments, partager ou fusionner 2 structures, ...

Hum... je pense que tu as perdu le PO...
01/11/2020, 09h00
IdiCoda

Merci à tous pour vos réponses, ça marche du feu de dieu pour le moment ! :D

Si j'ai d'autres problèmes concernant la suite de mon exercice, je peux continuer à m'adresser à vous ?
01/11/2020, 10h15
Sve@r

Citation:

Envoyé par IdiCoda

je peux continuer à m'adresser à vous ?

Ben... c'est un peu la raison d'être de ce forum. Et si on est là, c'est qu'on a envie d'aider les débutants à progresser en C, qu'on a tous quelque part une envie d'enseigner (certains d'entre nous sont ou ont été enseignants d'ailleurs)...
03/11/2020, 11h00
-Eks-
Citation:
Envoyé par foetus

lors d'échange de 2 éléments, il faut recopier tous les membres. Donc il faut avoir 1 fonction void swap_2_elements(t_SwiStruct*, t_SwiStruct*).
en, C, on peut copier le contenu d'une struct vers une autre de même type avec l'opérateur =

à moins d'avoir à faire des traitements intermédiaires (ce qui n'est pas le cas si la copie est à l'identique), créer une fonction ne me parait pas nécessaire.
03/11/2020, 13h43
Bousk

Faire un swap c'est assigner b à un temporaire, puis assigner a à b, puis assigner le temporaire à a.
Créer une fonction est un minimum de bon sens si on fait ça plus de 1 fois. Ou tout simplement pour avoir plus de lisibilité dans le code.
03/11/2020, 15h01
-Eks-

Citation:

Envoyé par Bousk

Faire un swap c'est assigner b à un temporaire, puis assigner a à b, puis assigner le temporaire à a.
Créer une fonction est un minimum de bon sens si on fait ça plus de 1 fois. Ou tout simplement pour avoir plus de lisibilité dans le code.

Certes, mais ce n'est pas mon point. Ce que que souligne, c'est que le C permet d'effectuer une affectation d'une struct avec l'opérateur = et qu'il est inutile de recopier tous les membres, car le C s'en charge, alors que le commentaire de foetus pouvait laisser penser le contraire.

J'ai donc essayé de clarifier ce point.

Une fonction peut être créée, bien sûr. Elle contiendra 3 instructions.

Je ne suis pas sûr que le fait qu'elles figurent dans une fonction appelée par une boucle ou dans la boucle qui les utilisera soit une affaire de bon sens dans notre cas de figure, ni une nécessité, mais plutôt de style ou de préférences personnelles et je ne vais pas me battre là dessus.
03/11/2020, 15h45
foetus

Citation:

Envoyé par -Eks-

une affectation d'une struct avec l'opérateur = et qu'il est inutile de recopier tous les membres, car le C s'en charge

Effectivement :mrgreen: ... mais c'est 1 copie binaire :mrgreen:

Donc avec les pointeurs, tu fais 1 "shallow copy" ... et donc potentiellement plantogène si ce n'est pas le comportement souhaité.
Mais, ici, c'est 1 structure avec des entiers/ flottants :mrgreen: ... donc pas trop de problème.

Citation:

Envoyé par -Eks-

une affaire de bon sens dans notre cas de figure, ni une nécessité, mais plutôt de style ou de préférences personnelles

Effectivement :mrgreen:

Cela va dépendre de la relecture de ton code :mrgreen: ... comme le dit également @Bousk
Tu en auras 1 plus "moderne" avec des swap, copy, ... des types Array_SwiStruct (par exemple) ... et donc qui attire l'attention sur le type de données que tu manipules.
Et 1 autre, 1 code C qu'il faut percuter que ta structure est 1 P.O.D. ("Plain Old Data") qui est manipulée comme 1 entier ou 1 flottant.
03/11/2020, 20h31
-Eks-
Peu importe la façon dont le C implémente la copie. C'est juste une copie du contenu de la struct que réalise l'opérateur = pas plus, pas moins

Citation:

Envoyé par foetus

Mais, ici, c'est 1 structure avec des entiers/ flottants :mrgreen: ... donc pas trop de problème.

Je serais plus affirmatif : il n'y a, dans le cas présent aucun problème, vu que ce que l'on cherche à copier est juste le contenu de la struct

Si je vois un code qui s'amuse inutilement à copier chaque membre de la struct à l'identique, cela ne me donne pas une bonne impression pour plusieurs raisons :
- c'est une ignorance des capacités du langage
- c'est du code inutile et qu'il faudra modifier demain si les membres de la struct changent, introduisant un risque de copie partielle du contenu si ce n'est pas fait
Citation:

Envoyé par foetus

Cela va dépendre de la relecture de ton code :mrgreen: ... comme le dit également @Bousk
Tu en auras 1 plus "moderne" avec des swap, copy, ... des types Array_SwiStruct (par exemple) ... et donc qui attire l'attention sur le type de données que tu manipules.
Et 1 autre, 1 code C qu'il faut percuter que ta structure est 1 P.O.D. ("Plain Old Data") qui est manipulée comme 1 entier ou 1 flottant.

Erm... J'ai du mal à saisir dans quel cas une struct en C ne serait pas POD ?

Personnellement, je ne suis pas pour la définition de "types" comme Array_SwiStruct. Si je peux éviter les typedef je le fais car ils masquent l'information. Le typedef crée un alias sur les types réels, qui ne sont plus visibles en utilisant cet alias.

C'est même tout à fait mal vu par certains :

https://www.kernel.org/doc/html/v4.1....html#typedefs
03/11/2020, 20h41
Sve@r
Citation:

Envoyé par -Eks-

Si je peux éviter les typedef je le fais car ils masquent l'information.... C'est même tout à fait mal vu par certains :https://www.kernel.org/doc/html/v4.1....html#typedefs

Euh... j'ai du mal avec l'anglais mais il me semble que le premier exemple de cet article met en opposition vps_t a face à struct virtual_container *a. Et donc ce n'est pas la création du type "vps_t" qui est mis au ban, mais le fait que le type créé masque l'étoile !!! Et là je suis en parfait accord avec cet ostracisation. On ne masque jamais un pointeur. De plus il me semble que le suffixe "_t" est réservé aux types "officiels" de la libc (ex size_t) et que pour les types perso, c'est le préfixe "t_" qui doit être utilisé => typedef ... t_vps.

En revanche je n'aurai rien à redire face à ce code...
Code:

1 2 3 4 5 6 7 typedef struct { unsigned short jj, mm, aa } t_date; ... t_date naissance;
... car je ne trouve pas que ce qu'il masque soit vraiment décisif face à cette écriture...
Code:

1 2 3 4 5 6 7 struct s_date { unsigned short jj, mm, aa }; ... struct s_date naissance;
Ok, dans la seconde écriture on voit que c'est une structure (on nous le jette à la figure à chaque variable créée en fait) et pas dans la première. Mais est-ce vraiment capital ?
03/11/2020, 21h26
foetus

Citation:

Envoyé par -Eks-

Peu importe la façon dont le C implémente la copie
...

Si je vois un code qui s'amuse inutilement à copier chaque membre de la struct à l'identique, cela ne me donne pas une bonne impression pour plusieurs raisons

Pour moi, il n'y a pas de bonne façon ni de mauvaise :zoubi:

En quoi faire 1 fonction de copy/ swap/ ... serait mauvais :koi: ? Justement cette fonction (qui peut être 1 macro) peut juste faire 1 affectation de structure.

Mais si tu rajoutes 1 pointeur ou 1 tableau dans ta structure, avec juste 1 affection tu seras quand même obligé de changer ton code, si tu veux faire des recopies ("deep copy")
D'autant + qu'1 fonction, c'est centralisé en 1 seul endroit au lieu de rechercher toutes les affectations pour modifier le code.

Mais on a bien compris ta façon de coder en C :zoubi: en mode "old school" et tout ce qui n'est pas nécessaire (typedef, petite bibliothèque pour mimiquer la POO, ...) on ne le fait pas.

Citation:

Envoyé par -Eks-

J'ai du mal à saisir dans quel cas une struct en C ne serait pas POD ?

Justement :whistle: je disais qu'1 structure est 1 POD, et donc que l'affectation fonctionne (mais c'est 1 copie binaire que justement je disais il faut connaître la subtilité)

Citation:

Envoyé par Sve@r

Euh... j'ai du mal avec l'anglais mais il me semble que le premier exemple de cet article met en opposition vps_t a face à struct virtual_container *a.

Le lien justement ne veut pas qu'on utilise typedef comme 1 sorte de type qu'on peut changer en cours de développement de façon + ou - transparente.

Par exemple passer d'1 type typedef int My_Flag à typedef long My_Flag.
C'est pour moi, 1 approche "old school" : chaque modification ne doit pas être transparente, et chaque modification dans 1 endroit de ton code doit être analysée pour éviter d'en louper 1 et de créer 1 bogue potentiel.
03/11/2020, 22h52
-Eks-

@foetus: cela m'a pris du temps de devenir "oldschool" comme tu dis ... je me suis beaucoup fourvoyé avec les typedef à mes débuts et l'encapsulation à outrance ;-)

Sur les changements "transparents" de types, l'un des cas où ce document reconnaît l'intérêt et la nécessité des typedef est justement lorsque l'on crée un type volontairement opaque. On doit avoir une bonne raison pour cela : l'utilisateur ne doit pas savoir ce qu'il y a sous le capot car les détails de l'implémentation peuvent changer. On lui demande alors de se reposer sur une interface qui permet à son code de fonctionner exactement de la même manière quelques soient les changements "internes" faits (dans la libc, les exemples typiques sont FILE ou time_t).

@Sve@r: c'est en anglais, mais il faut lire en entier et ne pas s'arrêter au premier exemple...

Citation:

In general, a pointer, or a struct that has elements that can reasonably be directly accessed should never be a typedef

Vas discuter de ton exemple avec Linus ;-)

Les exceptions sont listées de a) à e) et certaines ne concernent que le contexte du développement du noyau.

si tu écris :

Code:

struct date d;

Tu sais que d est de type struct date et tu n'as pas à te sentir obligé de préfixer l'étiquette d'un s_, car il n'y a pas d'équivoque pour le compilateur que "date" est l'étiquette d'une struct, ni pour le lecteur. En rajoutant un préfixe s_ tu introduis une répétition d'information redondante.

Si tu décides de créer un typedef, tu vas te sentir obligé de mettre un t_ devant ton nom de type (et un s_ devant le nom de l'étiquette de la struct comme tu le fais, histoire de marquer le fait que ce sont deux identifiants qualifiant des choses différentes). Puis, sauf à masquer cette information, pour savoir qu'il s'agit d'une struct en lisant le nom du type, tu rajouteras éventuellement cette information dans le nom du type, comme l'infortuné nom de type qu'on impose au PO SwiStruct, ou le t_SwiStruct qui a été cité dans ce fil.

je préfère écrire, dans ces conditions :

Code:

struct date d[9];

au lieu de :

Code:

t_date_struct d[9];

ou de :

Code:

t_array_date_struct d;

après, il faut dédramatiser, on parle de goûts et de couleurs :-)
04/11/2020, 08h59
Sve@r

Citation:

Envoyé par -Eks-

c'est en anglais, mais il faut lire en entier et ne pas s'arrêter au premier exemple...

Ouais j'avoue je devrais faire un effort de ce côté. D'autant plus qu'il y des traducteurs en ligne assez efficaces :oops:

Citation:

Envoyé par -Eks-

Vas discuter de ton exemple avec Linus ;-)

Oui, je l'ai fait et il est tombé d'accord avec moi. Il a dit "t'as raison, ce que tu as fait ca roxxe grave" :aie:

Citation:

Envoyé par -Eks-

si tu écris :

Code:

struct date d;

Tu sais que d est de type struct date et tu n'as pas à te sentir obligé de préfixer l'étiquette d'un s_, car il n'y a pas d'équivoque pour le compilateur que "date" est l'étiquette d'une struct, ni pour le lecteur. En rajoutant un préfixe s_ tu introduis une répétition d'information redondante.

Exact. Là effectivement c'est une bêtise de ma part. Le struct date suffit parfaitement.

Citation:

Envoyé par -Eks-

je préfère écrire, dans ces conditions :

Code:

struct date d[9];

au lieu de :

Code:

t_array_date_struct d;

Ah l'horreur !!! Aurais-tu masqué un tableau derrière un typedef ???

En fait, j'ai l'impression de ce dont on parle c'est la radicalisation d'un outil, et qu'on peut alors retrouver dans tous les outils (écrire tab[i++]=0 ou tab[i]=0; i++ ou encore memset(tab, 0, n*sizeof(*tab))). Est-ce l'outil qui est intrinsèquement mauvais ou bien juste la façon de s'en servir ? Un outil c'est bien quand on s'en sert correctement. J'ai déjà vu des cas où le goto se justifiait parfaitement. Ok il pouvait être supprimé mais rendait alors le code plus complexe avec des répétitions inutiles. Il reste à définir ce qu'on entend par "correctement" et ça c'est la partie la plus difficile du travail (goûts et couleurs comme tu le dis). Alors pour paraphraser je sais plus trop qui, je sais pas définir quand c'est correct mais je sais reconnaitre quand ça dépasse les limites, et cacher un tableau (ou un pointeur) dans un typedef c'est pour moi clairement une des choses à ne pas faire
Bien entendu, tout cela dit avec le respect d'un developpeur a un autre :chin:
04/11/2020, 16h09
kaitlyn

Bonjour,

Citation:

Envoyé par -Eks-

C'est même tout à fait mal vu par certains :

https://www.kernel.org/doc/html/v4.1....html#typedefs

C'est une convention de codage parmi tant d'autres. En dehors des personnes impliquées dans le code du noyau, elle n'a aucune valeur autoritative.
Mais de là à justifier des alias de types qui finissent en notation hongroise, c’est quand même une autre paire de manches.
04/11/2020, 21h38
Bktero

Je hais la notation hongroise :mouarf:

Un IDE moderne te met de l'auto-complétion, te montre l'aide d'un type quand tu passes ton curseur dessus, te permet de faire un CRTL+Clic sur un type ou une variable pour voir comme la déclaration / définition. J'ai même vu des IDE transformer un . en -> quand tu actives l'auto-complétion car il sait que c'est un pointeur.

En C++, quand tu fais struct Date {}, ça fait implicitement un typedef. Date date; et le tour est joué. Continuer en C à se promener avec struct date_t date;, c'est faire de la notation hongroise :aie:

Et puis, en toute honnêteté, si dans une fonction vous n'êtes pas capable de savoir de quel type est la variable x, est-ce que ce n'est pas que la fonction est beaucoup trop longue et a dépassé la taille critique au-delà laquelle il devient difficile de comprendre une fonction dans son ensemble ? :)
05/11/2020, 09h25
IdiCoda

Rebonjour !

Pour la suite de l'exercice, j'ai besoin d'ajouter la possibilité à l'utilisateur de soumettre sa propre commande, avec trois paramètres obligatoires : le nombre de ports, le débit et le prix. Le constructeur et l'énergie sont facultatifs, il peut les soumettre mais cela ne crée pas d'erreur si ce n'est pas le cas.

Si le nombre de paramètres est soit 3, 4 ou 5, la commande est valide; sinon un message d'erreur est renvoyé.

À la base, je pensais faire un scanf() puis une condition sur celui-ci mais cela me semble bien trop restrictif et je ne vois pas comment faire pour tester le nombre paramètres avec cette méthode.

Peut-être avec un case ?

Auriez-vous des idées ?

Les fonctions de la famille de scanf() sont très caprieuses, car elles tolèrent assez mal les chaines qui ne répondent pas au format attendu (le f à la fin de scanf, c'est pour "format" ;) ). Pour un code débutant, en connaissant ces limites, on peut se servir du fait que scanf() renvoie le nombre d'éléments convertis. Tu peux donc faire un code comme ceci :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 int main() { int a = 0; int b = 0; int c = 0; char buffer[1024]; fgets(buffer, 1024, stdin); int n = sscanf(buffer, "%d %d %d", &a, &b, &c); printf("Elements convertis %d\n", n); printf("%d %d %d", a, b, c); }
Avec la valeur de n, tu peux adapter ton code.

Par contre, clairement ce code n'accepte que des lignes comme "12 56 89", "12 56" ou "12". Cela correspond bien au format "%d %d %d" qui cherche 3 entiers.

Merci pour votre réponse !

Grâce à ce que vous m'avez proposé, j'ai adapté pour le besoin :
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 printf("Obligatoire : ports, debit, prix \n"); printf("Facultatif : constructeur, énergie \n"); char buffer[1024]; fgets(buffer, 1024, stdin); int n = sscanf(buffer, "%d %f %f %d %f" , &port, &debit, &prix, &constr, &eng); if ( n == 3) { printf("ports = %d, débit = %f et prix = %f \n", port, debit, prix); } else if (n == 4) { printf("ports = %d, débit = %f, prix = %f et constructeur = %d\n", port, debit, prix, constr); } else if (n == 5) { printf("ports = %d, débit = %f, prix = %f, constructeur = %d et énergie = %f\n", port, debit, prix, constr, eng); } else { printf("Le nombre d'informations renseignées n'est pas correct \n"); }
Ce n'est certes pas très optimisé mais ça fait ce qu'il faut, à un détail près. Si n est inférieur à 3, ça renvoie bien le message d'erreur mais s'il est supérieur à 5, ça ignore ce qui a été écrit et ça n'affiche que les 5 premiers paramètres. Je pense que c'est lié au scanf() qui ne prend en compte que les 5 informations rentrées, mais comment faire alors ?

EDIT :
En faisant ça, ça me donne le résultat voulu, mais je trouve pas bof comme alternative :

Code:

int n = sscanf(buffer, "%d %f %f %d %f %c" , &port, &debit, &prix, &constr, &eng, &buffer);

Des idées ?

05/11/2020, 12h14
Sve@r
Citation:

Envoyé par IdiCoda

Pour la suite de l'exercice, j'ai besoin d'ajouter la possibilité à l'utilisateur de soumettre sa propre commande, avec trois paramètres obligatoires : le nombre de ports, le débit et le prix. Le constructeur et l'énergie sont facultatifs, il peut les soumettre mais cela ne crée pas d'erreur si ce n'est pas le cas.

Si le nombre de paramètres est soit 3, 4 ou 5, la commande est valide; sinon un message d'erreur est renvoyé.

Auriez-vous des idées ?

En C, la notion "paramètre facultatif" n'existe pas. Une fonction qui attend 5 paramètre doit recevoir 5 paramètres. Il te faut donc ruser. Et là, il y a plusieurs possibilités
- certains paramètres pourront recevoir une valeur spéciale (ex 0 ou -1) signifiant "ne pas tenir compte". Ensuite dans ta fonction, tu testes et si 0 (ou -1 parce que "0" pouvant signifier "j'en veux spécifiquement aucun") alors tu gères le cas
- tu stockes les commandes dans un tableau de strings terminé par un NULL et tu passes ce tableau à ta fonction qui pourra alors compter combien de strings il y a avant le NULL et gérer selon le cas 3 ou 4 ou 5 strings. L'avantage de cette façon de faire c'est que tu n'as pas à créer de valeur spécifique comme le -1 de la solution précédente (c'est ce qu'on nomme une "valeur magique" et toute valeur magique doit ensuite être maintenue, c'est à dire la commenter et la suivre quand le code évolue) mais l'inconvénient, c'est que tu ne pourras pas commander de l'énergie sans spécifier aussi un constructeur car s'il n'y a que 4 strings, la 4° est obligatoirement celle du constructeur. Ou alors tu spécifies une syntaxe précise pour les strings (ex "constructeur=toto", "ports=18", ...) que tu peux ensuite analyser intelligement dans la fonction

Bonjour,

Citation:

Envoyé par IdiCoda

Code:

1
2
3
4
5
6
7
8
9
{34215637, 8, 001, 256, 1.8, 123.59, 12},
{34215641, 64, 001, 512, 4.2, 223.59, 45},
{34215117, 8, 010, 256, 2.1, 93.59, 116},
{34215118, 2, 001, 64, 0.8, 23.59, 2},
{34200007, 24, 001, 256, 1.8, 123.59, 212},
{34214637, 8, 012, 256, 1.8, 49.99, 342},
{34200017, 32, 100, 256, 2.58, 183.49, 89} ,
{34225637, 4, 101, 256, 1, 33.59, 62} ,
{14215637, 16, 101, 256, 1.4, 89.49, 592}

Il y a des valeurs octales dans ces structures, est-ce normal ?

05/11/2020, 13h15
Sve@r

Citation:

Envoyé par kaitlyn

Il y a des valeurs octales dans ces structures, est-ce normal ?

Je pencherais plutôt pour des valeurs décimales affichées sur 3 digits. Certes en C l'instruction int i=012 remplit "i" avec une valeur codée en octal, mais là il nous a présenté des valeurs "brutes" (comme si elles venaient d'un fichier ou d'une liste) donc sans lien avec une quelconque convention C et donc présumées être écrites en décimal.
De fait, si tu lis un fichier contenant une string "012" et que tu convertis cette string en int sans spécifier quoi que ce soit à la fonction de conversion, tu obtiens la valeur 12 et non 10.
05/11/2020, 13h21
IdiCoda

Citation:

Envoyé par kaitlyn

Bonjour,

Il y a des valeurs octales dans ces structures, est-ce normal ?

Oui. Certains champs ont des tailles spécifiques, notamment 3 chiffres pour le constructeur.
05/11/2020, 14h20
kaitlyn

Suite aux second et troisième messages de la discussion, tu sembles utiliser une source dans laquelle un tableau a été initialisé avec ces valeurs, d'où ma remarque.
Si tu affiches par exemple list_elements[2].constr (éventuellement avec un printf), tu comprendras.
05/11/2020, 14h24
IdiCoda

Les 00 situés devant disparaissent. Comment les forcer à rester ?
05/11/2020, 15h36
kaitlyn
Non c'est pire que ça, et ton modèle de structure étant imposé (int constr; ) tu ne peux remettre tes 0 qu'à l’affichage, par contre, pour une initialisation, il faut les enlever.
C'est là qu'est toute la différence entre la gestion en interne de données et leur présentation (à l'utilisateur). C'était peut-être un piège.

petite illustration
Code:

1 2 3 4 struct {int x, y;} s= { 010, 10}; printf("s.x: %03d (!)\ns.y: %03d", s.x, s.y);
05/11/2020, 15h40
foetus

Citation:

Envoyé par Sve@r

En C, la notion "paramètre facultatif" n'existe pas. Une fonction qui attend 5 paramètre doit recevoir 5 paramètres

:whistle: si cela existe. Ce sont les ellipses, utiliser notamment par les fonctions printf, scanf, ...

Documentation en anglais sur l'entête stdarg.h

Édit : Attention, ce n'est pas compliqué à utiliser mais cela peut être assez plantogène :zoubi:
Il faut soit passer le nombre d'arguments avant l'ellipse, soit mettre 1 valeur sentinelle. Si tu as le temps tu peux test :mrgreen:

Et tous les arguments doivent avoir le même type ... même si je pense qu'on peut bricoler :whistle:
05/11/2020, 15h42
IdiCoda

Oh, merci pour cette information ! Je ne le savais pas du tout.
05/11/2020, 16h13
Sve@r

Citation:

Envoyé par foetus

:whistle: si cela existe. Ce sont les ellipses, utiliser notamment par les fonctions printf, scanf, ...

Documentation en anglais sur l'entête stdarg.h

Arf en plus je les connaissais mais j'y ai pas pensé sur le coup !!! J'étais focalisé sur un truc type int fct(int x, int y=0) du C++ mais qu'on n'a pas en C
05/11/2020, 16h50
Bktero

Citation:

Envoyé par IdiCoda

Si n est inférieur à 3, ça renvoie bien le message d'erreur mais s'il est supérieur à 5, ça ignore ce qui a été écrit et ça n'affiche que les 5 premiers paramètres. Je pense que c'est lié au scanf() qui ne prend en compte que les 5 informations rentrées, mais comment faire alors ?

Tu viens de toucher les limites de scanf() ;)

Citation:

Envoyé par IdiCoda

En faisant ça, ça me donne le résultat voulu, mais je trouve pas bof comme alternative :

Code:

int n = sscanf(buffer, "%d %f %f %d %f %c" , &port, &debit, &prix, &constr, &eng, &buffer);

SI ça fonctionne et que c'est suffisant pour ton usage, tu peux t'en contenter ;)

J'ai pas d'idée simple et efficace à te proposer là, désolé.

Le scanf, écrit tel quel, on ne peut pas s'y fier. A l'inverse, la capture en chaines est moins faillible.
Dès lors, il est possible d'ajouter une étape et précompter les champs avant de tenter leurs conversions avec un second scanf.
Si ce dernier ne retourne pas le même nombre, c'est qu'il y a eu une erreur.
Code:

1 2 3 4 5 6 7 8 9 char buff[]= "12345 2.0 xxxx 3.999 xxx xxxx"; char tmp[79+1]= {0}; int nb_fields= sscanf(buff , "%79s %79s %79s %79s %79s %79s" , tmp, tmp, tmp, tmp, tmp, tmp ); printf("%s: %d\n", buff, nb_fields); //switch(nb_fields) {

07/11/2020, 17h53
Sve@r

Citation:

Envoyé par kaitlyn

Dès lors, il est possible d'ajouter une étape et précompter les champs avant de tenter leurs conversions avec un second scanf.
Si ce dernier ne retourne pas le même nombre, c'est qu'il y a eu une erreur.

Quel est l'avantage d'un second scanf par rapport à la méthode de Bktero ?
Si tu sais que tu dois avoir 6 nombres dès le départ, tu regardes si le scanf() t'en retourne 6 et ça devrait suffire non ?
07/11/2020, 21h15
kaitlyn

C'est parce que l'utilisateur peut saisir 3, 4 ou 5 données et que sscanf ne retourne que le nombre d'items correctement affectés.
08/11/2020, 09h27
Bktero

Et donc scanf renverra 3, 4 ou 5, et tu te rends compte que ça ne fait pas 6 nombres comme attendu...
08/11/2020, 15h36
kaitlyn

Le problème est que ce qui est attendu n'est pas 6, mais 3, 4 ou 5. Cf #20 sqq.
09/11/2020, 09h22
Bktero

Le posteur original est étudiant d'après son profil. On peut en déduire qu'il s'agit donc ici d'un exercice. Utiliser scanf() en supposant que la chaine est du format attendu (c'est ce que je mettais en prérequis dans mon message plus haut, c'était peut-être pas clair) est suffisant. Tu fais la convertion, tu regardes combien ça te renvoie, et tu sais quoi faire (3, 4 ou 5).

En revanche, tu as raison si on souhaite maintenant vérifier que la chaine est bien formattée, et rejeter par exemple "12 13 toto", appeler directement scanf() ne suffit pas.

Fuseau horaire GMT +2. Il est actuellement 21h55.