Récupérer résultat d'une boolean checkbox

Version imprimable

Bonjour alors voila mon bout de code :

fichier xhtml :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
<h:form>
     <h:dataTable value="#{personneService.personnes}" var="_pers" border="2">
         <f:facet name="header">
              <h:outputText value="Liste des joueurs" />
         </f:facet>
     <h:column>
          <h:selectBooleanCheckbox value="#{personneService.setpersonnesSelectionnees[_pers]}" />
     </h:column>
.
.
.
</h:dataTable>
    </h:form>
         <h:commandButton 
               action="#{personneService.supprimer()}" 
               value="Supprimer la slection">
         </h:commandButton>

fichier de service :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
private List<Personne> personnes = new ArrayList<Personne>();
private Map<Personne,Boolean> personnesSelectionnees;
 
public void supprimer() {
    if (personnesSelectionnees != null){
        System.out.println("personnes selectionnees pas null");
        for (Personne pers : personnesSelectionnees.keySet()){
              if(personnesSelectionnees.get(pers) == true){
                     Personne toDelete = em.merge(pers);
                     em.remove(toDelete);
              }
        }
    }
}

Ca ne fonctionne pas.
je suis débutant en JSF, si vous pouviez m'aider...

Merci beaucoup!

15/04/2008, 16h57
M€lK!oR

Assure toi déjà que ta Map ai un getter et un setter.

Ensuite, quand tu utilise une EL ( le #{ ... } ), il faut jamais prefixer avec set ou get, c'est géré tout seul.

Essaye en remplacant par :

Code:

<h:selectBooleanCheckbox value="#{personneService.personnesSelectionnees[_pers]}" />

Merci beaucoup pour cette réponse rapide.
Pour le set, oui désolé, j'avais essayé de la mettre ne voyant que ça ne fonctionnait pas autrement.;)
J'ai bien les getter et setter :
Code:

1 2 3 4 5 6 7 8 public Map<Personne, Boolean> getPersonnesSelectionnees() { return personnesSelectionnees; } public void setPersonnesSelectionnees( Map<Personne, Boolean> personnesSelectionnees) { this.personnesSelectionnees = personnesSelectionnees; }
je n'ai pas d'erreur lorsque j'ouvre la page mais quand je coche une ligne et que j'appuie sur supprimer, rien ne ce fait!
Je n'ai d'ailleur pas le System.out.println que j'ai mis dans la méthode supprimer !

15/04/2008, 17h08
romaintaz

action="#{personneService.supprimer}"
Il ne faut pas mettre les () à la fin.

Sinon je te conseille de lire quelques posts de ce forum concerne le design... Parce que lier directement la couche service avec les pages JSF c'est pas forcément la meilleure chose à faire ;)
15/04/2008, 17h11
heid

Ton bean doit être en scope request, soit tu fais en sorte de persister les données de ton arrayliste de personnes avec un savestate par exemple, soit tu passes ton managed bean en mode session.
15/04/2008, 21h35
monstor

Re bonsoir!

Merci pour vos réponses.
Alors j'ai enlever les parenthèses sur l'appel a la méthode supprimer.
J'ai aussi modifier le type du Scope et ai mis SESSION (il était à PAGE avant).
Je ne sais pas si c'est ce que tu voulais dire heid ?

Je pense que le problème viens plutôt de l'enregistrement de la Personne dans la Map personnesSelectionnees car dans le supprimer je fait un system.out.printlln si cette liste n'est pas nulle, ors il ne me l'affiche jamais! Donc pour moi, soit il n'y a ajmais d'enregistrement dans la Map personnesSelectionnees soit alors on ne rentre jamais dans la méthode supprimer :s
16/04/2008, 09h10
seddik_saber

primo y'a pas de scope page dans JSF.
<h:dataTable value="#{personneService.personnes}" var="_pers" border="2">
<f:facet name="header">
<h:outputText value="Liste des joueurs" />
</f:facet>
<h:column>
<h:selectBooleanCheckbox value="#{personneService.setpersonnesSelectionnees[_pers]}" />
</h:column>
.
.
.
</h:dataTable

a sa je comprend rien ??? tu veux faire quoi avec ton check box ???

personellement je croix que tu t'es lance sur jsf sans avoir lis quelque chose
car dans les EL on fais pas appelle a des methodes parametere a moin t'as fais une extension a l'EL.
16/04/2008, 09h25
monstor

Alors si j'ai lu des tuto JSF contrairement à ce que tu dis. Je débute simplement !

La partie que tu ne comprend pas est simplement un tableau dans lequel la première colonne est une checkBox et les autres me donnes les nom, prénom,... etc des personnes étant dans ma base.
Je veux que quand on coche ou décoche la chekbox, que la personne en question soit ajoutée ou supprimée dans ma Map personnesSelectionnees afin de pouvoir en supprimer plusieurs à la fois en cliquant sur le bouton supprimer.
J'avais auparavant mis le bouton supprimer sur chaque ligne de mon tableau et comme ceci ça fonctionnait très bien.

j'espère avoir été clair...
16/04/2008, 09h30
seddik_saber

ok :)

