避免硬编码 Hibernate 属性

Question

我没有在我的源代码中包含使用硬编码的 Hibernate 属性名称，以避免在更改数据库或 Hibernate 配置时出现潜在的错误。

我总是使用 Hibernate Criteria 和我创建的以下 HibernateUtils.getPropertyName() 方法来完成此操作。

/**
* Get a Hibernate property name.<p>
*
* <b>NOTE:</b> This method assumes all property names start with a lower-case character.
*
* @param methodChecker
 *        - An ignored value. This parameter is a place-holder for a call to the "get" method for the property of interest.
 * @param methodName
 *        - The name of the "get" method used to retrieve the property of interest
 * @return The property name
 */
public static String getPropertyName(Object methodChecker, String methodName)
{
    String propertyName;
    if (methodName.startsWith("get"))
    {
        propertyName = methodName.substring(3, 4).toLowerCase(Locale.ENGLISH)
                + methodName.substring(4);
    }
    else if (methodName.startsWith("is"))
    {
        propertyName = methodName.substring(2, 3).toLowerCase(Locale.ENGLISH)
                + methodName.substring(3);
    }
    else
    {
        throw new IllegalArgumentException("method name did not start with 'is' or 'get'");
    }

    return propertyName;
}

为了使用它，我将属性的“get”方法调用为第一个参数，然后硬编码第二个属性的“get”方法的名称。

使用这种方法，Hibernate 配置或数据库更改将导致 COMPILE-TIME ERRORS 而不是 RUN-TIME ERRORS。

例如，如果将 vendor 属性重命名为 vendorname，则以下代码将导致 RUN-TIME ERROR。

产品 product = (Product) session.createCriteria(Product.class).add(Property.forName("vendor").eq(vendor)).uniqueResult();

要修复代码，所有出现的 vendor 都必须替换为 vendorname。显然，这很容易出错并且可能非常耗时。

我使用以下语句实现了相同的功能：

Product product = (Product) session.createCriteria(Product.class).add(Property.forName(HibernateUtils.getPropertyName(myProduct.getVendor(), "getVendor")).eq(vendor)).uniqueResult ();

如果将 vendor 属性重命名为 vendorname，第二种方法将导致 COMPILE-TIME ERRORS，因为 getVendor() 方法将更改为getVendorname()。

我想知道是否有其他方法 - 可以让我完全消除 HibernateUtils.getPropertyName()。

谢谢！

Answer 1

我不认为这是一个好的设计。你说你没有硬编码属性名称，但你是。

首先，您不再只在实体中对属性名称进行一次硬编码，而是对它进行多次硬编码，从而增加了拼写错误的可能性，或者如果设计发生更改，则需要在另一个地方进行修复。

如果表或列发生更改，我不会担心将来对您的配置进行校对。您应该进行单元和集成测试（您确实有测试吗？）来实际测试数据是否被正确读取并加载到您的模式中（在测试数据库中）。任何更改都是数据库模式是一个巨大的更改，肯定会保证对您的代码进行新的更新，因此它不太可能意外或未经通知发生。最后，如果您只是适当地使用休眠属性映射，则只需在/如果列名称更改时更新单个位置。

最后，如果您想更改对象中的方法名称，所有优秀的 IDE 都将支持重构，以自动更新使用该方法的所有位置。如果您只是传递字符串名称并使用类似 Java Bean 的名称来确定它是什么方法，那将更难做到。