функция вызывается много раз без причины - PullRequest
0 голосов
/ 10 сентября 2011

У меня есть функция init (), которая возвращает список доменов, необходимых для заполнения таблицы данных, каждая строка этого таблицы данных имеет две командные ссылки: одну для редактирования, а другую для удаления, поэтому, когда я нажимаю ссылку на команду редактирования, появляется диалоговое окно. отображение информации Domaine с возможностью редактирования (есть еще один диалог для добавления новых доменов), поэтому проблема в том, что когда я нажимаю edit, функция init вызывается 8 раз, тогда устанавливаются атрибуты диалога добавления (даже если нет связь между ссылкой на команду редактирования и окном добавления) после повторного вызова функции init 4 раза. Я не понимаю, что. здесь код моей страницы:

    <?xml version='1.0' encoding='UTF-8' ?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml"
      xmlns:h="http://java.sun.com/jsf/html"
      xmlns:p="http://primefaces.prime.com.tr/ui"
      xmlns:ui="http://java.sun.com/jsf/facelets"
      xmlns:f="http://java.sun.com/jsf/core">
    <h:head>
        <title>Facelet Title</title>
    </h:head>
    <h:body>
         <ui:composition template="gpsiTemplate.xhtml">
            <ui:define name="left-menu">
                <ui:include src="admin-menuGauche.xhtml" />
            </ui:define> 
            <ui:define name="top">
                <ui:include src="menu-top.xhtml" />
            </ui:define>
            <ui:define name="content-top">
                <center>
                       <f:view>
                           <h:form id="topacteurs">
                               <h:panelGrid columns="14" styleClass="adminTopTable" >
                                   <h5 class="calendar">Direction :</h5> 
                                   <p:selectOneMenu id="typeacteurs" value="#{domaine.choixDirection}" style="width: 180px">  
                                        <f:selectItem itemLabel="Direction..." itemValue="" />  
                                        <f:selectItems value="#{direction.initcomb()}"  var="ta" itemValue="#{ta.codeDirection}" itemLabel="#{ta.libelleDirection}" />
                                        <p:ajax update=":mainform:domainlist" process="topacteurs" event="change" />
                                    </p:selectOneMenu>      

                                    <p:spacer width="20" height="0" />
                                    <p:commandButton title="Ajouter un domaine" image="ui-icon ui-icon-disk" value="Ajouter un domaine"  oncomplete="ajoutDomaine.show()"/>  
                                </h:panelGrid>  
                            </h:form>
                        </f:view>
                </center>
            </ui:define>
            <ui:define name="content-content">
                <h:form id="modifform">
                    <p:dialog header="Modifier un domaine" widgetVar="editDomaine" modal="true" width="400" height="200" showEffect="clip" hideEffect="clip">
                        <p:outputPanel id="editDomaineDetails" style="text-align:center;" layout="block">
                            <center>
                                <h:panelGrid  columns="2" cellpadding="5">
                                        <h:outputLabel  value="Intitulé        :"/>
                                        <p:inputText value="#{domaine.currentDomaine.libelleDomaine}" style="width: 180px"/>
                                        <h:outputLabel  value="Respensable        :"/>
                                        <p:inputText value="#{domaine.currentDomaine.nomDirecteur}" style="width: 180px"/>
                                        <h:outputLabel  value="Direction        :"/>
                                       <p:selectOneMenu id="editchoidirection" value="#{domaine.codeDirection}" style="width: 180px">  
                                            <f:selectItem itemLabel="Direction..." itemValue="" />  
                                            <f:selectItems value="#{direction.initcomb()}"  var="ta" itemValue="#{ta.codeDirection}" itemLabel="#{ta.libelleDirection}" />
                                        </p:selectOneMenu>
                                </h:panelGrid>
                                <h:panelGrid  columns="2" cellpadding="5">
                                    <p:commandButton value="Modifier" update="messages editDomaineDetails :mainform:domainlist" actionListener="#{domaine.update()}" oncomplete="editDomaine.hide()"/>
                                    <p:commandButton value="Annuler"  oncomplete="editDomaine.hide()"/>
                                </h:panelGrid>
                            </center>
                        </p:outputPanel>
                    </p:dialog>
                </h:form>
                <h:form id="mainform">
                    <p:growl id="messages" showDetail="true"/>
                    <p:confirmDialog message="Etes-vous sure?" width="200"  
                                        showEffect="clip" hideEffect="clip"  
                                        header="Confirmation" severity="alert" widgetVar="confirmation">  

                        <p:commandButton value="Oui sure" update="messages :mainform:domainlist" actionListener="#{domaine.delete()}" oncomplete="confirmation.hide()"/>  
                        <p:commandButton value="Non" onclick="confirmation.hide()" type="button" /> 
                    </p:confirmDialog>
                    <p:dialog header="Ajouter un domaine" widgetVar="ajoutDomaine" modal="true" width="400" height="200" showEffect="clip" hideEffect="clip">
                        <p:outputPanel id="ajoutDomaineDetails" style="text-align:center;" layout="block">
                            <center>
                                <h:panelGrid  columns="2" cellpadding="5">
                                        <h:outputLabel  value="Intitulé        :"/>
                                        <p:inputText value="#{domaine.nomDomaine}" style="width: 180px"/>
                                        <h:outputLabel  value="Respensable        :"/>
                                        <p:inputText value="#{domaine.nomDirecteur}" style="width: 180px"/>
                                        <h:outputLabel  value="Direction        :"/>
                                       <p:selectOneMenu id="ajoutchoidirection" value="#{domaine.codeDirection}" style="width: 180px">  
                                            <f:selectItem itemLabel="Direction..." itemValue="" />  
                                            <f:selectItems value="#{direction.initcomb()}"  var="ta" itemValue="#{ta.codeDirection}" itemLabel="#{ta.libelleDirection}" />
                                        </p:selectOneMenu>
                                </h:panelGrid>
                                <h:panelGrid  columns="2" cellpadding="5">
                                    <p:commandButton value="Ajouter" update="messages ajoutDomaineDetails :mainform:domainlist" actionListener="#{domaine.save()}" oncomplete="ajoutDomaine.hide()"/>
                                    <p:commandButton value="Annuler"  oncomplete="ajoutDomaine.hide()"/>
                                </h:panelGrid>
                            </center>
                        </p:outputPanel>
                    </p:dialog>

                    <p:dataTable id="domainlist" var="e" value="#{domaine.init()}">
                        <p:column headerText="Intitulé" filterBy="#{e.libelleDomaine}">
                            <h:outputText value="#{e.libelleDomaine}" />
                        </p:column>
                        <p:column headerText="Directeur" filterBy="#{e.nomDirecteur}">
                            <h:outputText value="#{e.nomDirecteur}" />
                        </p:column>
                        <p:column headerText="Direction" filterBy="#{e.directions.libelleDirection}">
                            <h:outputText value="#{e.directions.libelleDirection}" />
                        </p:column>
                        <p:column>
                            <h:panelGrid columns="3" style="border-color: #ffffff">
                                <p:commandLink update=":modifform:editDomaineDetails"  title="Editer" oncomplete="editDomaine.show()">  
                                    <p:graphicImage value="resources/images/edit.png"/>   
                                    <f:setPropertyActionListener value="#{e}" target="#{domaine.currentDomaine}" />  
                                </p:commandLink>
                                <p:commandLink title="Supprimer" oncomplete="confirmation.show()">  
                                    <p:graphicImage value="resources/images/delete.png"/>   
                                    <f:setPropertyActionListener value="#{e}" target="#{domaine.currentDomaine}" />  
                                </p:commandLink>
                            </h:panelGrid>
                        </p:column>
                    </p:dataTable>
                    <p:stack icon="resources/images/stack/stck.png">  
                        <p:menuitem value="Photo" icon="resources/images/stack/photo.png" url="http://localhost:8084/Gpsi/faces/profile-Photo.xhtml"/> 
                        <p:menuitem value="Profile" icon="resources/images/stack/profile.png" url="http://localhost:8084/Gpsi/faces/profile.xhtml"/>
                        <p:menuitem value="Administration" icon="resources/images/stack/administration.png" url="http://localhost:8084/Gpsi/faces/admin.xhtml"/>

                    </p:stack>
                </h:form>
            </ui:define>
        </ui:composition>
    </h:body>
