ID生成器代码重构问题

背景： 为每个i请求生成一个ID
即ID生成器:本机名的最后一个字段+当前时间戳精确到毫秒+8位随机字符串

通用方面
1.IdGenerator设计成了实现类，违反基于接口而非实现编程的设计思想,如果我们需要设计两个ID生成器算法就需要将IdGenerator改为接口并为不同的算法实现不同的实现类。
2.静态方法generate()测试难度更大。
3.没有编写单元测试代码
4.代码完全没有注释，不易理解，尤其是随机字符串部分
业务方面
1.没有涉及共享变量，所以代码线程安全。
2.接口只有generate()一个，不存在不易用问题。
3.多线程环境下调用generate()不存在并发问题。
4.每次生成 ID 都需要获取本机名，获取主机名会比较耗时，所以，这部分可以考虑优化一下。
5.randomAscii 的范围是 0～122，但可用数字仅包含三段子区间（0_9，az，A~Z），可优化。

代码结构
基于Maven下的程序

主函数

package org.example;

public class Main {
    public static void main(String[] args) {
        System.out.println(IdGenerator.generate());
    }

}
1
2
3
4
5
6
7
8

实现类IdGenerator

package org.example;
import java.net.InetAddress;
import java.net.UnknownHostException;
import java.util.Random;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

public class IdGenerator {
    private static final Logger logger = LoggerFactory.getLogger(IdGenerator.class);

    public static String generate() {
        String id = "";
        try {
            String hostName = InetAddress.getLocalHost().getHostName();
            String[] tokens = hostName.split("\\.");
            if (tokens.length > 0) {
                hostName = tokens[tokens.length - 1];
            }
            char[] randomChars = new char[8];
            int count = 0;
            Random random = new Random();
            while (count < 8) {
                int randomAscii = random.nextInt(122);
                if (randomAscii >= 48 && randomAscii <= 57) {
                    randomChars[count] = (char)('0' + (randomAscii - 48));
                    count++;
                } else if (randomAscii >= 65 && randomAscii <= 90) {
                    randomChars[count] = (char)('A' + (randomAscii - 65));
                    count++;
                } else if (randomAscii >= 97 && randomAscii <= 122) {
                    randomChars[count] = (char)('a' + (randomAscii - 97));
                    count++;
                }
            }
            id = String.format("%s-%d-%s", hostName,
                    System.currentTimeMillis(), new String(randomChars));
        } catch (UnknownHostException e) {
            logger.warn("Failed to get the host name.",e);
        }

        return id;
    }
}
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43

pom.xml依赖配置

<!--slf4j 依赖包 -->
        <dependency>
            <groupId>org.apache.logging.log4j</groupId>
            <artifactId>log4j-core</artifactId>
            <version>2.13.1</version>
        </dependency>
        <dependency>
            <groupId>log4j</groupId>
            <artifactId>log4j</artifactId>
            <version>1.2.17</version>
        </dependency>
        <dependency>
            <groupId>org.slf4j</groupId>
            <artifactId>slf4j-api</artifactId>
            <version>1.7.32</version>
        </dependency>
        <dependency>
            <groupId>org.slf4j</groupId>
            <artifactId>slf4j-log4j12</artifactId>
            <version>1.7.25</version>
        </dependency>
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21

原链接：34 | 实战一（上）：通过一段ID生成器代码，学习如何发现代码质量问题