From 75d63553c3eb0dad7a62fe08104ddb1825d17b26 Mon Sep 17 00:00:00 2001 From: Steve Ebersole Date: Wed, 3 Oct 2007 02:25:50 +0000 Subject: [PATCH] general code cleanup git-svn-id: https://svn.jboss.org/repos/hibernate/core/trunk@14043 1b8cb986-b30d-0410-93ca-fae66ebed9b2 --- .../id/IdentifierGeneratorFactory.java | 149 ++++++++++-------- 1 file changed, 86 insertions(+), 63 deletions(-) diff --git a/core/src/main/java/org/hibernate/id/IdentifierGeneratorFactory.java b/core/src/main/java/org/hibernate/id/IdentifierGeneratorFactory.java index 08ff96a7ed..07032bd190 100644 --- a/core/src/main/java/org/hibernate/id/IdentifierGeneratorFactory.java +++ b/core/src/main/java/org/hibernate/id/IdentifierGeneratorFactory.java @@ -9,6 +9,7 @@ import java.util.Properties; import org.slf4j.Logger; import org.slf4j.LoggerFactory; + import org.hibernate.HibernateException; import org.hibernate.MappingException; import org.hibernate.dialect.Dialect; @@ -21,111 +22,133 @@ import org.hibernate.util.ReflectHelper; * @author Gavin King */ public final class IdentifierGeneratorFactory { - - private static final Logger log = LoggerFactory.getLogger(IdentifierGeneratorFactory.class); + + private static final Logger log = LoggerFactory.getLogger( 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(); - try { - if (clazz==null) clazz = ReflectHelper.classForName(strategy); + Class clazz = ( Class ) GENERATORS.get( strategy ); + if ( "native".equals( strategy ) ) { + clazz = dialect.getNativeIdentifierGeneratorClass(); } - catch (ClassNotFoundException e) { - throw new MappingException("could not interpret id generator strategy: " + strategy); + try { + if ( clazz == null ) { + clazz = ReflectHelper.classForName( 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() { + } }