</html>

здесь тот же код на Pastebin, но он немного испорчен: http://pastebin.com/W0XGa0d2

и вот мой резервный боб:

......
 public List<Domaines> initComb()
{
    .....
}
 public List<Domaines> init()
 {

    if(choixDirection==0)
    {
        domaines=domainesService.getAllDomaines();
    }
    else
    {
        Directions direction=directionsService.getDirections(choixDirection);
        domaines=domainesService.getDirDomaines(direction);
    }
    return domaines;
 }
 public void save()
 {
   ......
 }
 public void delete()
 {
     ......

 }
 public void update()
 {
     ......
 }
  ////////////////////////////////////////////////////////// setters & getters \\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\
......
public void setCodeDirection(Integer codeDirection)
{
    this.codeDirection=codeDirection;
}
  public Integer getCodeDirection()
{
    return codeDirection;
}
  public void setNomDomaine(String nomDomaine)
{
    this.nomDomaine=nomDomaine;
}
  public String getNomDomaine()
{
    return nomDomaine;
}
   public void setNomDirecteur(String nomDirecteur)
{
    this.nomDirecteur=nomDirecteur;
}
  public String getNomDirecteur()
{
    return nomDirecteur;
}
  ......

}

Я отправил длинные коды, потому что проблема не в том.

1 Ответ

8 голосов
/ 10 сентября 2011

Ваша ошибка в том, что вы выполняете бизнес-действие в методе getter вместо метода действия, который вызывается только один раз. Вы не должны делать деловые действия в добытчиках. JSF EL использует геттеры для доступа к свойствам бина. EL не кеширует их, когда они когда-либо вызывались. Получатели должны только возвращать свойства бина (или, в крайнем случае, выполнять ленивую загрузку или быстрые и не интенсивные вычисления, например, простую сумму, например return foo + bar;), но определенно не иметь доступа к базе данных или чему-то еще.

Переместите это задание в конструктор или @PostConstruct метод или любой метод события компонента. В этом конкретном случае вы, вероятно, захотите использовать @PostConstruct для предварительной загрузки списка с помощью @EJB и позволить слушателю действия ajax также указать на тот же метод. В бине @ViewScoped этот способ вызывается только один раз при первом запросе представления, а также вызывается только один раз при изменении выбора.

@PostConstruct
public void init() {
    if (choixDirection == 0) {
        domaines = domainesService.getAllDomaines();
    } else {
        Directions direction = directionsService.getDirections(choixDirection);
        domaines = domainesService.getDirDomaines(direction);
    }
}

public List<Domaines> getDomaines() {
    return domaines;
}

и исправить вид соответственно

<p:ajax process="topacteurs" listener="#{domaine.init}" update=":mainform:domainlist" />
...
<p:dataTable id="domainlist" var="e" value="#{domaine.domaines}">

Смотри также:

...