Types: set the type field as a static final field 37/4637/1
Clément DAVID [Fri, 29 Jul 2011 10:17:14 +0000 (12:17 +0200)]
Change-Id: I6a5f7a07b034d3f7faf4d1c180ac3282e03fd34d

scilab/modules/types/src/java/org/scilab/modules/types/ScilabBoolean.java
scilab/modules/types/src/java/org/scilab/modules/types/ScilabDouble.java
scilab/modules/types/src/java/org/scilab/modules/types/ScilabInteger.java
scilab/modules/types/src/java/org/scilab/modules/types/ScilabList.java
scilab/modules/types/src/java/org/scilab/modules/types/ScilabMList.java
scilab/modules/types/src/java/org/scilab/modules/types/ScilabString.java
scilab/modules/types/src/java/org/scilab/modules/types/ScilabTList.java

index e9ee4e9..b79bec1 100644 (file)
@@ -28,12 +28,11 @@ import java.util.Arrays;
 public class ScilabBoolean implements ScilabType {
 
        private static final long serialVersionUID = 6511497080095473901L;
+       private static final ScilabTypeEnum type = ScilabTypeEnum.sci_boolean;
 
        /* the boolean data */
        private boolean[][] data;
 
-       private ScilabTypeEnum type = ScilabTypeEnum.sci_boolean;
-
        /**
         * Create an empty object
         */
@@ -83,7 +82,8 @@ public class ScilabBoolean implements ScilabType {
      * @return the type of Scilab
      * @since 5.4.0
      */
-    public ScilabTypeEnum getType() {
+    @Override
+       public ScilabTypeEnum getType() {
         return type;
     }
 
@@ -92,6 +92,7 @@ public class ScilabBoolean implements ScilabType {
         *
         * @return the height
         */
+       @Override
        public int getHeight() {
                if (data == null) {
                        return 0;
@@ -104,6 +105,7 @@ public class ScilabBoolean implements ScilabType {
         *
         * @return the width
         */
+       @Override
        public int getWidth() {
                if (data == null) {
                        return 0;
@@ -116,6 +118,7 @@ public class ScilabBoolean implements ScilabType {
         *
         * @return true if empty
         */
+       @Override
        public boolean isEmpty() {
                return (data == null);
        }
@@ -124,6 +127,7 @@ public class ScilabBoolean implements ScilabType {
        /**
         * @see org.scilab.modules.types.ScilabType#equals(Object)
         */
+       @Override
        public boolean equals(Object obj) {
                if (obj instanceof ScilabBoolean) {
                        return Arrays.deepEquals(this.getData(), ((ScilabBoolean)obj).getData());
@@ -138,6 +142,7 @@ public class ScilabBoolean implements ScilabType {
      *
         * @return the pretty print
         */
+       @Override
        public String toString() {
                StringBuilder result = new StringBuilder();
                
index 0e155ce..85d2d8e 100644 (file)
@@ -37,9 +37,10 @@ import java.util.Arrays;
 public class ScilabDouble implements ScilabType {
 
        private static final long serialVersionUID = 879624048944109684L;
+       private static final ScilabTypeEnum type = ScilabTypeEnum.sci_matrix;
+       
        private double[][] realPart;
        private double[][] imaginaryPart;
-       private ScilabTypeEnum type = ScilabTypeEnum.sci_matrix;
 
        /**
         * Default constructor
@@ -98,7 +99,8 @@ public class ScilabDouble implements ScilabType {
      * @return the type of Scilab
      * @since 5.4.0
      */
-    public ScilabTypeEnum getType() {
+    @Override
+       public ScilabTypeEnum getType() {
         return type;
     }
 
@@ -106,6 +108,7 @@ public class ScilabDouble implements ScilabType {
         * Check the emptiness of the associated data.
         * @return true, if the associated data array is empty. 
         */
+       @Override
        public boolean isEmpty() {
                return (realPart == null && imaginaryPart == null);
        }
@@ -203,6 +206,7 @@ public class ScilabDouble implements ScilabType {
        /**
         * @see org.scilab.modules.types.ScilabType#equals(Object)
         */
+       @Override
        public boolean equals(Object obj) {
                if (obj instanceof ScilabDouble) {
                        ScilabDouble sciDouble = ((ScilabDouble)obj);
index fefac10..a284a2f 100644 (file)
@@ -27,13 +27,13 @@ import java.util.Arrays;
 public class ScilabInteger implements ScilabType {
 
        private static final long serialVersionUID = 1759633801332932450L;
+       private static final ScilabTypeEnum type = ScilabTypeEnum.sci_ints;
 
        private long[][] longData = null;
        private short[][] shortData = null;
        private int[][] intData = null;
        private byte[][] byteData = null;
        private ScilabIntegerTypeEnum precision;
-       private ScilabTypeEnum type = ScilabTypeEnum.sci_ints;
        
        /**
         * Default constructor
@@ -191,7 +191,8 @@ public class ScilabInteger implements ScilabType {
      * @return the type of Scilab
      * @since 5.4.0
      */
-    public ScilabTypeEnum getType() {
+    @Override
+       public ScilabTypeEnum getType() {
         return type;
     }
 
@@ -421,6 +422,7 @@ public class ScilabInteger implements ScilabType {
        /**
         * @return true, if there is no values; false otherwise.
         */
+       @Override
        public boolean isEmpty() {
                if (this.getPrec() == null) {
                        return true;
@@ -446,6 +448,7 @@ public class ScilabInteger implements ScilabType {
        /**
         * @see org.scilab.modules.types.ScilabType#equals(Object)
         */
+       @Override
        public boolean equals(Object obj) {
                if (obj instanceof ScilabInteger) {
                        return Arrays.deepEquals(this.getData(), ((ScilabInteger)obj).getData());
index 4f66b46..571de87 100644 (file)
@@ -29,7 +29,7 @@ import java.util.Collection;
 public class ScilabList extends ArrayList<ScilabType> implements ScilabType {
        
        private static final long serialVersionUID = 6884293176289980909L;
-       private ScilabTypeEnum type = ScilabTypeEnum.sci_list;
+       private static final ScilabTypeEnum type = ScilabTypeEnum.sci_list;
 
        /**
         * Construct an empty Scilab list.
@@ -66,7 +66,8 @@ public class ScilabList extends ArrayList<ScilabType> implements ScilabType {
      * @return the type of Scilab
      * @since 5.4.0
      */
-    public ScilabTypeEnum getType() {
+    @Override
+       public ScilabTypeEnum getType() {
         return type;
     }
 
index 1b3b7ac..59ed9c6 100644 (file)
@@ -29,7 +29,7 @@ import java.util.Collection;
 public class ScilabMList extends ArrayList<ScilabType> implements ScilabType {
    
        private static final long serialVersionUID = 3224510024213901841L;
-       private ScilabTypeEnum type = ScilabTypeEnum.sci_mlist;
+       private static final ScilabTypeEnum type = ScilabTypeEnum.sci_mlist;
 
        /**
         * Construct an empty mlist.
@@ -90,7 +90,8 @@ public class ScilabMList extends ArrayList<ScilabType> implements ScilabType {
      * @return the type of Scilab
      * @since 5.4.0
      */
-    public ScilabTypeEnum getType() {
+    @Override
+       public ScilabTypeEnum getType() {
         return type;
     }
 
index f7c6f3a..74b7702 100644 (file)
@@ -27,10 +27,10 @@ import java.util.Arrays;
 public class ScilabString implements ScilabType {
 
        private static final long serialVersionUID = 359802519980180085L;
+       private static final ScilabTypeEnum type = ScilabTypeEnum.sci_strings;
 
        private String[][] data;
 
-       private ScilabTypeEnum type = ScilabTypeEnum.sci_strings;
        /**
         * Default constructor
         */
@@ -91,7 +91,8 @@ public class ScilabString implements ScilabType {
      * @return the type of Scilab
      * @since 5.4.0
      */
-    public ScilabTypeEnum getType() {
+    @Override
+       public ScilabTypeEnum getType() {
         return type;
     }
 
@@ -106,6 +107,7 @@ public class ScilabString implements ScilabType {
         * @return the height of the data matrix
         * @see org.scilab.modules.types.ScilabType#getHeight()
         */
+       @Override
        public int getHeight() {
                if (data == null) {
                        return 0;
@@ -117,6 +119,7 @@ public class ScilabString implements ScilabType {
         * @return the width of the data matrix
         * @see org.scilab.modules.types.ScilabType#getWidth()
         */
+       @Override
        public int getWidth() {
                if (data == null) {
                        return 0;
@@ -128,6 +131,7 @@ public class ScilabString implements ScilabType {
         * Check the emptiness of the associated data.
         * @return true, if the associated data array is empty. 
         */
+       @Override
        public boolean isEmpty() {
                return (data == null);
        }
@@ -135,6 +139,7 @@ public class ScilabString implements ScilabType {
        /**
         * @see org.scilab.modules.types.ScilabType#equals(Object)
         */
+       @Override
        public boolean equals(Object obj) {
                if (obj instanceof ScilabString) {
                        return Arrays.deepEquals(this.getData(), ((ScilabString)obj).getData());
@@ -150,6 +155,7 @@ public class ScilabString implements ScilabType {
         * @return a Scilab-like String representation of the data.
         * @see java.lang.Object#toString()
         */
+       @Override
        public String toString() {
                StringBuffer result = new StringBuffer();
                if (isEmpty()) {
index 2e427e6..77de07b 100644 (file)
@@ -29,8 +29,7 @@ import java.util.Collection;
 public class ScilabTList extends ArrayList<ScilabType> implements ScilabType {
 
        private static final long serialVersionUID = 8080160982092586620L;
-
-       private ScilabTypeEnum type = ScilabTypeEnum.sci_tlist;
+       private static final ScilabTypeEnum type = ScilabTypeEnum.sci_tlist;
 
        /**
         * Construct an empty tlist.
@@ -79,7 +78,8 @@ public class ScilabTList extends ArrayList<ScilabType> implements ScilabType {
      * @return the type of Scilab
      * @since 5.4.0
      */
-    public ScilabTypeEnum getType() {
+    @Override
+       public ScilabTypeEnum getType() {
         return type;
     }