做投票系统的经验

      写这篇经验之前，先总结下做这个系统出现的一些问题：

      忽现Unicode编码、票数清零、服务器当机、内存溢出、多线程处理文件造成文件错误更新、没有控制好用户狂刷、最囧的是，IO流没有close掉......

      在公司刚换了个项目组，接手了一个投票系统的任务，参于被投票的对象是一批经过多次评选的经纪人（一共60人），谁得票最高即可获得丰厚奖品的。

      投票的规则以每个IP每天最多可以投50票，投票者抓住这个规则之后，一直疯狂的刷点...

      我们开始把投票结果记录到.properties里面，也许因为刷点的原因，出现了莫名其妙的\u0000空格Unicode错误编码，造成对配置文件解析错误。严重的情况是，有某些参赛者的票数突然清零，狂飙冷汗，有不少的参赛者打电话来投诉（糟了，要被开除了...）看了下配置文件，还是一些无缘无故出现Unicode编码。

      原来做了几个投票页，1个IP只能投1票，所以很少同步操作的情况。但现在不同了，于是放弃.properties，改成xml，.properties最大的问题是更新后会引起顺序混乱，不好管理，而xml解决了这个问题，对于编码问题也容易处理。最重要数据被清零的问题也解决了。

<vote>
  <members>
    <member id="1" count="12839"></member>
    <member id="2" count="4550"></member>
    <member id="3" count="245"></member>
    <member id="4" count="20"></member>
    <member id="5" count="39"></member>

    ....
  </members>
  <dates>
    <date id="20090611">
        <ip addr="192.168.16.38" count="6"></ip>

        ....
    </date>

    ....
  </dates>
</vote>

      P.S.：这次新学了JDOM，用它来解析XML真的很方便，有兴趣可以学习一下。  http://www.ibm.com/developerworks/cn/java/j-jdom/index.html

      为了收集参赛者被投票的信息，在投票中加入了一个记录投票数据的xml，结构如下：

<vote>
  <members>
    <member id="1" name="张三" count="3">
      <ip addr="192.168.16.38" startTime="09:37:35" endTime="09:37:39" count="3"></ip>

      ....
    </member>

    ....
  </members>
  <ips>
    <ip addr="192.168.16.38" startTime="09:37:35" endTime="09:37:39" count="3">
      <info voteId="1" voteTime="09:37:35"></info>

      <info voteId="1" voteTime="09:37:37"></info>
      <info voteId="1" voteTime="09:37:39"></info>

      ....

    </ip>

    ....
  </ips>
</vote>

    这个投票详细记录的方法我们选择在晚上更新，但是更新之前服务器突然挂掉了，不知道什么原因，这个新的生成xml的方法更新以后，服务器挂掉的频率更高了，后来查看内存，一直狂涨，这下子就囧了。。。肯定是流忘记关闭了。然后回忆xml更新的一句代码，这是JDOM封装的保存更新xml的方法。代码如下：

public void updateXml(Document doc, String xmlUrl) {
   Format format = Format.getPrettyFormat();
   format.setEncoding("UTF-8");
   format.setExpandEmptyElements(true);
   XMLOutputter printDoc = new XMLOutputter();
   printDoc.setFormat(format);
   try {
     printDoc.output(doc, new FileOutputStream(xmlUrl));
   } catch (FileNotFoundException e) {
    e.printStackTrace();
   } catch (IOException e) {
    e.printStackTrace();
   }
 }

里面有个 new FileOutputStream(xmlUrl)， 问题就出在这里了，压力测试时没注意好内存的变化，于是重新初始化一个FileOutputStream对象，结束时关掉。

