[hibernate-commits] Hibernate SVN: r14042 - core/branches/Branch_3_2/src/org/hibernate/id.
hibernate-commits at lists.jboss.org
hibernate-commits at lists.jboss.org
Tue Oct 2 22:25:34 EDT 2007
Author: steve.ebersole at jboss.com
Date: 2007-10-02 22:25:34 -0400 (Tue, 02 Oct 2007)
New Revision: 14042
Modified:
core/branches/Branch_3_2/src/org/hibernate/id/IdentifierGeneratorFactory.java
Log:
general code cleanup
Modified: core/branches/Branch_3_2/src/org/hibernate/id/IdentifierGeneratorFactory.java
===================================================================
--- core/branches/Branch_3_2/src/org/hibernate/id/IdentifierGeneratorFactory.java 2007-10-03 02:12:41 UTC (rev 14041)
+++ core/branches/Branch_3_2/src/org/hibernate/id/IdentifierGeneratorFactory.java 2007-10-03 02:25:34 UTC (rev 14042)
@@ -9,6 +9,7 @@
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
+
import org.hibernate.HibernateException;
import org.hibernate.MappingException;
import org.hibernate.dialect.Dialect;
@@ -21,111 +22,133 @@
* @author Gavin King
*/
public final class IdentifierGeneratorFactory {
-
- private static final Log log = LogFactory.getLog(IdentifierGeneratorFactory.class);
+ private static final Log log = LogFactory.getLog( IdentifierGeneratorFactory.class );
+
+ private static final HashMap GENERATORS = new HashMap();
+ static {
+ GENERATORS.put( "uuid", UUIDHexGenerator.class );
+ GENERATORS.put( "hilo", TableHiLoGenerator.class );
+ GENERATORS.put( "assigned", Assigned.class );
+ GENERATORS.put( "identity", IdentityGenerator.class );
+ GENERATORS.put( "select", SelectGenerator.class );
+ GENERATORS.put( "sequence", SequenceGenerator.class );
+ GENERATORS.put( "seqhilo", SequenceHiLoGenerator.class );
+ GENERATORS.put( "increment", IncrementGenerator.class );
+ GENERATORS.put( "foreign", ForeignGenerator.class );
+ GENERATORS.put( "guid", GUIDGenerator.class );
+ GENERATORS.put( "uuid.hex", UUIDHexGenerator.class ); // uuid.hex is deprecated
+ GENERATORS.put( "sequence-identity", SequenceIdentityGenerator.class );
+ }
+
+ public static final Serializable SHORT_CIRCUIT_INDICATOR = new Serializable() {
+ public String toString() {
+ return "SHORT_CIRCUIT_INDICATOR";
+ }
+ };
+
+ public static final Serializable POST_INSERT_INDICATOR = new Serializable() {
+ public String toString() {
+ return "POST_INSERT_INDICATOR";
+ }
+ };
+
/**
* Get the generated identifier when using identity columns
+ *
+ * @param rs The result set from which to extract the the generated identity.
+ * @param type The expected type mapping for the identity value.
+ * @return The generated identity value
+ * @throws SQLException Can be thrown while accessing the result set
+ * @throws HibernateException Indicates a problem reading back a generated identity value.
*/
- public static Serializable getGeneratedIdentity(ResultSet rs, Type type)
- throws SQLException, HibernateException, IdentifierGenerationException {
+ public static Serializable getGeneratedIdentity(ResultSet rs, Type type) throws SQLException, HibernateException {
if ( !rs.next() ) {
throw new HibernateException( "The database returned no natively generated identity value" );
}
final Serializable id = IdentifierGeneratorFactory.get( rs, type );
- if ( log.isDebugEnabled() ) log.debug( "Natively generated identity: " + id );
+ if ( log.isDebugEnabled() ) {
+ log.debug( "Natively generated identity: " + id );
+ }
return id;
}
// unhappy about this being public ... is there a better way?
- public static Serializable get(ResultSet rs, Type type)
- throws SQLException, IdentifierGenerationException {
-
+ public static Serializable get(ResultSet rs, Type type) throws SQLException, IdentifierGenerationException {
Class clazz = type.getReturnedClass();
- if ( clazz==Long.class ) {
- return new Long( rs.getLong(1) );
+ if ( clazz == Long.class ) {
+ return new Long( rs.getLong( 1 ) );
}
- else if ( clazz==Integer.class ) {
- return new Integer( rs.getInt(1) );
+ else if ( clazz == Integer.class ) {
+ return new Integer( rs.getInt( 1 ) );
}
- else if ( clazz==Short.class ) {
- return new Short( rs.getShort(1) );
+ else if ( clazz == Short.class ) {
+ return new Short( rs.getShort( 1 ) );
}
- else if ( clazz==String.class ) {
- return rs.getString(1);
+ else if ( clazz == String.class ) {
+ return rs.getString( 1 );
}
else {
- throw new IdentifierGenerationException("this id generator generates long, integer, short or string");
+ throw new IdentifierGenerationException( "this id generator generates long, integer, short or string" );
}
-
- }
- private static final HashMap GENERATORS = new HashMap();
-
- public static final Serializable SHORT_CIRCUIT_INDICATOR = new Serializable() {
- public String toString() { return "SHORT_CIRCUIT_INDICATOR"; }
- };
-
- public static final Serializable POST_INSERT_INDICATOR = new Serializable() {
- public String toString() { return "POST_INSERT_INDICATOR"; }
- };
-
- static {
- GENERATORS.put("uuid", UUIDHexGenerator.class);
- GENERATORS.put("hilo", TableHiLoGenerator.class);
- GENERATORS.put("assigned", Assigned.class);
- GENERATORS.put("identity", IdentityGenerator.class);
- GENERATORS.put("select", SelectGenerator.class);
- GENERATORS.put("sequence", SequenceGenerator.class);
- GENERATORS.put("seqhilo", SequenceHiLoGenerator.class);
- GENERATORS.put("increment", IncrementGenerator.class);
- GENERATORS.put("foreign", ForeignGenerator.class);
- GENERATORS.put("guid", GUIDGenerator.class);
- GENERATORS.put("uuid.hex", UUIDHexGenerator.class); //uuid.hex is deprecated
- GENERATORS.put("sequence-identity", SequenceIdentityGenerator.class);
}
- public static IdentifierGenerator create(String strategy, Type type, Properties params, Dialect dialect)
- throws MappingException {
+ public static IdentifierGenerator create(String strategy, Type type, Properties params, Dialect dialect)
+ throws MappingException {
try {
Class clazz = getIdentifierGeneratorClass( strategy, dialect );
- IdentifierGenerator idgen = (IdentifierGenerator) clazz.newInstance();
- if (idgen instanceof Configurable) ( (Configurable) idgen).configure(type, params, dialect);
+ IdentifierGenerator idgen = ( IdentifierGenerator ) clazz.newInstance();
+ if ( idgen instanceof Configurable ) {
+ ( ( Configurable ) idgen ).configure( type, params, dialect );
+ }
return idgen;
}
- catch (Exception e) {
- throw new MappingException("could not instantiate id generator [entity-name=" + params.get( IdentifierGenerator.ENTITY_NAME ) + "]", e);
+ catch ( Exception e ) {
+ throw new MappingException(
+ "could not instantiate id generator [entity-name=" + params.get(
+ IdentifierGenerator.ENTITY_NAME
+ ) + "]", e
+ );
}
}
public static Class getIdentifierGeneratorClass(String strategy, Dialect dialect) {
- Class clazz = (Class) GENERATORS.get(strategy);
- if ( "native".equals(strategy) ) clazz = dialect.getNativeIdentifierGeneratorClass();
+ Class clazz = ( Class ) GENERATORS.get( strategy );
+ if ( "native".equals( strategy ) ) {
+ clazz = dialect.getNativeIdentifierGeneratorClass();
+ }
try {
- if (clazz==null) clazz = ReflectHelper.classForName(strategy);
+ if ( clazz == null ) {
+ clazz = ReflectHelper.classForName( strategy );
+ }
}
- catch (ClassNotFoundException e) {
- throw new MappingException("could not interpret id generator strategy: " + strategy);
+ catch ( ClassNotFoundException e ) {
+ throw new MappingException( "could not interpret id generator strategy: " + strategy );
}
return clazz;
}
public static Number createNumber(long value, Class clazz) throws IdentifierGenerationException {
- if ( clazz==Long.class ) {
- return new Long(value);
+ if ( clazz == Long.class ) {
+ return new Long( value );
}
- else if ( clazz==Integer.class ) {
- return new Integer( (int) value );
+ else if ( clazz == Integer.class ) {
+ return new Integer( ( int ) value );
}
- else if ( clazz==Short.class ) {
- return new Short( (short) value );
+ else if ( clazz == Short.class ) {
+ return new Short( ( short ) value );
}
else {
- throw new IdentifierGenerationException("this id generator generates long, integer, short");
+ throw new IdentifierGenerationException( "this id generator generates long, integer, short" );
}
}
- private IdentifierGeneratorFactory() {} //cannot be instantiated
+ /**
+ * Disallow instantiation.
+ */
+ private IdentifierGeneratorFactory() {
+ }
}
More information about the hibernate-commits
mailing list