De-localize Material and FlightDataType objects when persisting in ork files. Added...
[debian/openrocket] / core / src / net / sf / openrocket / preset / ComponentPresetFactory.java
index 5cf7f62e3a4560650d98b1c909f54a31821e8c57..7c4b2b9a0fac3f807b58e145458d31dce82cf94f 100644 (file)
@@ -1,20 +1,8 @@
 package net.sf.openrocket.preset;
 
-import static net.sf.openrocket.preset.ComponentPreset.FORE_OUTER_DIAMETER;
-import static net.sf.openrocket.preset.ComponentPreset.INNER_DIAMETER;
-import static net.sf.openrocket.preset.ComponentPreset.LENGTH;
-import static net.sf.openrocket.preset.ComponentPreset.MANUFACTURER;
-import static net.sf.openrocket.preset.ComponentPreset.MASS;
-import static net.sf.openrocket.preset.ComponentPreset.MATERIAL;
-import static net.sf.openrocket.preset.ComponentPreset.OUTER_DIAMETER;
-import static net.sf.openrocket.preset.ComponentPreset.PARTNO;
-import static net.sf.openrocket.preset.ComponentPreset.SHAPE;
-import static net.sf.openrocket.preset.ComponentPreset.THICKNESS;
-import static net.sf.openrocket.preset.ComponentPreset.TYPE;
+import static net.sf.openrocket.preset.ComponentPreset.*;
 import net.sf.openrocket.material.Material;
 import net.sf.openrocket.preset.ComponentPreset.Type;
-import net.sf.openrocket.rocketcomponent.BodyTube;
-import net.sf.openrocket.rocketcomponent.Bulkhead;
 import net.sf.openrocket.rocketcomponent.NoseCone;
 import net.sf.openrocket.rocketcomponent.Transition;
 
