|
锁定老帖子 主题：一个循环流水号实现，求评
精华帖 (0) :: 良好帖 (1) :: 新手帖 (0) :: 隐藏帖 (4)
|
|
|---|---|
| 作者 | 正文 |
|
发表时间：2011-09-19
最后修改：2011-09-19
1.要可以设初始值
2.补0这种事交给Format 去作 '0' The result will be zero-padded 又好懂又没bug 3.现在spring这么流行，你用static这种方式有点土了。 |
| 返回顶楼 | |
|
发表时间：2011-09-19
zuzong 写道 Crusader 写道 AnotherTomorrow 写道 hareamao 写道 方法的头一段可以改成：
int n = ++serialNumber;
serialNumber %= 999999;
另外，你的程序不考虑系统重启的可能性么？  小小纠正一下，按楼主要求，应该是
int n = serialNumber = serialNumber%999999 + 1;
应该是可以不同步整个方法的吧，n属于局部变量，只要同步serialNumber部分就可以了。 有问题吧，这样一直加下去，不考虑int溢出吗? 整个操作过程 check - save - get 应该封装为原子操作，同时可以考虑使用jdk的原子变量类来简化过程 因为到了999999就设为0，重新计数，系统重启没考虑了，小弟又修改了下，恳请各位大哥指导啊~
public class SerialNumber {
private static SerialNumber instance = new SerialNumber();
public static SerialNumber getInstance(){
return SerialNumber.instance;
}
private SystemParamDAO systemParamDAO;
/**
* 流水号
*/
private static volatile int serialNumber = 0;
/**
* 读取数据库中初始，或中断时记录的流水号
*/
public void initSerialNumber() {
SerialNumber.serialNumber = Integer.parseInt(this.systemParamDAO.search("serial_number"));
}
/**
* 生成流水号
* 从1 - 999999，不足六位，从右往左补0
* @return
*/
public synchronized String generateSerialNumber(){
int n = serialNumber = ++serialNumber;
if(n == (999999 + 1)){
serialNumber = n = 1;
}
SerialNumber.instance.systemParamDAO.updateBySerialNumber(n);
StringBuffer strbu = new StringBuffer(6);
strbu.append(n);
for(int i=0, length=6-strbu.length(); i<length; i++){
strbu.insert(0, 0);
}
return strbu.toString();
}
public void setSystemParamDAO(SystemParamDAO systemParamDAO) {
this.systemParamDAO = systemParamDAO;
}
}
这个貌似不是单利吧，没声明private的构造方法 |
| 返回顶楼 | |
|
发表时间：2011-09-19
晕，刚才写了半天回复，一提交，居然不见了
|
| 返回顶楼 | |
|
发表时间：2011-09-19
最后修改：2011-09-19
感谢大家的回复，
我用的spring，systemParamDAO是注进来的，initSerialNumber方法也是在spring配置文件里设置的init-method 调用方是个helper类，原本那里是没有数据库操作的，但流水号又需要保存。。。想来想去，干脆把systemParamDAO当参数传进去吧，总归是要注入的，注入systemParamDAO，总比注入我这个流水号的好些 synchronized 是为了保证++serialNumber的计算 改成了这样
public class SerialNumber {
private SystemParamDAO systemParamDAO;
/**
* 流水号
*/
private static volatile int serialNumber = 0;
/**
* 读取数据库中初始，或中断时记录的流水号
*/
public void initSerialNumber() {
SerialNumber.serialNumber = Integer.parseInt(this.systemParamDAO.search("serial_number"));
}
/**
* 生成流水号
* 从1 - 999999，不足六位，从右往左补0
* @return
*/
public synchronized static String generateSerialNumber(SystemParamDAO systemParamDAO){
int n = serialNumber = ++serialNumber;
if(n == (999999 + 1)){
serialNumber = n = 1;
}
systemParamDAO.updateBySerialNumber(n);
StringBuilder strbu = new StringBuilder(6);
strbu.append(n);
for(int i=0, length=6-strbu.length(); i<length; i++){
strbu.insert(0, 0);
}
return strbu.toString();
}
public void setSystemParamDAO(SystemParamDAO systemParamDAO) {
this.systemParamDAO = systemParamDAO;
}
}
|
| 返回顶楼 | |
|
发表时间：2011-09-19
多线程下，试试AtomicLong呢...
|
| 返回顶楼 | |
|
发表时间：2011-09-19
估计用的是MYSQL......
|
| 返回顶楼 | |
|
发表时间：2011-09-19
调用参数里面还得传一个dao，这有点怪异吧
getNextSerialNumber不一定非得是static的 可以每次取下一个序列的时候都检查一下有没有初始化过，如果没有初始化，那么先从数据库读取数据进行初始化 如果已经初始化了，那么直接返回(内存中保存的数据+1)%LIMIT private static volatile boolean INITED = false;
private static volatile int SN = 0;
private void init(){
SN = xxxDao.xx(); // 从数据库加载最后一次持久化的值
INITED = true;
}
public synchronized String getNextSerialNumber(){
if(!INITED) init();
SN = (SN + 1) % LIMIT;
// TODO: 更新数据库
return String.format("%06d", SN);
}
|
| 返回顶楼 | |
|
发表时间：2011-09-19
最后修改：2011-09-19
weiqiang.yang 写道 调用参数里面还得传一个dao，这有点怪异吧
getNextSerialNumber不一定非得是static的 可以每次取下一个序列的时候都检查一下有没有初始化过，如果没有初始化，那么先从数据库读取数据进行初始化 如果已经初始化了，那么直接返回(内存中保存的数据+1)%LIMIT private static volatile boolean INITED = false;
private static volatile int SN = 0;
private void init(){
SN = xxxDao.xx(); // 从数据库加载最后一次持久化的值
INITED = true;
}
public synchronized String getNextSerialNumber(){
if(!INITED) init();
SN = (SN + 1) % LIMIT;
// TODO: 更新数据库
return String.format("%06d", SN);
}如果再使用 java.util.concurrent.atomic.AtomicLong long old ＝ SN.longvalue long new = (old+1)%LIMIT; SN.weakCompareAndSet(old,new); 来作SN 就更加理想了， 去掉同步块大大增加多线程访问次数 最主要的是只看到业务逻辑 没看到丑陋的java底层代码 PS：为了好测试，可以把 if(!INITED) init(); 放到spring的 initmethod里面去。 |
| 返回顶楼 | |
|
发表时间：2011-09-19
最好从100000 开始,
|
| 返回顶楼 | |
|
发表时间：2011-09-19
这系统重启不就恢复了嘛
|
| 返回顶楼 | |
