我过去曾几次遇到这个问题,但是并没有真正找到一个好的解决方案/设计。

以下示例代码将从实体(公司或文章)生成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();
    }
}

10-07 19:42
查看更多