Meilleure collection pour filtrer

Version imprimable

25/02/2015, 11h01
Jb_One73

Meilleure collection pour filtrer

Bonjour à tous (désolé me revoilà),

J'ai une d'ordre existentielle, savez-vous qu'elle serait la collection la plus adapté à ce que je voudrais faire c'est à dire :
J'ai un fichier Xml qui contient x données, je voudrais récupérer uniquement les données correspondant à la chambre n après réflexion je me dis que j'ai peut-être pas besoin de tout charger dans une collection et ensuite filtrer ce que j'ai besoin ??

Merci d'avance à tous.
25/02/2015, 11h14
Népomucène

Je pense que oui : tu es dans le cadre d'un devoir et pas d'une application qui doit enregistrer dans une base.
En chargeant les chambres une fois en mémoire, tu n'auras pas à accéder à chaque échange à ton fichier XML
25/02/2015, 12h00
Jb_One73

Merci Népomucène pour ta réponse mais que veux tu dire exactement ? Que je dois lors de la 1ere lecture de mon fichier XML tout charger dans une collection (si tel est le cas, quelle collection serait la plus adapté ?) ou relire mon fichier XML et récupérer que ce dont j'ai besoin ?

Merci d'avance.
25/02/2015, 12h19
Népomucène

Je crois que tu as créé une classe pour représenter une chambre.
Comme il s'agit d'un devoir, je pense que la quantité de données n'est pas trop grande (< 500 Mo)
Donc tu peux te permettre de charger toutes tes chambres dans une Map qui a l'avantage de te donner l'accès par clé.
Tu pourras accéder à un objet "chambre" par son numéro si tu choisis (ce que je pense) de prendre le n° de chambre comme clé

regarde : http://java.developpez.com/faq/java/...TIONS_info_map
25/02/2015, 12h23
joel.drigo

Salut,

L'idée c'est de dire :

- au démarrage, je charge le xml dans quelque chose (on verra après ce que c'est) qui reste en mémoire
- à chaque fois que j'ajoute un nouvel élément, je l'ajoute dans ce quelque chose, et je sauvegarde dans le xml (en tâche de fond éventuellement, avec un Executor), comme ça le xml est toujours le plus à jour possible : mais comme le quelque chose est en mémoire et à jour, pas besoin de le charge.

Ensuite décrivons le quelque chose. Le mieux serait de faire un classe qui à une ou plusieurs variables, ce qui permet de changer la façon dont on stocke les données, sans avoir à changer toutes les classes et méthodes qui y ont accès.

Ensuite, la question de quelle collection. Evidemment, ça dépend de ce que tu veux en faire.

A là base une ArrayList est le choix le plus commun, est ça le fait le job pour la plupart des cas. Simplement une ArrayList est rapide en parcourt et accès direct à un élément dont on connait l'index, plutôt en ajout et suppression à la fin, mais lente pour les ajouts et suppressions au début et au milieu. Mais il faut quand même avoir un très grand nombre de manipulations très souvent pour que ça se ressente.

Il y a une alternative : le Set. L'avantage du Set est qu'il garantit l'unicité des éléments qu'on y met (mais on peut le garantir aussi avec un minimum de code, ou en faisant simplement attention à ce qu'on fait, avec l'ArrayList). Lorsqu'on utilise un HashSet, le test pour savoir si un élément est dans le set est très rapide. Mais l'inconvénient d'un Set est qu'on ne peut pas accèder à un élément directement par son index, mais les Stream peuvent nous aider, mais avec une performance moindre qu'avec un ArrayList (mais dans ton cadre, ça ne se verra pas vraiment).

Ensuite, il y a l'accès aux données selon une clef, ou un ensemble de clefs (un peu l'équivalent du where SQL), et, là, une ArrayList est relativement très lente pour ça : on peut l'assister avec une ou plusieurs Map selon les cas, et c'est assez simple à gérer.

Si tu as peu d'accès via l'index, mais beaucoup plus souvent par une clef, la Map suffira (on peut toujours parcourir son values()).

Ensuite, pour filter, pour faire des traitements en masse en particulier, tu as également les Stream. Ce qui peut te permettre également de générer des listes intermédiaires pour certains traitements : par exemple un traitement particulier sur toutes les chambres occupées.

[EDIT]Grillé par @Népomucène :P
25/02/2015, 12h31
Népomucène

Citation:

Envoyé par joel.drigo

[EDIT]Grillé par @Népomucène :P

