Lớp này có phải là singleton không?

Question

Lớp học có được liệt kê bên dưới một singleton không? Vì hàm tạo được khai báo là công khai, tôi có thể suy ra rằng lớp đó là một singleton có triển khai sai không?

public class CreateDevice extends Functionality{ 

    private static Simulator simulator; 
    ConnectionDB connect = ConnectionDB.getInstance(); 


    public CreateDevice(Simulator simulator){ 
    this.simulator = simulator; 
    } 

    private static CreateDevice instance; 
    synchronized public static CreateDevice getInstance() { 
    if(instance == null){ 
     instance = new CreateDevice(simulator); 
    }  
     return instance; 
    } 
} 

Answer 1

Bạn có thể làm cho nó thành một đĩa đơn, nhưng sau đó bạn cần phải tìm cách khác để đưa trình mô phỏng vào đó.

Trong triển khai hiện tại, trình mô phỏng được đặt lần đầu tiên ai đó gọi hàm tạo. Constructor đó rất lạ vì nó thiết lập một trường tĩnh. Nó cũng sẽ ngay lập tức mở một kết nối, tách biệt với kết nối được sử dụng bởi cá thể singleton.

Nếu phương thức getInstance() được gọi trước khi hàm tạo được gọi ít nhất một lần, trình giả lập không bao giờ được đặt.

Để làm cho nó trở thành một singleton thích hợp, bạn có thể loại bỏ hàm khởi tạo và thêm một hàm tạo riêng không-arg. Bạn cũng sẽ cần một phương thức setSimulator() tĩnh đặt trường tĩnh và đảm bảo nó được gọi trước khi bất kỳ tương tác nào khác với trình mô phỏng được yêu cầu.

Nếu bạn có phụ thuộc giữa các bộ đơn, tôi khuyên bạn nên sử dụng mẫu Inversion-of-Control, nơi một thùng chứa IoC tạo các đối tượng dịch vụ và nối chúng lại với nhau.

Answer 2

Trong từ - không. Vì hàm tạo là public, bất kỳ ai cũng có thể tạo phiên bản mới của nó ở bất kỳ đâu và bất cứ khi nào họ muốn. Làm cho hàm tạo private sẽ giải quyết vấn đề và biến lớp thành một singleton.

Answer 3

Tôi muốn nói rằng bạn đúng khi nói rằng đây không phải là Singleton trong quá trình triển khai hiện tại. Hoặc bạn cần phải làm cho hàm tạo riêng (đó là những gì thường được thực hiện), hoặc nếu vì lý do kế thừa bạn cần một hàm tạo công khai, bạn nên tạo một hàm tạo để kiểm tra xem cá thể đó có rỗng hay không và ném một ngoại lệ nếu nó đã tồn tại hoặc tạo nó bằng cách gọi một hàm tạo riêng khác, gán nó cho cá thể và trả về nó. Lựa chọn đầu tiên là thích hợp hơn, althouth khác hoạt động tốt cho hầu hết các dự án mà một refactor là quá đắt.