Discussion :

[Cpowa]

Salut,

J'ai encore une question.

Donc j'ai ma surcharge de l'opérateur >>

Code :

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

1
2
3
4
5
6
7
8
std::istream &operator>>( std::istream &fluxEntrant, Vecteur &vecteur )
{
 
    std::cout << "Entrez les " << vecteur.size() << " coordonnees : ";
    for(unsigned int i=0; i<vecteur.size();i++)
        fluxEntrant >> vecteur[i];
    return fluxEntrant;
}

Je me demandais comment je pouvais la protéger.
C'est à dire si l'utilisateur entre une lettre à la place d'un nombre. Idéalement il faudrait que ça se répète jusqu'à ce que l'utilisateur n'entre que des nombres.

Comment faire ?


Merci.

[Astraya]

Dans la FAQ

[Cpowa]

J'ai un peu de mal à comprendre.
ca veut dire que ma surcharge doit également renvoyer un bool ?

[Astraya]

Non mais tu test ton flux comme un bool.

Regarde ce bout de code tu vas comprendre:

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
int main()
{
    int tmp = 0;
 
    std::cin >> tmp ;
    if(std::cin.fail())
    {
        std::cout << "Vous n'avez pas entré un chiffre" << std::endl;
    }
    else
    {
        std::cout << "Vous avez entré un chiffre" << std::endl;
    }
 
    _getch();
}

[Cpowa]

j'ai tenté ceci:

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
std::istream &operator>>( std::istream &fluxEntrant, Vecteur &vecteur )
{
 
    std::cout << "Entrez les " << vecteur.size() << " coordonnees : ";
    for(unsigned int i=0; i<vecteur.size();i++)
    {
        fluxEntrant >> vecteur[i];
        while ( ! ( fluxEntrant >> vecteur[i] ))
        {
            if ( fluxEntrant.fail() )
            {
                std::cout << "Saisie incorrecte, recommencez : ";
                fluxEntrant.clear();
                fluxEntrant.ignore( std::numeric_limits<std::streamsize>::max(), '\n' );
            }
        }
 
    }
 
    return fluxEntrant;
}

Mais apparemment c'est pas bon puisque j'entre dans une boucle infini.

[gl]

<HS>

Donc j'ai ma surcharge de l'opérateur >>

Code :

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

1
2
3
4
5
6
7
8
std::istream &operator>>( std::istream &fluxEntrant, Vecteur &vecteur )
{
 
    std::cout << "Entrez les " << vecteur.size() << " coordonnees : ";
    for(unsigned int i=0; i<vecteur.size();i++)
        fluxEntrant >> vecteur[i];
    return fluxEntrant;
}

Je trouve assez malvenu que l'opérateur >> affiche des données. Ce n'est pas son rôle.

</HS>