Observer l'état d'une variable

**scheme** · 07/10/2010, 17h56

Salut à tous

Ma question va sûrement vous paraître triviale, mais je ne trouve pas de réponse.

Voici un bout de mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
int status;
 
status = fileReceptor.read(AUTO_DELAY, m_command_auto_delay);
if(status != SUCCESS_OPERATION) return status;
status = fileConfig.read(DURATION_CYCLE, m_duration_cycle);
if(status != SUCCESS_OPERATION) return status;
status = fileConfig.readDelays(m_date, m_date_passage_meridien, m_listDelays);
if(status != SUCCESS_OPERATION) return status;
 
return SUCCESS;

Comme vous le voyez c'est très répétitif, et j'aimerais améliorer ce code.
J'aimerais observer le status, et dès qu'il passe à une certaine valeur je stoppe.
Au début j'avais codé une boucle while, mais je me suis dit qu'il regardait le status seulement à la fin de la boucle, donc si ça plante au milieu, ça ne se verra pas.

Il doit sûrement y avoir un autre moyen, tout con en plus, mais je ne vois pas

Merci de m'apporter votre aide

**vikki** · 07/10/2010, 22h22

Hello,
Pourquoi pas enregistrer tes fonctions 'read' dans un vecteur de boost::function<int()>. Tu peux ensuite les appeler dans une simple boucle for:

Code :

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

1
2
3
4
5
6
 
for(int i=0; i< functions.size(); i++)
{
     if( !functions[i]() )
         break;
}

C'est peut être également faisable avec boost.signal s'il y a moyen de spécifier une condition d'arrêt.

EDIT: si tu n'as que trois conditions à tester, ca ne vaut peut être pas le coup de chercher à optimiser...

**white_tentacle** · 08/10/2010, 09h20

Pour éviter de tester partout les codes erreurs, on recourt généralement à une gestion par exception (par exemple, en wrappant la méthode read si tu ne peux pas la modifier, et en lançant une exception si le status_code est différent de SUCCESS_OPERATION).

**scheme** · 08/10/2010, 12h20

Salut à vous deux, vos réponses sont intéressantes

Je vais regarder le fonctionnement de boost::function. En fait j'ai plusieurs dizaines de conditions comme ça a tester, c'est pour ça que je cherche à économiser les lignes de code.

@white_tentacle: tu entends quoi par "wrapper la méthode read" ? L'idée serait de faire un throw(exception) dans la méthode read, et un catch dans les méthodes qui utilisent read ?

**white_tentacle** · 08/10/2010, 13h49

C'est l'idée.

C'est intéressant si tu as besoin de savoir qu'un traitement a réussi dans son intégralité, ou qu'il a échoué, mais pas forcément de savoir où il a échoué.

Du coup, tu peux te permettre un :

Code :

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

1
2
3
4
5
6
7
8
9
10
try 
{
  instruction1;
  instruction2;
  instruction3;
}
catch(readexception&)
{
  // traitement qui ne dépend pas de l'instruction qui a échoué
}

Ici, vu que tout ce que tu fais, c'est retourner l'erreur renvoyée en cas d'erreur, ça me semble approprié.

**scheme** · 08/10/2010, 16h18

Je trouve le principe super, mais en pratique j'ai des difficultés à lancer des throw :s

Par exemple j'ai une fonction read qui pour l'instant fonctionne comme ça:

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
 
//cette fonction ne lit pas dans un fichier mais dans un vector de string.
int File::read(int lineNumber, int & sortie) {
	if(!m_lines.empty() && lineNumber < (int) m_lines.size()) {
		try {
			sortie = atoi(m_lines.at(lineNumber).c_str());
			return SUCCESS_READING;
		} catch(exception &e) {
			cout << "You must clean the file before reading.";
			cout << e.what();
			return FAIL_READING_CONFIG;
		}
	} else {
		cout << "The container is empty, or the index is out of range." << endl;
		return FAIL_READING_CONFIG;
	}
}

Ici l'idée c'était de faire un throw si la conversion avec atoi a échouée, mais je ne vois pas comment procéder, à part relancer l'exception

**white_tentacle** · 08/10/2010, 16h24

