Почему JSF вызывает геттеры несколько раз - PullRequest
249 голосов
/ 19 января 2010

Допустим, я указываю компонент outputText следующим образом:

<h:outputText value="#{ManagedBean.someProperty}"/>

Если я распечатываю сообщение журнала, когда вызывается метод получения для someProperty, и загружаю страницу, тривиально заметить, что метод получения вызывается более одного раза за запрос (два или три раза - это то, что происходило в моем случае ):

DEBUG 2010-01-18 23:31:40,104 (ManagedBean.java:13) - Getting some property
DEBUG 2010-01-18 23:31:40,104 (ManagedBean.java:13) - Getting some property

Если значение someProperty слишком дорого для вычисления, это может стать проблемой.

Я немного погуглил и понял, что это известная проблема. Одним из обходных путей было включить проверку и посмотреть, был ли он уже рассчитан:

private String someProperty;

public String getSomeProperty() {
    if (this.someProperty == null) {
        this.someProperty = this.calculatePropertyValue();
    }
    return this.someProperty;
}

Основная проблема заключается в том, что вы получаете множество шаблонного кода, не говоря уже о частных переменных, которые вам могут не понадобиться.

Каковы альтернативы этому подходу? Есть ли способ достичь этого без такого большого количества ненужного кода? Есть ли способ остановить поведение JSF таким образом?

Спасибо за ваш вклад!

Ответы [ 9 ]

330 голосов
/ 19 января 2010

Это вызвано природой отложенных выражений #{} (обратите внимание, что «устаревшие» стандартные выражения ${} ведут себя точно так же, когда Facelets используется вместо JSP). Отложенное выражение не оценивается сразу , а создается как объект ValueExpression, а метод получения, стоящий за выражением, выполняется каждый раз, когда код вызывает ValueExpression#getValue().

Обычно он вызывается один или два раза в цикле запрос-ответ JSF, в зависимости от того, является ли компонент компонентом ввода или вывода ( изучите его здесь ). Однако это число может возрасти (намного) выше, когда используется при итерации компонентов JSF (таких как <h:dataTable> и <ui:repeat>), или кое-где в логическом выражении, таком как атрибут rendered. JSF (в частности, EL) вообще не будет кэшировать вычисленный результат выражения EL, так как может возвращать различные значения при каждом вызове (например, когда он зависит от текущей итерированной строки данных).

Оценка выражения EL и вызов метода get - это очень дешевая операция, поэтому вам вообще не следует об этом беспокоиться. Однако история меняется, когда вы по какой-то причине выполняете дорогостоящую DB / бизнес-логику в методе getter. Это будет повторяться каждый раз!

Методы получения в базовых компонентах JSF должны быть спроектированы таким образом, чтобы они возвращали уже подготовленное свойство и ничего более, в точности согласно спецификации Javabeans . Они вообще не должны делать дорогую БД / бизнес-логику. Для этого следует использовать методы слушателя bean @PostConstruct и / или (action). Они выполняются только один раз в какой-то момент жизненного цикла JSF на основе запроса, и это именно то, что вам нужно.

Вот сводка всех различных правильных способов предустановки / загрузки свойства.

public class Bean {

    private SomeObject someProperty;

    @PostConstruct
    public void init() {
        // In @PostConstruct (will be invoked immediately after construction and dependency/property injection).
        someProperty = loadSomeProperty();
    }

    public void onload() {
        // Or in GET action method (e.g. <f:viewAction action>).
        someProperty = loadSomeProperty();
    }           

    public void preRender(ComponentSystemEvent event) {
        // Or in some SystemEvent method (e.g. <f:event type="preRenderView">).
        someProperty = loadSomeProperty();
    }           

    public void change(ValueChangeEvent event) {
        // Or in some FacesEvent method (e.g. <h:inputXxx valueChangeListener>).
        someProperty = loadSomeProperty();
    }

    public void ajaxListener(AjaxBehaviorEvent event) {
        // Or in some BehaviorEvent method (e.g. <f:ajax listener>).
        someProperty = loadSomeProperty();
    }

    public void actionListener(ActionEvent event) {
        // Or in some ActionEvent method (e.g. <h:commandXxx actionListener>).
        someProperty = loadSomeProperty();
    }

    public String submit() {
        // Or in POST action method (e.g. <h:commandXxx action>).
        someProperty = loadSomeProperty();
        return "outcome";
    }

