Discussion :

[Neirbo]

Bonjour à tous

Je suis débutant programmation, et je bloque sur un petit problème pour une compilation.

J'ai la procédure suivante :

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
// PROCEDURE DE LECTURE D'UNE SUITE
void lire_suite (t_tab_r & g, int & lg)
{
	int cpt;
	char choix;
	cpt=0;
	do {
		cin >> g[cpt];
		cpt=cpt+1;
		cout << "Ajouter une valeur ? [O/N]";
		cin >> choix;
		if (choix !=o || choix !=n || choix !=O || choix !=N) {
			do {
				cout << "Les lettres acceptés sont 'O' ou 'N'. Ajouter une valeur ? [O/N]";
				cin >> choix;
			} while (choix!=o || choix!=n || choix!=O || choix!=N);
		}
	} while (choix !=o || choix !=O || cpt !=LGMAX);
	lg=cpt-1;
}

Comme vous le voyez c'est un truc très simple, cependant je ne comprend pas pourquoi à la compilation, Xcode me renvoit les erreurs suivante :

Code :

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

1
2
3
4
5
6
7
/Users/Adrien/Documents/Dev/S2_TP2_PART1/main.cpp: In function 'void lire_suite(float (&)[14], int&)':
/Users/Adrien/Documents/Dev/S2_TP2_PART1/main.cpp:37: error: 'o' was not declared in this scope
/Users/Adrien/Documents/Dev/S2_TP2_PART1/main.cpp:37: error: 'n' was not declared in this scope
/Users/Adrien/Documents/Dev/S2_TP2_PART1/main.cpp:37: error: 'O' was not declared in this scope
/Users/Adrien/Documents/Dev/S2_TP2_PART1/main.cpp:37: error: 'N' was not declared in this scope
/Users/Adrien/Documents/Dev/S2_TP2_PART1/main.cpp:43: error: 'o' was not declared in this scope
/Users/Adrien/Documents/Dev/S2_TP2_PART1/main.cpp:43: error: 'O' was not declared in this scope

je vous poste mon header pour peu que ça puisse aider :

Code :

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

1
2
3
4
5
6
 
#include <cstdlib>     // bibliothËque c standart
#include <iostream>    // flux d'entrÈe sortie
#include <iomanip>     // manipulateurs de flux              
 
using namespace std;   // permet d'utiliser les flux cin et cout

Voilà, je ne comprend pas ce qu'il se passe, c'est peut être une erreur bête mais j'avoue bloquer et en cherchant sur google je n'ai rien trouvé pouvant me dépanner, je ne sais même pas si ma procédure marche dans l'optique de ce que je veux faire je l'ai redigé en algo et ma traduction en C++ ne passe pas au compilateur, voilà.

D'avance merci !

[lboston]

Il manque les quote autours des caractères dans le if :

Code :

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

1
2
 
if (choix !='o' || choix !='n' || choix !='O' || choix !='N')

Sans les quote ces caractères sont consideres par le compilateur comme des noms de variable qui n'ont pas été déclarées. D'où l'erreur.

[Neirbo]

excellent, je te remercie. Je me doutais bien que c'était une erreur bête

[Médinoc]

Attention, ce test sera toujours vrai, car tu as mis des OU logiques au lieu de ET.

Si choix est égal à 'o', il sera forcément différent de 'n', donc...

Note: C'est une erreur commune. Tout le monde la fait un jour ou l'autre.

[3DArchi]

Attention, ce test sera toujours vrai, car tu as mis des OU logiques au lieu de ET.

Si choix est égal à 'o', il sera forcément différent de 'n', donc...

Note: C'est une erreur commune. Tout le monde la fait un jour ou l'autre.

Idem pour le do/while du niveau au dessus.

Sinon, tu peux aussi faire quelque chose comme ça :

Code :

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

1
2
3
4
5
6
7
8
   std::string const valide="oOnN";
   char choix;
   std::cout << "Ajouter une valeur ? [O/N]\n";
   while((std::cin>>choix)&&(valide.find(choix)==valide.npos))
   {
      std::cin.ignore( std::numeric_limits<std::streamsize>::max(), '\n' );
      std::cout << "Les lettres acceptés sont 'O' ou 'N'. Ajouter une valeur ? [O/N]\n";
   }

Ce qui permettrait d'exporter la chaine valide ailleurs et de pouvoir remplacer facilement o/n par y/n (valide="yYnN").