synchronize variable by lock

[Stefan Müllerheim]

hi, i try to synchronize the access to a variable and wrote something below. is that correct? see anybody any fault?

import java.util.concurrent.locks.ReentrantLock;

public class d
{
    private final ReentrantLock l1 = new ReentrantLock();

    private int a1 = 0;

    public int get()
    {	
	int a2; 
	l1.lock();
	a2 = a1;
	l1.unlock();
	return a2;
    }
    
    public void set( int a3 )
    {
	l1.lock();
	a1 = a3;
	l1.unlock();
    }
}

[Henry Wong]

BTW, there is no need to create a new local variable...

public int get()  
{     
    try {
        l1.lock();
        return a1;
    } finally {
        l1.unlock();  
    }
}  

Henry