使用单例模式避免重复创建LineSeparator是否合理？求优化建议

在项目中使用OpenPDF库生成PDF时，需要在不同位置添加换行分隔符，于是编写了一个独立方法实现：

private static void addLineSeparator(Document document) {
    LineSeparator separator = new LineSeparator();
    separator.setLineColor(Color.LIGHT_GRAY);
    separator.setLineWidth(2f);
    document.add(new Chunk(separator));
}

但每次调用该方法都会创建新的LineSeparator()实例，担心造成内存浪费。主调用代码如下：

Document document = new Document();
try {
    PdfWriter.getInstance(document, new FileOutputStream("HelloWorld.pdf"));
    document.open();
    document.add(new Paragraph("Hello World"));
    Paragraph newLine = new Paragraph(Chunk.NEWLINE);
    for (int i = 0; i < 5; ++i) {
        addLineSeparator(document);
        document.add(newLine);
    }
} catch (DocumentException de) {
    System.err.println(de.getMessage());
} catch (IOException ioe) {
    System.err.println(ioe.getMessage());
}
document.close();

为解决这个问题，编写了一个单例类：

public class LineSeparatorSingleton {
    private static LineSeparatorSingleton separatorSingleton;
    private static LineSeparator separator;
    private static Chunk chunk;

    private LineSeparatorSingleton() {
        separator = getLineSeparator();
        chunk = getChunk();
    }

    public static LineSeparatorSingleton getInstance() {
        if (separatorSingleton == null) {
            separatorSingleton = new LineSeparatorSingleton();
        }
        return separatorSingleton;
    }

    public LineSeparator getLineSeparator() {
        if (separator == null) {
            separator = new LineSeparator();
        }
        return separator;
    }

    public Chunk getChunk() {
        if (chunk == null) {
            chunk = new Chunk(getLineSeparator());
        }
        return chunk;
    }

    public void addLineSeparator(Document document) {
        getLineSeparator().setLineColor(Color.LIGHT_GRAY);
        getLineSeparator().setLineWidth(2f);
        document.add(getChunk());
    }
}

并在主代码中这样使用：

for (int i = 0; i < 5; ++i) {
    // addLineSeparator(document);
    LineSeparatorSingleton.getInstance().addLineSeparator(document);
    document.add(newLine);
}

请问这种避免重复创建对象的方式是否正确？希望得到相关建议或帮助。

你的单例实现思路方向是对的，但存在几个可以优化的点，同时也要考虑实际场景的必要性：

1. 单例的线程安全问题 #

当前的getInstance()方法是懒加载但未做线程同步，在多线程环境下可能会创建多个实例，违背单例的初衷。可以通过以下方式修复：

• 给getInstance()方法加上synchronized关键字，保证线程安全
• 改用饿汉式单例（类加载时就初始化实例），避免懒加载的线程问题
• 使用枚举单例，这是Java中最简洁且天然线程安全的单例实现方式

2. 冗余的属性设置 #

在addLineSeparator()方法中，每次调用都重复设置setLineColor和setLineWidth，这些属性是固定的，完全可以放在实例初始化时设置一次，不用每次调用都执行，减少冗余操作。

3. 更轻量的替代方案 #

其实没必要为这个场景单独写一个单例类，还有更简单的实现方式：静态常量初始化，直接在工具类中创建一个静态的LineSeparator实例，初始化时设置好固定属性，每次调用直接复用即可，代码更简洁：

public class PdfUtils {
    private static final LineSeparator LINE_SEPARATOR;
    private static final Chunk SEPARATOR_CHUNK;

    static {
        LINE_SEPARATOR = new LineSeparator();
        LINE_SEPARATOR.setLineColor(Color.LIGHT_GRAY);
        LINE_SEPARATOR.setLineWidth(2f);
        SEPARATOR_CHUNK = new Chunk(LINE_SEPARATOR);
    }

    public static void addLineSeparator(Document document) {
        document.add(SEPARATOR_CHUNK);
    }
}

调用时直接用PdfUtils.addLineSeparator(document)即可，比单例类更轻便，同样能达到复用对象的目的。

4. 关于内存浪费的实际考量 #

LineSeparator属于轻量对象，每次创建的内存开销极小，在循环次数不多（比如几十几百次）的场景下，完全没必要特意做复用优化。只有当循环次数极大（上万次以上）或者频繁调用这个方法时，复用才有实际意义。

内容的提问来源于stack exchange，提问作者Akshaya Amar