Trong một thư viện trong Java 7, tôi có một lớp cung cấp dịch vụ cho các lớp khác. Sau khi tạo một thể hiện của lớp dịch vụ này, một phương thức của nó có thể được gọi nhiều lần (hãy gọi nó là doWork()phương thức). Vì vậy, tôi không biết khi nào công việc của lớp dịch vụ được hoàn thành.

Vấn đề là lớp dịch vụ sử dụng các đối tượng nặng và nó sẽ giải phóng chúng. Tôi đặt phần này trong một phương thức (hãy gọi nó release()), nhưng không đảm bảo rằng các nhà phát triển khác sẽ sử dụng phương pháp này.

Có cách nào để buộc các nhà phát triển khác gọi phương thức này sau khi hoàn thành nhiệm vụ của lớp dịch vụ không? Tất nhiên tôi có thể làm tài liệu đó, nhưng tôi muốn ép buộc họ.

Lưu ý: Tôi không thể gọi release()phương thức trong doWork()phương thức, vì doWork() cần các đối tượng đó khi nó được gọi tiếp theo.

Giải pháp thực tế là tạo ra lớp AutoCloseablevà cung cấp một finalize()phương thức làm điểm dừng (nếu thích hợp ... xem bên dưới!). Sau đó, bạn dựa vào người dùng của lớp để sử dụng tài nguyên thử hoặc gọi close()một cách rõ ràng.

Tất nhiên tôi có thể làm tài liệu đó, nhưng tôi muốn ép buộc họ.

Thật không may, không có cách nào ¹ trong Java để buộc lập trình viên làm điều đúng đắn. Điều tốt nhất bạn có thể hy vọng làm là chọn cách sử dụng không chính xác trong máy phân tích mã tĩnh.

Về chủ đề quyết toán. Tính năng Java này có rất ít trường hợp sử dụng tốt . Nếu bạn dựa vào người hoàn thiện để dọn dẹp, bạn sẽ gặp phải vấn đề có thể mất một thời gian rất dài để việc dọn dẹp xảy ra. Bộ hoàn thiện sẽ chỉ được chạy sau khi GC quyết định rằng đối tượng không thể truy cập được nữa. Điều đó có thể không xảy ra cho đến khi JVM thực hiện một bộ sưu tập đầy đủ .

Do đó, nếu vấn đề bạn buộc phải giải quyết là lấy lại các tài nguyên cần được giải phóng sớm , thì bạn đã bỏ lỡ chiếc thuyền bằng cách sử dụng quyết toán.

Và chỉ trong trường hợp bạn không có những gì tôi đang nói ở trên ... gần như không bao giờ thích hợp để sử dụng bộ hoàn thiện trong mã sản xuất, và bạn không bao giờ nên dựa vào chúng!

