struts 的radio标记的增强

写struts的html标记的人不知道是不是对html标记不熟，写出来的标记怎么怪里怪气的，id标记被用来绑定bean了，我还以为不支持html的id属性了，不小心看到org.apache.struts.taglib.html.BaseHandlerTag的prepareStyles()方法里面，居然赫然写着：
        if (styleId != null) {
            styles.append(" id=/"");
            styles.append(getStyleId());
            styles.append("/"");
        }
原来要用styleId属性来定义html的id属性，不看源码你敢这么猜吗，怎么看styleId都象是个样式有关的属性嘛，要定义html的id属性，如果不能用id至少也应该叫HTMLID或者tagId之类的名字嘛。ok，源码都看了，那我们用styleId也成，可是看看radio标记就实在是笨了：
    public int doAfterBody() throws JspException {
        if (this.bodyContent != null) {
            String value = this.bodyContent.getString().trim();
            if (value.length() > 0) {
                this.text = value;
            }
        }
        return (SKIP_BODY);
    }
你说我把文字括到<html:radio>text</html:radio>标记中间为的是什么？当然这个文字要和radio绑定啦。可是上面的代码居然直接把文字往生成的<input type="radio"...>标记后面一丢了事，有这么不负责的嘛？
还好咱java和html都还懂一点。自己写个类：
package hospital.tongren.oa.common.web.tag;
import javax.servlet.jsp.JspException;
public class RadioTag extends org.apache.struts.taglib.html.RadioTag{
  public int doAfterBody() throws JspException {
    if (this.bodyContent != null) {
        String value = this.bodyContent.getString().trim();
        if (value.length() > 0) {
            this.text = "<label for=/""+getStyleId()+"/" "+
                prepareEventHandlers()+
                prepareStyles().replaceAll(" id=/""+getStyleId()+"/"","")+
                ">"+value+"</label>";
        }
    }
    super.setStyleId(null);
    return (SKIP_BODY);
  }
  static int idCount=0;
  public int doStartTag() throws JspException {
    if (super.getStyleId() == null){
      super.setStyleId("RadioTag_"+ ++idCount);
     }
    return super.doStartTag();
  }
}
然后再在WEB-INF下面找到struts-html.tld,把
<tagclass>org.apache.struts.taglib.html.RadioTag</tagclass>
改为
<tagclass>myPackage.RadioTag</tagclass>

这下整个世界都清凉多了。回家过五一了，下次再对checkbox开刀吧。

author: emu(黄希彤)