java - 私の生産者/消費者ソリューションは正しいですか?

Question

私はスレッドについてもっと学ぼうとしており、生産者/消費者の問題に対する解決策を考え出すことが良い出発点になると考えました. 私がソリューションに課した制約の 1 つは、生産者がどれだけ生産しているかを消費者が事前に知らないということでした。コードは期待どおりに動作し、何度も実行しましたが、欠陥がないわけではありません。このソリューションに問題はありますか?

package Multithreading.ProducerConsumer;

import java.util.LinkedList;
import java.util.concurrent.Semaphore;

public class ProducerConsumer
{
    private class Producer implements Runnable
    {
        @Override
        public void run()
        {
            for(int i = 0; i < 1000; i++)
            {
                try 
                {
                    canProduce.acquire();
                    mutex.acquire();
                    queue.add(i);
                    mutex.release();
                    canConsume.release();
                } 
                catch (InterruptedException ex) 
                {
                    ;
                }
            }
            try
            {
                canConsume.acquire();
                isTryingToFinish = true;
                canConsume.release();
            }
            catch (InterruptedException ex)
            {
                ;
            }
        }
    }

    private class Consumer implements Runnable
    {
        @Override
        public void run()
        {
            while(!isDone)
            {
                try
                {
                    canConsume.acquire();
                    mutex.acquire();
                    System.out.println(queue.pop());
                    if(isTryingToFinish && queue.isEmpty())
                    {
                        isDone = true;
                    }
                    mutex.release();
                    canProduce.release();
                }
                catch (InterruptedException ex)
                {
                    ;
                }
            }
        }
    }

    Semaphore canProduce;
    Semaphore canConsume;
    Semaphore mutex;
    boolean isTryingToFinish = false;
    boolean isDone = false;
    final static int bufferSize = 100;
    LinkedList<Integer> queue;

    public ProducerConsumer()
    {
        queue = new LinkedList<>();
        canProduce = new Semaphore(bufferSize);
        canConsume = new Semaphore(0);
        mutex = new Semaphore(1);
    }

    public void Go() throws InterruptedException
    {
        Thread p = new Thread(new Producer());
        Thread c = new Thread(new Consumer());
        p.start();
        c.start();
        p.join();
        c.join();
        System.out.println("Job Complete!");
    }

    public static void main(String[] args) throws InterruptedException
    {
        ProducerConsumer p = new ProducerConsumer();
        p.Go();
    }
}