    public SomeObject getSomeProperty() {
        // Just keep getter untouched. It isn't intented to do business logic!
        return someProperty;
    }

}

Обратите внимание, что вы не должны использовать конструктор bean-компонента или блок инициализации для задания, потому что он может вызываться несколько раз, если вы используете инфраструктуру управления bean-компонентом, которая использует прокси, такие как CDI.

Если для вас действительно нет других способов из-за некоторых ограничительных требований к дизайну, то вам следует ввести ленивую загрузку в методе получения. То есть если свойство null, загрузите его и назначьте свойству, иначе верните его.

    public SomeObject getSomeProperty() {
        // If there are really no other ways, introduce lazy loading.
        if (someProperty == null) {
            someProperty = loadSomeProperty();
        }

        return someProperty;
    }

Таким образом, дорогая БД / бизнес-логика не будет выполняться без необходимости при каждом вызове геттера.

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

16 голосов
/ 31 октября 2010

С JSF 2.0 вы можете подключить слушателя к системному событию

<h:outputText value="#{ManagedBean.someProperty}">
   <f:event type="preRenderView" listener="#{ManagedBean.loadSomeProperty}" />
</h:outputText>

В качестве альтернативы вы можете заключить страницу JSF в тег f:view

<f:view>
   <f:event type="preRenderView" listener="#{ManagedBean.loadSomeProperty}" />

      .. jsf page here...

<f:view>
9 голосов
/ 05 января 2011

Я написал статью о том, как кэшировать средство получения JSF-компонентов с помощью Spring AOP.

Я создаю простой MethodInterceptor, который перехватывает все методы, отмеченные специальной аннотацией:

public class CacheAdvice implements MethodInterceptor {

private static Logger logger = LoggerFactory.getLogger(CacheAdvice.class);

@Autowired
private CacheService cacheService;

@Override
public Object invoke(MethodInvocation methodInvocation) throws Throwable {

    String key = methodInvocation.getThis() + methodInvocation.getMethod().getName();

    String thread = Thread.currentThread().getName();

    Object cachedValue = cacheService.getData(thread , key);

    if (cachedValue == null){
        cachedValue = methodInvocation.proceed();
        cacheService.cacheData(thread , key , cachedValue);
        logger.debug("Cache miss " + thread + " " + key);
    }
    else{
        logger.debug("Cached hit " + thread + " " + key);
    }
    return cachedValue;
}


public CacheService getCacheService() {
    return cacheService;
}
public void setCacheService(CacheService cacheService) {
    this.cacheService = cacheService;
}

}

Этот перехватчик используется в файле конфигурации пружины:

    <bean id="advisor" class="org.springframework.aop.support.DefaultPointcutAdvisor">
    <property name="pointcut">
        <bean class="org.springframework.aop.support.annotation.AnnotationMatchingPointcut">
            <constructor-arg index="0"  name="classAnnotationType" type="java.lang.Class">
                <null/>
            </constructor-arg>
            <constructor-arg index="1" value="com._4dconcept.docAdvance.jsfCache.annotation.Cacheable" name="methodAnnotationType" type="java.lang.Class"/>
        </bean>
    </property>
    <property name="advice">
        <bean class="com._4dconcept.docAdvance.jsfCache.CacheAdvice"/>
    </property>
</bean>

Надеюсь, это поможет!

6 голосов
/ 14 апреля 2013

Первоначально опубликовано на форуме PrimeFaces @ http://forum.primefaces.org/viewtopic.php?f=3&t=29546

Недавно я был одержим оценкой производительности моего приложения, настройкой JPA-запросов, заменой динамических SQL-запросов именованными запросами, и только сегодня утром я понял, что метод геттера был скорее ГОРЯЧИМ СПОТОМ в Java Visual VM, чем остальная часть моего кода (или большая часть моего кода).

Метод получения:

PageNavigationController.getGmapsAutoComplete()

Ссылка на пользовательский интерфейс: включить в index.xhtml

Ниже вы увидите, что PageNavigationController.getGmapsAutoComplete () является ГОРЯЧИМ СПОТОМ (проблема производительности) в Java Visual VM. Если вы посмотрите дальше вниз, на снимке экрана вы увидите, что getLazyModel (), метод получения ленивых данных с помощью PrimeFaces, также является горячей точкой, только когда конечный пользователь выполняет много «ленивых данных» типа вещей / операций / задач в приложении. :)

