【JSF专家Dennis Byrne】JSF反模式与陷井(二)

本节主要包括JSF的XML配置，线程安全等问题，是JSF开发人员不可多得的指导性文章。Dennis Byrne绝对是一位实战型JSF专家，不仅仅能对JSF规范提出不满，而且还对存在问题的JSF作一一解答。 

正文:

盲目使用XML

JSF的XML配置文件就算没有几千行，一般也得个几百行了。我曾经见过一个这样的项目：他们的“联系我们”这个页面充斥在几乎所有的单独的navigation-rule中：

<navigation-rule>
	<from-view-id>/home.xhtml</from-view-id>
	<navigation-case>
		<from-outcome>contact_us</from-outcome>
		<to-view-id>/contact.xhtml</to-view-id>
	</navigation-case>
</navigation-rule>
<navigation-rule>
	<from-view-id>/site_map.xhtml</from-view-id>
	<navigation-case>
		<from-outcome>contact_us</from-outcome>
		<to-view-id>/contact.xhtml</to-view-id>
	</navigation-case>
</navigation-rule>
<navigation-rule>
	<from-view-id>/about_us.xhtml</from-view-id>
	<navigation-case>
		<from-outcome>contact_us</from-outcome>
		<to-view-id>/contact.xhtml</to-view-id>
	</navigation-case>
</navigation-rule>
<!—下面还有很多……. -->

 

 

而现在仅仅只需要一个全局navigation-rule就可以搞定了：

 

<navigation-rule>
	<from-view-id>*</from-view-id>
	<navigation-case>
		<from-outcome>contact_us</from-outcome>
		<to-view-id>/contact.xhtml</to-view-id>
	</navigation-case>
</navigation-rule>

 

 

XML地狱还远不止这些，还有很多运行期异常，我们来看几个例子：

 

<managed-bean>
   	<managed-bean-name>invalid</managed-bean-name>
      <!-- misspelled class name throws ClassNotFoundException -->
      <managed-bean-class>net.dbyrne.misspelled.Invalid</managed-bean-class>
    	<managed-bean-scope>session</managed-bean-scope>
    	<managed-property>
      	<!-- one side of cyclical reference -->
    		<property-name>incorrect</property-name>
    		<value>#{incorrect}</value>
      </managed-property>
    </managed-bean>

    <managed-bean>
    	<managed-bean-name>incorrect</managed-bean-name>
     	<managed-bean-class>net.dbyrne.validate.Incorrect</managed-bean-class>
  	<managed-bean-scope>session</managed-bean-scope>
      <managed-property>
      	<!-- other side of cyclical reference -->
    		<property-name>invalid</property-name>
    		<value>#{invalid}</value>
      </managed-property>
    </managed-bean>

 

 

心细一点，很快能看出这是一个依赖注入死循环！！！而根据JSF实现，如果发生这样情况应用程序启动时不会得到任何警告，但运行期肯定会抛异常。（注：如果使用Spring来故意这样配置的话，启动时就会报错，由此可见JSF的依赖注入还不是很成熟）。

 

 

再来看一个配置重名的情况：

 

<managed-bean>
<!-- first duplicate -->
    	<managed-bean-name>duplicate</managed-bean-name>
      <managed-bean-class>
      	net.dbyrne.validate.FirstDuplicate
      </managed-bean-class>
    	<managed-bean-scope>session</managed-bean-scope>
    </managed-bean>

    <managed-bean> <!-- second duplicate -->
    	<managed-bean-name>duplicate</managed-bean-name>
      <managed-bean-class>
      	net.dbyrne.validate.SecondDuplicate
      </managed-bean-class>
    	<managed-bean-scope>application</managed-bean-scope>
    </managed-bean>

 

 

 

这个问题JSF1.2已经修正了。但使用JSF的仍然有不少是1.1版本。这让开发人员相当头痛了——调试难度相当大。大多数1.1的JSF实现版本都会忽略第一个重名的bean，而只是简单的调用第二个。（注：不光JSF，Struts2同样也存在这样的话，特别是数百行以上的XML配置文件，如果真有人不小心这么写了，出现的问题常常摸不到头脑，我已经遇到过两次，受有体会。）

 

