【问题标题】:How do I implement Observer design pattern for a Multi-Threaded Java server?如何为多线程 Java 服务器实现观察者设计模式?
【发布时间】:2016-04-21 15:17:47
【问题描述】:

我有以下情况:

Java 服务器必须通过套接字响应多个客户端。服务器响应实际上是数据库查询的结果。客户端也可以要求服务器更新数据库内容。

为每个新连接的客户端分配一个新线程。我希望每个连接的客户端都“知道”其他客户端何时更新数据库内容。

我试图实现观察者设计模式来实现这一点,但我没有成功:当更新查询应用于数据库时不会通知观察者 -> 永远不会调用观察者中的更新方法。 实际的Java代码比较复杂,所以我将尝试通过以下代码sn-p来总结一下观察者设计模式的实现:

public class DBDataPersistance<E> extends Observable implements DataPersistance<E>{
private static Connection conn;
protected Statement stmt;
protected ResultSet res;
protected List<Observer> obs = new LinkedList<Observer>();

//The method that alters the database;
public void updateData(String parameters) {
    try {
        this.setChanged();
        this.notifyObservers();
        this.stmt.executeUpdate(parameters);
    } catch (SQLException e) {
        JOptionPane.showMessageDialog(null, e.getMessage());
    }
}

public void addObserver(Observer o){
    System.out.println("New observer added!");
    this.obs.add(o);
    }
}

//MultiThreadServer implements Runnable and Observer and contains an instance of DBDataPersistance
public class PizzaMultiThreadServer implements Runnable, Observer{
    private static PizzaDBDataPersistance sDB;
    private static ObjectOutputStream sendClientResponse;
    private static ObjectInputStream isc;


    MultiThreadServer(Socket s) {
        this.cSocket = s;
        try {
            sendClientResponse = new ObjectOutputStream(cSocket.getOutputStream());
            sendClientResponse.flush();
            isc = new ObjectInputStream(cSocket.getInputStream());
        } catch (IOException e) {
            e.printStackTrace();
        }
    }


    public static void main(String[] args) {
    sDB = new DBDataPersistance("jdbc:sqlserver://LAPTOPPC;database=BD;integratedSecurity=true;");
        while (true) {
             //client-server communication
             MultiThreadServer newClient = new MultiThreadServer(csock);
             sDB.addObserver(newClient);
             Thread t = new Thread(newClient);
             t.start();
        }
    }


 @Override
 public void run() {
 //Client server communication on a separate thread
 }


 @Override
 public void update(Observable o, Object arg) {
         System.out.println("Thread " + Thread.currentThread().getId()+ ":Noticed on update!");
 }
}

你能告诉我我错在哪里,并指出在这种情况下正确实现观察者设计模式的步骤吗?

非常感谢。

【问题讨论】:

  • 多线程代码很难正确处理,即使对于优秀的程序员也是如此。我的建议是让这项工作在没有线程的情况下同步进行。测量性能,查看是否需要异步处理,仅在需要时添加线程。你可能会让问题变得比不需要的更难。更好地分层问题:将数据库代码放在一个单独的类中,您可以单独测试并放在一边。如果您确实需要线程,最好使用 Executor 类而不是原始线程。为了使并发更容易,进行了很多更改。使用它们。
  • 除了 duffymo 所说的之外,使用观察者模式进行数据库更新听起来像是一个可疑的解决方案。大多数数据库使用缓存,甚至允许缓存暂时过时(过时)以获得更高的性能。如果用户看到几秒钟前的数据,许多类型的更新都无关紧要。设计是必须的。
  • 感谢您的意见。 :)

标签: java database multithreading observer-pattern


【解决方案1】:

你的问题是因为你override你的类DBDataPersistance中的方法addObserver这样你没有Observers注册到类Observable所以你没有Observer可以通知。

代码应该是:

public void addObserver(Observer o){
    // Calls the method addObserver of the class Observable
    super.addObserver(o);
    System.out.println("New observer added!");
    this.obs.add(o);
}

请注意,您不需要自己管理Observer 的列表,因为它已经由Observable 类管理,因此您应该摆脱obs,代码将是:

 public void addObserver(Observer o){
    // Calls the method addObserver of the class Observable
    super.addObserver(o);
    System.out.println("New observer added!");
}

关于您的代码的最后一句话是,您应该在通知Observers 之前调用executeUpdate,所以它应该是:

public void updateData(String parameters) {
    try {
        this.stmt.executeUpdate(parameters);
        this.setChanged();
        this.notifyObservers();        
    } catch (SQLException e) {
        JOptionPane.showMessageDialog(null, e.getMessage());
    }
}

【讨论】:

  • 我进行了建议的修改并且它有效。再次感谢。 :)
  • 观察者列表不感兴趣,因为它已经由Observable管理,可以删除。
  • @Jean-BaptisteYunès 是正确的,我更新了我的答案以表明,谢谢
【解决方案2】:

不要自己实现,有很多库可以帮助你。

我建议https://github.com/ReactiveX/RxJava

(不确定这应该是一个答案还是一个抱歉的评论)

【讨论】:

  • 很高兴知道未来的情况。感谢您的建议。
猜你喜欢
  • 1970-01-01
  • 2012-07-21
  • 2013-03-05
  • 1970-01-01
  • 2012-02-03
  • 1970-01-01
  • 1970-01-01
  • 2010-12-24
  • 1970-01-01
相关资源
最近更新 更多