Discussion :

[airness86180]

Bonjour,

Je reprends un code qui compilait bien avec un ancien compilateur g++ 4.9 mais qui ne compile plus avec la 6.3. Il s'agit de tests de comparaison avec NULL pour tester l'existence d'un objet (voir message ci-dessous) :

error: no match for ‘operator==’ (operand types are ‘std::fstream {aka std::basic_fstream<char>}’ and ‘long int’)

if ( fbin == NULL )

Quelques précisions : fbin est déclaré comme fstream.

Code :

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

1
2
3
4
5
6
7
8
9
fstream fbin;
fbin.open(filename, ios::in);
 
	if ( fbin == NULL )
	{
		cerr << "Pb d'ouverture du fichier " << filename <<
				" en lecture." << endl;
		exit(1);
	}

Ne connaissant pas grand chose en C++, pouvez-vous m'aider s'il vous plaît ? Merci d'avance

airness86180

[foetus]

C'est pourtant explicite Tu testes si ta variable fbin est 1 pointeur valide, or ce n'est pas 1 pointeur

Documentation de std::fstream::open, lien cplusplus.com en anglais.

Pour savoir si le fichier est ouvert, il faut coder 1 test comme cela :

Code :

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

1
2
3
    if (ios::fail() == true) {
        return -1;
    }

Édit : effectivement @dalfab a raison : il existe 1 opérateur booléen std::ios::operator bool pour tester le flux (<- lien cplusplus.com en anglais)

[dalfab]

Bonjour,

La norme a en effet évolué. avant le C++11 il était assez complexe d'avoir une conversion vers bool sans effet de bord.
Le moyen le plus simple était d'effectuer une conversion vers pointeur (nul ou non nul) qui se convertissait ensuite en boolOn pouvait tester si un objet est valide par le test if ( fbin ) et tester le invalide par if ( !fbin ).
Sauf que certains voyant cette conversion en pointeur on cru comprendre qu'il fallait faire les tests if ( fbin != NULL ) ou if ( fbin == NULL ).
Depuis la conversion booléenne propre est possible, donc tous les codes bien écrits (ceux avec if ( fbin) ou if ( !fbin)) fonctionnent sans problème.
Pour les autre il faut corriger donc chez toi cela donne :

Code :

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

if ( !fbin )

[airness86180]

Super. Merci c'est très clair