Discussion :

[Glork]

Bonjour,

j'aimerais trouver la définition de la méthode public boolean add(E) qui est déclarée dans l'interface Collection<E> si j'ai bien compris. J'ai regardé sur internet et ce serait :

Code :

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

1
2
3
public boolean add(E element){
    throw new UnsupportedOperationException();
}

Cette méthode est définie ds AbstractCollection<E>, mais je comprends pas très bien le code: elle se résume à lever une exception ok mais où est le code correspondant à ajouter l'élément ???
Merci.

[thelvin]

- Il n'y a pas de définition pour Collection, parce que Collection est une interface, et que le principe d'une interface c'est d'annoncer les méthodes qu'elles propose, sans les définir.

- La définition dans AbstractCollection ne fait rien d'autre que lancer une Exception, parce que c'est le comportement le plus utile pour une Collection dont on a pas précisé si c'est un Set, une List, autre chose, et s'il n'y a pas des contraintes dessus (comme par exemple que cette collection ne puisse pas être modifiée ou que sa taille doive être constante.)

- Si tu veux un exemple de méthode add(), il faut regarder pour quelle classe concrètement, et non pas pour les interfaces qu'elle implémente ou pour ses classes parentes. Parce que ce ne sera jamais pareil.
Par exemple HashSet, TreeSet, ArrayList, LinkedList, etc.

[JoeChip]

cela dit, "add" pour une Collection, en principe ça ajoute un élément à une Collection, ou alors les mots n'ont plus de sens...

[thelvin]

Oui, et "démarrer" pour un appareil, en principe ça démarre l'appareil, sinon ça ne veut plus dire grand-chose.
Il n'en reste pas moins que démarrer une télé et démarrer un tracteur se font de manière complètement différente et ont des effets qui n'ont rien à voir l'un avec l'autre. Pourtant l'appareil a bel et bien été démarré et l'un des effets et qu'il est à présent utilisable.

[tchize_]

cela dit, "add" pour une Collection, en principe ça ajoute un élément à une Collection, ou alors les mots n'ont plus de sens...

D'ou le "UnsupportedOperationException" qui exprimer clairement que l'AbstractCollection n'a pas la moindre idée de comment effectuer le add().

Ceci dit, il aurait probablement été plus propre que la méthode reste abstraite

[thelvin]

Propre certainement, pratique ça se discute.

À chaque fois que je crée une nouvelle Collection, je suis bien content de ne pas devoir me copier/coller encore une fois la méthode add() avec son throw new UnsupportedOperationException.

[tchize_]

je crée rarement des nouvelles classes collections, et encore plus rarement des nouvelles collections en lecture seule

[thelvin]

Bah dès qu'on a un objet dont le but est essentiellement d'en contenir d'autres... Moi si j'ai une Matrice => extends Collection. Une grille à coordonnées pseudo-infinies => extends Collection.
Certes des méthodes toArrayList() ou juste étendre Iterable plutôt que Collection, c'est plus facile. Mais c'est aussi moins confortable pour le filer à tout ce qui existe déjà.

(Accessoirement, read-only et refuser qu'on ajoute un élément sans dire où, ce sont deux choses différentes. Exemples : Arrays.asList(), Map.keySet()...)

[tchize_]

C'est vrai qu'en général, je travail en sens inverse. Je crée ma classe modèle, elle contient un champ genre List<Item> items; puis je fais bouton droit -> generate delegate methods, puis au final je rejoute le implement List
Du coup j'ai rarement besoin de passer par AbstractCollection.

Par contre, pour les tableau, je préfère utiliser le asList(), qui fait déjà tout le travail

[Glork]

Ok.
Je suis allé dans la classe ArrayList, dans le "Constructor detail" je trouve bien la méthode public boolean add(E e) mais où est le code de cette méthode ?
Merci.

[tchize_]

Tu ouvre ton eclipse

tu écrit

ArrayList a = new ArrayList();
a.add("Hello");

et tu fait ctrl-click tu "add", eclipse va t'ammener directement au code de la méthode

[Glork]

J'ai mis ce code dans une classe quelconque (en important java.util.ArrayList), mais quand je fait CTRL-CLICK sur add, ça m'ouvre ArrayList.class où il y a marqué:
Source not found
The jar file C/.../rt.jar has no source attachment
You can attach the source by clicking Attach Source below.

Je clique dessus, mais je vois pas trop quoi faire (ss faire de bêtise).

[joel.drigo]

Salut,

Tu actionnes le bouton "Attach source", tu vas dans le dossier d'installation du JDK, tu sélectionnes le fichier src.zip...

[Glork]

je suis allé ds le dossier d'installation du jdk parcouru pas mal de dossiers mais j'ai pas trouvé de fichier src.zip: je suppose qu'il faut le télécharger ?
(décidément rien n'est simple ^^)

[thelvin]

Le code source fait partie de l'installation du JDK. Il est possible de demander à ne pas l'installer, et du coup tu te retrouves sans src.zip. Mais c'est uniquement parce que tu l'as voulu quand tu as installé le JDK. Relance l'installeur et demande-lui, cette fois, de mettre les sources avec.

[tchize_]

et normalement eclipse devrait trouver les sources comme un grand. Sauf si tu as associé un jre à ton projet au lieu d'associer un jdk

Sinon, le code dans ArrayList est tout con:

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 boolean add(E e) {
        ensureCapacityInternal(size + 1);  // Increments modCount!!
        elementData[size++] = e;
        return true;
    }
 
 
    private void ensureCapacityInternal(int minCapacity) {
        modCount++;
        // overflow-conscious code
        if (minCapacity - elementData.length > 0)
            grow(minCapacity);
    }
 
 
    private void grow(int minCapacity) {
        // overflow-conscious code
        int oldCapacity = elementData.length;
        int newCapacity = oldCapacity + (oldCapacity >> 1);
        if (newCapacity - minCapacity < 0)
            newCapacity = minCapacity;
        if (newCapacity - MAX_ARRAY_SIZE > 0)
            newCapacity = hugeCapacity(minCapacity);
        // minCapacity is usually close to size, so this is a win:
        elementData = Arrays.copyOf(elementData, newCapacity);
    }

[Glork]

Ok pour le code de add (enfin^^)
Merci !