Java Visual VM: showing HOT SPOT

См. (Оригинальный) код ниже.

public Boolean getGmapsAutoComplete() {
    switch (page) {
        case "/orders/pf_Add.xhtml":
        case "/orders/pf_Edit.xhtml":
        case "/orders/pf_EditDriverVehicles.xhtml":
            gmapsAutoComplete = true;
            break;
        default:
            gmapsAutoComplete = false;
            break;
    }
    return gmapsAutoComplete;
}

Ссылка на следующее в index.xhtml:

<h:head>
    <ui:include src="#{pageNavigationController.gmapsAutoComplete ? '/head_gmapsAutoComplete.xhtml' : (pageNavigationController.gmaps ? '/head_gmaps.xhtml' : '/head_default.xhtml')}"/>
</h:head>

Решение: так как это метод 'getter', переместите код и присвойте значение gmapsAutoComplete до вызова метода; см. код ниже.

/*
 * 2013-04-06 moved switch {...} to updateGmapsAutoComplete()
 *            because performance = 115ms (hot spot) while
 *            navigating through web app
 */
public Boolean getGmapsAutoComplete() {
    return gmapsAutoComplete;
}

/*
 * ALWAYS call this method after "page = ..."
 */
private void updateGmapsAutoComplete() {
    switch (page) {
        case "/orders/pf_Add.xhtml":
        case "/orders/pf_Edit.xhtml":
        case "/orders/pf_EditDriverVehicles.xhtml":
            gmapsAutoComplete = true;
            break;
        default:
            gmapsAutoComplete = false;
            break;
    }
}

Результаты теста: PageNavigationController.getGmapsAutoComplete () больше не является HOT SPOT в Java Visual VM (даже больше не отображается)

Обмен этой темой, так как многие опытные пользователи посоветовали начинающим разработчикам JSF НЕ добавлять код в методы 'getter'. :)

4 голосов
/ 11 июня 2012

Если вы используете CDI, вы можете использовать методы Producers. Он будет вызываться много раз, но результат первого вызова кэшируется в области действия компонента и эффективен для получателей, которые вычисляют или инициализируют тяжелые объекты! См. здесь , для получения дополнительной информации.

3 голосов
/ 19 января 2010

Возможно, вы могли бы использовать AOP для создания какого-либо аспекта, который кэшировал результаты наших методов получения в течение настраиваемого промежутка времени. Это избавит вас от необходимости копировать и вставлять шаблонный код в десятки методов доступа.

0 голосов
/ 26 июня 2019

Я бы также посоветовал использовать такие фреймворки как Primefaces вместо стандартных JSF, они решают такие проблемы перед командой JSF e. g в простейших лицах вы можете установить частичную подачу. В остальном BalusC это хорошо объяснил.

0 голосов
/ 06 января 2012

Это все еще большая проблема в JSF. Например, если у вас есть метод isPermittedToBlaBla для проверки безопасности, и, по вашему мнению, у вас есть rendered="#{bean.isPermittedToBlaBla}, то метод будет вызываться несколько раз.

Проверка безопасности может быть сложной, например. LDAP-запрос и т. Д. Так что вы должны избегать этого с

Boolean isAllowed = null ... if(isAllowed==null){...} return isAllowed?

и вы должны обеспечить в сессионном компоненте это для каждого запроса.

Я думаю, JSF должен реализовать здесь некоторые расширения, чтобы избежать нескольких вызовов (например, аннотация @Phase(RENDER_RESPONSE), вызывать этот метод только один раз после RENDER_RESPONSE phase ...)

0 голосов
/ 19 января 2010

Если значение someProperty слишком дорого для вычисления, это может стать проблемой.

Это то, что мы называем преждевременной оптимизацией.В тех редких случаях, когда профилировщик говорит вам, что вычисление свойства настолько дорого обходится, что вызов его трижды, а не один раз оказывает значительное влияние на производительность, вы добавляете кеширование, как вы описываете.Но если вы не делаете что-то действительно глупое, например, деление простых чисел или получение доступа к базе данных в геттере, ваш код, скорее всего, будет иметь десяток худших неэффективностей в местах, о которых вы никогда не задумывались.

...