从不同的类访问私有访问变量数据

Question

我的项目中有这个线程，它不断运行并接受新符号

public class StreamThread extends Thread {
    private Set<String> allSymbolSet = new HashSet<String>(Arrays.asList("USBC", "TCSD", "PCLJ"));
    private PriorityBlockingQueue<String> priorityBlocking = new PriorityBlockingQueue<String>();

    public void addSymbols(String str) {
        if (str != null) {
            priorityBlocking.add(str);
        }
    }

    public void run() {
        while (true) {
            try {
                while (priorityBlocking.peek() != null) {
                    String symbol = priorityBlocking.poll();
                    allSymbolSet.add(symbol);
                }
            } catch (Exception e) {
                e.printStackTrace();
            }
        }
    }
}

我的问题是，我想从另一个类访问变量 allSymbolSet

从另一个类访问这个名为allSymbolSet 的变量的最佳方法是什么，为此我有两个选择

1. 将 allSymbolSet 的访问说明符从私有修改为默认。

2. 编写一个用于返回 Set 的 get 方法

请建议我，在这种情况下有什么好的方法？？

Answer 1

最好的方法是 getter 方法并同步对对象 allSymbolSet 的访问，如下所示：

public Set<String> getAllSymbolSet() {
    synchronized(allSymbolSet) {
        return allSymbolSet;
    }
}

并在你的线程中同步对allSymbolSet的访问。

Answer 2

几个cmets：

• 如果您将设置设为非私有，则某些代码可能会修改它（错误地或故意地），这可能会导致 StreamThread 类中的行为不一致。不要那样做。
• 提供简单的 getter 并不能解决上述问题。最好退回您的套装副本。
• 在多线程环境中尽可能将变量设为最终变量 - 它解决了许多线程安全问题。
• 宁愿实现 Runnable 而不是扩展 Thread
• 您将需要同步对您的集合的所有访问（读取和写入），例如使用synchronizedSet，甚至更好地包装ConcurrentHashMap，这通常会提供更好的性能。
• 您可以在队列中简单地使用take 而不是peek+poll

所以你的最后一课可能是这样的：

public class StreamTask implements Runnable {

    private final Set<String> allSymbolSet;
    private final PriorityBlockingQueue<String> priorityBlocking = new PriorityBlockingQueue<String>();

    public StreamTask() {
         String[] symbols = {"USBC", "TCSD", "PCLJ"};
         //use a thread safe set, for example based on ConcurrentHashMap
         allSymbolSet = Collections.newSetFromMap(new ConcurrentHashMap<String, Boolean> ());
         Collections.addAll(allSymbolSet, symbols);
    }

    public void addSymbols(String str) {
        if (str != null) {
            priorityBlocking.add(str);
        }
    }

    public Set<String> getSymbols() {
        return new HashSet<> (allSymbolSet); //return a copy
    }

    public void run() {
        while (true) {
            try {
                allSymbolSet.add(priorityBlocking.take());
            } catch (Exception e) {
                e.printStackTrace();
            }
        }
    }
}

最后，我可能遗漏了一些东西，但该类看起来等同于更简单的：

public class StreamTask {

    private final Set<String> allSymbolSet;

    public StreamTask() {
         String[] symbols = {"USBC", "TCSD", "PCLJ"};
         //use a thread safe set, for example based on ConcurrentHashMap
         allSymbolSet = Collections.newSetFromMap(new ConcurrentHashMap<String, Boolean> ());
         Collections.addAll(allSymbolSet, symbols);
    }

    public void addSymbols(String str) {
        if (str != null) {
            allSymbolSet.add(str);
        }
    }

    public Set<String> getSymbols() {
        return new HashSet<> (allSymbolSet); //return a copy
    }
}

Answer 3

更好的方法是方法 2。编写一个 getter 方法。如果您想允许设置值，请稍后使用设置器。然后你的数据将被封装。

Answer 4

编写一个应该返回 Set 的 get 方法。通过使用它，您的 private 仍然是私有的，并且您还可以使用同一类的 Object 从外部访问它。