java.lang.ClassCastException

[weljo]

hab diese Exception:

Exception in thread "AWT-EventQueue-0" java.lang.ClassCastException: Container cannot be cast to Combo

... in der zeile:

if(MainFrame.table.getValueAt(MainFrame.table.getSelectedRow(), e.getColumn()).toString()
  .compareTo(MainFrame.combos.elementAt(v).getName().toString()) == 0)

Container und Combo sind 2 eigene klassen welche nur ein paar daten beinhalten(containerklassen)

mach ja eigentlich nur ein compareTo(...), meiner meinung nach sind ja nicht einmal die toString() nötig aber wollt auf nummer sicher gehn

sieht da jemand einen fehler bin schon fast am verzweifeln

 

[SlaterB]

zunächst musst du herausfinden, wo die Exception auftritt, ob innnerhalb von compareTo oder bei einem Cast in elementAt(v),
in der geposteten Zeile jedenfalls nicht,

wie jedem würde auch dir dennoch hoffentlich helfen, aus einem Unfall sauberen Code zu machen:
Typ a = ..;
Typ b = ..;
if (a.compareTo(b)) {
..
}

schon kann man sogar ein bisschen Struktur erkennen


edit:
also was genau ist MainFrame.combos?
ein generischer Container? bitte dessen Definition posten und alle Stellen prüfen, an denen dort etwas eingefügt wird

 

[weljo]

ok der fehler liegt in meinem fall im b teil ... poste jetzt mal alles relevante villeicht hab ich ja einen groben denkfehler

ok ich habe eine JTable in der die erste spalte eine Combobox als editor beinhaltet. Eine Reihe ergibt immer einen Datensatz welcher aus verschiedenen kombinationen zusammengesetzt sein kann. die kombinationen speichere ich bei beenden des programms. ist die kombination bereits vorhanden erhöhe ich eine variable, welche mitzählt wie oft die kombination verwendet wurde. Vor dem speichern sortiere ich die Kombinationen noch nach dem wie oft sie verwendet wurden.

boolean got = false;
			int id;
			for(int r=0; r<container.size(); r++)
			{
				Container cont = container.elementAt(r);
				id = 0;
				while(got==false || id!=combos.size())
				{// sucht bis eine übereinstimmung gefunden wurde
					Combo combo = combos.elementAt(id);
					if(cont.getName().compareTo(combo.getName())==0 &&
 cont.getTemp().compareTo(combo.getTemp())==0 &&
 cont.getTime().compareTo(combo.getTime())==0 &&
 cont.getDilution().compareTo(combo.getDilution())==0)
// datensatz vergleichen vergleichen
					{
						combo.setCounter(combo.getCounter()+1);
						got = true;
					}
				}
				if(!got)//wenn die kombination nicht vorhanden ist erzeuge ich einen neuen eintrag
				{
					combos.add(new Combo(cont.getName(),cont.getTemp(),
cont.getTime(),cont.getDilution()));//neuen datensatz anlegen
				}
			}
			//combos sortieren nach count
				for (int i = 0; i < (combos.size()-1); i++) {
					for (int j = i+1; j < combos.size(); j++) {
						if (combos.elementAt(i).getCounter() > combos.elementAt(j).getCounter()) {
							Combo tmp;
							tmp = combos.elementAt(i);
							combos.set(i, combos.elementAt(j));
							combos.set(j, tmp);
						}//end if
					}//end for (j...)
				}//end for (i...)

			save();

ok ... soweit so gut ... natürlich lade ich diese daten auch wieder sobald ich das programm starte und wenn jetzt jemand eine Combobox ändet sollten sich in der Reiher der Tabelle auch die anderen daten auf die meist gewählte kombination anpassen, was ich so gelöst habe

private class MyTableModelListener implements TableModelListener
		{
        	boolean found;
    		int v;
    		int selectedrow = -1;
    		String selecteditem;
    		Container selectedcell;
    		
		    public void tableChanged(TableModelEvent e)
		    {	    		
		        found = false;
		        v=0;		    	
		        selectedrow = MainFrame.table.getSelectedRow();
		        selectedcell = MainFrame.container.elementAt(selectedrow);
				selecteditem = MainFrame.table.getValueAt(MainFrame.table.getSelectedRow(), e.getColumn()).toString();				

				    	switch(e.getColumn())
				    	{
				    		case 0:	selectedcell.setName(selecteditem);
						    		while(!found)// sucht die meistgewählten zusatzinfos
						    		{
						    									    			if(v<MainFrame.combos.size())
						    			{						    			Combo combo = MainFrame.combos.elementAt(v);

						    				String b = combo.getName().toString();
						    				if(selecteditem.compareTo(b) == 0)
						    				{
						    					selectedcell.setTemp(combo.getTemp().toString());
						    					selectedcell.setTime(combo.getTime().toString());
						    					selectedcell.setDilution(combo.getDilution().toString());
						    					found=true;
						    				}
						    				else
						    				{
						    					v++;
						    				}
						    			}
						    			else
						    			{
						    				found = true;
						    			}							
						    		}
				    				break;
				    		case 1:	selectedcell.setTemp(selecteditem);	
			    					break;
				    		case 2:	selectedcell.setTime(selecteditem);
			    					break;
				    		case 3:	selectedcell.setDilution(selecteditem);
	    							break;
	    					case 4:	selectedcell.setInfo(selecteditem);
	    							break;
			    			default:System.out.println("???");break;
			    		}
		     }
		}

