美丽的错误

　　“善意的谎言、美丽的错误”，这些事情在我们的生活及工作经常都在发生。最近花了三天多的时间学习了EasyJF开源官网的Blog程序源码，振奋人心之处就不说了，看过的都知道。同时也存在很多的错误，这些错误有的是由我不知何时亲自导演，这里就捡一些“美丽”的错误及Bub来说说，为了自己以后不再犯这样错误。

　　技术构架：EasyJWeb+Spring2+JPA　视图模板使用:Velocity

　1、很Cool的循环　来自:Blog的圈子显示　错误等级★★★★★

#foreach($!subList in $!CommUtil.toRowChildList($myOnweCircle,2))
   < tr >
 #foreach($!info in $subList)
  < td >
$!info.title
</ td >
 #end
< tr >
 #end

　　其中$myOnwerCircle是从后台传过来的一个List。这个程序是遍历一个列集合，把集合的元素循环显示到<tr>中，第行最多显示两列<td>。

　　但无论如何，都无法得到想要的结果，由于准备换成类似$BlogQuery.circleQuery.user($blog.owner).number(-1).list这样的标签来查询数据，所以一直认为是标签设计的问题。经过后来的一阵折腾证明是脚本写法的问题，你知道是哪些地方出的问题了吗？先说明：$!CommUtil.toRowChildList这个标签没任何问题。

2、easy的echo　来自:Blog的圈子显示　错误等级★★★★★☆
　　页面内容:

< span  class ="strong0" > $!info.title($!info.blogs.size()) </ span >< br >

　　这里打算显示圈子的主题以及圈子下面的blog数量。但如果不亲身体会，鬼都不知道他会给你显示什么出来。你知道吗？

　3、非菜鸟的equals　来自:文章及照片评论
　　

String user  =! "" .equals(obj.getInputUser())  ?  obj.getInputUser().getName() :  " 匿名用户 " ; 

　　数据结构永远就比算法重要，这个是不容质疑的，可以搞出很漂亮的算法，但没有一个好的模型，你就瞎忙吧。你觉得上面的代码哪儿错了？

　4、一不小心就OO及ORM　来自:评论系统设计 ★★☆

@Entity
@javax.persistence.Inheritance(strategy  =  javax.persistence.InheritanceType.JOINED)
@Cache(usage = CacheConcurrencyStrategy.NONSTRICT_READ_WRITE)
@FormPO(inject  =   " sequence,content,owner " )
public   abstract   class  Review  implements  java.io.Serializable  ... {

 @Id
 @GeneratedValue(strategy = GenerationType.TABLE)
 private Long id;// 主键
..}


@FormPO(inject = " blogOwner,inputUser " )
@Entity
@javax.persistence.MappedSuperclass
public   class  BlogReview  extends  Review  implements  IBlogComponment  ... {
 @POLoad(name="userId")
 @ManyToOne
 private User blogOwner;

 private String inputUser;
}


@FormPO(inject = " newsDoc " )
@Entity
public   class  BlogNewsReview  extends  BlogReview  ... {
 @POLoad(name = "docId")
 @ManyToOne
 @JoinColumn
 private BlogNewsDoc newsDoc;// 评论的对象

}

public   interface  IBlogReviewDAO   extends  GenericDAO < BlogReview >   ... {

}

  private  IBlogReviewDAO dao;
     public  BlogReview getBlogReview(Long id)  ... {
  return this.dao.get(id);
 }

　　评论系统，管理员能管理所有评论，能对Blog文章、像册等进行评论。
这个示例要是搞好了可以当成是一个比较经典的OO及ORM示例，但是现在这样的写法你知道JPA会创建几张表吗，各个表之间关联字段是是什么? Service中的getBlogReview返回什么？答对了有奖。

　4、有事无事的事务　来自blog信息发表 
  

<!-- 事务配置 -->
  < aop:config >
   < aop:pointcut id = " saveMethods "
   expression = " execution(* com.easyjf.blog.service.BlogNewsDocService.*(..)) "   />
    < aop:advisor advice - ref = " txBlogServiceAdvice "
   pointcut - ref = " saveMethods "   />  
  </ aop:config >
  < tx:advice id = " txBlogServiceAdvice "
  transaction - manager = " transactionManager " >
   < tx:attributes >
    < tx:method name = " wirteBlog* "  propagation = " REQUIRES_NEW "   />
    < tx:method name = " update* "  propagation = " REQUIRES_NEW "   />
    < tx:method name = " get* "  propagation = " SUPPORTS "
    read - only = " true "   />
    < tx:method name = " query* "  propagation = " SUPPORTS "
    read - only = " true "   />
    < tx:method name = " * "  propagation = " REQUIRED "   />
   </ tx:attributes >
  </ tx:advice >

代码：
this .service.addBlogReview(review);
  if  (review  instanceof  BlogNewsReview)  ... {
   BlogNewsReview r = (BlogNewsReview) review;
   //this.docService.updateBlogNewsDoc(r.getNewsDoc());
   this.htmlGenerator.process(r.getNewsDoc());
   r.getNewsDoc().getReviews().remove(r);// 很怪也滑稽的设计
   return new Page("html", r.getNewsDoc().getStaticUrl(), "html");
 }

 　　为什么Blog更新后出来的html页面还是上一次更新的内容？为什么在添加新闻的评论里面要加一句r.getNewsDoc().getReviews().remove(r)，是无奈还是真的滑稽。

　5、究竟是要IP、IQ还是IC卡　来自:blog首页 ☆
　

　后台Action代码
public  Page doInit(WebForm form, Module module)  ... {
  form.addResult("blogInfoService", this.blogInfoService);
  form.addResult("bqu", bqu);
  form.addResult("bdqu", bdqu);
  form.addResult("bcqu", bcqu);
form.addResult("bb” bb);
  if (UserContext.getUser() != null) ...{
   form.addResult("user", "true");
  }
  return new Page("index", "/blog/index.html");
 }
前台页面
#foreach($info in $bcu.thisYear.doc.list)
$info.title
#end

　　这段代码是想把一些实用工具比如bb,bq,bcqu,bdqu等暴露给页面，由页面人员根据自己的喜爱想要展示什么数据就展示什么数据。严格来说，这没什么错，只是害苦了美工MM而已，难怪她们不喜欢呆办公室。

　7、真空还是没空(is NULL or EMPTY)　来自Blog发布程序

业务层代码：
QueryObject query  =   new  QueryObject();
java.util.Calendar ca  =  java.util.Calendar.getInstance();
ca.setTimeInMillis(ca.getTimeInMillis()  -  blogIntervals  *   1000 );  
query.addQuery( " (obj.updateTime<? or obj.updateTime is EMPTY) " , new  Object[] ... {ca.getTime()} );
query.setPageSize( - 1 );  
List list  =   this .blogService.getBlogInfo(query).getResult();

知道执行这段代码生成的SQL吗？一定会让你大吃一惊，把EMPTY改成NULL难道是我们想要的？

　8、F想要干什么 来自:blog新闻管理　
 

Javascript脚本代码：
function  init() ... {
   F=new FORM("blogNewsDoc.ejf","ListForm");
   windowInit();
  }
  window.onload = init;  
  F.doList  =  F.doOneCommand( " listDoc " );

　
　　古老的new FORM曾经让我们非常欣喜，但为什么现在换来的却是大家抱怨呢？代码有时候也是有生命的，你如何都不用心好好对待他，他怎么会一直按部就班的运转呢？

　9.想擦掉js给你留下的伤疤吗 来自blog管理后台
　

　　Html代码
< form name = ”ListForm” id = ”ListForm” />
< input type = " button "  name = " submit "  value = " 彻底删除 "  onClick = " F.doRemove(); "  class = " pushbutton "  style = " cursor:pointer  " >
</ form >
     F.doRemove()里面会执行类似$(“ListForm”).submit()这样的代码来提交表单。

你觉得这段代码能让你真正的彻底删除掉js曾经留给你的诸多痛苦回忆吗？你继续痛哭吧，呵呵。其实我们只要仔细看一看就知道问题了。w3c可不是war3，不能随便乱来的。

10.你拥有什么？　来自Blog域对象 
 
　　

@FormPO(disInject = " id,owner,inputTime,readTimes,status,elite,updated,docNum,photoNum " )
@MappedSuperclass
public   class  AbstractBlog  implements  Serializable  ... { 
 @Id
 @GeneratedValue(strategy = GenerationType.TABLE)
 protected Long id;
 
 @OneToOne
 protected User onwer;
 
 @Column(length = 200)
 protected String title;// 标题

 @Column(length = 200)
 protected String subtitle;// 副标题

}
页面标签：
#foreach($info $BlogQuery.photoQuery.blogOwner($blog.owner).number( 6 ).orderBy( " inputTime " ).desc().list)

#end

　　呵呵，今天就这么多了，先听听大家的高见！下次发几个能让人眼前一亮的。