Ne me dis pas que j'ai été plus rapide que toi ??? ça doit être un bug du forum :mouarf:
25/02/2015, 13h01
Jb_One73
Ok ! Merci à tous les 2 pour ces précisions. Je vais quand vous apporter quelques précisions sur le fonctionnement de mon appli après ce sera certainement plus claire pour vous, à savoir qu'une grande partie a été faite en contribution avec Joël (d'ailleurs je crois qu'à la fin de mon projet je vais mettre fait par joel.drigo).

Commençons par son commencement :
1 - J'ai une fenêtre principale qui me sert à créer, supprimer et visualiser des réservations, je vais insister sur le mot visualiser car c'est sur lui que je veux faire un filtre.
2 - Le chargement des données ce fait 1 seule fois et uniquement à l'ouverture du logiciel dans des collections de type ArrayList.
3 - A chaque mise à jour des données (ajouter, modifier et supprimer) les collections sont modifiés en fonction de l'action à faire et les fichiers XML sont mis à jour à leur tour mais en aucun cas je refais une relecture de ceux-ci.

Donc je pense que tel que la décrit Joël, je dois être bon dans mon fonctionnement.

Pour finir et c'est là le sujet de ma question :
Lorsque l'utilisateur clique sur visualiser, je dois être capable de récupérer toutes les réservations de la chambre en question, exemple :
Si je veux voir toutes les réservations de la chambre 1 je dois être de les récupérer dans une collection de type sont ensuite j'en ferai la lecture pour les afficher dans une jTable.

Je crois que les collections de type Map n'accepte pas les clés en doublons donc si je dois m'en servir il va falloir que je fasse un truc du genre :
Code:

1 2 HashMap<String, List<Booking>> cache = new HashMap<String, List<Booking>>();
Qu'est ce que vous en pensez ?

EDIT : Dactylo ce n'est pas mon fort, donc je met un peu de temps à écrire quelques lignes contrairement à Joël qui arrive à écrire des romans en 5mn

Merci d'avance
25/02/2015, 13h09
Népomucène

Oui, cela a l'air bien :
Tu accèdes à la liste des résa par la clé de la chambre. L'objectif est atteint.
Pareil pour ajouter une résa : cache.get(numChambre).add(monObjetRésa)

Je complète la réponse de @Népomucène :

Tu peux également faire une Map<Chambre, List<Booking>>, voire Map<Chambre, Bookings>, ou Bookings est une classe qui encapsule une List, ou un Map, ou tout autre instance qui t'arrange pour gérer les Booking.

Pour avoir utiliser Chambre en tant que clef, il te faudra respecter 3 choses :

avoir une méthode equals qui garantit que si chambre1.equals(chambre2) alors chambre2.equals(chambre1)
une méthode hashcode qui garantit que si chambre1.equals(chambre2) alors chambre1.hashCode()==chambre2.hashCode()
et que tout ce qui est utilisé pour calculer le hashCode et pour déterminer l'égalité vraie entre 2 chambres est immutable (ne peut jamais être changé tant que les chambres sont en clefs dans la map)

