Objekte in Vector werden überschrieben

[crosskay]

Hi,

vielleicht kann mir jemand helfen.
Ich hab folgendes Problem:

Beim Einfügen eines Objekts in einen Vector werden alle bisher eingefügten Objekte durch das neu eingefügte Objekt überschrieben.
Hier ein Code-Auszug

// Programmauszug

private void jConExtractButtonMouseClicked(java.awt.event.MouseEvent evt) {
        // TODO add your handling code here:
        ContourData condata;

        img = (BinaryImage)ImageFilesContainer.getBinaryImages().get(index-1);
        condata = ContourExtract.extract(img);
        //condata[index-1] = new ContourData(ContourExtract.getCoords());     // ContourData Objekt für jeweiliges Bild
        DataContainer.addContourData(condata);
        //contourData.add(condata);

        //int width = jMainPanel.getInsets().right - jMainPanel.getInsets().left;
        //int height = jMainPanel.getInsets().bottom - jMainPanel.getInsets().top;
        Dimension dim = new Dimension(img.getWidth(),img.getHeight());

        panel.setActive(condata.getCoords(), dim);
        jMainPanel.add(panel, java.awt.BorderLayout.CENTER);
        int anz = condata.getCoordsNr();
        String nr = String.valueOf(anz);
        jContourPtTextField.setText(nr);

    }

//Ausschnitt DataContainer

import java.util.*;

public class DataContainer {

    private static Vector conData = new Vector();
    private static CamData[] camData;

    /** Creates a new instance of DataContainer */
    public DataContainer() {
    }

    public static void addContourData(ContourData data)
    {
        conData.add(data);
    }

kurze Erklärung:

In der Methode "jConExtractButtonMouseClicked" wird ein die Kontur eines Bild (img) bestimmt
-> Objekt/Klasse: condata, diese Objekt will ich in einem Vector ablegen (über Methode DataContainer.addContourData); Die Berechnungen funktionieren alle, das Objekt wird auch im vector gespeichert
Aber: sobald methode "jConExtractButtonMouseClicked" nochmals ausgeführt wird, wird das bereits im Vector vorhandene Objekt durch das neue Objekt überschrieben, das neue Objekt dann angehängt

Es wäre schön, wenn mir jemand bei meinem problem helfen kann.

P.S.: ich hab es auch schon mit nem array versucht, hilft alles nichts

 

[Ives]

Du musst eine Instanz von DataContainer benutzen (new DataContainer ()), asserdem gehört in die Methodendeklaration von addContourData kein static. Dann sollte es gehen.

 

[crosskay]

danke für den tipp, Ives.
Aber leider hat das auch nicht funktioniert. Ich habe das Objekt je nach index einen neuen Objektnamen zugewiesen, das half aber auch nicht. das aktuelle condata-Objekt überschreibt immer die vorherigen.

 

[bygones]

die Container Klasse wuerde ich erstmal komplett umschreiben - wie schon erwaehnt lass das static

die Klasse in der diese buttonMethode ist (meiner Ansicht nach ein aeusserst merkwuerdiger Name fuer eine Methode) hat eine Instanz der Container Klasse. Eigentlich sollte dann ein einfachs add ueber die Klasse, was ein Objekt an den Vector anfuegt reichen....

 

[mic_checker]

zeig auch mal welche änderungen du an deinem code vorgenommen hast....was die namensgebung betrifft wirst du sicher nicht froh mit deinem aktuellen ansatz

 

[crosskay]

Hier ist die Button-Methode:

private void jConExtractButtonMouseClicked(java.awt.event.MouseEvent evt)
{
   // TODO add your handling code here:
   ContourData condata;

   img = (BinaryImage)ImageFilesContainer.getBinaryImages().get(index-1);
   condata = ContourExtract.extract(img);
   container.addContourData(condata);

   Dimension dim = new Dimension(img.getWidth(),img.getHeight());

   panel.setActive(condata.getCoords(), dim);
   jMainPanel.add(panel, java.awt.BorderLayout.CENTER);
   int anz = condata.getCoordsNr();
   String nr = String.valueOf(anz);
   jContourPtTextField.setText(nr);

}

Die Methode ist Teil eines Dialogs. Als Klassenvariable hab ich genommen:

 public DataContainer container;

und im Konstruktor steht

 container = new DataContainer();

condata hat je nach index unterschiedlichen Inhalt, wenn aber mehr als ein condata-Objekt in den Vector eingefügt wird, wird das alte durchs neue überschrieben und das neue angehängt.

Hier der DataContainer:

import java.util.*;

public class DataContainer {

    private Vector conData;
    private CamData[] camData;

    /** Creates a new instance of DataContainer */
    public DataContainer()
    {
        conData = new Vector();
        camData = null;
    }

    public void addContourData(ContourData data)
    {
        conData.add(data);
    }

    public Vector getContourData()
    {
        return conData;
    }

    public void setCamData(CamData[] data)
    {
        camData = new CamData[data.length];
        for(int i=0; i<data.length; i++)
            camData[i] = data[i];
    }

    public CamData[] getCamData()
    {
        return camData;
    }

}

 

[mic_checker]

Die Methode ist Teil eines Dialogs. Als Klassenvariable hab ich genommen:

