Доступ частного доступ переменные данные из другого класса

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 из anoter класса, для этого у меня есть два варианта

1. изменить спецификатор доступа allSymbolSet от частного до стандартного.

2. Написать метод GET, который supoused вернуть Set

Пожалуйста, предложите мне, что будет хороший подход в этом случае ??

Answer 1

Несколько комментариев:

• Если вы сделаете установить, не частный, какой-то код может изменить его (по ошибке или с целью), которые могут привести к противоречивому поведению в классе StreamThread. Не делай этого.
• Предоставление простого геттера не решает проблему выше. Предпочитаете возвращать копию вашего набора.
• Сделайте свои переменные окончательными, когда сможете, в многопотоковой среде - он решает многие проблемы безопасности потоков.
• Предпочитает реализацию Runnable, чем простирающийся тему
• Вам нужно будет синхронизировать все доступы к вашему набору (чтение и запись), например, с помощью synchronizedSet или даже лучше, обернув ConcurrentHashMap, который обычно обеспечивает более высокую производительность.
• вместо peek + poll вы можете просто take из очереди

Так что ваш конечный класс может выглядеть следующим образом:

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 2

Лучший подход был бы метод получения и синхронизации доступа к объекту allSymbolSet, что-то вроде этого:

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

, а также синхронизировать доступ к allSymbolSet внутри вашего потока.

Answer 3

Лучший способ - это метод 2. Написание метода геттера. Если вы хотите разрешить устанавливать значения, используйте позднее сеттер. Затем ваши данные будут инкапсулированы.

Answer 4

Напишите метод получения, который должен вернуть набор., используя это личное личное личное, и вы также получаете доступ к нему извне, используя объект того же класса.