Java Generics vấn đề nhà máy trừu tượng

Question

Tôi đang đấu tranh để làm cho công việc này:

public abstract class MapperFactory<M extends TaskMapper<? extends Message, ? extends Message, ? extends TaskForm>> { 

    public static <M extends TaskMapper<? extends Message, ? extends Message, ? extends TaskForm>> MapperFactory<M> getMapperFactory(Message msgIn, Message msgOut) { 

     if (msgIn.isMyMapper()) 
      return new MyTaskMapperFactory(); 

     throw new IllegalStateException("Mapper not found!"); 
    } 

    public abstract TaskMapper<? extends Message, ? extends Message, ? extends TaskForm> getTaskMapper(); 

    public static class MyTaskMapperFactory extends MapperFactory<MyTaskMapper> { 

     @Override 
     public TaskMapper<? extends Message, ? extends Message, ? extends TaskForm> getTaskMapper() { 
      return new MyTaskMapper(); 
     } 

    } 
} 

public interface TaskMapper<I extends Message, O extends Message, F extends TaskForm> { 

    public F fillForm(I msgIn, O msgOut, F taskForm); 

    public O fillMsgOut(F taskForm); 
} 

public class MyTaskMapper implements TaskMapper<IncomingMessage, OutgoingMessage, MyTaskForm > { 

    public MyTaskForm fillForm(IncomingMessage msgIn, OutgoingMessage msgOut, 
      MyTaskForm taskForm) { 
     return null; 
    } 

    public OutgoingMessage fillMsgOut(MyTaskForm taskForm) { 
     return null; 
    } 

} 

Vấn đề là một lỗi biên dịch:

Type mismatch: cannot convert from MapperFactory.MyTaskMapperFactory to MapperFactory

trong MapperFactory tôi ở đây:

if (msgIn.isMyMapper()) 
      return new MyTaskMapperFactory(); 

Bất kỳ ý tưởng làm thế nào để sửa lỗi này?

Dĩ nhiên thay thế:

public static <M extends TaskMapper<? extends Message, ? extends Message, ? extends TaskForm>> MapperFactory<M> getMapperFactory(Message msgIn, Message msgOut) { 

     if (msgIn.isMyMapper()) 
      return new MyTaskMapperFactory(); 

     throw new IllegalStateException("Mapper not found!"); 
    } 

với:

public static MapperFactory<?> getMapperFactory(Message msgIn, Message msgOut) { 

     if (msgIn.isMyMapper()) 
      return new MyTaskMapperFactory(); 

     throw new IllegalStateException("Mapper not found!"); 
    } 

sẽ làm việc, nhưng đó không phải là câu trả lời mà tôi đang tìm kiếm.

Điều này có vẻ là một vấn đề với mô hình nhà máy chung trừu tượng nói chung. Câu trả lời cung cấp các mẫu nguồn bằng cách sử dụng các đối tượng được tạo tùy chỉnh cũng được hoan nghênh.

Answer 1

Theo Effective Java, 2nd edition, mục 28:

If a type parameter appears only once in a method declaration, replace it with a wildcard.

phương pháp getMapperFactory bạn chỉ sử dụng các tham số loại M trong kiểu trả về. Sau lời khuyên này cho chữ ký phương pháp sau đây, và phương pháp biên dịch:

public static MapperFactory<? extends TaskMapper<Message, ? extends Message, ? extends String>> getMapperFactory(Message msgIn, Message msgOut) 

EDIT: Tôi càng nhìn vào mã, tôi càng nghĩ MapperFactory không nên được tham số. Tham số không được sử dụng trong mã ở đây, getTaskMapper trả về một TaskMapper.

Answer 2

Lệnh return hoạt động tốt với một thợ đúc chư:

return (BpmMapperFactory<MAPPER>)new Bpm007PrepareDocTaskMapperFactory(); 

Đó là mã sẽ không bao giờ thực hiện mặc dù trong hình thức hiện tại của nó, bởi vì Bpm007PrepareDocTaskMapper không mở rộng BpmCommonMessageDto, vì vậy msgIn có thể không có khả năng là một thể hiện của Bpm007PrepareDocTaskMapper.

Answer 3

Giải pháp của tôi sẽ được giết càng nhiều các Generics càng tốt với một ngọn lửa:

abstract class MapperFactory<M extends TaskMapper<?, ?, ?>> { 

    public static MapperFactory<?> getMapperFactory(Message msgIn, Message msgOut) { 
     if (msgIn.isMyMapper()) return new MyTaskMapperFactory(); 
     throw new IllegalStateException("Mapper not found!"); 
    } 

    public abstract M getTaskMapper(); 
} 


class MyTaskMapperFactory extends MapperFactory<MyTaskMapper> { 

    @Override 
    public MyTaskMapper getTaskMapper() { 
     return new MyTaskMapper(); 
    } 

} 


interface TaskMapper<I extends Message, O extends Message, F extends TaskForm> { 

    public F fillForm(I msgIn, O msgOut, F taskForm); 

    public O fillMsgOut(F taskForm); 

} 

class MyTaskMapper implements TaskMapper<IncomingMessage, OutgoingMessage, MyTaskForm> { 

    public MyTaskForm fillForm(IncomingMessage msgIn, OutgoingMessage msgOut, MyTaskForm taskForm) { 
     return null; 
    } 

    public OutgoingMessage fillMsgOut(MyTaskForm taskForm) { 
     return null; 
    } 

} 

Nó thực sự không cần thiết phải lặp lại các tham số kiểu của một lớp trong mỗi phương pháp có sử dụng nó nếu bạn không thực sự quan tâm những gì họ đang có hoặc không cần phải hạn chế họ nhiều hơn so với chữ ký lớp.