做投票系统的经验
写这篇经验之前，先总结下做这个系统出现的一些问题：
忽现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、流对象不能单独在参数中实例化，不然会很囧...它不会自己关掉的，必须手动关掉.