再来看看最后一个例子：

 

 

<managed-bean>
    	<managed-bean-name>requestScopeManagedBean</managed-bean-name>
    	<managed-bean-class>
    		net.dbyrne.RequestScopeManagedBean
    	</managed-bean-class>
    	<managed-bean-scope>request</managed-bean-scope>
    </managed-bean>

    <managed-bean>
    	<managed-bean-name>sessionScopeManagedBean</managed-bean-name>
    	<managed-bean-class>
    		net.dbyrne.SessionScopeManagedBean
    	</managed-bean-class>
    	<managed-bean-scope>session</managed-bean-scope>
	    <managed-property>
		     <!-- throws runtime exception -->
		    <property-name>requestScopeManagedBean</property-name>
		    <value>#{requestScopeManagedBean}</value>
	    </managed-property>
    </managed-bean>

 

 

乍一看，没什么嘛。但再请看看两个bean的作用域：一个是request，一个是session。JSF规定：如果一个受托管的bean的作用域必须小于其依赖注入的属性的作用域。也就是说上面的的这个例子sessionScopeManagedBean的作用域是session，而其一个注入的属性requestScopeManagedBean却是request，与JSF规范不符，运行期肯定出错。

 

大多数像这些配置上的缺陷是可以通过JUnit，JBossJSFUnit检测到的。当然，你还可以使用Seam或Shale的Annotation类库，从而避免XML问题。

 

 

 

 

线程安全

 

不是所有的代码都要考试线程安全问题。通过调用FacesContext.getCurrentInstance()返回的数据是线程内安全的，而且request或unscoped作用域所托管的bean也是安全的。但有些代码你就需要注意了，它们可能被多个线程同时访问。

 

现在我们拿PhaseListeners和Renderers举例。每一个PhaseListener生存周期都是application作用域并且为每一个request至少保持一个状态。组件实例(component)与Renderer实例之间是多对一的关系——相同类型的组件可以共享同一个Renderer实例。处在Session和application作用域的bean显然会不止一个线程访问。

 标签一般是不需要考虑线程安全的，但你所运行的容器却可以将先前的标签实例重用。(注：这一点听去似乎有点可怕啊……)。因此将标签所涉及到的字段复位总是需要的。 

你可以随便使不使用Converter标签来保证线程安全。一个Converter标签的使用永远不会导致线程问题，组件会每次调用时帮你返回一个实例，请看下面代码：

 

 

<converter>
		<converter-id>threadUnsafeConverter</converter-id>
        	<converter-class>net.dbyrne.ThreadUnsafeConverter</converter-class>
    	</converter>

	<h:inputText value="#{managedBean.value}" >
		<!-- 这里使用的是converter标签 -->
		<f:converter converterId="threadUnsafeConverter" >
	</h:inputText>

 

 

如果使用的是标签里的converter属性，那可能就没这么幸运了。这个时候，很有可能这个实例会同时被多个request访问到。

<managed-bean>
    	<managed-bean-name>threadUnsafeConverter</managed-bean-name>
    	<managed-bean-class>net.dbyrne.ThreadUnsafeConverter</managed-bean-class>
    	<managed-bean-scope>session</managed-bean-scope>
    </managed-bean>
<!-- 这里使用的是converter属性 -->
    <h:inputText value="#{managedBean.value}" converter="#{threadUnsafeConverter}" />

 

 

验证标签存在同样的问题。

 

 

 

迁徙到Facelets的挑战：自定义标签是否应该与行为绑定？？？

Java开发者们使用自己的JSP标签有些年头了。JSF继承发扬JSP标签是因为JSF的默认视图就是JSP！！！JSF触发是由JSP标签调用set方法生效的。当调用标签时，标签上的属性值将传递到其映射的UI组件上去。

public class WidgetTag extends UIComponentELTag{

	private String styleClass = "default_class";
	private String title;

	public String getComponentType() {
		return "net.dbyrne.widget";
	}

	public String getRendererType() {
		return "net.dbyrne.widget";
	}

