一个JAVA单例模式的典型错误应用的分析和解决方法

最新推荐文章于 2022-11-04 22:23:08 发布

原创最新推荐文章于 2022-11-04 22:23:08 发布 · 7k 阅读

6 ·

CC 4.0 BY-SA版权

文章标签：

#java #manager #tree #exception #null #string

Java 专栏收录该内容

254 篇文章

订阅专栏

本文探讨了在Java中使用单例模式时遇到的线程安全问题，特别是针对StringBuffer对象在多线程环境下被多个线程同时访问时引发的数据混乱问题。通过将StringBuffer从实例变量改为局部变量，并将其作为参数传递给递归方法，有效解决了线程安全问题。

问题来自论坛，

http://topic.youkuaiyun.com/u/20090927/23/497372f4-af98-4b7b-8489-3eb3a8de43b5.html

其代码如下：

import java.sql.Connection; import java.sql.PreparedStatement; import java.sql.ResultSet; import java.sql.SQLException; import com.cfd.drp.utils.Utils; public class ClientManager { // 此处使用了实例级的属性 StringBuffer sb = new StringBuffer(); // 此处实现了单例模式 private static ClientManager manager = new ClientManager(); private ClientManager() { } public static ClientManager getInstance() { return manager; } // 这里没有同步，多个线程将同时调用这个方法 public String showTree() { Connection connection = null; sb.delete(0, sb.length()); try { connection = Utils.getConnection(); tree(connection, 0, 1); } catch (Exception e) { e.printStackTrace(); } finally { Utils.close(connection); } return sb.toString(); } public void tree(Connection connection, int id, int level) throws SQLException { String sql = "select * from t_client where pid = ?"; int i = 0; PreparedStatement ps = null; ResultSet rs = null; try { ps = connection.prepareStatement(sql); ps.setInt(1, id); rs = ps.executeQuery(); while (rs.next()) { sb.append(rs.getString("name") + "<br>" + "/n"); if (rs.getString("is_leaf").equals("N")) { tree(connection, rs.getInt("id"), level); } } } finally { Utils.slose(rs); Utils.close(ps); } } }

具体的错误我已经在代码里标注出来了。

单例，就是系统只有一份实例，那么多个线程使用这一个实例，当然也就同时使用这一个StringBuffer了。

大家一起胡乱的往里面倾倒垃圾，怎么可能不混乱呢！

解决方法就是将StringBuffer做成局部变量

import java.sql.Connection; import java.sql.PreparedStatement; import java.sql.ResultSet; import java.sql.SQLException; import com.cfd.drp.utils.Utils; public class ClientManager { // 此处实现了单例模式 private static ClientManager manager = new ClientManager(); private ClientManager() { } public static ClientManager getInstance() { return manager; } // 这里没有同步，多个线程将同时调用这个方法 public String showTree() { // 此处使用局部变量 StringBuffer sb = new StringBuffer(); Connection connection = null; try { connection = Utils.getConnection(); tree(connection, 0, 1, sb); // 作为参数调用 } catch (Exception e) { e.printStackTrace(); } finally { Utils.close(connection); } return sb.toString(); } public void tree(Connection connection, int id, int level, StringBuffer sb) throws SQLException { String sql = "select * from t_client where pid = ?"; int i = 0; PreparedStatement ps = null; ResultSet rs = null; try { ps = connection.prepareStatement(sql); ps.setInt(1, id); rs = ps.executeQuery(); while (rs.next()) { sb.append(rs.getString("name") + "<br>" + "/n"); if (rs.getString("is_leaf").equals("N")) { tree(connection, rs.getInt("id"), level,sb); } } } finally { Utils.slose(rs); Utils.close(ps); } } }