voila moi ce que je te propose :
Cree une Classe PersonneRow qui est comme suit

public class PersonneRow{
private boolean checked;
private String nom;

// faire les accessor
}

puis t'as ton tableau
<h:dataTable value="#{personneService.personnes}" var="pers" border="2">
<f:facet name="header">
<h:outputText value="Liste des joueurs" />
</f:facet>
<h:column>
<h:selectBooleanCheckbox value="#{pers.checked}" />
</h:column>
<h:column>
<h:selectBooleanCheckbox value="#{pers.nom}" />
</h:column>
</h:dataTable>

dans ton backing bean:

private List<PersonneRow> personnes = new ArrayList<PersonneRow>();

et vire le hashmap

lors de la suppression parcours la liste personnes et supprime ce avec checked true
16/04/2008, 09h39
monstor

Ok, merci, je vais essayé tout ça!
Au vu de de que tu fais, on ne pourrait pas simplement rajouter un attribut boolean checked a ma classe Personne?

PS : on peut mettre des choses comme ca :
<h:commandButton action="#{personneService.enregistrer()}" value="Enregistrer"></h:commandButton>
Ce qui va bien appeler la méthode enregistrer() de ma classe PersonneService (et ça fonctionne bien ;) )
16/04/2008, 09h58
seddik_saber

oui tu peux mais personellement je laisse les nonm comme personne, et tous ce qui reflete le business a mes entite jPA :)

sinon pour ton bouton enleve les parathese a enregistrer

Voila, ca fonctionne, enfin non mais les methodes sont appelées correctement et il voit les joueurs qui sont cochés.

J'ai une erreure lors de la suppression :

Citation:

GRAVE: java.lang.IllegalArgumentException: Removing a detached instance testNico.Personne#1

Voila le code de ma méthode supprimer :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 public void supprimer() { Iterator<Personne> it = personnes.iterator(); while (it.hasNext()){ Personne p = it.next(); if(p.isChecked()){ em.merge(p); System.out.println("em.merge okkkkkkkkkk"); em.remove(p); System.out.println("em.remove okkkkkkkkkk"); personnes.remove(p); } } em.flush(); }
Le problème c'est le em.remove(), pourtant j'ai bien rattaché l'objet p avant avec le em.merge() ?

16/04/2008, 11h41
seddik_saber

c'est du jpa ca ??????
qu'es qu'il fais ici??,

le problem c'est que ton entite est detache du contexte de persistence.

il faux eviter d'utiliser du business dans les backings beans
16/04/2008, 12h02
romaintaz

Citation:

Envoyé par seddik_saber

il faux eviter d'utiliser du business dans les backings beans

J'aurais plutôt dit qu'il faut éviter d'appeller le service depuis son JSF et donc de créer le backing bean adéquat, mais on est d'accord ;)
16/04/2008, 12h03
monstor
Et sinon à par demander ce que je fou ici (je pense venir chercher de l'aide...) et me traduire le message d'erreur que j'avais très bien compris quelle serait la solution à mon soucis.

Comme je l'ai dit, je pensais que le merge permettait de lier mon objet!

Merci de m'expliquer plutôt que de dénigrer, c'est souvent plus constructif ;)

Bon, j'ai résolu mon problème tout seul, j'ai fait
Code:

1 2 3 4 5 6 if(p.isChecked()){ Personne pers = em.merge(p); em.remove(pers); personnes.remove(pers); }
Merci
16/04/2008, 14h28
seddik_saber

:D

Citation:

Et sinon à par demander ce que je fou ici (je pense venir chercher de l'aide...)

je parlais pas de toi mais du entity manager ;)
17/04/2008, 10h13
heid

Scope page + lier le jsf à un EJB.... Tu serais pas sur SEAM toi par hazard?
17/04/2008, 11h03
monstor

Si si ;)