Discussion :

[scolopendra]

Bonjour,

j'ai une boucle dans une methode1() qui permet de parcourir un fichier et extraire donnée par donnée un "for(Event e : file)" par ex.

ma question c'est je veux savoir si on peux mettre cette boucle dans une autre methode : methode2(Event e) et au lieu d'integrer la boucle dans methode1() .j'integre methode2(e) dans methode1().
j'espere que c'est compri au moin

et merci

[KiLVaiDeN]

Salut, oui tu peux appeler une méthode paramétré à l'intérieur d'une autre méthode, c'est même conseillé de séparer ton application avec des méthodes atomiques, pour faciliter la réutilisation.

Je ne vois pas bien ce qui pourrait te faire croire que ça n'est pas possible ? Est-ce un souci de performance?

Si oui, la performance est gérée par le compilateur, en théorie, et le compilateur de bytecode de sun est assez performant pour optimiser ce genre de choses.

A+

[scolopendra]

non c'est juste parceque j'ai jammais entendu parlé pourraige avoir l'algorithme simplifié de cette methode ?et merci

[CyberChouan]

Voilà un exemple de division d'une grosse méthode en sous-méthodes: comme tu vois, ça n'a rien de sorcier.

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
 
public void methodePrincipale() {
 
   List<Event> maListe = null;
   // recuperation de ta liste;
 
   traitementListe(maListe);
 
}
 
private void traitementListe(List<Event> maListe) {
   for(Event e: maListe) {
      traitementUnitaire(e);
   }
}
 
private void traitementUnitaire(Event e) {
   // ici la méthode de traitement d'un élément de ta boucle
}

[scolopendra]

en fait moi c'est plutot comme ca :

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
public void methodePrincipale() {
 
   Event event = null
 
   this.traitementListe(event);
    //traitement sur e
 
}
 
private void traitementListe(Event e) {
   for(Event e: file) 
     event=e;
     return event;
 
}

[KiLVaiDeN]

Salut,

Tu as des grosses confusion entre variables locales à une fonction, membres de classe, paramètres de méthode, et variables de retour.

Quand dans ta méthode tu écris "event = e", le compilateur va te dire qu'il ne connait pas event; Normal, il n'existe pas dans la portée de ta méthode.

De plus, l'interêt d'appeller une méthode pour traiter une liste, n'est jamais de retourner un élément de celle-ci, mais j'imagine dans ton cas plutot de traiter chacun des Event, et eventuellement de retourner une collection d'objets Event.

Regarde ce code :

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
public class MaClasseDeTest {
  private Collection<Event> mesEvents;
 
  // Le getter de ton membre privé
  public Collection<Event> getMesEvents() {
    return mesEvents;
  }
 
  public void chargerMonFichier(String fichier) {
    // lire le fichier
    BufferedReader monFichier = new BufferedReader(new FileReader(fichier));
 
    String ligne = null;
    while(ligne = monFichier.readLine() != null) {
      Event event = new Event();
      event.setName(ligne.substring(1,10));
      mesEvents.add(event);
    }
  }
 
  public static void main(String[] args) {
    MaClasseDeTest maClass = new MaClasseDeTest();
    maClass.chargerMonFichier("maListeDEvent.txt");
    // A ce stade, maClass.getMesEvents() te retournera ta collection chargée.
  }
}

A+

[Rayndell]

Ta variable event, déclare-la plutôt comme attribut de ta classe, et ne fournit aucun paramètre à ta méthode traitementListe, et ça devrait marcher.