Par exemple :
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 31 32 33 34 35 public class Chambre { private final int numero; private boolean occupee; public Chambre(int numero) { this.numero=numero; } public final int getNumero() { return numero; } public boolean estOccupee() { return occupee; } public void setOccupee(boolean occupee) { this.occupee=occupee; } public int hashCode() { return Objects.hash(numero); } public boolean equals(Object obj) { if ( obj instanceof Chambre ) { // ou plus restrictif : ( obj!=null && obj.getClass().equals(Chambre.class) ) final Chambre chambre = (Chambre)obj; return chambre.numero==numero; } return false; } }

25/02/2015, 13h34
Népomucène

Citation:

Je complète la réponse de @Népomucène :

si après ça tu n'as pas une bonne note !!!
25/02/2015, 14h35
Jb_One73

Effectivement je l'espère surtout avec tout le mal que je me suis donnée, ben évidemment votre contribution m'a été forte utile et ça je ne pourrais jamais l'oublier.
Après on ne connait pas les gens et peut être que les attentes de mon prof ne sont pas dans mon projet donc ??? J'aurai quand même la fierté d'avoir fait un programme en JAVA et rencontré des gens comme vous.

Merci pour tout

Désolé de revenir encore sur sujet mais en regardant la structure de mon fichier XML, je me pose la question de savoir comment m'organiser pour affecter à la collection Map<Chambre, List<Booking>> les données correspondant à celle-ci ?
Je vais mettre la représentation du fichier XML pour que cela soit plus compréhensible :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 <?xml version="1.0" encoding="utf-8"?> <booking> <customer id="ZXHAU" checkin="2015/1/24" numberofnights="3"> <firstname>John</firstname> <lastname>Rambo</lastname> <id type="passeport">JR14231</id> <address/> <phone>0112365478</phone> <email>John.Rambo@gmail.com</email> <room id="103" type="rt1" state="available"/> </customer> </booking>
Comme je l'avais un peu plus haut dans mes messages, je charge une seule fois dans une collection de type List mon fichier XML donc, faut-il lors de la visualisation des réservations pour une chambre donnée que je parcours ma collection de type puis que j'ajoute les éléments trouvés dans Map<Chambre, List<Booking>> ? Ça fait pas un peu trop redondant tout ça ?

Merci d'avance.

Citation:

Envoyé par Jb_One73

Comme je l'avais un peu plus haut dans mes messages, je charge une seule fois dans une collection de type List mon fichier XML donc, faut-il lors de la visualisation des réservations pour une chambre donnée que je parcours ma collection de type puis que j'ajoute les éléments trouvés dans Map<Chambre, List<Booking>> ? Ça fait pas un peu trop redondant tout ça ?

A priori, tu as une List<Chambre> que tu as créé lorsque tu as chargé ton xml. Et je ne vois pas trop le rapport avec la map. Tu vois où une notion de redondance ?

La méthode reserver pourrait être
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 public void reserver(Chambre chambre, Booking booking) { List<Booking> roomBookings = bookings.get(chambre); if( bookings==null ) { // première réservation roomBookings = new ArrayList<>(); bookings.put( chambre, roomBookings ); } // stockage de la réservation roomBookings.add( booking ); }
pour la sauvegarde on peut avoir, par exemple :
Code:

1 2 3 4 for(Chambre chambre : listeChambres) { List<Booking> roomBookings = bookings.get(chambre); save(chambre, roomBookings...); }

25/02/2015, 16h11
Jb_One73

Mince ! J'ai l'impression que je n'ai pas dû m'expliquer correctement donc je vais essayé de repartir de 0 et si jamais c'est pas bon je ferai des captures d'écran.

Alors, mon problème de base c'est de savoir comment charger des données de réservation correspondant à un n° de chambre en gros cela reviendrait à faire ceci :
SELECT * FROM BOOKING WHERE ROOM = '1', j'exagère mais ça serait ça sauf que c'est sur un fichier XML dont j'ai posté la structure dans mon précédent message.

Pour continuer :
Avant d'en arriver à la visualisation des réservations, je charge les données contenu dans le fichier XML nommé booking.xml dans une List<Booking> array = new ArrayList<>(), cette collection va donc me contenir l'intégralité des données du fichier booking.xml c'est à dire le customer, la room et le booking donc tout ça est mélangé et tout ça me fait revenir à ma question principale que j'ai cité plus haut dans ce message.

Je sais pas si vous avez réussi à comprendre ce que j'essai de dire ? Juste une petite parenthèse : une jFrame qui pour le moment contient 1500 lignes de code est-ce que c'est beaucoup sachant qu'un l'intérieur il y a la conception du GUI ?

Merci d'avance.

Je vais apporter une précision concernant le fichier XML.
L'attribut <customer> est celui qui sert de séparation pour une réservation, là entre autre je n'en ai qu'une pour une seconde j'aurai la même chose avec <customer><firstname><.....></customer>.
J'ai donc créé une classe Booking qui contient tous les get du fichiers xml (firstname, lastname, ..., room).
25/02/2015, 16h40
joel.drigo

Citation:

Envoyé par Jb_One73

Alors, mon problème de base c'est de savoir comment charger des données de réservation correspondant à un n° de chambre en gros cela reviendrait à faire ceci :
SELECT * FROM BOOKING WHERE ROOM = '1', j'exagère mais ça serait ça sauf que c'est sur un fichier XML dont j'ai posté la structure dans mon précédent message.

On avait pas dit qu'on chargeait toutes les données au démarrage ? Pour moi ça voulait dire les chambres, les réservations, etc. Tu charges tout une fois, tu sauvegarde ce qu'il faut au fur et à mesure qu'il y a des modifications( ou des créations, ou suppressions), et donc le SELECT * FROM BOOKING WHERE ROOM = '1', revient à faire un get() sur ta map.

Citation:

Envoyé par Jb_One73

Pour continuer :
Avant d'en arriver à la visualisation des réservations, je charge les données contenu dans le fichier XML nommé booking.xml dans une List<Booking> array = new ArrayList<>(), cette collection va donc me contenir l'intégralité des données du fichier booking.xml c'est à dire le customer, la room et le booking donc tout ça est mélangé et tout ça me fait revenir à ma question principale que j'ai cité plus haut dans ce message.

ce n'est plus vraiment un problème si tu charges tout au démarrage. Le seul problème que tu peux avoir, c'est un problème de mémoire : la JVM est configurée par défaut pour 64Mo. Faudra voir si ce n'est pas un peu juste. Mettre 256 Mo (paramètre -Xmx) n'est pas trop exagéré vu la RAM qu'on a disposition sur les machines aujourd'hui, et ça devrait déjà être pas mal.

Citation:

Envoyé par Jb_One73

Je sais pas si vous avez réussi à comprendre ce que j'essai de dire ? Juste une petite parenthèse : une jFrame qui pour le moment contient 1500 lignes de code est-ce que c'est beaucoup sachant qu'un l'intérieur il y a la conception du GUI ?

Ça dépend comment elles sont réparties entre les méthodes. avoir 50 méthodes de 30 lignes, il n'y a rien d'anormal, Si tu as 2 méthodes de 750 lignes, il faut probablement factoriser et répartir. En plus ça dépend un peu comment tu comptes les lignes (ça comprends les lignes blanches, les commentaires, les déclarations de classe, les lignes où il y a juste une } ? L'application sur laquelle je bosse à 3,5 millions de lignes de code (sans les lignes blanches et les commentaires - et ce n'est que le client RCP, avec plein de bibliothèques - donc avec énormément de code que tu dois faire toi-même) : tu as encore de la marge :P Bon d'accord, elle fait nettement plus de choses que la tienne.

Merci d'avance.

Citation:

Envoyé par Jb_One73

Je vais apporter une précision concernant le fichier XML.
L'attribut <customer> est celui qui sert de séparation pour une réservation, là entre autre je n'en ai qu'une pour une seconde j'aurai la même chose avec <customer><firstname><.....></customer>.
J'ai donc créé une classe Booking qui contient tous les get du fichiers xml (firstname, lastname, ..., room).

[/QUOTE]
Ça, faut voir : si tu veux faire un suivi de client (ceux qui reviennent plusieurs fois), il faudrait peut-être faire un fichier à part et pointer dessus avec un id. Mais ce n'est peut-être pas essentielle pour ton projet. Tout dépend du temps qu'il te restera quand tu auras fait l'essentiel : je ne sais pas si tu dois faire un mémoire de présentation, mais ça pourrait être simplement une bonne idée de dire dedans que, comme amélioration possible, on pourrait le faire en donnant quelques extraits de code, quelques données techniques (format xml) etc...
25/02/2015, 17h09
Jb_One73

J'avais poursuivi dans la modif de mon message mais je me rend compte que tu as répondu après donc je vais remettre ce que j'ai noté dans celui-ci pour que tu le prenne en compte :

Je viens de me rendre compte que cela allait être plus compliqué que ce que je pensais. Après réflexion, je me rend compte que la structure du fichier est pas très top dans le sens ou tout est mélangé pour le client, j'ai donc demandé à mon prof si je pouvais modifier la structure du fichier et il m'a dit qu'il n'y avait aucun problème, donc pour aller plus loin dans ma réflexion il me semblerait plus judicieux de créer 2 fichiers xml.
1 - booking customer (n° de résa, client)
2 - booking room (n° de résa, room)
Les 2 seraient liés par un même id de résa et de cette façon, je pourrais charger dans une nouvelle collection mes chambres dans laquelle seraient stockées les dates de résa.

Qu'en penses-tu ?

Pour continuer, 3.5 millions de lignes !!!! Moi j'en ai un peux plus de 5000 en tout (lignes blanches, commentaires, etc inclus) et je deviens dingue donc toi avec ce que tu m'annonces je saute par la fenêtre ... bon ça va ! Il n'y a pas d'étage mais bon ....

Toujours pour continuer :
Si ! Si ! On a bien dit que l'on chargé toutes les données au démarrage et c'est ce que je fais, je charge bien les chambres, les clients et les résas et ensuite je sauvegarde que ce dont j'ai besoin lorsque je fais une modifs (si c'est pour un client je modifie que le fichier xml du client etc.)

Normalement, si j'ai écouté tout ce que tu m'as dit de faire ça doit être correct mais demande moi pas d'en faire 3.5millions car là ça va pas être possible ou peut être que si avec des copier/coller rébarbatif ;)
25/02/2015, 17h25
joel.drigo

Citation:

Envoyé par Jb_One73

J'avais poursuivi dans la modif de mon message mais je me rend compte que tu as répondu après donc je vais remettre ce que j'ai noté dans celui-ci pour que tu le prenne en compte :

Je viens de me rendre compte que cela allait être plus compliqué que ce que je pensais. Après réflexion, je me rend compte que la structure du fichier est pas très top dans le sens ou tout est mélangé pour le client, j'ai donc demandé à mon prof si je pouvais modifier la structure du fichier et il m'a dit qu'il n'y avait aucun problème, donc pour aller plus loin dans ma réflexion il me semblerait plus judicieux de créer 2 fichiers xml.
1 - booking customer (n° de résa, client)
2 - booking room (n° de résa, room)
Les 2 seraient liés par un même id de résa et de cette façon, je pourrais charger dans une nouvelle collection mes chambres dans laquelle seraient stockées les dates de résa.

Qu'en penses-tu ?

Actuellement tu as un xml pour la description de l'hotel et un pour les résa, c'est ça ? Faire 2 fichiers n'a pas grand intérêt pour le chargement, mais pour la sauvegarde, ça peut aider à éviter d'avoir à sauvegarde sans arrêt quelque chose qui change très peu. Mais scinder le fichier résa en 2, j'ai du mal à voir comme ça ce que ça pourrait apporter. Sauf si tu fais un suivi des clients (ce dont je t'ai parlé dans le précédent message). Même si tu as lien n-n (un même client peut réserver plusieurs chambres et une chambre peut être réservée par plusieurs clients différents : en plus, le problème quand on gère une liste d'ids dans 2 fichiers, ce sont les problèmes d'incohérences (normalement ça ne devrait jamais arrivé, si le code est parfaitement écrit, mais quand ça arrive, et ça peut, on se retrouve avec la moitié des données qu'on ne sais pas raccrocher). Mais vaut-il mieux perdre un couple réservation/client, ou n'avoir que l'un des eux dont on ne peut rien faire. Remarque, on sait toujours au moins qu'on a perdu quelque chose.

Par contre, toujours dans l'optique d'un chargement unique au démarrage de l'application, avoir 2 fichiers ou 2 ne changent rien.
25/02/2015, 17h35
Jb_One73

Pour être assez claire, j'ai en tout 5 fichiers xml (client, hôtel, réservation, chambre et type de chambre (celui-ci indique si c'est chambre simple, double, etc et le prix de la chambre)). Maintenant je me suis un peu trop embrouillé dans mon raisonnement et je sais plus s'il faut que je reste avec 1 seul fichier ou que je le scinde en 2 ?
Ce qui m'embête dans le fichier de résa (booking.xml) que j'ai donné plus haut, c'est que la chambre est renseignée dans le customer du client qui la réservé (en soit c'est normal), donc comment faire quand je vais sélectionner une période pour vérifier si la chambre 50 est dispo ou pas ? Il va falloir que je parcours toute la collection en cherchant toutes les chambres n° 50 et leur date de réservation c'est un peut un travail de titan !!!
Je viens de me rendre compte de ça, il faut que je vérifie de la dispo de la chambre pour la période donnée ... un véritable casse tête !!!
25/02/2015, 17h40
Népomucène

Citation:

Je viens de me rendre compte de ça

Tu viens surtout de te rendre compte qu'il vaut mieux utiliser une base de données dans ce cas.
(comme dans tous les cas de gestion d'ailleurs)

Combien de temps te reste-t-il avant de rendre le boulot ?
25/02/2015, 17h53
Jb_One73

Il me reste exactement 11 jours pour le rendre mais je ne peux pas utiliser de BDD (ce qui est dommage) car les fichiers XML me sont imposés donc, soit je vais brûler un cierge tout de suite en espérant avoir fini à temps soit je rend le travail au point où il en sera.
La deuxième solution m'ennuie un peu mais je ne suis pas Dieu et on en revient à la 1ere solution, là aussi ça devient compliqué. Celui qui a inventé JAVA à pas dû penser à moi :(
25/02/2015, 18h06
Jb_One73

J'ai réfléchit à une possibilité et vous me direz ce que vous en pensez :
Je créé de 2 fichiers (désolé Joël tu n'es pas d'accord mais c'est juste une réflexion après tu pourras m'insulter si tu veux) :
1 - booking qui va contenir les infos clients et le n° de résa.
1 - bookingroom qui va contenir la n° de la chambre, la date et le n° de résa.

Ensuite, je pensais faire comme ceci :
Le client réserve la chambre 1 du 25/02 au 28/02, j'enregistre donc le client dans booking et je lui génère un n° de résa 2502TOTO puis j'enregistre dans bookingroom room=1, chekin=25/02/2015, numberofnights=3.
Un autre client réserve la même chambre mais du 01/03 au 02/03 à ce moment là ! Le gars (c'est une phrase culte de Bigard) je modifie boockingroom room=1 en lui rajoutant uniquement checkin et numberofnights ce qui me donnerait

<bookingroom>
<room id=1>
<resa checkin=25/02/2015 numberofnights=3>2502TOTO</resa>
<resa checkin=01/03/2015 numberofnights=1>0103BOBY</resa>
</room>
</boockingroom>

Ainsi je pourrais charger dans une collection MAP boockingroom et pour faire des filtres ce sera plus facile, qu'en pensez vous ?
25/02/2015, 18h10
joel.drigo
Ok, tu as 5 fichiers. Ça ne change rien si tout est chargé en mémoire. C'est justement la map qui va t'éviter de parcourir toutes les listes. Si ton problème est que tu as un numéro (donc un int, ou peut-être un String), et que la map est Map<Chambre, List<Bookings>>, c'est sûr que pour accèder à la List<Bookings> il te faut le Chambre correspondant, mais il suffit d'une map Map<Integer, Chambre> pour retrouver l'instance de Chambre par son numéro.

Reprenons, au début tu charges tout :
- un xml d'hotel (les informations générales, je suppose, son nom, son lieu, etc...) -> chargé dans un objet Hotel
- un xml de type de chambres (ça c'est de la config : pourquoi pas, ça permet à ton programme d'être paramétrable) -> chargé dans une liste de type de chambres
- un xml de chambre, un hôtel j'espère (qui contient, pour chaque chambre, donc un numéro, un type de chambre, et je ne sais quoi (s'il y a une télé de 4 m de diamètre, un jacuzi, un minibar...) -> chargé dans une map de Chambre Map<Integer, Chambre>, et éventuellement dans une liste de Chambre (mais ce n'est pas nécessaire, la map suffisant)
- un xml de client : (voir plus loin) -> chargé dans une map de client Map<Integer (ou String), Customer>
- un xml de réservations ( qui contient, un id de résa, une période, l'id d'un client, un numéro de chambre) -> chargé dans Map<Chambre, List<Booking>>
Le problème peut être éventuellement avec le client : charger les 3000 clients qu'on a eu les 10 dernières années, alors qu'on a 30 chambres peut-être un peu lourd pour l'application. Dans ce cas, et pour ce cas uniquement, je pourrais procéder différemment. (En fait, j'avais compris que tu n'avais pas de fichier de client, et que pour chaque réservation tu stockais le client complet.) Donc qu'un même client qui réservait 2 fois, était dupliqué.
Si on faisait le chargement du client à la volée, donc pas au début, mais uniquement à la demande : ça prendrait un peu de temps (à recharger tout le fichier) à chaque fois qu'on prend une réservation, mais faut voir, ça peut être tolérable. Éventuellement, un chargement partiel en SAX ou Stax pourrait faire gagner globalement du temps.
Mais puisqu'on charge toutes les réservations, on a au moins en mémoire les clients qui ont une réservation en court : ça permet de gagner pas mal de temps, en particulier, parce que chaque fois qu'on affiche une réservation, on a déjà les infos du client en mémoire (pas besoin de recharger tout le fichier de clients). Et ça me semble largement suffisant, parce que, à part procédure particulière, on s'en fout des clients qui sont venus il y a 10 ans, et jamais revenu. Ceux qui sont important c'est ceux qui ont une réservation, et ceux qui occupent une chambre actuellement, et éventuellement ceux qui auraient fait leur checkout et qui aurait une facturation en cours (ce que ton programme ne gère pas, donc on s'en fout aussi).

Donc, pour résoudre ça, on peut charger le dom complet du fichier customer.xml, charger le dom du fichier resa.xml, construire la map Map<Chambre, List<Bookings>>, et Map<,Customer> qui ne contient que les clients qui sont dans Bookings (et peut-être aussi dans l'objet qui décrit l'occupation actuelle), et on ensuite on ne conserve pas les DOM en mémoire. Pour sauvegarder un nouveaux client, par contre, il faut recharger tout le DOM du fichier customer.xml. Maintenant, pour simplifier tout ça, peut être que charger les 3000 clients il ne faut pas tant de mémoire que ça.
Donc peut-être qu'on peut tout simplifier en chargeant toute la map de customer au démarrage. Je veux dire on se prend la tête peut-être pour 100Ko de données maximum pour 10 ans d'utilisation. Sinon, on peut aussi avoir un fichier d'archives avec les vieux clients qui ne reviennent plus depuis 10 ans (qu'on pourrait virer éventuellement) et un fichier des clients actifs, ce qui reviennent régulièrement, et qu'on va charger systématiquement en mémoire au démarrage.

Si tu as besoin d’accéder à une réservation par une autre clef que la chambre, tu peux utiliser d'autres map : une map par id de réservation Map<Integer/String, Booking>, une map par id de client, etc. Chaque map prend peu de mémoire par rapport aux données réelles qu'elles stockent, qui elles sont en communs, et n'existent en mémoire qu'en une seule version.
25/02/2015, 18h14
joel.drigo

Citation:

Envoyé par Jb_One73

Il me reste exactement 11 jours pour le rendre mais je ne peux pas utiliser de BDD (ce qui est dommage) car les fichiers XML me sont imposés donc, soit je vais brûler un cierge tout de suite en espérant avoir fini à temps soit je rend le travail au point où il en sera.
La deuxième solution m'ennuie un peu mais je ne suis pas Dieu et on en revient à la 1ere solution, là aussi ça devient compliqué. Celui qui a inventé JAVA à pas dû penser à moi :(

Pour avoir fait un serveur d'application avec tout stocker dans des fichiers xml, je peux te dire qu'il n'y a aucun problème pour gérer ça en Java. Et il y avait des milliers d'objet avec des liens entre, et tout. Simplement, chaque objet était dans un fichier différent, chaque objet avec un type et un id, et il y avait juste un index qui faisait le le lien entre un id et un path de fichier xml, qui pouvait être en mémoire ou dans une base de données. Et un cache MRU à base de map pour stocker en mémoire les objets.
25/02/2015, 18h28
Jb_One73

Ok ! Joël ! J'ai bien tout lu après avoir tout compris c'est tout autre cependant, l'indication que tu me donnes dans le chargement de mes collections n'est pas tout ça pour être claire toutes mes collections sont de type List.
Il n'y pas de Integer c'est que du String après je ne me suis jamais servis des collections MAP, il faut que je contrôle que le n° de chambre n'est pas déjà existant et si oui ... ???
Je me suis dit ce matin en me levant, cool ! je vais faire les résas et les visu aujourd'hui et comme ça il ne me restera plus que les résas auto et facture demain et en fait depuis ce matin je suis sur les résas et je n'ai même pas avancé. c'est un truc de dingue à chaque que j'ai l'impression de voir la fin du tunnel et bien non il y a autre chose.
Joël ! Toi qui a fait 3.5millions de lignes de code, tu dors bien la nuit ou tu as déjà cassé 15 TV ???
Ca va faire 3 semaines que je là dessus et j'avance pas d'un yota

Citation:

Envoyé par Jb_One73

Ok ! Joël ! J'ai bien tout lu après avoir tout compris c'est tout autre cependant, l'indication que tu me donnes dans le chargement de mes collections n'est pas tout ça pour être claire toutes mes collections sont de type List.

Comme on a parlé de Map avant, je pensais que tu étais passé aux Map.
Avec une liste tu es obligé de tout parcourir...

Citation:

Envoyé par Jb_One73

Il n'y pas de Integer c'est que du String

ça n'a pas grande importance pour le principe.

Citation:

après je ne me suis jamais servis des collections MAP, il faut que je contrôle que le n° de chambre n'est pas déjà existant et si oui ... ???

Ça n'a rien de sorcier. 2 façons pour l'écriture :
Si on charge l'ensemble de données (donc on est à l'initialisation), par exemple, des Clients, des Rerservations, des Chambres, on doit avoir une clef (donc l'id pour le clients, le numéro pour la chambre, etc.

Là ou aujourd'hui tu as un code qui fait :

Code:

list.add( machin );

tu vas avoir un code qui fait

Code:

map.put( machin.getId(), machin )

Pas compliqué, si ?
Si on a un système du genre : je veux créer un nouvel objet s'il existe pas dans ma list ou map map, on fait un truc du genre
Code:

1 2 s'il existe pas dans ma map, je le créé, le le met dans le map et je le retourne sinon je le prend dans ma map et je le retourne
donc là ou tu as aujourd'hui :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 Machin chercher( id ) { for(Machin machin : list ) { if ( machin.getId().equals( id) ) { return machin; } } Machin nouveaumachin = new Machin(id); list.add( nouveaumachin ); return nouveaumachin }
tu auras :
Code:

1 2 3 4 5 6 7 8 9 10 11 Machin chercher( id ) { if ( map.containsKey(id) ) { return map.get(id); } Machin nouveaumachin = new Machin(id); map.put( id, nouveaumachin ); return nouveaumachin }
ou :
Code:

1 2 3 4 5 6 7 8 9 10 11 Machin chercher( id ) { Machin machin = map.get(id); if ( machin == null ) { machin = new Machin(id); map.put( id, machin ); } return machin }

Bonjour Joël,

De retour sur mes collections, alors j'ai un petit avancé mais comme d'habitude je but sur un nouveau problème (ça devient une fâcheuse habitude). Comment fais-tu pour récupérer les valeurs d'une Hashtable dont la structure est la suivante :

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
 
public void AddXMLBooking (Hashtable<String,Booking> dataBooking) throws ParserConfigurationException, SAXException, IOException{
 
        /** Etape 1 : récupération d'une instance de la classe "DocumentBuilderFactory" */
        final DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance();
 
        try{      
            /** Etape 2 : création d'un parseur */
            final DocumentBuilder builder = factory.newDocumentBuilder();
 
            /** Etape 3 : création d'un Document */
            final Document document = builder.parse(new File(XmlDoc));
 
            // Element racine
            Element racine = document.getDocumentElement();
 
            Enumeration e = dataBooking.elements();
 
            while (e.hasMoreElements()) {
 
                // Début du corps
                final Element customer = document.createElement("customer");
                customer.setAttribute("id",e.nextElement());                             // <--- LE PROBLEME EST ICI, IL ME DEMANDE DE CASTER EN STRING
                customer.setAttribute("checkin",next.getCheckin());
                customer.setAttribute("numberofnights",next.getNumberofnights());
                racine.appendChild(customer);
 
/** Je ne met pas la suite car c'est la même chose */

Je n'arrive ni à récupérer la clé, ni à récupérer mes valeurs de ma classe Booking qui contient : firstname, lastname, etc avec une list c'était plus simple car il m'afficher directement toutes les méthodes de ma classe Booking.

Voilà mon nIème problème et il n'est que 8h10 (ça commence mal).

Merci d'avance.

26/02/2015, 09h20
Jb_One73
Cool ! J'ai trouvé ! En fait il ne faut pas que je passe par "Enumeration" car je n'est pas la possibilité de récupérer mes boonking, j'ai fait directement ceci est ça fonctionne :
Code:

1 2 3 4 for (int i = 0; i < htBooking.size(); i++) { htBooking.get(i).getIdbooking(); }
J'avance ! Pas trop vite mais bon ...

A toute à l'heure ! C'est sur !
26/02/2015, 09h52
joel.drigo
Pourquoi utilises-tu des Hashtable ? On t'a parlé de HashMap. Ce n'est pas gravissime, mais une Hashtable c'est un type qui est prévu pour fonctionner en multithread (elle est synchronisée), ce qui plombe un peu ses performances. En plus elle ne permet ni clef null, ni valeur null.

Pour
Code:

1 2 3 for (int i = 0; i < htBooking.size(); i++) { htBooking.get(i).getIdbooking(); }
Tu peux aussi faire :
Code:

1 2 3 for (Bookings booking : htBooking) { booking.getIdbooking(); }
A l'occasion, va voir aussi les possibilités des streams.
26/02/2015, 10h39
Jb_One73

Merci pour ta réponse, je viens d'appliquer les modifs donc c'est tout bon, en même temps il n'y avait pas grand chose à faire. J'ai encore une question :
Comment fait-on pour modifier une valeur et/ou clé dans une TreeMap ?

Merci d'avance.
26/02/2015, 10h43
Jb_One73

Je crois que j'ai trouvé ça doit être ça ?

Code:

tmRoom.replace(entree.getKey(),new Room(jTFIdRoom.getText(), jCBIdTypeRoom.getSelectedItem().toString(), state));

Exact ?

Citation:

Envoyé par Jb_One73

Je crois que j'ai trouvé ça doit être ça ?

Code:

tmRoom.replace(entree.getKey(),new Room(jTFIdRoom.getText(), jCBIdTypeRoom.getSelectedItem().toString(), state));

Exact ?

Là, sorti du contexte, je ne sais pas trop. C'est quoi tmRoom ?

Plus généralement :
pour remplacer une valeur associée à une clef, il suffit de remettre la nouvelle valeur associée à la clef, l'ancienne valeur est enlevée de la map :
Code:

1 2 3 4 map.put( "1", 2 ); // la map à 2 associé à "1" map.put( "1", 3 ); // la map à 3 associé à "1", et le 2 n'y est plus associé à "1"
Autrement dit : il ne peut y avoir qu'une seule valeur associée à une clef à un instant t
On peut aussi faire des remplacements lorsqu'on parcourt l'entrySet :
Code:

1 2 3 4 5 6 7 8 for(Map.Entry<String, String> entry : map.entrySet() ) { // exemple de principe if( "truc".equals( entry.getKey() ) && "machin".equals( entry.getvalue() ) { entry.setValue("bidule"); } }
On ne peut pas remplacer une clef. On enlèvera l'ancienne association, puis on créera une nouvelle :
Code:

1 2 valeur = map.remove(oldKey); map.put(newKey, valeur);