ReSQL.ru
     
powered by simpleCommunicator - 2.0.61     © 2026 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / Java [игнор отключен] [закрыт для гостей] / Объясните пример их книги Concurency in practice
9 сообщений из 9, страница 1 из 1
Объясните пример их книги Concurency in practice
    #37541923
Usik
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Usik
Участник
Разбираю пример из книги Concurency in practice: программа диспетчер транспортных средств. Т/с характеризуется идентификатором и координатами взятого с GPS. Один поток показывает как-то информацию, другой ее обновляет.Код ниже, взят из книги
Код: plaintext
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
37.
38.
39.
40.
41.
42.
43.
44.
45.
46.
@ThreadSafe
 public   class  MonitorVehicleTracker {
    @GuardedBy("this")
     private   final  Map<String, MutablePoint> locations;

     public  MonitorVehicleTracker(
            Map<String, MutablePoint> locations) {
         this .locations = deepCopy(locations);
    }

     public   synchronized  Map<String, MutablePoint> getLocations() {
         return  deepCopy(locations);
    }

     public   synchronized   MutablePoint getLocation(String id) {
        MutablePoint loc = locations.get(id);
         return  loc ==  null  ?  null  :  new  MutablePoint(loc);
    }

     public   synchronized    void  setLocation(String id,  int  x,  int  y) {
        MutablePoint loc = locations.get(id);
         if  (loc ==  null )
             throw   new  IllegalArgumentException("No such ID: " + id);
        loc.x = x;
        loc.y = y;
    }

     private   static  Map<String, MutablePoint> deepCopy(
            Map<String, MutablePoint> m) {
        Map<String, MutablePoint> result =
                 new  HashMap<String, MutablePoint>();
         for  (String id : m.keySet())
            result.put(id,  new  MutablePoint(m.get(id)));
         return  Collections.unmodifiableMap(result);
    }
}

 public   class  MutablePoint {
     public   int  x, y;

     public  MutablePoint() { x =  0 ; y =  0 ; }
     public  MutablePoint(MutablePoint p) {
         this .x = p.x;
         this .y = p.y;
    }
}

Я не могу понять зачем нужен этот метод deepCopy? Ведь все действия с location находятся в блоках
synchronized. На кой нужен этот метод? Почему нельзя сделать так как ниже


Код: plaintext
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
 public   class  MonitorVehicleTracker {
    @GuardedBy("this")
     private   final  Map<String, MutablePoint> locations;

     public  MonitorVehicleTracker(
            Map<String, MutablePoint> locations) {
         this .locations = locations;
    }

     public   synchronized  Map<String, MutablePoint> getLocations() {
         return  locations;
    }

     public   synchronized   MutablePoint getLocation(String id) {
        MutablePoint loc = locations.get(id);
         return  loc ==  null  ?  null  :  new  MutablePoint(loc);
    }

     public   synchronized    void  setLocation(String id,  int  x,  int  y) {
        MutablePoint loc = locations.get(id);
         if  (loc ==  null )
             throw   new  IllegalArgumentException("No such ID: " + id);
        loc.x = x;
        loc.y = y;
    }
}
...
Рейтинг: 0 / 0
24.11.2011, 11:00
| Ответить | Цитировать | Написать  
Объясните пример их книги Concurency in practice
    #37541933
rabiter
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
rabiter
Участник
Usik,

Ну дык может потому что у клиента, который создает новый экземпляр MonitorVehicleTracker остается ссылка на коллекцию locations? И он с этой коллекцией может играться как хочет в любое время после того как объект MonitorVehicleTracker был создан? Вмешиваясь этим самым в состояние объекта MonitorVehicleTracker, что может привести MonitorVehicleTracker в inconsistent состояние. А если мы при создании MonitorVehicleTracker создадим копию locations, то уже никак не будем зависеть от исходной коллекции.
...
Рейтинг: 0 / 0
24.11.2011, 11:07
| Ответить | Цитировать | Написать  
Объясните пример их книги Concurency in practice
    #37541945
rabiter
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
rabiter
Участник
Причем коллекция состоит из String и из MutablePoint, т.е. при копировании коллекции, String можно копировать по ссылке (ибо String immutable - его никто не сможет поменять), а MutablePoint - надо тоже делать deep copy:
Код: plaintext
result.put(id,  new  MutablePoint(m.get(id)));
Вообще про immutable и mutable объекты хорошо написано в книге Joshua Bloch "Effective Java. Programming Language Guide"
...
Рейтинг: 0 / 0
24.11.2011, 11:14
| Ответить | Цитировать | Написать  
Объясните пример их книги Concurency in practice
    #37542053