修改后，代码如下：

 public void updateXml(Document doc, String xmlUrl) {
   FileOutputStream out = null;
   Format format = Format.getPrettyFormat();
   format.setEncoding("UTF-8");
   format.setExpandEmptyElements(true);
   XMLOutputter printDoc = new XMLOutputter();
   printDoc.setFormat(format);
   try {
     out = new FileOutputStream(xmlUrl);
    printDoc.output(doc, out);
   } catch (FileNotFoundException e) {
    e.printStackTrace();
   } catch (IOException e) {
    e.printStackTrace();
   } finally {
    try {
     out.flush();
     out.close();
    } catch (IOException e) {
     e.printStackTrace();
    }
 }

       哎，真失败吖。。搞了两年程序出了这样一个失误。。悲哀吖。。问题还是没有完全解决，投票的人还是一直狂点狂刷，于是在页面加了这样段很简单的脚本，限制了投票人的疯狂操作。js代码如下：

var num = 0;
var oId = "";
function toVote(id){
    oId = id;
     if(num == 5) {
        alert("很抱歉，您投票的频率太快，请稍后再试。");
        return;
   }
   if(num!=1){
       num = 1;
       createxmlHttpuest("update.jsp?id="+id+"&time"+new Date());
       num = 5;
       fangshua();
   } else {
       alert("正在投票中,请稍后");
   }
}
 
function fangshua() {
    window.setTimeout(function(){num = 0;} ,5000);
}

    红色部分就是新加进去的防刷脚本，虽然简单，但是很实用。

    但是为了避免刷票而控制程序问题，还是治标不治本，于是在调用更新票数的方法上加上同步锁，避免多线程引起的错误。代码如下：

Object obj = new Object();
....
synchronized (obj) {
    String cTime = xml.CurrentlyTimes("yyyyMMdd");
    try {
        content = xml.getVoteUpdate(id, cTime, ip);             
    } catch (Exception e) {
        content = null;
    }
    if (content.equals("ok")) {
        xml.statVote(ip, id);
    }

}

 

   对更新的方法经过压力测试，50个同步线程执行，统计的xml与详细记录的xml文件更新也不会出现多线程写入的错误。但是还有一个细节，处理一个文件，建议单独使用一个File对象，避免出现引用错误。

 

File file = new File(this.statVoteXmlURL);    
Document newDoc = null;
SAXBuilder sb = new SAXBuilder();
sb.setIgnoringElementContentWhitespace(true);     
if (file.exists()) {
    sb.setIgnoringElementContentWhitespace(true);
    try {
        newDoc = sb.build(file);
    } catch(Exception e) {
        e.printStackTrace();
        return ;
    }
....

 

   总结一下之前做投票系统需要注意的问题：

   1、一定要做防刷分脚本.

   2、频繁被调用的方法要做同步，做多线程压力测试.

   3、处理文本的对象要单独初始化，避免引用冲突.

   4、使用xml记录数据比使用文本记录要好.

   5、流对象不能单独在参数中实例化，不然会很囧...它不会自己关掉的，必须手动关掉.

 

评论

3 楼 printfabcd 2011-12-24  

挺好的，不过如果对压力比较大的情况下，用xml存储的话，肯定是扛不住的，不过防刷这个挺不错的，学习了function fangshua() {
    window.setTimeout(function(){num = 0;} ,5000);
}

2 楼 joewalker 2009-09-23  

hemin108 写道

就是被投票者的信息（票数）都存在在xml文件么？还想问下如果我们采用将数据存储在数据库要注意什么问题？

    恩，都是保存在xml里面，至于结构可以按照投票项目的形式去设置。除非投票的信息需要长期保存，数据对日后有帮助，才考虑保存到数据库，短期就无所谓了，哪个都一样。
    其实不管是保存在数据库也好还是xml也好，一定要控制好防止用户刷分的情况出现，对update的方法进行同步，可以的话做下多线程的测试，避免出现死锁、或者写文件错误（用xml的时候）。其实投票系统需要注意的地方，我上面都已经全部说明了，只要控制好以上几点，就OK了。

1 楼 hemin108 2009-09-19  

嗯学习了 最近正要写一个投票系统，还没下手，看到楼主的帖子，学到了些知识，但还有些问题想请教，就是被投票者的信息（票数）都存在在xml文件么？还想问下如果我们采用将数据存储在数据库要注意什么问题？