我过去曾几次遇到这个问题,但是并没有真正找到一个好的解决方案/设计。
以下示例代码将从实体(公司或文章)生成PDF文档
public class Entity
{
int id;
}
public class Company extends Entity
{
private String HQ;
}
public class Article extends Entity
{
private String title;
}
public interface EntityPDFGenerator
{
void generate(Entity entity);
}
public class ArticlePDFGenerator implements EntityPDFGenerator
{
public void generate(Entity entity)
{
Article article = (Article) entity;
// create Article related PDF from entity
}
}
public class CompanyPDFGenerator implements EntityPDFGenerator
{
public void generate(Entity entity)
{
Company company = (Company) entity;
// create Company related PDF
}
}
主班:
public class PDFGenerator
{
public void generate(Entity entity)
{
EntityPDFGenerator pdfGenerator = getConcretePDFGenerator(entity);
pdfGenerator.generate(entity);
}
// lets make the factory task simple for now
EntityPDFGenerator getConcretePDFGenerator(Entity entity)
{
if(entity instanceof Article){
return new ArticlePDFGenerator();
}else{
return new CompanyPDFGenerator();
}
}
}
在上述方法中,问题在于将Entity强制转换为具体类型(在代码的后期阶段进行铸造可能很危险)。我尝试使用泛型来实现,但是随后得到警告
未经检查的“ generate(T)”调用
我可以改善此代码吗?
最佳答案
简短答案
泛型不是这里的正确工具。您可以使转换明确:
public class CompanyPDFGenerator implements EntityPDFGenerator
{
public void generate(Entity entity)
{
if (! (entity instanceof Company)) {
throw new IllegalArgumentException("CompanyPDFGenerator works with Company object. You provided " + (entity == null ? "null" : entity.getClass().getName()));
}
Company company = (Company) entity;
System.out.println(company);
// create Company related PDF
}
}
或者,您可以在实体类中定义某种数据结构,并仅在打印机中使用它:
public abstract class Entity
{
int id;
public abstract EntityPdfData getPdfData();
}
// ...
public class CompanyPDFGenerator implements EntityPDFGenerator
{
public void generate(Entity entity)
{
EntityPdfData entityPdfData = entity.getPdfData();
// create Company related PDF
}
}
长答案
如果您在编译时知道类型,则泛型很有用。即如果您可以将实际类型写入程序。对于列表,它看起来很简单:
// now you know at compile time that you need a list of integers
List<Integer> list = new ArrayList<>();
在您的示例中,您不知道:
public void generate(Entity entity)
{
// either Article or Company can come it. It's a general method
EntityPDFGenerator pdfGenerator = getConcretePDFGenerator(entity);
pdfGenerator.generate(entity);
}
假设您要向
EntityPDFGenerator
添加类型,如下所示:public static interface EntityPDFGenerator<T extends Entity>
{
void generate(T entity);
}
public static class ArticlePDFGenerator implements EntityPDFGenerator<Article>
{
public void generate(Article entity)
{
Article article = (Article) entity;
// create Article related PDF from entity
}
}
public static class CompanyPDFGenerator implements EntityPDFGenerator<Company>
{
public void generate(Company entity)
{
Company company = (Company) entity;
// create Company related PDF
}
}
看起来不错但是,找到合适的发电机将非常棘手。 Java泛型是不变的。甚至
ArrayList<Integer>
也不是ArrayList<Number>
的子类。因此,ArticlePdfGenerator
不是EntityPDFGenerator<T extends Entity>
的子类。即这将无法编译:<T extends Entity> EntityPDFGenerator<T> getConcretePDFGenerator(T entity, Class<T> classToken)
{
if(entity instanceof Article){
return new ArticlePDFGenerator();
}else{
return new CompanyPDFGenerator();
}
}