Гость
Войти | Профиль | Очистить
Целевая тема:
Создать новую тему:
Автор:
Действия ...
Форумы / Java [игнор отключен] [закрыт для гостей] / Объясните пример их книги Concurency in practice / 9 сообщений из 9, страница 1 из 1
24.11.2011, 11:00
    #37541923
Usik
Usik
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Объясните пример их книги Concurency in practice
Разбираю пример из книги 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:07
    #37541933
rabiter
rabiter
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Объясните пример их книги Concurency in practice
Usik,

Ну дык может потому что у клиента, который создает новый экземпляр MonitorVehicleTracker остается ссылка на коллекцию locations? И он с этой коллекцией может играться как хочет в любое время после того как объект MonitorVehicleTracker был создан? Вмешиваясь этим самым в состояние объекта MonitorVehicleTracker, что может привести MonitorVehicleTracker в inconsistent состояние. А если мы при создании MonitorVehicleTracker создадим копию locations, то уже никак не будем зависеть от исходной коллекции.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
24.11.2011, 11:14
    #37541945
rabiter
rabiter
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Объясните пример их книги Concurency in practice
Причем коллекция состоит из 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, 12:00
    #37542053
Usik
Usik
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Объясните пример их книги Concurency in practice
Ок, а зачем в этом методе вызывать Collections.unmodifiableMap(result), мы же ведь скопировали коллекцию?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
24.11.2011, 12:11
    #37542079
rabiter
rabiter
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Объясните пример их книги Concurency in practice
UsikОк, а зачем в этом методе вызывать Collections.unmodifiableMap(result), мы же ведь скопировали коллекцию?

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

Обратите внимание на этот метод, deepCopy используется не только для того, чтобы отдать копию локейшенов клиенту но и в конструкторе. Поэтому и возвращается unmodifyableMap, по-большому счету чтобы уберечь самого себя от модификации коллекции внутри методов VehicleTracker, ну и немного паранойи:) Это хорошая практика. В случае если бы deepCopy отдавался только наружу - то вполне можно было обойтись без оборочавания в немодифайебл мап
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
24.11.2011, 16:27
    #37542731
rabiter
rabiter
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Объясните пример их книги Concurency in practice
забыл ник
Код: 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
| Ответить | Цитировать | Написать  
Период между сообщениями больше года.
10.02.2017, 20:22
    #39402478
questioner
questioner
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Объясните пример их книги Concurency in practice
Как-то в этом примере мапа скопирована супер странно. почему бы просто в коструктор было не передать locations ?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
10.02.2017, 20:33
    #39402481
questioner
questioner
Гость
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Объясните пример их книги Concurency in practice
ой вру, там же deepCopy
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
Форумы / Java [игнор отключен] [закрыт для гостей] / Объясните пример их книги Concurency in practice / 9 сообщений из 9, страница 1 из 1
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение  
Полит. конфиден.  
созд. / загр.: 388ms
Закрыть
start [/forum/topic.php?fid=59&mobile=1&tid=2123161]:
 0ms
get settings:
 9ms
get forum list:
 14ms
check forum access:
 3ms
check topic access:
 3ms
track hit:
 62ms
get topic data:
 11ms
get forum data:
 3ms
get page messages:
 52ms
get tp. blocked users:
 2ms
others: 229ms
total:  388ms
x
x
Закрыть


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