Is this producer-consumer scenario safe about dead-lock?

Question

Hi friends!
I'm developing an application with a producer-consumer scenario. It produces a dead-lock 10% of times approx I execute it . I think the real reason of the crashing resides on the concurrency, because executing only the following code it gets the "main" waiting blocked during its execution after looping the do{...}while(true) sometimes. I've developed this code:
public class CubbyHole
    private int nFinishedThreads;
    private int nTotalThreads;
    public CubbyHole(int TotalThreads)
        this.nTotalThreads = nTotalThreads;
        this.nFinishedThreads = 0;
    public synchronized void put()
        this.nFinishedThreads++;
    public synchronized void get()
        while(nFinishedThreads < nTotalThreads)
            try
                wait();
            catch(InterruptedException ie){}
        notifyAll();
//import java.util.Random;
public class Producer implements Runnable
    private CubbyHole container;
    public Producer(CubbyHole container)
        this.container = container;
    public void run()
        System.out.println(Thread.currentThread().getName() + " has begun");
        try{
            Random rand = new Random();
            Thread.sleep(Math.abs(rand.nextInt()*10));
        catch(InterruptedException ie){}
        System.out.println(Thread.currentThread().getName() + " has finished");
        container.put();
public class Consumer implements Runnable
    private CubbyHole container;
    public Consumer(CubbyHole container)
        this.container = container;      
    public void run()
        System.out.println(Thread.currentThread().getName() + "is waiting for all threads to be finished");
        container.get();
        System.out.println(Thread.currentThread().getName() + "says all threads have finished");
public class Application
    public static void main(String Args[])
        do{
            int nThreads = 2;
            CubbyHole container = new CubbyHole(nThreads);
            Thread[] ProducersArray = new Thread[nThreads];
            for(int i=0; i<nThreads; i++)
                ProducersArray=
new Thread(
new Producer(container));
ProducersArray[i].start();
Thread finalWait = new Thread( new Consumer(container));
finalWait.start();
try
finalWait.join();
catch(InterruptedException ie){}
System.out.println("Application has finished correctly");
while(true);
Please, execute it and give me your results: is the "main" waiting blocked to you?
Is this producer-consumer code safe about dead-lock?
If the answer is no... Why is it waiting blocked?
Thank you in advance.
P.D. I'm spanish, so my english is horrible! I'm sorry!

Answer 1

Thanks ejp:
Moreover, I suppose nothing happens if I write notifyAll() in both methods, put() and get(). In fact this matters only if the producer overwrites the memory space where data is stored, which is not my case.
Is this so?