[Jérémy]

Bonjour à tous,

Toujours sur mon module GSM, j'ai repris depuis le debut !

Une fonction me turlupine car je n'arrive pas à la simplifier

Voici la fonction d'origine

Code : Tout sélectionner

//------------------------------------------------------------------------------
// Get GSM response, if there is any
short Get_response() {
    if (response_rcvd) {   // Si il y a une reponse qui est finie  (flag levé)
      response_rcvd = 0;   // RAZ du flag
      return responseID;   // On renvoie l'identification de la réponse
    }
    else
      return -1;           // Sinon on renvoie -1
}

// Attente de la réponse du GSM (boucle infinie)
void Wait_response(char rspns) {
char test = 1;

  while (test){
  test = Get_response();
  if ((test == rspns) || (test == GSM_ERROR))
    test = 0;
  else
    test = 1;
  }

} 

En paramètre on lui dit la réponse que l'on attend
On va chercher la réponse et on la place dans la variable "test".

Si cette réponse est égale a la réponse attendue ou on reçoit une erreur alors on sort, sinon on attend encore !

Plutôt bien fait mais je pense qu'on doit pouvoir raccourcir .
je pensais faire comme cela en pensant que ça revenait au même mais apparemment non car car ne fonctionne pas !

Tant que la réponse est différente de celle attendue ET différente d'erreur alors je boucle!

Code : Tout sélectionner

//------------------------------------------------------------------------------
// Attente de la réponse du GSM (boucle infinie)
void Wait_response(char rspns) {

     while ( (Get_response() != rspns) && (Get_response() != GSM_ERROR) );

} 

Je me demande si ce n'est pas a cause de la fonction qui est deux fois dans les conditions de test !

Qu'en pensez vous ?

Merci

[Jérémy]

J'ai trouvé une piste mais je comprends pas trop pourquoi ça ne fonctionne pas avec ma première solution .

Comme ceci ça fonctionne : pensez vous que ce soit fonctionnel !

Code : Tout sélectionner

//------------------------------------------------------------------------------
// Attente de la réponse du GSM (boucle infinie)
void Wait_response(char rspns) {

     while ( (Get_response() != (rspns && GSM_ERROR) ) );

} 

[brunog]

J'ai trouvé une piste mais je comprends pas trop pourquoi ça ne fonctionne pas avec ma première solution .

Comme ceci ça fonctionne : pensez vous que ce soit fonctionnel !

Code : Tout sélectionner

//------------------------------------------------------------------------------
// Attente de la réponse du GSM (boucle infinie)
void Wait_response(char rspns) {

     while ( (Get_response() != (rspns && GSM_ERROR) ) );

}

Hello,
non ça ne fonctionne pas, les deux expressions sont différentes car il n'y a pas de distributivité sur les opérateurs utilisés.

ta fonction Get_response() retourne un résultat qui est dépendant d'une variable qui est elle-même modifiée dans cette fonction : deux appels successifs à cette fonction peuvent donc donner des résultats différents, c'est le cas dans ton bout de code :

Code : Tout sélectionner

     while ( (Get_response() != rspns) && (Get_response() != GSM_ERROR) );

il ne faut faire qu'un seul appel à cette fonction, et stocker le résultat dans une variable intermédiaire. pour cela il faut remplacer le bloc while par un bloc do...while :

Code : Tout sélectionner

//------------------------------------------------------------------------------
// Attente de la réponse du GSM (boucle infinie)
void Wait_response(char rspns) {

     do
         {
         char rep ;
         rep = Get_response() ;
         }
     while ( (rep != rspns) && (rep != GSM_ERROR) );

}

[Jérémy]

Génial ca !



Que penses-tu de ma deuxième solution car j'ai un doute sur mes conditions ?

Code : Tout sélectionner

//------------------------------------------------------------------------------
// Attente de la réponse du GSM (boucle infinie)
void Wait_response(char rspns) {

     while ( (Get_response() != (rspns && GSM_ERROR) ) );

}  

[brunog]

Génial ca !



Que penses-tu de ma deuxième solution car j'ai un doute sur mes conditions ?

Code : Tout sélectionner

//------------------------------------------------------------------------------
// Attente de la réponse du GSM (boucle infinie)
void Wait_response(char rspns) {

     while ( (Get_response() != (rspns && GSM_ERROR) ) );

}

voir dans ma réponse ci-dessus, ça ne marchera pas.
il suffit de prendre un contre exemple :
supposons que get_response() retourne 1 (lors des deux appels successifs), rspns égale 2 et GSM_ERROR égale 3 :
la première expression :
(Get_response() != rspns) && (Get_response() != GSM_ERROR)
donnera :
(1 != 2) && (1 != 3)
= (true && true)
= true
alors que l'expression :
Get_response() != (rspns && GSM_ERROR) 
donnera :
(1 != (2 && 3)) = (true != (true && true))
= true != true
= false
ces deux expressions ne sont donc pas identiques

[Jérémy]

D'accord ! Je pense avoir saisi avec tes explications claires !

et