h:commandLink behaviour - decode, encode

[msznapka]

Hi,
I have very simple Seam application:
numberGuess.jspx:

<h:form>
 <h:inputText value="#{numberGuess.input}"/>
 <h:commandLink action="#{numberGuess.doSomething}" value="submit"/>
</h:form>

NumberGuess.java

@Name("numberGuess")
@Scope(ScopeType.EVENT)
public class NumberGuess implements Serializable {

 private String input;

 public String getInput() {
 System.out.println("getInput()");
 return input;
 }

 public void setInput(String input) {
 System.out.println("setInput()");
 this.input = input;
 }

 public String doSomething() {
 System.out.println("doSomething()");
 return null;
 }

 @Create
 public void begin() {
 System.out.println("created");
 randomNumber = new Random().nextInt(100);guessCount = 0;biggest = 100;smallest = 1;
 }

 @Destroy
 public void end() {
 System.out.println("destroyed");
 System.out.println("----------");
 }

...

PROBLEM:
results are:

 created
 getInput() <--
 setInput()
 doSomething()
 getInput()
 destroyed
 ----------

which is strange for me, because I expected result without first getInput, which is called in encode phase of the component. the next getInput is called in decode phase

it is right behaviour? because i have application where are lot of inputs generated with ui:repeat and data for inputs are loaded from database TWICE (two getXXX())

are there any solution to have behaviour like this?:

 created
 setInput()
 doSomething()
 getInput()
 destroyed
 ----------

Thanks!

[msznapka]

I forget my configuration:
jboss-seam-2.0.0.BETA1
jboss-4.2.0.GA
facelets-1.1.13

[pmuir]

This is just the way the JSF lifecycle works. Make sure not to do a lot of work in a getter.

[msznapka]

Thank you for answer Pete

So you think that loading data from database should be done inside action method (here doSomething)?

What about first page load, where no action was fired?

first page load - getData
action load - getData, action, getData

For information - this is event scoped seam component and data are loaded from database. Such loaded data depends on your action, if no action, than load default data.

So here is code:

@Name("event")
@Scope(EVENT)
public class Event {
 List<Item> data = null;
 getData ... return data;
 setData ... this.data = data;

 doSomething() {
 if(linkParameter == null)
 loadDefaultDataFromDatabase();
 else
 loadDataFromDatabase(linkParameter);
 }
...

so if you access page for first time, null data are returned, that is why i thought that loading data must be inside getters methods

[pmuir]

This is what @Factory is for.

[msznapka]

It works, thanks a lot!