 public DataContainer container;

und im Konstruktor steht

 container = new DataContainer();

In dem Fall ist es aber eine Instanzvariable , keine Klassenvariable

import java.util.*;

public class DataContainer {

    private Vector conData;
    private CamData[] camData;

wozu brauchst du camData?

 

[crosskay]

in dem DataContainer sollten nicht nur die Daten über die Kontur eines Bildes, sondern auch Kameraparameter stehen, deshalb camdata

 

[mic_checker]

poste bitte mal mehr von dem aktuellen code...sieht so aus als würdest du irgendwo ein neues Objekt vom Typ DataContainer erstellen, bzw. den Vector neu erstellen und in diesen dann das neue Objekt reinhauen...

 

[crosskay]

condata wird ja immer wieder überschrieben bzw. neu angelegt, aber wenn ich das mit add() in enien Vektor packe, müsste jedes Objekt im Vektor doch erhalten bleiben, stattdessen werden alle Objekte immer wieder durch das neue aktualisiert, also überschrieben. Muss ich vielleicht ne tiefe Kopie machen ?

 

[crosskay]

einen Moment, muss erst an nen anderen rechner, dann kommt der code.

 

[crosskay]

Hier der code für ContourData-Objekt:

/*
 * ContourData.java
 *
 * Created on 22. Juni 2005, 17:54
 */

/**
 *
 * @author  Arne Kesting
 */
public class ContourData {

    private int[][] cc;    // Konturkoordinaten
    private double[][] cs;    // Samplingkoordinaten
    private double sampDist;

    /** Creates a new instance of ContourData */
    public ContourData()
    {
        cc = null;
        cs = null;
        sampDist = 0;
    }

    public ContourData(int[][] coord)
    {
        int cnt = coord.length;

        cc = new int[cnt][2];
        for(int i=0; i<cnt; i++)
        {
            cc[i][0] = coord[i][0];
            cc[i][1] = coord[i][1];
        }
        cs = null;
        sampDist = 0;
    }

    public int[][] getCoords()
    {
       return cc;
    }

    public int getCoordsNr()
    {
        return cc.length;
    }

    public int getSampCoordsNr()
    {
        return cs.length;
    }

    public void setCoords(int[][] coord)
    {
        int cnt = coord.length;

        cc = new int[cnt][2];
        for(int i=0; i<cnt; i++)
        {
            cc[i][0] = coord[i][0];
            cc[i][1] = coord[i][1];
        }
     }

    public void setSampCoords(double[][] coord)
    {
        int cnt = coord.length;
        cs = new double[cnt][2];
        for(int i=0; i<cnt; i++)
        {
            cs[i][0] = coord[i][0];
            cs[i][1] = coord[i][1];
        }
    }

    public double[][] getSampCoords()
    {
        return cs;
    }

    public void setSampDistance(double dist)
    {
        sampDist = dist;
    }

    public double getSampDistance()
    {
        return sampDist;
    }

}

und hier für ContourExtract.ectract(...)

/*
 * ContourExtract.java
 *
 * Created on 20. März 2005, 17:53
 */

/**
 *
 * @author  Arne Kesting
 */

import java.util.*;
import java.awt.Point;

public class ContourExtract
{

    protected int count = 0;
    protected int[][] coord = null;

    /** Creates a new instance of ContourExtract */
    public ContourExtract()
    {
    }

    public ContourData extract(BinaryImage img)
    {
        ContourData cData;
        byte[][] data;
        Vector coord_list;
		Point pkt = new Point();

        data = img.getData();
        coord_list = Contour.MooreTracing(data);
		count = coord_list.size();

		coord = new int[count][2];

 		for(int i=0; i<count; i++)
 		{
            pkt = (Point)coord_list.get(i);
            coord[i][0] = pkt.x;
            coord[i][1] = pkt.y;
 		}
        cData = new ContourData(coord);
        return cData;
    }

    public int getPtNr()
    {
        return count;
    }

    public int[][] getCoords()
    {
        return coord;
    }

}

das funktioniert auch alles. Für Bild(img) wird Kontur berechnet, Konturdaten in condata übergeben. Nur beim mehrmaligen einfügen in den Vector macht das Programm Mist.

 

[crosskay]

der Code für den DataContainer sieht so aus:

/*
 * DataContainer.java
 *
 * Created on 1. Juli 2005, 10:16
 */

/**
 *
 * @author  Arne Kesting
 */

import java.util.*;

public class DataContainer {
    
    private Vector conData;
    private CamData[] camData;
    
    /** Creates a new instance of DataContainer */
    public DataContainer() 
    {
        conData = new Vector();
        camData = null;
    }
    
    public void addContourData(ContourData data)
    {
        conData.add(data);
    }
    
    public Vector getContourData()
    {
        return conData;
    }
    
    public void setCamData(CamData[] data)
    {
        camData = new CamData[data.length];
        for(int i=0; i<data.length; i++)
            camData[i] = data[i];
    }
    
    public CamData[] getCamData()
    {
        return camData;
    }
    
}

 

[crosskay]

DataContainer wird im Konstruktor des Dialogs (in dem steht die Button-Methode) initialisiert.