Déjà, tu es mal barré : le atoi que je connais le lance pas d'exception en cas d'échec

. Tu peux regarder du côté de boost::lexical_cast pour faire ce que tu veux (et là, tu auras une belle exception).

Ensuite, il faut savoir que dans un bloc catch, écrire "throw;" tel que relance la dernière exception attrapée.

Enfin, tout dépend de ton besoin... Mais ta fonction read peut très bien lancer deux exceptions : LineOutOfRangeException et InvalidDataException, qui héritent toutes les deux de FileReadException. En fonction des besoins, le client choisira de gérer indifféremment les deux, ou de faire la distinction.

**scheme** · 08/10/2010, 16h32

En fait je n'arrive pas à utiliser boost. La raison est stupide: j'ai un makefile très compliqué qui vient du framework qu'on me force à utiliser, et je n'arrive pas à inclure boost dedans.

C'est pour ça que je passe par atoi, qui effectivement retourne un 0 en cas d'échec. C'est d'autant plus nul que dans plusieurs cas je vais lire des 0 ... Je ne peux donc pas savoir si atoi m'a bien retourné la valeur, ou un échec. C'est vraiment pas top pour lancer un throw donc :s

Au final je ne vois pas comment gérer les exceptions autrement que ce que j'ai fais

**white_tentacle** · 08/10/2010, 16h52

Envoyé par scheme

Au final je ne vois pas comment gérer les exceptions autrement que ce que j'ai fais

Est-ce que tu réalises bien qu'en fait, là, tu ne gères pas du tout le cas où atoi échoue ?

**scheme** · 11/10/2010, 00h19

Non je viens juste de réaliser

Aw je vais avoir du mal à m'en sortir

**white_tentacle** · 12/10/2010, 08h54

il y a plusieurs solutions pour t'en sortir :

- si tu sais que la valeur que tu lis n'es jamais 0, alors tu peux faire quelque chose du type

Code :

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

1
2
3
 
int ret = atoi(val);
if(ret == 0) { throw fail_reading_exception(); }

Si maintenant, ta valeur peut être zéro, il va falloir identifier les cas où atoi renvoie 0 parce qu'il a échoué, et ceux où il renvoie 0 parce qu'il a lu 0 (ah, les joies des api pourries...). Tu peux faire quelque chose comme :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
function monAtoiQuiLeveUneException(char const * val)
{
  int ret = atoi(val);
  if(ret == 0)
  {
    if(val[0] == 0) // cas d'une chaîne vide
      throw fail_reading_exception();
    else if(val[0] == '0' && val[1] == 0) // cas d'un 0 suivi d'un \0, c'est bien 0 qu'il faut lire
      return 0;
    else
      throw fail_reading_exception();
  }
}

**Jean-Marc.Bourguet** · 12/10/2010, 09h45

Envoyé par white_tentacle

il y a plusieurs solutions pour t'en sortir :

Pour commencer, me pas utiliser atoi mais strtol si on veut faire de la detection d'erreur.

**white_tentacle** · 12/10/2010, 10h04

Envoyé par Jean-Marc.Bourguet

Pour commencer, me pas utiliser atoi mais strtol si on veut faire de la detection d'erreur.

D'après ma page man, EINVAL n'est pas garanti...

. Ca marche sur toutes les implémentations que tu connais ?

**scheme** · 12/10/2010, 10h06

J'ai procédé de cette manière:

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
int File::read(int lineNumber, int & sortie) throw(out_of_range) {
	if(lineNumber >= (int) m_lines.size()) throw out_of_range("out of range");
	try {
		sortie =fromStringTo<int>(m_lines.at(lineNumber));
	} catch(BadConversionError &e) {
		cerr << "You must clean the file before reading." << " (int)"<< endl;
		cerr << e.what();
		return FAIL_READING_CONFIG;
	}
        return SUCCESS_READING;
}
 
template<typename T>
static T fromStringTo(string const& s, bool failIfLeftoverChars = true) throw(BadConversionError&){
	T x;
	istringstream i(s);
	char c;
	if (!(i >> x) || (failIfLeftoverChars && i.get(c)))
		throw BadConversionError(s);
	return x;
}

Qu'en pensez-vous ?