Всем привет, помогите реализовать синхронизацию, второй день капаю и ничего не получается. Спасибо заранее



Это сообщение отредактировал(а) Vizor - 4.11.2008, 22:02

lock(this) никогда не следует писать. см. Рихтера
Для синхронизации он рекомендовал использовать приватный филд - если такого нету, можно самому его создать, и определить для него тип object (3 байта памяти, если не ошибаюсь). Это делаеться для того чтобы можно было рекурсивно входить в метод с блокировкой. Да и блокировать такой большой объект, кажись плохо для производительности.

Но тут явно дело в другом. Я так понимаю, проблема в том что ты пытаешься изменить список айтомеов листа в другом потоке - а изменения в любом контроле должны выполняться в том же потоке в кором создался контрол.. (см BeginInvoke) (посмотрел внимательнее - ты ничего не изменяешь, так что все пока ок  - но мало ли  )

Но и это еще не все. В даном случае вообще плохо написано. Во-первых ты блокируешь себя в цикле, и там же делаешь слип - чего делать категорически нельзя (выходит твой езкемпляр листа постоянно заблокирован)

А если посмотреть с точки зрения архитектуры - то тут вообще все не так как должно быть - контролл создает какой-то поток, который постоянно работает, слипаеться иногда - по-моему это неудачное решение. В контролла полно всяких событий - в данном случае помогло бы ItemChanged (или как оно там - влом смотреть в мсдн). Подписуешься на него и делаешь все изменения только тогда, когда изменяються итемы в контролле, а не каждый раз. 

ВОобще представь себе что я кинул таких контролов на форму 200 штук - выйдет что они создут 200 потоков.  

Добавлено @ 01:56
И вообще, почему тема в FAQ?   

Это сообщение отредактировал(а) archeg - 5.11.2008, 01:59

Что-то я не вижу, что с чем тут должно синхронизироваться.

Насколько я читаю этот код, я вижу, что в конструкторе списка запускается поток, который захватывает список. Поток что-то непонятное делает с его айтемами. Что такое pTable и u непонятно. Затем поток ждёт одну секунду и завершается, освобождая список.

В результате, я думаю, должен получиться список, который появляется на экране с запозданием. Потому что во время его появления он может быть захвачен потоком и не реагировать на сообщения графического интерфейса. Затем, когда поток завершается, список становится живым. 

Но зачем был нужен поток -- непонятно.

Добавлено через 1 минуту и 28 секунд
А, да, while(true) не заметил.

Добавлено через 4 минуты и 46 секунд
Это вообще вилы. Получается, поток никогда не завершается и постоянно держит список. Список освобождается на микросекунду после закрывающей скобки lock и до захода на новый виток цикла while. За это время, если повезёт, графическая оболочка успеет сама захватить список и что-то с ним сделать. Но что-то незначительное. А потом, когда она его отпустит, то данный поток снова его захватит и станет сканировать элементы.

То есть, данный код представляет собой код "мёртвого" списка, который почти ни на что не реагирует.