From ad8bd5c3b55adc4c5052e850ad407cc04a70d246 Mon Sep 17 00:00:00 2001
From: "Brian S. O'Neill" <bronee@gmail.com>
Date: Sun, 21 Oct 2007 16:14:05 +0000
Subject: Assign number to properties.
---
.../amazon/carbonado/gen/StorableGenerator.java | 68 ++++++----------------
.../carbonado/info/StorableIntrospector.java | 24 +++++++-
.../amazon/carbonado/info/StorableProperty.java | 8 +++
.../repo/jdbc/JDBCStorableIntrospector.java | 4 ++
4 files changed, 52 insertions(+), 52 deletions(-)
(limited to 'src/main/java/com/amazon')
diff --git a/src/main/java/com/amazon/carbonado/gen/StorableGenerator.java b/src/main/java/com/amazon/carbonado/gen/StorableGenerator.java
index e28f07c..9db49ad 100644
--- a/src/main/java/com/amazon/carbonado/gen/StorableGenerator.java
+++ b/src/main/java/com/amazon/carbonado/gen/StorableGenerator.java
@@ -113,7 +113,7 @@ public final class StorableGenerator<S extends Storable> {
/**
* Name prefix of protected method in generated storable that returns false
* if a specific alternate key is uninitialized. The complete name is
- * formed by the prefix appended with the zero-based alternate key ordinal.
+ * formed by the prefix appended with the zero-based alternate key number.
*/
public static final String IS_ALT_KEY_INITIALIZED_PREFIX = "isAltKeyInitialized$";
@@ -146,10 +146,10 @@ public final class StorableGenerator<S extends Storable> {
*/
public static final String PROPERTY_STATE_FIELD_NAME = "propertyState$";
- /** Adapter field names are propertyName + "$adapter$" + ordinal */
+ /** Adapter field names are propertyName + "$adapter$" + number */
public static final String ADAPTER_FIELD_ELEMENT = "$adapter$";
- /** Constraint field names are propertyName + "$constraint$" + ordinal */
+ /** Constraint field names are propertyName + "$constraint$" + number */
public static final String CONSTRAINT_FIELD_ELEMENT = "$constraint$";
/** Reference to TriggerSupport instance */
@@ -278,8 +278,8 @@ public final class StorableGenerator<S extends Storable> {
*
* Property state field names are defined by the concatenation of
* {@code PROPERTY_STATE_FIELD_NAME} and a zero-based decimal
- * ordinal. To determine which field holds a particular property's state,
- * the field ordinal is computed as the property ordinal divided by 16. The
+ * number. To determine which field holds a particular property's state,
+ * the field number is computed as the property number divided by 16. The
* specific two-bit state position is the remainder of this division times 2.
*
* @throws com.amazon.carbonado.MalformedTypeException if Storable type is not well-formed
@@ -451,12 +451,11 @@ public final class StorableGenerator<S extends Storable> {
// Also remember ordinal of optional version property for use later.
int versionOrdinal = -1;
{
- int ordinal = -1;
int maxOrdinal = mAllProperties.size() - 1;
boolean requireStateField = false;
for (StorableProperty<S> property : mAllProperties.values()) {
- ordinal++;
+ int ordinal = property.getNumber();
if (!property.isDerived() && property.isVersion()) {
versionOrdinal = ordinal;
@@ -853,7 +852,7 @@ public final class StorableGenerator<S extends Storable> {
Label setInternalProp = b.createLabel();
// Access state of internal property directly.
- int ord = findPropertyOrdinal(internal);
+ int ord = internal.getNumber();
b.loadThis();
b.loadField(PROPERTY_STATE_FIELD_NAME + (ord >> 4), TypeDesc.INT);
b.loadConstant(PROPERTY_STATE_MASK << ((ord & 0xf) * 2));
@@ -1792,7 +1791,6 @@ public final class StorableGenerator<S extends Storable> {
LocalVariable target = CodeBuilderUtil.uneraseGenericParameter(b, storableTypeDesc, 0);
LocalVariable stateBits = null;
- int ordinal = 0;
int mask = PROPERTY_STATE_DIRTY;
for (StorableProperty property : mAllProperties.values()) {
@@ -1803,6 +1801,8 @@ public final class StorableGenerator<S extends Storable> {
!property.isPrimaryKeyMember() && dataProperties);
if (shouldCopy) {
+ int ordinal = property.getNumber();
+
if (stateBits == null) {
// Load state bits into local for quick retrieval.
stateBits = b.createLocalVariable(null, TypeDesc.INT);
@@ -1855,7 +1855,6 @@ public final class StorableGenerator<S extends Storable> {
skipCopy.setLocation();
}
- ordinal++;
if ((mask <<= 2) == 0) {
mask = 3;
stateBits = null;
@@ -2288,7 +2287,7 @@ public final class StorableGenerator<S extends Storable> {
}
if (properties.size() == 1) {
- int ordinal = findPropertyOrdinal(properties.values().iterator().next());
+ int ordinal = properties.values().iterator().next().getNumber();
b.loadThis();
b.loadField(PROPERTY_STATE_FIELD_NAME + (ordinal >> 4), TypeDesc.INT);
b.loadConstant(PROPERTY_STATE_MASK << ((ordinal & 0xf) * 2));
@@ -2349,18 +2348,6 @@ public final class StorableGenerator<S extends Storable> {
b.returnValue(TypeDesc.BOOLEAN);
}
- private int findPropertyOrdinal(StorableProperty<S> property) {
- int ordinal = 0;
- for (StorableProperty<S> p : mAllProperties.values()) {
- if (p == property) {
- return ordinal;
- }
- ordinal++;
- }
- throw new IllegalArgumentException
- ("Unable to find property " + property + " in " + mAllProperties);
- }
-
/**
* Generates code that verifies that all primary keys are initialized.
*
@@ -2443,15 +2430,6 @@ public final class StorableGenerator<S extends Storable> {
b.switchBranch(cases, switchLabels, noMatch);
- // Gather property ordinals.
- Map<StorableProperty<?>, Integer> ordinalMap = new HashMap<StorableProperty<?>, Integer>();
- {
- int ordinal = 0;
- for (StorableProperty<?> prop : mAllProperties.values()) {
- ordinalMap.put(prop, ordinal++);
- }
- }
-
// Params to invoke String.equals.
TypeDesc[] params = {TypeDesc.OBJECT};
@@ -2502,8 +2480,7 @@ public final class StorableGenerator<S extends Storable> {
}
b.branch(joinMatch);
} else {
- int ordinal = ordinalMap.get(prop);
-
+ int ordinal = prop.getNumber();
b.loadThis();
b.loadField(PROPERTY_STATE_FIELD_NAME + (ordinal >> 4), TypeDesc.INT);
int shift = (ordinal & 0xf) * 2;
@@ -2964,21 +2941,14 @@ public final class StorableGenerator<S extends Storable> {
b.loadConstant(detail);
invokeAppend(b, TypeDesc.STRING);
- // First pass, just print primary keys. Also gather ordinals.
+ // First pass, just print primary keys.
- Map<Object, Integer> ordinals = new IdentityHashMap<Object, Integer>();
- int ordinal = 0;
-
LocalVariable commaCountVar = b.createLocalVariable(null, TypeDesc.INT);
b.loadConstant(-1);
b.storeLocal(commaCountVar);
- for (StorableProperty property : mAllProperties.values()) {
- ordinals.put(property, ordinal);
- if (property.isPrimaryKeyMember()) {
- addPropertyAppendCall(b, property, commaCountVar, ordinal);
- }
- ordinal++;
+ for (StorableProperty property : mInfo.getPrimaryKeyProperties().values()) {
+ addPropertyAppendCall(b, property, commaCountVar);
}
// Second pass, print non-primary keys.
@@ -2989,7 +2959,7 @@ public final class StorableGenerator<S extends Storable> {
if (!property.isPrimaryKeyMember() &&
(!property.isDerived()) && (!property.isJoin()))
{
- addPropertyAppendCall(b, property, commaCountVar, ordinals.get(property));
+ addPropertyAppendCall(b, property, commaCountVar);
}
}
}
@@ -3010,10 +2980,9 @@ public final class StorableGenerator<S extends Storable> {
StorableKey<S> key = mInfo.getAlternateKey(i);
- ordinal = 0;
for (OrderedProperty<S> op : key.getProperties()) {
StorableProperty<S> property = op.getChainedProperty().getPrimeProperty();
- addPropertyAppendCall(b, property, commaCountVar, ordinals.get(property));
+ addPropertyAppendCall(b, property, commaCountVar);
}
b.loadConstant('}');
@@ -3032,8 +3001,7 @@ public final class StorableGenerator<S extends Storable> {
private void addPropertyAppendCall(CodeBuilder b,
StorableProperty property,
- LocalVariable commaCountVar,
- int ordinal)
+ LocalVariable commaCountVar)
{
Label skipPrint = b.createLabel();
@@ -3042,6 +3010,8 @@ public final class StorableGenerator<S extends Storable> {
addSkipIndependent(b, null, property, skipPrint);
}
+ int ordinal = property.getNumber();
+
// Check if property is initialized, and skip if not.
b.loadThis();
b.loadField(PROPERTY_STATE_FIELD_NAME + (ordinal >> 4), TypeDesc.INT);
diff --git a/src/main/java/com/amazon/carbonado/info/StorableIntrospector.java b/src/main/java/com/amazon/carbonado/info/StorableIntrospector.java
index b7f7744..12fbc01 100644
--- a/src/main/java/com/amazon/carbonado/info/StorableIntrospector.java
+++ b/src/main/java/com/amazon/carbonado/info/StorableIntrospector.java
@@ -228,9 +228,16 @@ public class StorableIntrospector {
// Now that the StorableInfo object has been constructed, assign it
// to all properties to prevent it from being prematurely uncached.
- for (StorableProperty property : properties.values()) {
- if (property instanceof SimpleProperty) {
- ((SimpleProperty)property).setEnclosingInfo(info);
+ // Also assign number now that properties have been sorted.
+ {
+ int number = 0;
+ for (StorableProperty property : properties.values()) {
+ if (property instanceof SimpleProperty) {
+ SimpleProperty sp = (SimpleProperty) property;
+ sp.setEnclosingInfo(info);
+ sp.setNumber(number);
+ }
+ number++;
}
}
@@ -1584,6 +1591,9 @@ public class StorableIntrospector {
// Resolved derived to properties.
private ChainedProperty<S>[] mDerivedTo;
+ // Resolved number.
+ private int mNumber = -1;
+
// Reference to enclosing StorableInfo. This reference exists to
// prevent the StorableInfo from being uncached so as long as a
// reference from a property exists.
@@ -1621,6 +1631,10 @@ public class StorableIntrospector {
return mBeanProperty.getType();
}
+ public final int getNumber() {
+ return mNumber;
+ }
+
public final Class<S> getEnclosingType() {
return mEnclosingType;
}
@@ -1838,6 +1852,10 @@ public class StorableIntrospector {
app.append('}');
}
+ void setNumber(int number) {
+ mNumber = number;
+ }
+
void setEnclosingInfo(StorableInfo<S> info) {
mEnclosingInfo = info;
}
diff --git a/src/main/java/com/amazon/carbonado/info/StorableProperty.java b/src/main/java/com/amazon/carbonado/info/StorableProperty.java
index e61c5e9..53b5098 100644
--- a/src/main/java/com/amazon/carbonado/info/StorableProperty.java
+++ b/src/main/java/com/amazon/carbonado/info/StorableProperty.java
@@ -39,6 +39,14 @@ public interface StorableProperty<S extends Storable> extends Appender {
*/
Class<?> getType();
+ /**
+ * Returns the zero-based numerical position of this property within its
+ * enclosing type.
+ *
+ * @since 1.2
+ */
+ int getNumber();
+
/**
* Returns the enclosing type of this property.
*/
diff --git a/src/main/java/com/amazon/carbonado/repo/jdbc/JDBCStorableIntrospector.java b/src/main/java/com/amazon/carbonado/repo/jdbc/JDBCStorableIntrospector.java
index f9fb6a3..a2d9181 100644
--- a/src/main/java/com/amazon/carbonado/repo/jdbc/JDBCStorableIntrospector.java
+++ b/src/main/java/com/amazon/carbonado/repo/jdbc/JDBCStorableIntrospector.java
@@ -1184,6 +1184,10 @@ public class JDBCStorableIntrospector extends StorableIntrospector {
return mMainProperty.getType();
}
+ public int getNumber() {
+ return mMainProperty.getNumber();
+ }
+
public Class<S> getEnclosingType() {
return mMainProperty.getEnclosingType();
}
--
cgit v1.2.3