Discussion :

[4r19415]

bonjour
Je fais du c++ depuis déjà un moment je suis tombé sur un os pour un projet (un drone autonome):
quand je communique avec mon module wifi, je n'ai pas besoin de paralyser tout le programme pour attendre sa réponse

donc dans ma fonction de mise_a_jour, j'envoie un message et je repart dans la boucle stabiliser mon drone

mais, quand je reçois la réponse, il faut que je reprenne là ou je me suis arrêté dans mise_a_jour.... ça ressemble beaucoup à un GOTO !!!

mais on m'a toujours dit de ne jamais utiliser de goto pour des raisons évidentes, auriez vous une solution pour les remplacer ?

ps1 : mon programme doit être très rapide, je suis sur un ARM donc pas de switch avec un case pas ligne...
ps2 : ce code n'est clairement pas fonctionnel, c'est juste pour vous donner une idée du problème, il faut que je revoie le type de emplacement_actuel

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
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
 
void loop() 
{
  //plein d'instructions
 
  wifi.loop();
}
 
class wifi
{
  public:
    void loop();
 
  private:
    void reception();
    void mise_a_jours();
    String emplacement_actuel;
};
 
void wifi::loop();
{
  if (attente de réponse ) reception();
 
  else mise_a_jours();
}
 
void wifi::reception()
{
  if( la réponse n'est pas arrivée )
  {
    return(0);
  }
  else
  {
    goto(emplacement_actuel);
  }
}
 
void wifi::mise_a_jours()
{
  emplacement_actuel = "flag1:"
  write_serie("hey");
  return
  flag1:
  emplacement_actuel = "flag2:"
  write_serie("ho");
  return
  flag2:
  emplacement_actuel = "flag3:"
  write_serie("hey");
  return
  flag3:
}

Vous pouvez ragez sur le goto mais si vous avez mieux, je prend !

[ternel]

Et pourtant, switch serait le plus efficace.
Il faut bien réaliser que switch, break, continue, et même l'accolade fermante d'une boucle, sont compilés par un goto.

Le problème principal d'un goto, c'est qu'on peut faire n'importe quoi avec, et que c'est vraiment difficile de s'en apercevoir.

Utilise une enum plutot que des strings.
tu pourrais avoir des pointeurs de fonctions membres.

D'une manière générale, la performance s'améliore après mesure, jamais avant.

[Bousk]

Et ça marche ça ?!?
Parce que goto, il attend un label, et ça n'a strictement rien à voir avec une string. Donc ton code, j'ai surtout bien envie de dire qu'il ne marche pas du tout.
Ce que tu veux faire c'est une bête machine à état, un enum, un switch, chaque case est un état.
Quant à la rapidite.. un switch case c'est (le plus souvent?) une table et un jump, donc instant, la comparaison de string par contre..

[koala01]

Salut,

Je code depuis ... oufff, un sérieux bail maintenant, dans différents langages de programmation, même si je me suis peu à peu spécialisé en C++ et je peux t'assurer que je n'ai jamais été dans une situation dans laquelle le recours à un goto pouvait s'avérer nécessaire.

Je suis donc tout à fait de l'avis de mes collègues : une énumération et un swich... case semble beaucoup plus adapté, et sera -- quoi qu'il en soit -- toujours plus sécurisant qu'un goto qui risque de transformer ton code en un immonde plat de spagettis (et qui, de plus, risque de te faire faire n'importe quoi).

[4r19415]

Merci beaucoup

Zut, je pensait avoir trouvé le 1% où le goto était approprié ...

merci de vos réponse rapide, je pense que je vais utiliser un enum avec toutes mes fonctions (en pointeurs) et avancer en incrémentant l'indice, vous en pensez quoi ?
Mais ca risque de prendre pas mal de place mémoire et beaucoup de temps d’appel de fonctions non ?

[JolyLoic]

bonjour
donc dans ma fonction de mise_a_jour, j'envoie un message et je repart dans la boucle stabiliser mon drone

mais, quand je reçois la réponse, il faut que je reprenne là ou je me suis arrêté dans mise_a_jour.... ça ressemble beaucoup à un GOTO !!!

Ca ressemble surtout beaucoup à une coroutine. Ce n'est pas (encore) supporté directement par le langage, mais des implémentations à base de bibliothèques existent.

[Christophe]

Ce post compètera les réponses :

http://www.developpez.net/forums/d77.../goto-label-c/

[ternel]

Tu peux aussi regarder du coté du pattern "état" ("state").
C'est assez précisément ce que tu veux faire.

Il y a plusieurs variantes d'implémentations

[RPGamer]

Comme le disais leternel, goto est historiquement lié au fait qu'il s'agit d'une traduction direct d'un "jump" en langage assembleur, en tout cas dans le concept. A l'origine, on pouvait encore beaucoup programmer en assembleur et cette façon de penser les programmes par des sauts directs ou des sauts conditionnels à une adresse (ou un label) était très répandue. Instaurer une instruction goto dans les langages haut niveau comme C ou BASIC était alors naturel.

Les structures de langage qui ont été offertes par la suite on rendu cette manière de penser totalement caduc pour un programmeur "haut niveau". C'est du reste l'intérêt principal de ces langages, pouvoir remplacer des jump et des instructions logiques par des boucles for.

[4r19415]

Meri de toutes vos réponses,
je crois que j'ai assez d’élément pour m'en sortir maintenant, j'uploaderais la solution quand j'aurais quelque chose de propre