Usik
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Usik
Участник
Ок, а зачем в этом методе вызывать Collections.unmodifiableMap(result), мы же ведь скопировали коллекцию?
...
Рейтинг: 0 / 0
24.11.2011, 12:00
| Ответить | Цитировать | Написать  
Объясните пример их книги Concurency in practice
    #37542079
rabiter
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
rabiter
Участник
UsikОк, а зачем в этом методе вызывать Collections.unmodifiableMap(result), мы же ведь скопировали коллекцию?

Ну да, я с первого раза тоже не могу понять для чего это могло бы быть нужно... По-моему можно было бы и не делать unmodifiableMap.
...
Рейтинг: 0 / 0
24.11.2011, 12:11
| Ответить | Цитировать | Написать  
Объясните пример их книги Concurency in practice
    #37542618
забыл ник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
забыл ник
Участник
Код: plaintext
1.
2.
3.
4.
 public  MonitorVehicleTracker(
            Map<String, MutablePoint> locations) {
         this .locations = deepCopy(locations);
    }

Обратите внимание на этот метод, deepCopy используется не только для того, чтобы отдать копию локейшенов клиенту но и в конструкторе. Поэтому и возвращается unmodifyableMap, по-большому счету чтобы уберечь самого себя от модификации коллекции внутри методов VehicleTracker, ну и немного паранойи:) Это хорошая практика. В случае если бы deepCopy отдавался только наружу - то вполне можно было обойтись без оборочавания в немодифайебл мап
...
Рейтинг: 0 / 0
24.11.2011, 15:36
| Ответить | Цитировать | Написать  
Объясните пример их книги Concurency in practice
    #37542731
rabiter
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
rabiter
Участник
забыл ник
Код: plaintext
1.
2.
3.
4.
 public  MonitorVehicleTracker(
            Map<String, MutablePoint> locations) {
         this .locations = deepCopy(locations);
    }

Обратите внимание на этот метод, deepCopy используется не только для того, чтобы отдать копию локейшенов клиенту но и в конструкторе. Поэтому и возвращается unmodifyableMap, по-большому счету чтобы уберечь самого себя от модификации коллекции внутри методов VehicleTracker, ну и немного паранойи:) Это хорошая практика. В случае если бы deepCopy отдавался только наружу - то вполне можно было обойтись без оборочавания в немодифайебл мап

Да, присоединяюсь. На мой взгляд в данном случае это просто хорошая практика и не более. Но я бы вынес создание unmodifyableMap из метода deepCopy, ибо название метода как бы не подразумевает :-) Но это уже придирки. А книжка очень хорошая (я про Java Concurrency in Practice)
...
Рейтинг: 0 / 0
24.11.2011, 16:27
| Ответить | Цитировать | Написать  
Период между сообщениями больше года.
Объясните пример их книги Concurency in practice
    #39402478
questioner
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
questioner
Гость
Как-то в этом примере мапа скопирована супер странно. почему бы просто в коструктор было не передать locations ?
...
Рейтинг: 0 / 0
10.02.2017, 20:22
| Ответить | Цитировать | Написать  
Объясните пример их книги Concurency in practice
    #39402481
questioner
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
questioner
Гость
ой вру, там же deepCopy
...
Рейтинг: 0 / 0
10.02.2017, 20:33
| Ответить | Цитировать | Написать  
9 сообщений из 9, страница 1 из 1
Форумы / Java [игнор отключен] [закрыт для гостей] / Объясните пример их книги Concurency in practice
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение   Полит. конфиден.  
NoSQL.ru       Кролег: Проекты, Новости, Чат       РЕД ФОРУМ       Фотоальбом: Участники, Встречи
Закрыть
start [/forum/topic.php?fid=59&msg=37541923&tid=2123161]:
 0ms
get settings:
 8ms
get forum list:
 13ms
check forum access:
 3ms
check topic access:
 3ms
track hit:
 63ms
get topic data:
 10ms
get forum data:
 2ms
get page messages:
 50ms
get tp. blocked users:
 1ms
others: 241ms
total:  394ms
созд. / загр.: 394ms
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]