Re: Mutable Objects and Thread Boundaries

Alan Gutierrez wrote:


Actually, the LinkedBlockingQueue is synchronized. Why do you say it's not?

It's direct parent, BlockingQueue, makes the memory consistency guarantee.

On the other hand, this is a problem:

 > try {
....
 > } catch (InterruptedException e) {
 > continue;
 > }

Interrupts do not happen spuriously or for no reason. If you get an
interrupt, someone has requested that your thread exit. It's best to do
so. I'd recommend something like:

             public void run() {
                 try {
                     for (;;) {

                           List<Integer> list = queue.take();
                         if (list.get(0) != 0) {
                             continue;
                         }
                     }
                } catch (InterruptedException e) {
                }

             }

In other words, go ahead and catch the exception to prevent ugly
messages, but "handle" it by exiting. This is almost always the correct
response.