La syntaxe de goto en Java

Version imprimable

Voir 40 message(s) de cette discussion en une page

11/10/2007, 11h22
ghervouin

C'était juste histoire de chambrer un peu...

Ca fait 6 ans que je développe avec les patterns et tout le bazar et c'est vrai que dès que tu manipules des frameworks techniques, t'es bien obligé de cracher de la ligne de code à la suite...

Je suis bien d'accord avec toi en tout cas que de factoriser du code pour le plaisir n'a absolument aucun intérêt. D'ailleurs concevoir n'a rien à avoir avec factoriser du code !

Voilà mon avis à 2 balles...
11/10/2007, 11h28
OButterlin

Citation:

Envoyé par ghervouin

Voilà mon avis à 2 balles...

Et voilà l'travail ! Tu t'es fait corriger par des gens qui ne sont pas là pour rigoler ou qui ont également la tête dans le c.. (Je parle pour moi les gars, inutile de m'envoyer bouler, j'rigole :mrgreen:)

1 pièce(s) jointe(s)

Je déterre ce message – j'adore déterrer – pour exposer mon expérience récente (la dernière heure) :

Pour moi, l'absence de goto en Java, bien que venant d'intention louables, pose quand même problème dans certaines situations un peu particulières.

Pièce jointe 165692

Quand ce flux est contenu dans une seule et unique méthode, j'utilise return (attention, c'est du EBean et tout un foutoir de Json...) :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public void setAdresse(JsonNode adresse) {
        if (adresse.has("id")) {
            this.adresse = Adresse.find.byId(UUID.fromString(adresse.get("id")
                    .asText()));
            if (this.adresse != null)
                return;
        }
        List<Adresse> adresseList = Adresse.search(adresse);
        if (adresseList.size() == 1) {
            this.adresse = adresseList.get(0);
            return;
        }
        this.adresse = Adresse.fromJson(adresse);
    }

Quand je ne peux pas sortir d'une méthode (la logique n'est qu'une partie d'une méthode un peu plus vaste), c'est moche, j'utilise une fausse boucle while, un label et des break :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
bloc: while (true) {
                if (contactNode.has("id")) {
                    contact = Contact.find.byId(contactNode.get("id").asInt());
                    if (contact != null)
                        break bloc;
                }
                List<Contact> contactList = Contact.search(contactNode);
                if (contactList.size() == 1) {
                    contact = contactList.get(0);
                    break bloc;
                }
                contact = Contact.fromJson(contactNode);
                break bloc;
            }

Bien sûr, il y a probablement d'autres solutions, c'est la plus claire que j'ai trouvée pour le moment. Je vois difficilement comment factoriser ce genre de truc.

Citation:

Envoyé par Pierre-Yves Le Dévéhat

Quand je ne peux pas sortir d'une méthode (la logique n'est qu'une partie d'une méthode un peu plus vaste),

Situation parfaitement illégitime et indéfendable.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
address = extractAddress(contactNode);
 
/* ... */
 
private static String extractAddress(JsonNode adresse) {
  if (adresse.has("id")) {
    String candidate = Adresse.find.byId(UUID.fromString(adresse.get("id")
      .asText()));
    if (candidate != null) {
      return candidate;
    }
  }
 
  List<Adresse> adresseList = Adresse.search(adresse);
  if (adresseList.size() == 1) {
    return adresseList.get(0);
  }
  return Adresse.fromJson(adresse);
}

Ou alors :

Code:

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
address = AddressExtraction.fromNode(contactNode);
 
/* ... */
 
class AddressExtraction {
 
  static String fromNode(JsonNode adresse) {
    String candidate = byId(adresse);
    if(candidate == null) {
 
      List<Adresse> adresseList = Adresse.search(adresse);
      if(adresseList.size() == 1) {
        return adresseList.get(0);
      } else {
        return Adresse.fromJson(adresse);
      }
    } else {
      return candidate;
    }
  }
 
  private static String byId(JsonNode adresse) {
    if (adresse.has("id")) {
      return Adresse.find.byId(UUID.fromString(adresse.get("id")
        .asText()));
    } else {
      return null;
    }
  }
}

Ce ne sont pas les solutions simples, claires et donc par définition bien meilleures que du goto, qui manquent.

10/01/2015, 15h48
professeur shadoko
pour les amateurs de curiosités (et sans rapport direct avec les gotos)
Code:

1 2 3 4 SI_TOUT_OK: { //codes }
petit challenge à la communauté: exhiber des exemples pertinents d'utilisation de ce truc en Java :roll:
(de mémoire je crois que ça m'est arrivé deux fois ... mais il faudrait que je me plonge dans mes archives :oops:)
15/01/2015, 12h04
Pierre-Yves Le Dévéhat

Citation:

Envoyé par thelvin

Ce ne sont pas les solutions simples, claires et donc par définition bien meilleures que du goto, qui manquent.

Effectivement, je trouvais juste que devoir créer une méthode byId était un peu de trop, mais c'est toujours plus élégant que ce que je proposais.
Sinon, juste pour info, dans mes exemples, on n'attend pas une String mais un objet Adresse. J'ai adapté, notamment en incorporant les méthodes extract et byId à la classe Adresse (évitant ainsi de devoir les reproduire dans toutes les autres classes nécessitant ce mécanisme de récupération d'Adresse). Merci, en tous cas, pour le coup de main.

Voir 40 message(s) de cette discussion en une page