bekomm jedoch immer meine exception wie oben beschrieben bei

Combo combo = MainFrame.combos.elementAt(v);

 

[SlaterB]

du kürzt immer noch nicht genug ab

in MyTableModelListener steht 7x
> MainFrame.container.elementAt(selectedrow)

wenn du da vorher eine Variable selCon definieren würdest..,
immerhin hast du schon eine Variable selectedrow,
sonst würde da 7x MainFrame.container.elementAt(MainFrame.table.getSelectedRow())
stehen..

--------

if(container.elementAt(r).getName().compareTo(combos.elementAt(id).getName())==0 &&
container.elementAt(r).getTemp().compareTo(combos.elementAt(id).getTemp())==0 &&
container.elementAt(r).getTime().compareTo(combos.elementAt(id).getTime())==0 &&
container.elementAt(r).getDilution().compareTo(combos.elementAt(id).getDilution())==0)

liest sich ähnlich schlimm, das könnte

con = ..;
combo = ..;
if(con.getName().compareTo(comb.getName())==0 &&
...
lauten

---------

zum Thema sehe ich persönlich kaum Fortschritte,
die Definition von combos fehlt weiterhin

 

[weljo]

hab jetzt die leslichkeit verbessert ich glaub wirkilch viel mehr geht nicht

 

[SlaterB]

das war nur nebenbei angemerkt, ist wie gesagt für die Frage weniger entscheidend,

zum dritten und letzten Mal:
was ist combos? welche Klasse, welcher generischer Parameter, falls vorhanden?

 

[weljo]

combos ist ein Vector<Combo> in dem die jeweiligen kombinationen(alias datensatz alias zeile) wie oben geschrieben gespeichert werden

Combo ist lediglich eine containerklasse

import java.io.Serializable;


public class Combo implements Serializable
{	
	String name;
	String temp;
	String time ;
	String dilution;
	long counter =0;
	
	public Combo(String name, String temp, String time, String dilution) {
		super();
		this.name = name;
		this.temp = temp;
		this.time = time;
		this.dilution = dilution;
	}

	public String getName() {
		return name;
	}

	public String getTemp() {
		return temp;
	}

	public String getTime() {
		return time;
	}

	public String getDilution() {
		return dilution;
	}

	public long getCounter() {
		return counter;
	}

	public void setCounter(long conter) {
		this.counter = counter;
	}
}

 

[SlaterB]

gut, also das Problem ist, dass irgendwie ein Container-Objekt in den Vector hineingekommen ist,
bei generischer Typprüfung sollte das gar nicht möglich sein, wenn aber der Vector noch in einer anderen Variablen steckt, dann geht das,

ich mag nun gar nicht vermuten, was alles in deinem Programm denkbar ist,
eine Variante zur Lösung wäre, alle add-Methoden des Vectors testweise zu überschreiben und die eingefügten Elemente zu testen:

public class Test
{
    public static void main(String[] args)
        throws Exception
    {
        Vector v = new Vector()
            {
                public synchronized boolean add(Object o)
                {
                    if (!Combo.class.isInstance(o))
                    {
                        new Error("kein Combo: " + o.getClass()).printStackTrace();
                    }
                    return super.add(o);
                }

                public synchronized Object set(int index, Object o)
                {
                    if (!Combo.class.isInstance(o))
                    {
                        new Error("kein Combo: " + o.getClass()).printStackTrace();
                    }
                    return super.set(index, o);
                }

                public synchronized void setElementAt(Object o, int index)
                {
                    if (!Combo.class.isInstance(o))
                    {
                        new Error("kein Combo: " + o.getClass()).printStackTrace();
                    }
                    super.setElementAt(o, index);
                }

            };
        Vector<Combo> vc = v;

        vc.add(new Combo());
        vc.set(0, new Combo());

        v.add(new Container()); // Zeile 47

        Combo c = vc.get(1);
        System.out.println(c);
    }
}
class Combo
{
}
class Container
{
}

java.lang.Error:  kein Combo: class test.Container
	at test.Test$1.add(Test.java:18)
	at test.Test.main(Test.java:47)
Exception in thread "main" java.lang.ClassCastException: test.Container cannot be cast to test.Combo
	at test.Test.main(Test.java:49)

die ClassCastException ist in etwa das was du siehst,
aber der Error kommt von einer der add-Methoden und zeigt genau die Stelle im Code an, die das falsche Objekt einfügt (Zeile 47)

versuche mal, statt deines Vectors die überschriebene Klasse einzusetzen

 

[weljo]

schon mal danke für den ansatz werds versuchen

 

[weljo]

ok es funktioniert jetzt .... der fehler lag nicht am code sondern an der datei die ich geladen hab ... die hatte noch von einer alten version einen falschen inhalt und ich hab aus versehn noch von der alten datei geladen jedoch bereits in eine neue gespeichert ... mein fehler aber thx für die bemühungen(jetzt sieht zumindest mein code etwas schöner aus)

 

[Ollek]

Ich erhalte ebenfalls eine ClassCastException und zwar diese:

Uncaught error fetching image:
java.lang.ClassCastException: [I cannot be cast to [B
	at java.awt.image.ColorModel.getAlpha(Unknown Source)
	at java.awt.image.ColorModel.getRGB(Unknown Source)
	at sun.awt.image.ImageRepresentation.convertToRGB(Unknown Source)
	at sun.awt.image.ImageRepresentation.setPixels(Unknown Source)
	at java.awt.image.AreaAveragingScaleFilter.accumPixels(Unknown Source)
	at java.awt.image.AreaAveragingScaleFilter.setPixels(Unknown Source)
	at sun.awt.image.ImageDecoder.setPixels(Unknown Source)
	at sun.awt.image.JPEGImageDecoder.sendPixels(Unknown Source)
	at sun.awt.image.JPEGImageDecoder.readImage(Native Method)
	at sun.awt.image.JPEGImageDecoder.produceImage(Unknown Source)
	at sun.awt.image.InputStreamImageSource.doFetch(Unknown Source)
	at sun.awt.image.ImageFetcher.fetchloop(Unknown Source)
	at sun.awt.image.ImageFetcher.run(Unknown Source)

Ich lade eine Blob Datei in ein ByteArray, über dieses ByteArray erstelle ich ein ImageIcon. Von dem ImageIcon hole ich mit mir getImage() das Image um dieses anzuzeigen.
Habe schon einige Try und Catch Blöcke eingebaut, aber keiner war richtig positioniert und hat diese Exception abgefangen. Eventuell könnt ihr mir dort ja helfen, ich komme nicht weiter...

Hier mal die Sourcen.

public class ArticleImage extends JPanel implements Observer {

private Image scaledImage = null;
private Article article = null;
private Image defaultImage = null;
private BufferedImage bufferedImage;

public ArticleImage(){
		super();
		init();
		ArticleManagement.getInstance().addObserver(this);
		
		defaultImage = createImageIcon("/icons/defaultArtImg.jpg").getImage();
}

@Override
	public void paintComponent(Graphics g) {
		super.paintComponent(g);
		g.drawImage(scaledImage, 0, 0, this);
	}

@Override
	public void update(Observable arg0, Object arg1) {
		article = (Article) arg1;
		
		// wenn kein Image auf der Datenbank verfügbar ist, wird das Default Image angezeigt
		if(article.getImageBytes() != null){
			try{
				ImageIcon imageIcon = new ImageIcon(article.getImageBytes());
				scaledImage = scaledImage(imageIcon.getImage());
			}catch (ClassCastException e){
				System.out.println("Probleme beim Casten");
				e.printStackTrace();
			}
		}else
			scaledImage = defaultImage;
		repaint();
}

/**
	 * Skaliert das Bild in die Größe
	 * 300 * 225 Pixel.
	 * 
	 * @param Image - Bilddatei zum skalieren
	 * @return - skalierte Bilddatei
	 */
	public Image scaledImage(Image image){
		try{
			scaledImage = image.getScaledInstance(300, 225, Image.SCALE_SMOOTH);	
		}catch (ClassCastException e){
			System.out.println("Probleme beim Skalieren");
			e.printStackTrace();
		}
		return scaledImage;
}

/** Gibt ein ImageIcon oder null wenn das 
	 * Verzeichnis nicht verfügabr ist zurück.
	 */
	protected static ImageIcon createImageIcon(String path) {
	    URL imgURL = ArticleImage.class.getResource(path); 
	    if (imgURL != null) {
	        return new ImageIcon(imgURL);
	    } else {
	        System.err.println("Konnte Datei im Verzeichnis nicht finden: " + path);
	        return null;
	    }
}

}

Das ist mein ImagePanel, woanders benutze ich kein Image...