Refatorando Webwork Actions

10 de fevereiro de 2004 6 respostas

dukejeffrie 10 de fevereiro de 2004

eu tenho um código assim no método doExecute() das minhas Actions:

if &#40;!this.isSimulationAvailable&#40;&#41;&#41; &#123;
	return LOGIN;
&#125;
this.openHibernateSession&#40;&#41;;
if &#40;this.getHasErrorMessages&#40;&#41;&#41; &#123;
	return ERROR;
&#125;
try &#123;
	model = loadEmployee&#40;&#41;;
	return SUCCESS;
&#125;
catch &#40;HibernateException e&#41; &#123;
	this.log.error&#40;"&#91;" + this.getActionName&#40;&#41; + "&#93;&#58;doDefault&#40;&#41; failed", e&#41;;
	this.addErrorMessage&#40;e.getMessage&#40;&#41;&#41;;
	return ERROR;
&#125;
finally &#123;
	this.closeHibernateSession&#40;&#41;;
&#125;

Eu quero refatorar isso pra colocar num lugar só a parte que mexe com o hibernate, mas eu não posso/devo/quero usar AOP.

Também está muito tarde pra passar tudo pro WW2.

Todas as minhas actions já extendem HibernateAction, que tem os métodos de abrir e fechar a session.

O problema de reescrever o método execute() é que eu não quero abrir a session antes de chamar o doValidation(), pq é meio sem nexo, e eu não tô a fim de copiar o execute() da ActionSupport pra dentro da minha action.

Alguém tem alguma idéia??

[]s!

6 Respostas

smota 10 de fev. de 2004

Alguem ae … o WW 1.x não tem interceptors? se tiver é só criar um :lol:

dukejeffrie 10 de fev. de 2004

pois eh, acho que não.

eu acho que “tuuudo não dá”. Se eu quiser minha session aberta antes do doExecute() mas depois do doValidation(), eu tenho mesmo que sabotar a ActionSupport.

mas vai que alguém já passou por isso, queria ver como o pessoal resolve esse tipo de encrenca…

Minhas actions tão feiamente duplicadas…

smota 10 de fev. de 2004

Porque ao invés de sabotar a ActionSupport você não faz uma classe extendendo a danada e escreve o doExecute() nela, ae vc muda o nome nas sua classes do método pra myDoExecute() :shock:

Entonces é só chamar o danado a partir do doExecute() padrão que trata tudo que tem que tratar com o Hibernate & cia (inclusive os erros) …

(vc podria tb chamar super.doExecute() no doexecute() de cada action, mas teria duplicado 1 linha em cada action 8) )

Esse principio é o mesmo dos interceptors, mais simples mas eh a mesma ideia.

hummm … funciona, mas se eh a melhor maneira nao sei … socorro, alguem que usa o 1.x … outra idéia?

ricardolecheta 10 de fev. de 2004

eu faço deste jeito que o smota disse com o Struts, deixo o controle de transação do Hibernate em uma action mãe…

nesta action eu inicio uma transação do Hibernate, depois chamo um método abstrato para as subclasses tratarem a requisição, depois disto fecho a sessao/commit/rollback…

cv1 10 de fev. de 2004

Eh, nao tem mto jeito pra se virar com isso sem interceptors alem do que o smota mencionou...

Talvez usar a tal da action-mae como decorator? ;)

public class Mae ... &#123;

  ...

  public final String doExecute&#40;&#41; ... &#123;
    Action a = new HibernateDecorator&#40;this&#41;;
    return a.doExecute&#40;&#41;;
  &#125;

&#125;

Nao pensei muuuuito bem nisso, mas acho que da pra passar a ideia. Isso da certo? :D

dukejeffrie 10 de fev. de 2004

cv, viajei nessa que vc falou.

HibernateDecorator é a minha classe?

Então, galera… eu tb pensei nisso, ter um hibExecute() que seria executado só em modo “online”…

… mas daí eu tenho que implementar tb o hibDefault(), o hibPreview(), e todos os outros commands da classe.

Eu posso mexer na ActionFactory, colocar minha própria factory, botar um decorator nas actions que eu crio: se ela implementar HibernateAction (eu transformo a classe em interface, e a atual vira AbstractHibernateAction ou HibernateActionSupport), eu intercepto os métodos doXXX():

public HibernateDecorator implements InvocationHandler &#123;
	public Object invoke&#40;Object proxy, Method m, Object&#91;&#93; args&#41; &#123;
		boolean intercept =  &#40;m.getName&#40;&#41;.beginsWith&#40;"do"&#41;
		&amp;&amp; proxy instanceof HiebrnateAction&#41;;
		if &#40;intercept&#41; &#123;
			&#40;&#40;HibernateAction&#41;proxy&#41;.openHibernateSession&#40;&#41;;
			try &#123;
				// chamo o método &#40;requer uma certa promiscuidade&#41;
			&#125;
			catch &#40;HibernateException e&#41; &#123;...&#125;
			finally &#123;
				&#40;&#40;HibernateAction&#41;proxy&#41;.closeHibernateSession&#40;&#41;;
			&#125;
		&#125;
		else &#123;
			// chamo o método
		&#125;
	&#125;
&#125;

mas olha o trampo que dá… editar webwork.properties!! : )
implementar mais várias classes…

se eu terminar antes do prazo (tá, tá, podem rir!) eu faço algo mais nessa linha…

ou então o cv explica a do decorator…

[]s!!

Criado 10 de fevereiro de 2004

Ultima resposta 10 de fev. de 2004

Respostas 6

Participantes 4

6 Respostas

Topicos relacionados