2

I'm trying to have a enum type on the database and convert it to java, I wrote a EnumUserType class to do the conversion, but it doesn't recognize the PGobject class.

public Object nullSafeGet(ResultSet rs, String[] names, SessionImplementor si, Object owner)
        throws HibernateException, SQLException {
    Object object = rs.getObject(names[0]);
    if (rs.wasNull()) {
        return null;
    }

    if (object instanceof PGobject) {
        //code doesn't reach this line
    }

    log.info(object.getClass()); // prints class org.postgresql.util.PGobject
    return null;
}

I checked and I have the exact same postgresql driver version. I saw this post: Java enum with Eclipselink. It is a solution that I will also try, but my main question is: apparently it is the same class, why it is not being recognized as such? Can I have two different classes with the same name and package? If I still have to use enums in Postgres, how can I fix it to properly map to my Java enum?

EDIT:

I tried to do a:

PGobject pg = (PGobject) object;

and it throws a class cast exception:

org.postgresql.util.PGobject cannot be cast to org.postgresql.util.PGobject

Thanks

Community
  • 1
  • 1
Migore
  • 1,477
  • 3
  • 19
  • 40
  • 2
    Why don't you use `rs.getString()` instead? It should return the enum value as a String, AFAIK. – JB Nizet Mar 07 '13 at 22:51
  • Just tried it and it throws an exception: org.postgresql.util.PGobject cannot be cast to java.lang.String – Migore Mar 08 '13 at 12:37
  • Disregard my previous comment, there was something else missing. Just tried it and it work, thanks! This solve my issue but I'm still curious about the problem. – Migore Mar 08 '13 at 12:56
  • 1
    Judging by the exception, it looks like a classloader problem - the same class loaded by two different classloaders are considered different classes by java. – Grim Mar 08 '13 at 13:18
  • I'm using JBoss AS 7.1.1 as a container, I copied the same jar to jboss repository and to my local maven repository that is used by my application. – Migore Mar 08 '13 at 13:43

1 Answers1

0

I use a generic class to use as a type to map a enum. Then, you can map all your enums using it.

The class is this:

public class GenericEnumUserType implements UserType, ParameterizedType {

private static final String DEFAULT_IDENTIFIER_METHOD_NAME = "name";
private static final String DEFAULT_VALUE_OF_METHOD_NAME = "valueOf";

private Class enumClass;
private Class identifierType;
private Method identifierMethod;
private Method valueOfMethod;
private NullableType type;
private int[] sqlTypes;

@Override
public void setParameterValues(Properties parameters) {
    String enumClassName = parameters.getProperty("enumClassName");
    try {
        enumClass = Class.forName(enumClassName).asSubclass(Enum.class);
    } catch (ClassNotFoundException cfne) {
        throw new HibernateException("Enum class not found", cfne);
    }

    String identifierMethodName = parameters.getProperty("identifierMethod", DEFAULT_IDENTIFIER_METHOD_NAME);

    try {
        identifierMethod = enumClass.getMethod(identifierMethodName, new Class[0]);
        identifierType = identifierMethod.getReturnType();
    } catch (Exception e) {
        throw new HibernateException("Failed to obtain identifier method", e);
    }

    type = (NullableType) TypeFactory.basic(identifierType.getName());

    if (type == null) {
        throw new HibernateException("Unsupported identifier type " + identifierType.getName());
    }

    sqlTypes = new int[] { type.sqlType() };

    String valueOfMethodName = parameters.getProperty("valueOfMethod", DEFAULT_VALUE_OF_METHOD_NAME);

    try {
        valueOfMethod = enumClass.getMethod(valueOfMethodName, new Class[] { identifierType });
    } catch (Exception e) {
        throw new HibernateException("Failed to obtain valueOf method", e);
    }
}

@Override
public Class returnedClass() {
    return enumClass;
}

@Override
public Object nullSafeGet(ResultSet rs, String[] names, Object owner) throws HibernateException, SQLException {
    Object identifier = type.get(rs, names[0]);
    if (rs.wasNull()) {
        return null;
    }

    try {
        return valueOfMethod.invoke(enumClass, new Object[] { identifier });
    } catch (Exception e) {
        throw new HibernateException("Exception while invoking " + "valueOf method " + valueOfMethod.getName()
                + " of enumeration class " + enumClass, e);
    }
}

@Override
public void nullSafeSet(PreparedStatement st, Object value, int index) throws HibernateException, SQLException {
    try {
        if (value == null) {
            st.setNull(index, type.sqlType());
        } else {
            Object identifier = identifierMethod.invoke(value, new Object[0]);
            type.set(st, identifier, index);
        }
    } catch (Exception e) {
        throw new HibernateException("Exception while invoking identifierMethod " + identifierMethod.getName()
                + " of enumeration class " + enumClass, e);
    }
}

@Override
public int[] sqlTypes() {
    return sqlTypes;
}

@Override
public Object assemble(Serializable cached, Object owner) throws HibernateException {
    return cached;
}

@Override
public Object deepCopy(Object value) throws HibernateException {
    return value;
}

@Override
public Serializable disassemble(Object value) throws HibernateException {
    return (Serializable) value;
}

@Override
public boolean equals(Object x, Object y) throws HibernateException {
    return x == y;
}

@Override
public int hashCode(Object x) throws HibernateException {
    return x.hashCode();
}

@Override
public boolean isMutable() {
    return false;
}

@Override
public Object replace(Object original, Object target, Object owner) throws HibernateException {
    return original;
}}

Enum class should be like this:

public enum AccountStatus {
ACTIVE(1), BLOCKED(2), DELETED(3);

private AccountStatus(int id) {
    this.id = id;
}

private int id;

public int getId() {
    return id;
}

public void setId(int id) {
    this.id = id;
}

public static AccountStatus valueOf(int id) {
    switch (id) {
    case 1:
        return ACTIVE;
    case 2:
        return BLOCKED;
    case 3:
        return DELETED;
    default:
        throw new IllegalArgumentException();
    }
}}

The static method "valueOf" is necessary to convert from id stored in database to java object.

Then, hibernate mapping is like this:

<hibernate-mapping>
<typedef class="path.to.GenericEnumUserType" name="accountStatusType">
    <param name="enumClassName">com.systemonenoc.hermes.ratingengine.persistence.constants.AccountStatus</param>
    <param name="identifierMethod">getId</param>
</typedef>
<class name="package.to.class.with.enum.Account" table="account" schema="public">
    <property name="accountStatus" type="accountStatusType" column="account_status" not-null="true" />
[...]
</hibernate-mapping>

So you have to declare as a type the class GenericEnumUserType with typedef, and a method to get the id of the enum (in this case, getId()). In your database wil be store the id as value in a integer column, and in java you will have the enum object.

Isthar
  • 433
  • 3
  • 14