^{1 - Có nhiều cách. Nếu bạn chuẩn bị "ẩn" các đối tượng dịch vụ khỏi mã người dùng hoặc kiểm soát chặt chẽ vòng đời của họ (ví dụ: https://softwareengineering.stackexchange.com/a/345100/172 ) thì mã người dùng không cần phải gọi release(). Tuy nhiên, API trở nên phức tạp hơn, bị hạn chế ... và IMO "xấu xí". Ngoài ra, điều này không buộc các lập trình viên phải làm điều đúng đắn . Nó đang loại bỏ khả năng của lập trình viên để làm điều sai!}

Điều này có vẻ giống như trường hợp sử dụng cho AutoCloseablegiao diện và try-with-resourcescâu lệnh được giới thiệu trong Java 7. Nếu bạn có một cái gì đó như

public class MyService implements AutoCloseable {
    public void doWork() {
        // ...
    }

    @Override
    public void close() {
        // release resources
    }
}

sau đó người tiêu dùng của bạn có thể sử dụng nó như

public class MyConsumer {
    public void foo() {
        try (MyService myService = new MyService()) {
            //...
            myService.doWork()
            //...
            myService.doWork()
            //...
        }
    }
}

và không phải lo lắng về việc gọi một cách rõ ràng releasebất kể mã của họ có ném ngoại lệ hay không.

Câu trả lời bổ sung

Nếu những gì bạn thực sự tìm kiếm là để đảm bảo rằng không ai có thể quên sử dụng chức năng của bạn, bạn phải làm một vài điều

1. Đảm bảo tất cả các nhà xây dựng MyServicelà riêng tư.
2. Xác định một điểm duy nhất để sử dụng MyServiceđể đảm bảo nó được dọn sạch sau.

Bạn sẽ làm một cái gì đó như

public interface MyServiceConsumer {
    void accept(MyService value);
}

public class MyService implements AutoCloseable {
    private MyService() {
    }


    public static void useMyService(MyServiceConsumer consumer) {
        try (MyService myService = new MyService()) {
            consumer.accept(myService)
        }
    }
}

Sau đó, bạn sẽ sử dụng nó như là

public class MyConsumer {
    public void foo() {
        MyService.useMyService(new MyServiceConsumer() {
            public void accept(MyService myService) {
                myService.doWork()
            }
        });
    }
}

Tôi đã không đề xuất đường dẫn này ban đầu vì nó gớm ghiếc nếu không có lambdas Java-8 và các giao diện chức năng (nơi MyServiceConsumertrở thành Consumer<MyService>) và nó khá áp đặt đối với người tiêu dùng của bạn. Nhưng nếu những gì bạn chỉ muốn là release()phải được gọi, nó sẽ hoạt động.

Vâng bạn có thể

Bạn hoàn toàn có thể buộc họ phát hành, và làm cho nó không thể quên 100%, bằng cách khiến họ không thể tạo ra các Servicethể hiện mới .

Nếu họ đã làm một cái gì đó như thế này trước đây:

Service s = s.new();
try {
  s.doWork("something");
  if (...) s.doWork("something else");
  ...
}
finally {
  s.release(); // oops: easy to forget
}

Sau đó thay đổi nó để họ phải truy cập nó như thế này.

// Their code

Service.runWithRelease(new ServiceConsumer() {
  void run(Service s) {
    s.doWork("something");
    if (...) s.doWork("something else");
    ...
  }  // yay! no s.release() required.
}

// Your code

interface ServiceConsumer {
  void run(Service s);
}

class Service {

   private Service() { ... }      // now: private
   private void release() { ... } // now: private
   public void doWork() { ... }   // unchanged

   public static void runWithRelease(ServiceConsumer consumer) {
      Service s = new Service();
      try {
        consumer.run(s);
      }
      finally {
        s.release();
      } 
    } 
  }

Hãy cẩn thận:

• Hãy xem xét mã giả này, nó đã có từ rất lâu kể từ khi tôi viết Java.
• Có thể có nhiều biến thể thanh lịch hơn trong những ngày này, có thể bao gồm AutoCloseablegiao diện mà ai đó đã đề cập; nhưng ví dụ đã cho sẽ hoạt động tốt, và ngoại trừ sự thanh lịch (bản thân nó là một mục tiêu tốt đẹp) không nên có lý do chính để thay đổi nó. Lưu ý rằng điều này dự định nói rằng bạn có thể sử dụng AutoCloseablebên trong triển khai riêng tư của mình; lợi ích của giải pháp của tôi đối với việc người dùng của bạn sử dụng AutoCloseablelà nó có thể, một lần nữa, không bị lãng quên.
• Bạn sẽ phải bổ sung nó khi cần thiết, ví dụ để đưa các đối số vào new Servicecuộc gọi.
• Như đã đề cập trong các ý kiến, câu hỏi liệu một cấu trúc như thế này (nghĩa là thực hiện quá trình tạo và hủy a Service) có thuộc về người gọi hay không nằm ngoài phạm vi của câu trả lời này. Nhưng nếu bạn quyết định rằng bạn hoàn toàn cần điều này, đây là cách bạn có thể làm nó.
• Một người bình luận đã cung cấp thông tin này về việc xử lý ngoại lệ: Google Sách

Bạn có thể thử sử dụng mẫu Command .

class MyServiceManager {

    public void execute(MyServiceTask tasks...) {
        // set up everyting
        MyService service = this.acquireService();
        // execute the submitted tasks
        foreach (Task task : tasks)
            task.executeWith(service);
        // cleanup yourself
        service.releaseResources();
    }

}

Điều này cho phép bạn kiểm soát hoàn toàn việc thu thập và phát hành tài nguyên. Người gọi chỉ gửi các tác vụ đến Dịch vụ của bạn và chính bạn chịu trách nhiệm thu nhận và dọn dẹp tài nguyên.

Có một nhược điểm, tuy nhiên. Người gọi vẫn có thể làm điều này:

MyServiceTask t1 = // some task
manager.execute(t1);
MyServiceTask t2 = // some task
manager.execute(t2);

Nhưng bạn có thể giải quyết vấn đề này khi nó xảy ra. Khi có vấn đề về hiệu suất và bạn phát hiện ra rằng một số người gọi thực hiện việc này, chỉ cần chỉ cho họ cách thức phù hợp và giải quyết vấn đề:

MyServiceTask t1 = // some task
MyServiceTask t2 = // some task
manager.execute(t1, t2);

Bạn có thể làm cho điều này trở nên phức tạp một cách tùy tiện bằng cách thực hiện các lời hứa cho các nhiệm vụ phụ thuộc vào các nhiệm vụ khác, nhưng sau đó việc phát hành nội dung cũng trở nên phức tạp hơn. Đây chỉ là một điểm khởi đầu.

Không đồng bộ

Như đã được chỉ ra trong các ý kiến, ở trên không thực sự hoạt động với các yêu cầu không đồng bộ. Đúng rồi. Nhưng điều này có thể dễ dàng được giải quyết trong Java 8 bằng cách sử dụng CompleteableFuture , đặc biệt là CompleteableFuture#supplyAsyncđể tạo ra các tương lai riêng lẻ và CompleteableFuture#allOfđể hoàn thiện việc giải phóng các tài nguyên khi tất cả các nhiệm vụ đã hoàn thành. Ngoài ra, người ta luôn có thể sử dụng Chủ đề hoặc ExecutorService để triển khai triển khai Tương lai / Lời hứa của riêng họ.

Nếu bạn có thể, không cho phép người dùng tạo tài nguyên. Để người dùng chuyển đối tượng khách truy cập vào phương thức của bạn, phương thức này sẽ tạo tài nguyên, chuyển nó sang phương thức của đối tượng khách truy cập và phát hành sau đó.

Ý tưởng của tôi tương tự như của Polygnome.

Bạn có thể có các phương thức "do_s Something ()" trong lớp của bạn chỉ cần thêm các lệnh vào danh sách lệnh riêng. Sau đó, có một phương thức "commit ()" thực sự hoạt động và gọi "release ()".

Vì vậy, nếu người dùng không bao giờ gọi commit (), công việc không bao giờ được thực hiện. Nếu họ làm, các tài nguyên được giải phóng.

public class Foo
{
  private ArrayList<ICommand> _commands;

  public Foo doSomething(int arg) { _commands.Add(new DoSomethingCommand(arg)); return this; }
  public Foo doSomethingElse() { _commands.Add(new DoSomethingElseCommand()); return this; }

  public void commit() { 
     for(ICommand c : _commands) c.doWork();
     release();
     _commands.clear();
  }

}

Sau đó, bạn có thể sử dụng nó như foo.doSthing.doSomethineElse (). Commit ();

Một cách khác để thay thế những thứ được đề cập sẽ là việc sử dụng "tài liệu tham khảo thông minh" để quản lý tài nguyên được đóng gói của bạn theo cách cho phép trình thu gom rác tự động dọn dẹp mà không cần giải phóng tài nguyên theo cách thủ công. Tất nhiên tính hữu dụng của chúng phụ thuộc vào việc thực hiện của bạn. Đọc thêm về chủ đề này trong bài đăng blog tuyệt vời này: https://community.oracle.com/bloss/enicholas/2006/05/04/under Hiểu-weak-report

Đối với bất kỳ ngôn ngữ Định hướng Lớp nào khác tôi muốn nói hãy đặt nó vào hàm hủy, nhưng vì đó không phải là một tùy chọn nên tôi nghĩ bạn có thể ghi đè phương thức hoàn thiện. Theo cách đó, khi cá thể đi ra khỏi phạm vi và là rác được thu thập, nó sẽ được phát hành.

@Override
public void finalize() {
    this.release();
}

Tôi không quá quen thuộc với Java, nhưng tôi nghĩ rằng đây là mục đích để hoàn thiện ().

http://docs.oracle.com/javase/7/docs/api/java/lang/Object.html#finalize ()