Question

在以下程序中，LoggerThread类中的this关键字是否引用LoggerThread对象或LogService对象？从逻辑上讲，它应该引用LogService以便进行同步，但是从语义上讲，它似乎是在引用LoggerThread。

public class LogService {
    private final BlockingQueue<String> queue;
    private final LoggerThread loggerThread;
    private final PrintWriter writer;
    @GuardedBy("this") private boolean isShutdown;
    @GuardedBy("this") private int reservations;
    public void start() { loggerThread.start(); }
    public void stop() {
        synchronized (this) { isShutdown = true; }
        loggerThread.interrupt();
    }
    public void log(String msg) throws InterruptedException {
        synchronized (this) {
            if (isShutdown)
                throw new IllegalStateException("...");
            ++reservations;
        }
        queue.put(msg);
    }
    private class LoggerThread extends Thread {
        public void run() {
            try {
                while (true) {
                    try {
                        synchronized (this) {
                            if (isShutdown && reservations == 0)
                                break;
                        }
                        String msg = queue.take();
                        synchronized (this) { --reservations; }
                        writer.println(msg);
                    } catch (InterruptedException e) { /* retry */ }
                }
            } finally {
                writer.close();
            }
        }
    }
}

谢谢您的帮助

Answer 1

this方法中的

LoggerThread引用一个LoggerThread实例。
LogService.this是指外部类。

isShutdown和reservations都通过不同的锁（LoggerThread.this和LogService.this）同步，因此@GuardedBy("this")不能反映现实。

Answer 2

这段代码来自《Java Concurrency In Practice》一书，代码清单 7.15
这是一个错字，在“勘误”部分中提到：
http://jcip.net.s3-website-us-east-1.amazonaws.com/errata.html

Answer 3

this是指直接封闭类的当前实例。 JLS #15.8.4。

从逻辑上讲，它应该引用LogService以便进行同步，但从语义上看，它是在指LoggerThread。

正确。这是一个错误。

使用此关键字作为并发锁定

3 个答案: