美丽的错误

　　“善意的谎言、美丽的错误”，这些事情在我们的生活及工作经常都在发生。最近花了三天多的时间学习了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

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