Discussion :

[NiouNiou]

Bonjour,

J'ai fait un TP de programmation dans le cadre de mes etudes et j'aimerais améliorer cette fonction:.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
void disciplinechoix(string *disc,string *noms, int n)
{
    string discchoisie;
    cout<<"Choisir la discipline : "<<endl;
    cin>>discchoisie;
    cout<<endl<<"Laureat pratiquant la discipline choisie sont: "<<endl;
    for(int i=0;i<n;i++)
    {
        if(discchoisie==disc[i])
        {
            cout<<noms[i]<<endl;
        }
    }
}

En fait j'aimerais contraindre l'utilisateur à retaper la discipline si elle est différente de "Physique" ou de "Chimie". J'ai essayé avec un do while mais la boucle me redemande la discipline a l'infini.
Voici le code modifié:

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
void disciplinechoix(string *disc,string *noms, int n)
{
    string discchoisie;
    do
    {
        cout<<"Choisir la discipline : "<<endl;
        cin>>discchoisie;
    }
    while(discchoisie!="Physique"||discchoisie!="Chimie");
    cout<<endl<<"Laureat pratiquant la discipline choisie sont: "<<endl;
    for(int i=0;i<n;i++)
    {
        if(discchoisie==disc[i])
        {
            cout<<noms[i]<<endl;
        }
    }
}

Sachant qu'on vient à peine de voir le chapitre sur les strings j'ai du louper un truc mais j'ai beau me creuser la tête je ne vois pas...
Merci d'avance pour vos conseils!

[ternel]

Comme c'est du C++, commence déjà par utiliser des références sur des string, plutot que des pointeurs.

ton problème, c'est que tu as un mis un ou logique (||) au lieu d'un et logique (&&).
discchoisie est forcément différente de "Physique" ou différente de "Chimie".

[NiouNiou]

J'ai utilisé des pointeurs car c'est l'énoncé du TP qui le demande.

Quand je remplace || par &&, j'ai le message d'erreur suivant:

error: no match for 'operator &&'

[ternel]

lis ton message d'erreur complet, il te dit le types des arguments, et t'énonce les différentes opérateurs && qu'il peut envisager, et pourquoi chacun ne va pas.
Au besoin, ajoute des parenthèses.

[ternel]

Concernant les pointeurs, je n'avais pas regardé correctement.
Le réflexe en C++ correct est de ne pas avoir de pointeurs.

En l'occurence, tes pointeurs (et le int qui les accompagne) sont des adresses de tableaux.
La solution "C++-esque" de ce C-isme serait d'avoir deux références sur des vecteurs.

La bonne solution serait d'avoir une référence sur un vecteur de Laureat, qui serait une structure ayant un nom et une discipline.
J'espère que ton professeur/cours t'en parleras très rapidement.


note: La solution idéale serait d'en faire une template, pour laisser à l'appeleur le choix du type de conteneur.

[Pyramidev]

Quand je remplace || par &&, j'ai le message d'erreur suivant:

error: no match for 'operator &&'

Chez moi, ça compile.
Es-tu sûr de ne pas avoir fait autre chose que de remplacer || par && ?