Author: tchemit Date: 2009-10-16 19:17:29 +0200 (Fri, 16 Oct 2009) New Revision: 187 Modified: trunk/src/main/java/org/nuiton/math/matrix/AbstractMatrixND.java trunk/src/main/java/org/nuiton/math/matrix/DoubleBigVector.java trunk/src/main/java/org/nuiton/math/matrix/DoubleVector.java trunk/src/main/java/org/nuiton/math/matrix/FloatBigVector.java trunk/src/main/java/org/nuiton/math/matrix/FloatVector.java trunk/src/main/java/org/nuiton/math/matrix/MapFunction.java trunk/src/main/java/org/nuiton/math/matrix/MatrixEncoder.java trunk/src/main/java/org/nuiton/math/matrix/MatrixFactory.java trunk/src/main/java/org/nuiton/math/matrix/MatrixHelper.java trunk/src/main/java/org/nuiton/math/matrix/MatrixIterator.java trunk/src/main/java/org/nuiton/math/matrix/MatrixIteratorImpl.java trunk/src/main/java/org/nuiton/math/matrix/MatrixND.java trunk/src/main/java/org/nuiton/math/matrix/MatrixNDImpl.java trunk/src/main/java/org/nuiton/math/matrix/MatrixStringEncoder.java trunk/src/main/java/org/nuiton/math/matrix/SemanticList.java trunk/src/main/java/org/nuiton/math/matrix/SubMatrix.java trunk/src/main/java/org/nuiton/math/matrix/gui/MatrixPanelEditor.java trunk/src/main/java/org/nuiton/math/matrix/gui/MatrixPopupMenu.java trunk/src/test/java/org/nuiton/math/matrix/gui/MatrixTableModelTest.java Log: - fix some generics issues - improve code (add @Override, @Depreacted, serialVersionUID where needed) Modified: trunk/src/main/java/org/nuiton/math/matrix/AbstractMatrixND.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/AbstractMatrixND.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/AbstractMatrixND.java 2009-10-16 17:17:29 UTC (rev 187) @@ -52,10 +52,13 @@ /** to use log facility, just put in your code: log.info(\"...\"); */ private static Log log = LogFactory.getLog(AbstractMatrixND.class); + @Override public abstract MatrixIterator iterator(); + @Override public abstract double getValue(int[] coordinates); + @Override public abstract void setValue(int[] coordinates, double d); protected transient DimensionHelper dimHelper = new DimensionHelper(); @@ -83,7 +86,7 @@ protected void init(int[] dim) { this.dim = new int[dim.length]; System.arraycopy(dim, 0, this.dim, 0, dim.length); - semantics = new List[dim.length]; + semantics = new List<?>[dim.length]; dimNames = new String[dim.length]; // par defaut chaine vide pour le nom des dimensions Arrays.fill(dimNames, ""); @@ -149,6 +152,7 @@ /* * @see org.nuiton.math.matrix.MatrixND#copy() */ + @Override public MatrixND copy() { MatrixND result = getFactory().create(this); return result; @@ -181,6 +185,7 @@ * * @deprecated Use #getSemantics(dim) */ + @Deprecated @Override public List<?> getSemantics(int dim) { return getSemantic(dim); @@ -196,6 +201,7 @@ * * @deprecated Use #setSemantic(dim, List<E>) */ + @Deprecated @Override public <E> void setSemantics(int dim, List<E> sem) { setSemantic(dim, sem); @@ -203,7 +209,7 @@ @Override public <E> void setSemantic(int dim, List<E> sem) { - if (!(sem instanceof SemanticList)) { + if (!(sem instanceof SemanticList<?>)) { sem = new SemanticList<E>(sem); } // else SemanticList is immutable and can be used in many matrix in @@ -238,6 +244,7 @@ * * @deprecated Use #getDimensionNames() */ + @Deprecated @Override public String[] getDimensionName() { return getDimensionNames(); @@ -248,6 +255,7 @@ * * @deprecated Use #setDimensionName(String[]) */ + @Deprecated @Override public void setDimensionName(String[] names) { setDimensionNames(names); @@ -500,17 +508,17 @@ @Override public List<?> toList() { - List<Object> result = new ArrayList<Object>(); + List<?> result = new ArrayList<Object>(); // [3,2,5,4] for (MatrixIterator i = iterator(); i.next();) { int[] coord = i.getCoordinates(); double value = i.getValue(); - List<Object> tmp = result; + List<? super Object> tmp = (List<Object>) result; for (int dim = 0; dim < coord.length - 1; dim++) { while (tmp.size() <= coord[dim]) { tmp.add(new ArrayList<Object>()); } - tmp = (List<Object>) tmp.get(coord[dim]); + tmp = (List<? super Object>) tmp.get(coord[dim]); } while (tmp.size() <= coord[coord.length - 1]) { tmp.add(NumberUtils.DOUBLE_ZERO); @@ -528,9 +536,9 @@ // toutes de la meme dimension pour une dimension donnée. ArrayIntList dim = new ArrayIntList(); List<?> tmp = list; - while (tmp.get(tmp.size() - 1) instanceof List) { + while (tmp.get(tmp.size() - 1) instanceof List<?>) { dim.add(tmp.size()); - tmp = (List<Object>) tmp.get(tmp.size() - 1); + tmp = (List<?>) tmp.get(tmp.size() - 1); } dim.add(tmp.size()); MatrixND mat = getFactory().create(dim.toArray()); @@ -539,7 +547,7 @@ int[] coord = i.getCoordinates(); tmp = list; for (int d = 0; d < coord.length - 1; d++) { - tmp = (List<Object>) tmp.get(coord[d]); + tmp = (List<?>) tmp.get(coord[d]); } Double value = (Double) tmp.get(coord[coord.length - 1]); @@ -559,16 +567,12 @@ public boolean isValidCoordinates(Object[] semantics) { boolean result = getDimCount() == semantics.length; for (int i = 0; result && i < semantics.length; i++) { - result = getSemantic(i).contains(semantics[i]); + List<?> semantic = getSemantic(i); + result = semantic.contains(semantics[i]); } return result; } - /* - * (non-Javadoc) - * - * @see org.codelutin.math.matrix.MatrixND#sumAll() - */ @Override public double sumAll() { double result = 0; @@ -595,7 +599,7 @@ // le nombre d'element qu'il y aura dans la dim pour le resultat int nbDim = getDim(dim) / step; - List<Object>[] semantics = new List[getDimCount()]; + List<?>[] semantics = new List<?>[getDimCount()]; System.arraycopy(getSemantics(), 0, semantics, 0, getDimCount()); semantics[dim] = semantics[dim].subList(0, nbDim); @@ -616,7 +620,7 @@ @Override public MatrixND sumOverDim(int dim, int start, int nb) { // copie de l'ancienne semantique - List<Object>[] semantics = new List[getDimCount()]; + List<? super Object>[] semantics = new List[getDimCount()]; System.arraycopy(getSemantics(), 0, semantics, 0, getDimCount()); semantics[dim] = new ArrayList<Object>(semantics[dim]); @@ -798,7 +802,7 @@ result = getFactory() .create( getName(), - new List[] { Collections.nCopies(1, null), + new List<?>[] { Collections.nCopies(1, null), getSemantic(0) }, new String[] { "Dimension 0", getDimensionName(0) }); for (int x = 0; x < getDim(0); x++) { @@ -806,7 +810,7 @@ } } else { result = getFactory().create(getName(), - new List[] { getSemantic(1), getSemantic(0) }, + new List<?>[] { getSemantic(1), getSemantic(0) }, new String[] { getDimensionName(1), getDimensionName(0) }); for (int x = 0; x < getDim(0); x++) { for (int y = 0; y < getDim(1); y++) { @@ -951,6 +955,7 @@ @Override public MatrixND mults(final double d) { map(new MapFunction() { + @Override public double apply(double val) { return val * d; } @@ -961,6 +966,7 @@ @Override public MatrixND divs(final double d) { map(new MapFunction() { + @Override public double apply(double val) { return val / d; } @@ -971,6 +977,7 @@ @Override public MatrixND adds(final double d) { map(new MapFunction() { + @Override public double apply(double val) { return val + d; } @@ -981,6 +988,7 @@ @Override public MatrixND minuss(final double d) { map(new MapFunction() { + @Override public double apply(double val) { return val - d; } Modified: trunk/src/main/java/org/nuiton/math/matrix/DoubleBigVector.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/DoubleBigVector.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/DoubleBigVector.java 2009-10-16 17:17:29 UTC (rev 187) @@ -38,22 +38,27 @@ data = new double[capacity]; } + @Override public int size() { return data.length; } + @Override public double getMaxOccurence() { return MatrixHelper.maxOccurence(data); } + @Override public double getValue(int pos) { return data[pos]; } + @Override public void setValue(int pos, double value) { data[pos] = value; } + @Override public boolean equals(Object o) { boolean result = false; if (o instanceof DoubleBigVector) { @@ -69,27 +74,33 @@ return result; } + @Override public boolean isImplementedPaste(Vector v) { return v instanceof DoubleBigVector; } + @Override public boolean isImplementedAdd(Vector v) { return v instanceof DoubleBigVector; } + @Override public boolean isImplementedMinus(Vector v) { return v instanceof DoubleBigVector; } + @Override public boolean isImplementedMap() { return true; } + @Override public void paste(Vector v) { DoubleBigVector fbv = (DoubleBigVector) v; System.arraycopy(fbv.data, 0, this.data, 0, this.size()); } + @Override public void add(Vector v) { DoubleBigVector fbv = (DoubleBigVector) v; for (int i = 0; i < data.length; i++) { @@ -97,6 +108,7 @@ } } + @Override public void minus(Vector v) { DoubleBigVector fbv = (DoubleBigVector) v; for (int i = 0; i < data.length; i++) { @@ -104,6 +116,7 @@ } } + @Override public void map(MapFunction f) { for (int i = 0; i < data.length; i++) { data[i] = f.apply(data[i]); Modified: trunk/src/main/java/org/nuiton/math/matrix/DoubleVector.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/DoubleVector.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/DoubleVector.java 2009-10-16 17:17:29 UTC (rev 187) @@ -61,12 +61,14 @@ this.defaultValue = defaultValue; } + @Override public int size() { return capacity; } // poussin 20060827 TODO: verifier l'implantation, il semble quelle soit // fausse et ne puisse pas recherche le nombre max correctement + @Override public double getMaxOccurence() { double result = defaultValue; @@ -126,6 +128,7 @@ } } + @Override public double getValue(int pos) { checkPos(pos); @@ -137,9 +140,7 @@ return result; } - /** - * On ajoute dans l'o - */ + @Override public void setValue(int pos, double value) { checkPos(pos); @@ -165,6 +166,7 @@ } } + @Override public boolean equals(Object o) { boolean result = false; if (o instanceof DoubleVector) { @@ -184,6 +186,7 @@ /** * retourne la position dans le tableau position de la position lineaire * + * @param pos * @return la position ou < 0 donnant la position de l'element s'il etait * present */ @@ -222,20 +225,24 @@ return oldval; } + @Override public boolean isImplementedPaste(Vector v) { return v instanceof DoubleVector; } + @Override public boolean isImplementedAdd(Vector v) { // FIXME une fois la methode implanter supprimer le false return false && v instanceof DoubleVector; } + @Override public boolean isImplementedMinus(Vector v) { // FIXME une fois la methode implanter supprimer le false return false && v instanceof DoubleVector; } + @Override public boolean isImplementedMap() { return true; } @@ -244,6 +251,7 @@ * On recopie tous les attributs pour que le vector ressemble exactement a * celui passé en argument */ + @Override public void paste(Vector v) { DoubleVector fbv = (DoubleVector) v; this.capacity = fbv.capacity; @@ -258,6 +266,7 @@ // poussin 20060827 FIXME a refaire car v.data et date n'ont pas forcement // leur element qui se correspondent, cette implatation est donc fausse + @Override public void add(Vector v) { DoubleVector fbv = (DoubleVector) v; for (int i = 0; i < data.size(); i++) { @@ -268,6 +277,7 @@ // poussin 20060827 FIXME a refaire car v.data et date n'ont pas forcement // leur element qui se correspondent, cette implatation est donc fausse + @Override public void minus(Vector v) { DoubleVector fbv = (DoubleVector) v; for (int i = 0; i < data.size(); i++) { @@ -279,6 +289,7 @@ /** * on applique sur chaque donnée existante et sur default */ + @Override public void map(MapFunction f) { // on commence toujours par modifier la valeur par defaut // car les valeurs suivante pourrait prendre cette valeur Modified: trunk/src/main/java/org/nuiton/math/matrix/FloatBigVector.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/FloatBigVector.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/FloatBigVector.java 2009-10-16 17:17:29 UTC (rev 187) @@ -38,22 +38,27 @@ data = new float[capacity]; } + @Override public int size() { return data.length; } + @Override public double getMaxOccurence() { return MatrixHelper.maxOccurence(data); } + @Override public double getValue(int pos) { return data[pos]; } + @Override public void setValue(int pos, double value) { data[pos] = (float) value; } + @Override public boolean equals(Object o) { boolean result = false; if (o instanceof FloatBigVector) { @@ -69,27 +74,33 @@ return result; } + @Override public boolean isImplementedPaste(Vector v) { return v instanceof FloatBigVector; } + @Override public boolean isImplementedAdd(Vector v) { return v instanceof FloatBigVector; } + @Override public boolean isImplementedMinus(Vector v) { return v instanceof FloatBigVector; } + @Override public boolean isImplementedMap() { return true; } + @Override public void paste(Vector v) { FloatBigVector fbv = (FloatBigVector) v; System.arraycopy(fbv.data, 0, this.data, 0, this.size()); } + @Override public void add(Vector v) { FloatBigVector fbv = (FloatBigVector) v; for (int i = 0; i < data.length; i++) { @@ -97,6 +108,7 @@ } } + @Override public void minus(Vector v) { FloatBigVector fbv = (FloatBigVector) v; for (int i = 0; i < data.length; i++) { @@ -104,6 +116,7 @@ } } + @Override public void map(MapFunction f) { for (int i = 0; i < data.length; i++) { data[i] = (float) f.apply(data[i]); Modified: trunk/src/main/java/org/nuiton/math/matrix/FloatVector.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/FloatVector.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/FloatVector.java 2009-10-16 17:17:29 UTC (rev 187) @@ -61,10 +61,12 @@ this.defaultValue = defaultValue; } + @Override public int size() { return capacity; } + @Override public double getMaxOccurence() { float result = defaultValue; @@ -124,6 +126,7 @@ } } + @Override public double getValue(int pos) { checkPos(pos); @@ -135,9 +138,7 @@ return result; } - /** - * On ajoute dans l'o - */ + @Override public void setValue(int pos, double dValue) { checkPos(pos); @@ -164,6 +165,7 @@ } } + @Override public boolean equals(Object o) { boolean result = false; if (o instanceof FloatVector) { @@ -183,6 +185,7 @@ /** * retourne la position dans le tableau position de la position lineaire * + * @param pos * @return la position ou < 0 donnant la position de l'element s'il etait * present */ @@ -221,20 +224,24 @@ return oldval; } + @Override public boolean isImplementedPaste(Vector v) { return v instanceof FloatVector; } + @Override public boolean isImplementedAdd(Vector v) { // FIXME une fois la methode implanter supprimer le false return false && v instanceof FloatVector; } + @Override public boolean isImplementedMinus(Vector v) { // FIXME une fois la methode implanter supprimer le false return false && v instanceof FloatVector; } + @Override public boolean isImplementedMap() { return true; } @@ -243,6 +250,7 @@ * On recopie tous les attributs pour que le vector ressemble exactement a * celui passé en argument */ + @Override public void paste(Vector v) { FloatVector fbv = (FloatVector) v; this.capacity = fbv.capacity; @@ -256,6 +264,7 @@ } // FIXME a faire + @Override public void add(Vector v) { FloatVector fbv = (FloatVector) v; for (int i = 0; i < data.size(); i++) { @@ -264,6 +273,7 @@ } // FIXME a faire + @Override public void minus(Vector v) { FloatVector fbv = (FloatVector) v; for (int i = 0; i < data.size(); i++) { @@ -274,6 +284,7 @@ /** * on applique sur chaque donnée existante et sur default */ + @Override public void map(MapFunction f) { // on commence toujours par modifier la valeur par defaut // car les valeurs suivante pourrait prendre cette valeur Modified: trunk/src/main/java/org/nuiton/math/matrix/MapFunction.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/MapFunction.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/MapFunction.java 2009-10-16 17:17:29 UTC (rev 187) @@ -39,6 +39,6 @@ * @return la nouvelle valeur à mettre dans la matrice à la place de * l'ancienne. */ - public double apply(double value); + double apply(double value); } // MapFunction Modified: trunk/src/main/java/org/nuiton/math/matrix/MatrixEncoder.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/MatrixEncoder.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/MatrixEncoder.java 2009-10-16 17:17:29 UTC (rev 187) @@ -50,6 +50,7 @@ * méthode n'est pas surchargée. Le seul objet convenablement supporté sont * les représentation objet des types primitifs. * + * @param o * @throws IOException * @return semntics as xml */ Modified: trunk/src/main/java/org/nuiton/math/matrix/MatrixFactory.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/MatrixFactory.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/MatrixFactory.java 2009-10-16 17:17:29 UTC (rev 187) @@ -213,7 +213,7 @@ protected Vector createVector(int length) { try { Constructor<?> c = vectorClass - .getConstructor(new Class[] { Integer.TYPE }); + .getConstructor(new Class<?>[] { Integer.TYPE }); return (Vector) c.newInstance(new Object[] { length }); } catch (Exception eee) { throw new RuntimeException("Can't create vector", eee); Modified: trunk/src/main/java/org/nuiton/math/matrix/MatrixHelper.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/MatrixHelper.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/MatrixHelper.java 2009-10-16 17:17:29 UTC (rev 187) @@ -247,6 +247,7 @@ */ public static MatrixND fill(MatrixND mat, final double value) { mat.map(new MapFunction() { + @Override public double apply(double v) { return value; } @@ -279,6 +280,8 @@ /** * le tableau en entre est trie durant l'execution de la methode, il est * donc modifié + * @param tmp TODO + * @return TODO */ protected static double maxOccurence1(double[] tmp) { if (tmp.length == 0) { Modified: trunk/src/main/java/org/nuiton/math/matrix/MatrixIterator.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/MatrixIterator.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/MatrixIterator.java 2009-10-16 17:17:29 UTC (rev 187) @@ -36,7 +36,7 @@ * * @return current element semantics */ - public Object[] getSemanticsCoordinates(); + Object[] getSemanticsCoordinates(); } // MatrixIterator Modified: trunk/src/main/java/org/nuiton/math/matrix/MatrixIteratorImpl.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/MatrixIteratorImpl.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/MatrixIteratorImpl.java 2009-10-16 17:17:29 UTC (rev 187) @@ -47,26 +47,32 @@ pos = 0; } + @Override public boolean hasNext() { return iterator.hasNext(); } + @Override public boolean next() { return iterator.next(); } + @Override public int[] getCoordinates() { return iterator.getCoordinates(); } + @Override public double getValue() { return iterator.getValue(); } + @Override public void setValue(double value) { iterator.setValue(value); } + @Override public Object[] getSemanticsCoordinates() { Object[] result = null; if (semantics != null) { Modified: trunk/src/main/java/org/nuiton/math/matrix/MatrixND.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/MatrixND.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/MatrixND.java 2009-10-16 17:17:29 UTC (rev 187) @@ -63,6 +63,7 @@ * * @deprecated use #getSemantic(int) instead */ + @Deprecated public List<?> getSemantics(int dim); /** @@ -77,16 +78,19 @@ /** * Modifie la semantique d'une dimension * + * @param <E> data type * @param dim dimension to modify semantic * @param sem new semantic to set * * @deprecated use #setSemantics(int, List<E>) instead */ + @Deprecated public <E> void setSemantics(int dim, List<E> sem); /** * Modifie la semantique d'une dimension * + * @param <E> data type * @param dim dimension to modify semantic * @param sem new semantic to set */ @@ -113,6 +117,7 @@ * * @deprecated (since 1.0.3) Use #setDimensionNames(String[]) */ + @Deprecated public void setDimensionName(String[] names); /** @@ -131,6 +136,7 @@ * * @deprecated (since 1.0.3) Use #getDimensionNames() */ + @Deprecated public String[] getDimensionName(); /** Modified: trunk/src/main/java/org/nuiton/math/matrix/MatrixNDImpl.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/MatrixNDImpl.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/MatrixNDImpl.java 2009-10-16 17:17:29 UTC (rev 187) @@ -78,22 +78,27 @@ this.paste(matrix); } + @Override public MatrixIterator iterator() { return new MatrixIteratorImpl(matrix.iterator(), getSemantics()); } + @Override public double getValue(int[] coordinates) { return matrix.getValue(coordinates); } + @Override public void setValue(int[] coordinates, double d) { matrix.setValue(coordinates, d); } + @Override public double getMaxOccurence() { return matrix.getMaxOccurence(); } + @Override public String toString() { return getName() + " " + matrix.toString(); } @@ -117,6 +122,7 @@ * Si on peut on utilise le paste du vector de BasicMatix, sinon on utilise * la methode de AbstractMatrixND. */ + @Override public MatrixND paste(int[] origin, MatrixND mat) { // permet de savoir si l'origin est bien le point 0 de la matrice boolean origin0 = true; @@ -135,6 +141,7 @@ return this; } + @Override public MatrixND add(MatrixND m) { if (m instanceof MatrixNDImpl && matrix.data.isImplementedAdd(((MatrixNDImpl) m).matrix.data)) { @@ -151,6 +158,7 @@ * nombre de dimension, et chacune de ses dimensions doit avoir un nombre * d'element au moins egal a cette matrice. */ + @Override public MatrixND minus(MatrixND m) { if (m instanceof MatrixNDImpl && matrix.data Modified: trunk/src/main/java/org/nuiton/math/matrix/MatrixStringEncoder.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/MatrixStringEncoder.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/MatrixStringEncoder.java 2009-10-16 17:17:29 UTC (rev 187) @@ -235,7 +235,7 @@ } String[] sems = StringUtil.split(localStr, ","); - List<?>[] result = new List[sems.length]; + List<?>[] result = new List<?>[sems.length]; for (int i = 0; i < sems.length; i++) { result[i] = splitObjects(sems[i]); Modified: trunk/src/main/java/org/nuiton/math/matrix/SemanticList.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/SemanticList.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/SemanticList.java 2009-10-16 17:17:29 UTC (rev 187) @@ -69,8 +69,8 @@ */ @Override public int indexOf(Object o) { - Map<Object, Integer> index = getIndex(); - Integer result = index.get(o); + Map<Object, Integer> i = getIndex(); + Integer result = i.get(o); int resultIndex = -1; if (result != null) { resultIndex = result.intValue(); Modified: trunk/src/main/java/org/nuiton/math/matrix/SubMatrix.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/SubMatrix.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/SubMatrix.java 2009-10-16 17:17:29 UTC (rev 187) @@ -70,14 +70,17 @@ getDim()[dim] = elem.length; } + @Override public MatrixIterator iterator() { return new SubMatrixIteratorImpl(this); } + @Override public double getValue(int[] coordinates) { return matrix.getValue(converter.convertCoordinates(coordinates)); } + @Override public void setValue(int[] coordinates, double d) { matrix.setValue(converter.convertCoordinates(coordinates), d); } @@ -100,10 +103,12 @@ } + @Override public boolean hasNext() { return !Arrays.equals(cpt, last); } + @Override public boolean next() { boolean result = hasNext(); int ret = 1; @@ -116,10 +121,12 @@ return result; } + @Override public int[] getCoordinates() { return cpt; } + @Override public Object[] getSemanticsCoordinates() { int[] coordinates = getCoordinates(); Object[] result = MatrixHelper.dimensionToSemantics(subMatrix @@ -127,10 +134,12 @@ return result; } + @Override public double getValue() { return subMatrix.getValue(getCoordinates()); } + @Override public void setValue(double value) { subMatrix.setValue(getCoordinates(), value); } @@ -162,6 +171,7 @@ this.nb = nb; } + @Override public int[] convertCoordinates(int[] coordinates) { int[] result = null; if (coordinates[dim] < nb) { @@ -193,6 +203,7 @@ System.arraycopy(elem, 0, this.elem, 0, elem.length); } + @Override public int[] convertCoordinates(int[] coordinates) { int[] result = null; if (coordinates[dim] < elem.length) { Modified: trunk/src/main/java/org/nuiton/math/matrix/gui/MatrixPanelEditor.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/gui/MatrixPanelEditor.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/gui/MatrixPanelEditor.java 2009-10-16 17:17:29 UTC (rev 187) @@ -478,7 +478,7 @@ */ MatrixND m = MatrixFactory.getInstance().create("name", - new List[]{sem1, sem2, sem3}, + new List<?>[]{sem1, sem2, sem3}, new String[]{"dim1", "dim2", "dim3"}); m.setValue(0, 0, 0, 1); Modified: trunk/src/main/java/org/nuiton/math/matrix/gui/MatrixPopupMenu.java =================================================================== --- trunk/src/main/java/org/nuiton/math/matrix/gui/MatrixPopupMenu.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/main/java/org/nuiton/math/matrix/gui/MatrixPopupMenu.java 2009-10-16 17:17:29 UTC (rev 187) @@ -298,6 +298,7 @@ if (fileChooser == null) { fileChooser = new JFileChooser(); FileFilter filter = new FileFilter() { + @Override public boolean accept(File pathname) { if (pathname.isDirectory()) { return true; @@ -315,6 +316,7 @@ return false; } + @Override public String getDescription() { return "Texte CSV (*.csv)"; } @@ -331,6 +333,8 @@ public Action getSendToClipBoardAllCopyAction() { if (sendToClipBoardAllCopyAction == null) { sendToClipBoardAllCopyAction = new AbstractAction() { + private static final long serialVersionUID=1L; + @Override public void actionPerformed(ActionEvent e) { sendToClipBoardAllCopyPerformed(); } @@ -346,6 +350,8 @@ public Action getSendToClipBoardAllPasteAction() { if (sendToClipBoardAllPasteAction == null) { sendToClipBoardAllPasteAction = new AbstractAction() { + private static final long serialVersionUID=1L; + @Override public void actionPerformed(ActionEvent e) { sendToClipBoardAllPastePerformed(); } @@ -361,6 +367,8 @@ public Action getSendToClipBoardSelectionCopyAction() { if (sendToClipBoardSelectionCopyAction == null) { sendToClipBoardSelectionCopyAction = new AbstractAction() { + private static final long serialVersionUID=1L; + @Override public void actionPerformed(ActionEvent e) { sendToClipBoardSelectionCopyPerformed(); } @@ -376,6 +384,8 @@ public Action getSendToClipBoardCurrentPasteAction() { if (sendToClipBoardCurrentPasteAction == null) { sendToClipBoardCurrentPasteAction = new AbstractAction() { + private static final long serialVersionUID=1L; + @Override public void actionPerformed(ActionEvent e) { sendToClipBoardCurrentPastePerformed(); } @@ -458,6 +468,8 @@ public Action getSendToFileAllCopyAction() { if (sendToFileAllCopyAction == null) { sendToFileAllCopyAction = new AbstractAction() { + private static final long serialVersionUID=1L; + @Override public void actionPerformed(ActionEvent e) { sendToFileAllCopyPerformed(); } @@ -473,6 +485,8 @@ public Action getSendToFileAllPasteAction() { if (sendToFileAllPasteAction == null) { sendToFileAllPasteAction = new AbstractAction() { + private static final long serialVersionUID=1L; + @Override public void actionPerformed(ActionEvent e) { sendToFileAllPastePerformed(); } @@ -488,6 +502,8 @@ public Action getSendToFileSelectionCopyAction() { if (sendToFileSelectionCopyAction == null) { sendToFileSelectionCopyAction = new AbstractAction() { + private static final long serialVersionUID=1L; + @Override public void actionPerformed(ActionEvent e) { sendToFileSelectionCopyPerformed(); } @@ -503,6 +519,8 @@ public Action getSendToFileCurrentPasteAction() { if (sendToFileCurrentPasteAction == null) { sendToFileCurrentPasteAction = new AbstractAction() { + private static final long serialVersionUID=1L; + @Override public void actionPerformed(ActionEvent e) { sendToFileCurrentPastePerformed(); } Modified: trunk/src/test/java/org/nuiton/math/matrix/gui/MatrixTableModelTest.java =================================================================== --- trunk/src/test/java/org/nuiton/math/matrix/gui/MatrixTableModelTest.java 2009-10-16 15:22:47 UTC (rev 186) +++ trunk/src/test/java/org/nuiton/math/matrix/gui/MatrixTableModelTest.java 2009-10-16 17:17:29 UTC (rev 187) @@ -71,7 +71,7 @@ } /* - * Test method for 'org.codelutin.math.matrix.gui.MatrixTableModelND.isCellEditable(int, int)' + * Test method for 'org.nuiton.math.matrix.gui.MatrixTableModelND.isCellEditable(int, int)' */ @Test public void testIsCellEditable() { @@ -84,7 +84,7 @@ } /* - * Test method for 'org.codelutin.math.matrix.gui.MatrixTableModelND.tableToMatrix(int, int)' + * Test method for 'org.nuiton.math.matrix.gui.MatrixTableModelND.tableToMatrix(int, int)' */ @Test public void testTableToMatrix() { @@ -100,7 +100,7 @@ } /* - * Test method for 'org.codelutin.math.matrix.gui.MatrixTableModelND.getValue(int, int)' + * Test method for 'org.nuiton.math.matrix.gui.MatrixTableModelND.getValue(int, int)' */ @Test public void testGetValue() { @@ -128,7 +128,7 @@ /* * Test method for - * 'org.codelutin.math.matrix.gui.MatrixTableModelND.getColumnName(int)' + * 'org.nuiton.math.matrix.gui.MatrixTableModelND.getColumnName(int)' */ @Test public void testGetColumnNameInt() { @@ -137,7 +137,7 @@ /* * Test method for - * 'org.codelutin.math.matrix.gui.MatrixTableModelND.getRowCount()' + * 'org.nuiton.math.matrix.gui.MatrixTableModelND.getRowCount()' */ @Test public void testGetRowCount() { @@ -146,7 +146,7 @@ /* * Test method for - * 'org.codelutin.math.matrix.gui.MatrixTableModelND.getColumnCount()' + * 'org.nuiton.math.matrix.gui.MatrixTableModelND.getColumnCount()' */ @Test public void testGetColumnCount() { @@ -155,7 +155,7 @@ /* * Test method for - * 'org.codelutin.math.matrix.gui.MatrixTableModelND.getValueAt(int, int)' + * 'org.nuiton.math.matrix.gui.MatrixTableModelND.getValueAt(int, int)' */ @Test public void testGetValueAt() { @@ -164,7 +164,7 @@ /* * Test method for - * 'org.codelutin.math.matrix.gui.MatrixTableModelND.setValueAt(Object, int, + * 'org.nuiton.math.matrix.gui.MatrixTableModelND.setValueAt(Object, int, * int)' */ @Test