I'm not using synclock correctly...

[j3ko]

Hi, I'm trying to start a thread that constantly iterates through an
arraylist of items that the main thread adds and removes from...how
would I accomplish this?

Here's the gist of what I have: (can anybody give me pointers on what
i'm doing wrong?)

Class SecondThread
aList as new arraylist
innerthread as new thread(addresso f task)
public sub task()
while true
synclock aList.syncroot
for i as integer = 0 to aList.count-1
' do something
end for
end synclock

innerthread.sle ep(10000)
end while
end sub

public sub start()
innerthread.sta rt()
end sub

public sub add(item)
synclock aList.syncroot
aList.add(item)
end synclock
end sub

public sub remove(item)
synclock aList.syncroot
aList.remove(it em)
end synclock
end sub
end class
my main thread calls start when I load and calls add and remove
periodically... It seems to add just fine...but a call to remove does
nothing :\

[José Manuel Agüero]

Hello j3ko,

You should use a synchronized arraylist:
Dim aList as ArrayList = ArrayList.Synch ronized(New ArrayList)

Using this, you don't have to synchronize read and write operations yourself:
public sub remove(item)
aList.remove(it em)
end sub

You still need to synchronize enumeration or iterations through the array:
synclock aList.syncroot
for i as integer = 0 to aList.count-1
' do something
next
end synclock

Anyway, as you wrap the methods for the collection, you can make your background thread responsive using an AutoResetEvent. Your code would look like this:

Class SecondThread
Dim aList as ArrayList = ArrayList.Synch ronized(New ArrayList)
Dim innerthread as new thread(addresso f task)
Dim sync as new AutoResetEvent( False)

public sub task()
Do

sync.WaitOne 'Stop until sync.Set() is called anywhere.
synclock aList.syncroot
for i as integer = 0 to aList.count-1
' do something
Next
end synclock
Loop
end sub

public sub start()
innerthread.IsB ackground = True 'So it ends when main thread ends.
innerthread.sta rt()
end sub

public sub add(item)
aList.add(item)
sync.Set 'Let the background thread make a loop.
end sub

public sub remove(item)
aList.remove(it em)
sync.Set
end sub
end class
Regards.
"j3ko" <je*******@gmai l.com> escribió en el mensaje news:11******** **************@ g43g2000cwa.goo glegroups.com.. .
| Hi, I'm trying to start a thread that constantly iterates through an
| arraylist of items that the main thread adds and removes from...how
| would I accomplish this?
|
| Here's the gist of what I have: (can anybody give me pointers on what
| i'm doing wrong?)
|
| Class SecondThread
| aList as new arraylist
| innerthread as new thread(addresso f task)
|
|
| public sub task()
| while true
| synclock aList.syncroot
| for i as integer = 0 to aList.count-1
| ' do something
| end for
| end synclock
|
| innerthread.sle ep(10000)
| end while
| end sub
|
| public sub start()
| innerthread.sta rt()
| end sub
|
| public sub add(item)
| synclock aList.syncroot
| aList.add(item)
| end synclock
| end sub
|
| public sub remove(item)
| synclock aList.syncroot
| aList.remove(it em)
| end synclock
| end sub
| end class
|
|
| my main thread calls start when I load and calls add and remove
| periodically... It seems to add just fine...but a call to remove does
| nothing :\

[j3ko]

Hi Jose,

Thanks! Just tried it, works like a charm!