	public void setStyleClass(String styleClass) {
		this.styleClass = styleClass;
	}

	public void setTitle(String title) {
		this.title = title;
	}

	public void release() {
		super.release();
		styleClass = null;
		title = null;
	}

	protected void setProperties(UIComponent component) {

		super.setProperties(component);

		Widget span = (Widget) component;

		span.setStyleClass(styleClass);
		span.setTitle(title == null ? "no title" : title);

		FacesContext ctx = FacesContext.getCurrentInstance();
		Map sessionMap = ctx.getExternalContext().getSessionMap();
		span.setStyle((String) sessionMap.get("style"));

	}

}

 

 

 

稍微有经验的JSF程序会注意到这段代码有点问题。styleClass字段默认设置成“default_style”，title字段依赖于三元运算符，还有一个组件值要通过父类来计算出来 (注意这里说的应该是release()方法) 。很遗憾，这个标签的实现与具体行为绑的死死的。但我现在要说的是，我不鼓励这种做法。当你使用类似于Facelets的可选视图技术后，组件绑定行为可以放弃不用了。我们在开发JSF组件的时候，一个重要的原则就是一标签只处理一个逻辑! Apache MyFaces的代码就是一个很好的例子。源文件有25个组件和25个标签，在构建期时，通过元数据就将它们生成好了，不包含任何行为。

 

 

得墨忒耳法则

在一次JavaOne大会上，Mathias需要借些钱。“Craig” Mathias说到：“我欠了Martin一瓶啤酒，能借我5美元吗？” 。于是Craig被迫让Mathias的从自己的钱包里瞎翻出5美元给了Martin。这时Craig说到：“Mathias，如果我把钱放在鞋子或钱包里的话，你也许可换种方式以从我这里借钱了。”　Mathias接着说：“很好，我借钱的逻辑与你将钱放在哪里的逻辑是有关系的的。下次当我再向你借钱时，你只要关心钱在哪就行了。”Craig知道Mathias是个精明的家伙，他的这个建议立即使得这个借钱与给钱立即得到解藕了。这就是经典的得墨忒耳法则。

 

人人都知道没有当事人的允许，是不可以去翻别人的钱包。但很多开发人员却在开发时常常无法清醒的认识到这一点：

 

// highly sensitive to changes of the domain model
employee.getDepartment().getManager().getOffice().getAddress().getZip()

 <!-- highly sensitive to changes of the domain model -->
#{employee.department.manager.office.address.zip}

 

 

 

缺少经验的程序员需要明白：上面的代码违背了面向对象的原则，随时会在其中某个地方不加判断而掉链子。但有一些人并没有认识到这么长的EL表达式也是相当糟糕的。由于使用了EL表达式，这段代码不会出现编译错误。那么MethodNotFoundErrors和NullPointerExceptions异常对类似于这样的视图来说就是家常便饭了。重构是很痛苦的，因为EL表达式本身是不是强类型的。当EL表达式要间接访问视图model多级属性时，还不如先封装一下，再调用：

<!-- encapsulated, insensitive to changes -->
#{employee.departmentalManagerOfficeZip}

 

 

17
顶

6
踩

分享到：

【JSF专家Dennis Byrne】JSF反模式与陷井 ... | 【JSF专家Dennis Byrne】JSF反模式与陷井 ...

• 2008-03-04 22:30
• 浏览 6218
• 评论(4)
• 分类:企业架构
• 查看更多

评论

4 楼 kyo100900 2008-03-05  

本文不是一篇教你一步一步怎么JSF入门的文章，而是你在运用JSF过程中，作者Dennis Byrne将可能会遇到的问题做了一些说明，以及解决办法。在今年The Server Side大会上，Dennis Byrne会有他的关于本文个案研究，到时候，我看看能不能结合实例，再给大家看看。

3 楼 xuejianshan 2008-03-05  

真可惜我的英语不好,要不然的话我也要去翻译外国人的文章~~~ 

2 楼 xuejianshan 2008-03-05  

看不懂呀,大哥能不能把你的调通的原码也献出来~~ 

1 楼 liubinduo 2008-03-05  

GOOD