@@ -22,66 +10,87 @@ public abstract class ComponentPresetFactory {
 
        public static ComponentPreset create( TypedPropertyMap props ) throws InvalidComponentPresetException {
 
+               InvalidComponentPresetException exceptions = new InvalidComponentPresetException("Invalid preset specification.");
+               
                ComponentPreset preset = new ComponentPreset();
                // First do validation.
-               if ( !props.containsKey(TYPE)) {
-                       throw new InvalidComponentPresetException("No Type specified " + props.toString() );
-               }
-
                if (!props.containsKey(MANUFACTURER)) {
-                       throw new InvalidComponentPresetException("No Manufacturer specified " + props.toString() );
+                       exceptions.addInvalidParameter(MANUFACTURER, "No Manufacturer specified"); 
                }
-
                if (!props.containsKey(PARTNO)) {
-                       throw new InvalidComponentPresetException("No PartNo specified " + props.toString() );
+                       exceptions.addInvalidParameter(PARTNO,"No PartNo specified");
+               }
+               if ( !props.containsKey(TYPE)) {
+                       exceptions.addInvalidParameter(TYPE, "No Type specified" );
+                       // We can't do anything else without TYPE so throw immediately.
+                       throw exceptions;
                }
 
+
                preset.putAll(props);
 
                // Should check for various bits of each of the types.
                Type t = props.get(TYPE);
                switch ( t ) {
                case BODY_TUBE: {
-                       makeBodyTube(preset);
+                       makeBodyTube(exceptions,preset);
                        break;
                }
                case NOSE_CONE: {
-                       makeNoseCone(preset);
+                       makeNoseCone(exceptions,preset);
                        break;
                }
                case TRANSITION: {
-                       makeTransition(preset);
+                       makeTransition(exceptions,preset);
                        break;
                }
                case BULK_HEAD: {
-                       makeBulkHead(preset);
+                       makeBulkHead(exceptions,preset);
                        break;
                }
                case TUBE_COUPLER: {
-                       makeTubeCoupler(preset);
+                       // For now TUBE_COUPLER is the same as BODY_TUBE
+                       makeBodyTube(exceptions,preset);
                        break;
                }
                case CENTERING_RING: {
-                       makeCenteringRing(preset);
+                       makeCenteringRing(exceptions,preset);
                        break;
                }
                case ENGINE_BLOCK: {
-                       makeEngineBlock(preset);
+                       makeEngineBlock(exceptions,preset);
+                       break;
+               }
+               case LAUNCH_LUG: {
+                       // Same processing as BODY_TUBE
+                       makeBodyTube(exceptions,preset);
+                       break;
+               }
+               case STREAMER: {
+                       makeStreamer(exceptions,preset);
+                       break;
+               }
+               case PARACHUTE: {
+                       makeParachute(exceptions,preset);
                        break;
                }
                }
 
+               if ( exceptions.hasProblems() ) {
+                       throw exceptions;
+               }
+               
                preset.computeDigest();
 
                return preset;
 
        }
 
-       private static void makeBodyTube( ComponentPreset preset ) throws InvalidComponentPresetException {
+       private static void makeBodyTube( InvalidComponentPresetException exceptions, ComponentPreset preset ) throws InvalidComponentPresetException {
                
-               checkRequiredFields( preset, LENGTH );
+               checkRequiredFields( exceptions, preset, LENGTH );
 
-               checkDiametersAndThickness(preset);
+               checkDiametersAndThickness(exceptions, preset);
                
                double volume = computeVolumeOfTube( preset );
                
@@ -91,16 +100,16 @@ public abstract class ComponentPresetFactory {
                        if ( preset.has(MATERIAL) ) {
                                materialName = preset.get(MATERIAL).getName();
                        }
-                       Material m = Material.newMaterial(Material.Type.BULK, materialName, preset.get(MASS)/volume, false);
+                       Material m = Material.newUserMaterial(Material.Type.BULK, materialName, preset.get(MASS)/volume);
                        preset.put(MATERIAL, m);
                }
 
 
        }
 
-       private static void makeNoseCone( ComponentPreset preset ) throws InvalidComponentPresetException {
+       private static void makeNoseCone( InvalidComponentPresetException exceptions, ComponentPreset preset ) {
 
-               checkRequiredFields( preset, LENGTH, SHAPE, OUTER_DIAMETER );
+               checkRequiredFields( exceptions, preset, LENGTH, SHAPE, AFT_OUTER_DIAMETER );
 
                if ( preset.has(MASS) ) {
                        // compute a density for this component
@@ -114,15 +123,15 @@ public abstract class ComponentPresetFactory {
                                materialName = preset.get(MATERIAL).getName();
                        }
 
-                       Material m = Material.newMaterial(Material.Type.BULK, materialName,density, false);
+                       Material m = Material.newUserMaterial(Material.Type.BULK, materialName,density);
                        preset.put(MATERIAL, m);
 
                }
 
        }
 
-       private static void makeTransition( ComponentPreset preset ) throws InvalidComponentPresetException {
-               checkRequiredFields(preset, LENGTH, OUTER_DIAMETER, FORE_OUTER_DIAMETER);
+       private static void makeTransition( InvalidComponentPresetException exceptions, ComponentPreset preset ) {
+               checkRequiredFields(exceptions, preset, LENGTH, AFT_OUTER_DIAMETER, FORE_OUTER_DIAMETER);
 
                if ( preset.has(MASS) ) {
                        // compute a density for this component
@@ -136,21 +145,19 @@ public abstract class ComponentPresetFactory {
                                materialName = preset.get(MATERIAL).getName();
                        }
 
-                       Material m = Material.newMaterial(Material.Type.BULK, materialName,density, false);
+                       Material m = Material.newUserMaterial(Material.Type.BULK, materialName,density);
                        preset.put(MATERIAL, m);
 
                }
 
        }
 
-       private static void makeBulkHead( ComponentPreset preset ) throws InvalidComponentPresetException {
-               checkRequiredFields(preset, LENGTH, OUTER_DIAMETER );
+       private static void makeBulkHead( InvalidComponentPresetException exceptions, ComponentPreset preset ) {
+               checkRequiredFields(exceptions, preset, LENGTH, OUTER_DIAMETER );
 
                if ( preset.has(MASS) ) {
                        // compute a density for this component
                        double mass = preset.get(MASS);
-                       // FIXME - Bulkhead.getComponentVolume does not exist!
-                       // double density = mass / tr.getComponentVolume();
 
                        double volume = computeVolumeOfTube(preset);
                        double density = mass / volume;
@@ -160,79 +167,69 @@ public abstract class ComponentPresetFactory {
                                materialName = preset.get(MATERIAL).getName();
                        }
 
-                       Material m = Material.newMaterial(Material.Type.BULK, materialName,density, false);
+                       Material m = Material.newUserMaterial(Material.Type.BULK, materialName,density);
                        preset.put(MATERIAL, m);
 
                }
 
        }
 
-       private static void makeTubeCoupler( ComponentPreset preset ) throws InvalidComponentPresetException {
-               
-               checkRequiredFields( preset, LENGTH );
+       private static void makeCenteringRing( InvalidComponentPresetException exceptions, ComponentPreset preset ) throws InvalidComponentPresetException {
+               checkRequiredFields( exceptions, preset, LENGTH );
 
-               checkDiametersAndThickness( preset );
-               
-               double volume = computeVolumeOfTube( preset );
-
-               // Need to translate Mass to Density.
-               if ( preset.has(MASS) ) {
-                       String materialName = "TubeCouplerCustom";
-                       if ( preset.has(MATERIAL) ) {
-                               materialName = preset.get(MATERIAL).getName();
-                       }
-                       Material m = Material.newMaterial(Material.Type.BULK, materialName, preset.get(MASS)/volume, false);
-                       preset.put(MATERIAL, m);
-               }
-       }
-
-       private static void makeCenteringRing( ComponentPreset preset ) throws InvalidComponentPresetException {
-               checkRequiredFields( preset, LENGTH );
-
-               checkDiametersAndThickness( preset );
+               checkDiametersAndThickness( exceptions, preset );
 
                double volume = computeVolumeOfTube( preset );
 
                // Need to translate Mass to Density.
                if ( preset.has(MASS) ) {
-                       String materialName = "TubeCouplerCustom";
+                       String materialName = "CenteringRingCustom";
                        if ( preset.has(MATERIAL) ) {
                                materialName = preset.get(MATERIAL).getName();
                        }
-                       Material m = Material.newMaterial(Material.Type.BULK, materialName, preset.get(MASS)/volume, false);
+                       Material m = Material.newUserMaterial(Material.Type.BULK, materialName, preset.get(MASS)/volume);
                        preset.put(MATERIAL, m);
                }
 
        }
        
-       private static void makeEngineBlock( ComponentPreset preset ) throws InvalidComponentPresetException {
-               checkRequiredFields( preset, LENGTH );
+       private static void makeEngineBlock( InvalidComponentPresetException exceptions, ComponentPreset preset ) throws InvalidComponentPresetException {
+               checkRequiredFields( exceptions, preset, LENGTH );
 
-               checkDiametersAndThickness( preset );
+               checkDiametersAndThickness( exceptions, preset );
 
                double volume = computeVolumeOfTube( preset );
 
                // Need to translate Mass to Density.
                if ( preset.has(MASS) ) {
-                       String materialName = "TubeCouplerCustom";
+                       String materialName = "EngineBlockCustom";
                        if ( preset.has(MATERIAL) ) {
                                materialName = preset.get(MATERIAL).getName();
                        }
-                       Material m = Material.newMaterial(Material.Type.BULK, materialName, preset.get(MASS)/volume, false);
+                       Material m = Material.newUserMaterial(Material.Type.BULK, materialName, preset.get(MASS)/volume);
                        preset.put(MATERIAL, m);
                }
 
        }
        
-       private static void checkRequiredFields( ComponentPreset preset, TypedKey<?> ... keys ) throws InvalidComponentPresetException {
+       private static void makeStreamer( InvalidComponentPresetException exceptions, ComponentPreset preset ) {
+               checkRequiredFields( exceptions, preset, LENGTH, WIDTH );
+       }
+
+       private static void makeParachute( InvalidComponentPresetException exceptions, ComponentPreset preset ) {
+               checkRequiredFields( exceptions, preset, DIAMETER, LINE_COUNT, LINE_LENGTH );
+       }
+       
+
+       private static void checkRequiredFields( InvalidComponentPresetException exceptions, ComponentPreset preset, TypedKey<?> ... keys ) {
                for( TypedKey<?> key: keys ) {
                        if (! preset.has(key) ) {
-                               throw new InvalidComponentPresetException( "No " + key.getName() + " specified for " + preset.getType().name() + " preset " + preset.toString());
+                               exceptions.addInvalidParameter(key, "No " + key.getName() + " specified");
                        }
                }
        }
 
-       private static void checkDiametersAndThickness( ComponentPreset preset ) throws InvalidComponentPresetException {
+       private static void checkDiametersAndThickness( InvalidComponentPresetException exceptions, ComponentPreset preset ) throws InvalidComponentPresetException {
                // Need to verify contains 2 of OD, thickness, ID.  Compute the third.
                boolean hasOd = preset.has(OUTER_DIAMETER);
                boolean hasId = preset.has(INNER_DIAMETER);
@@ -252,11 +249,13 @@ public abstract class ComponentPresetFactory {
                                thickness = preset.get(THICKNESS);
                                innerRadius = outerRadius - thickness;
                        } else {
-                               throw new InvalidComponentPresetException("Preset underspecified " + preset.toString());
+                               exceptions.addMessage("Preset dimensions underspecified");
+                               throw exceptions;
                        }
                } else {
-                       if ( ! hasId && ! hasThickness ) {
-                               throw new InvalidComponentPresetException("Preset underspecified " + preset.toString());
+                       if ( ! hasId || ! hasThickness ) {
+                               exceptions.addMessage("Preset dimensions underspecified");
+                               throw exceptions;
                        }
                        innerRadius = preset.get(INNER_DIAMETER)/2.0;
                        thickness = preset.get(THICKNESS);