Bug in JComboBox ?

[vidi]

hallo,

mir ist aufgefallen das die jbombox (editable) ein problem damit hat wenn im selben panel eine andere komponente mit einem jpopupmenu vorhanden ist.

folgendes ist mir aufgefallen:

beim programm start bekommt die textcomponent der jcombobox den focus.

man drück tab (combobox verliert den focus). kommt zur nächsten komponenten. (in meinem fall ein TextField, dem ein popup hinzugefügt ist). das popup von dem textfield öffnet sich.
klick auf den textbereich der jcombobox (jetzt sollte das popup vom textfield ein canceled event bekommen, zugehn und die textcomponente von der jcombobox den focus bekommen) möp und genau das tut sie nicht, die textcomponente bekommt zwar den focus, aber das popup vom textfield bleibt offen.

dachte mir ok vieleicht nen fehler in meiner komponente, aber ich hab mal die jcombobox gegen ein normales jtextfield getauscht und siehe da es funktioniert..

nun frag ich mich woran das liegt ?!?

hab ihr vieleicht schon ähnliches bemerkt ?

greetz

vidi

 

[Wildcard]

Liegt wohl eher an deinem Popup. Code würde helfen.

 

[vidi]

so habe grade mal ein bsp. fertig gemacht aus dem das deutlich wird.
falls in der matchbox nicht alles super schön ist bitte drüber hinwegsehen, ich bin wie gesagt grade am debugen.

hier das hauptprogramm:

import de.comp.*;

import java.awt.*;
import java.util.Vector;

import javax.swing.*;

public class ComboTest extends JFrame
{
	
	public ComboTest()
	{
			
		super("Test");
		
		Dimension frameSize = new Dimension(1024, 768);
		Dimension screenSize = Toolkit.getDefaultToolkit().getScreenSize();

		int top = (screenSize.height - frameSize.height) / 2;
		int left = (screenSize.width - frameSize.width) / 2;
		
		setSize(frameSize);
		setLocation(left, top);
		
		loadGui();
				
		setVisible(true);
		setDefaultCloseOperation(EXIT_ON_CLOSE);
					
	}
	
	private void loadGui()
	{
		
		JPanel panel = new JPanel();
		
		String[] data = {"1", "2", "3", "4", "5", "6", "7", "8", "9", "10"};
		JComboBox comboBox = new JComboBox(data);
		comboBox.setEditable(true);
		
		MatchBox matchBox = new MatchBox(18, 1);
		
		Vector <Vector> rowData = new Vector<Vector>();
		
		for (int i =0; i < 100; i++)
		{
			Vector<String> vector = new Vector<String>();
			vector.add("Name " + i);
			vector.add("Vorname " + i);
			vector.add("0190/333" + i);
			rowData.add(vector);
		}	
		
		
		Vector <String> columnNames = new Vector <String>();
		
		columnNames.removeAllElements();
		columnNames.add("Name");
		columnNames.add("Vorname");
		columnNames.add("Telefonnr.");
			
		matchBox.init(rowData, columnNames);

		JTextField textField = new JTextField(18);
		
		panel.add(comboBox);
		panel.add(matchBox);
		panel.add(textField);
		
		add(panel);
		
	}
	
	public static void main(String argscv[])
	{
		new ComboTest();
	}

}

und hier die "matchBox"

import java.awt.*;
import java.awt.event.*;

import java.util.Vector;

import javax.swing.*;

import javax.swing.table.DefaultTableModel;
import javax.swing.text.Document;
import javax.accessibility.Accessible;

public class MatchBox extends JComponent implements MouseListener, 
													KeyListener, 
													ActionListener, 
													MouseMotionListener, 
													Accessible,
													FocusListener
													
{
    
	private JTextField textField = new JTextField();
   
	private JTable table = new JTable();
	private JScrollPane scrollPane = new JScrollPane(table);
	
	private JPopupMenu popup = new JPopupMenu();
	private JButton button = new JButton("Suche");
	
	private Vector <Vector> rowData = new Vector <Vector>();
	private Vector <String> columnNames = new Vector <String>();
		
	private int coltToSelect = 0;
			
    public MatchBox(int columns, int coltToSelect)
    { 
    	  
    	super();
    	setLayout(new FlowLayout(FlowLayout.TRAILING,0,0));
    	
    	this.coltToSelect = coltToSelect;
    	
		textField.setColumns(columns);
		textField.addKeyListener(this);
		textField.addFocusListener(this);
								
		ImageIcon suchenIcon = new ImageIcon("images/search.png");
	  	button.setIcon(suchenIcon);
	  	button.addActionListener(this);
	  	button.setPreferredSize(new Dimension(button.getPreferredSize().width, textField.getPreferredSize().height -1));
		  	
		table.setAutoscrolls(true);
		table.addMouseListener(this);
		table.addMouseMotionListener(this);
				    			
		popup.add(scrollPane);
		
		add(textField);
		add(button);
		
		textField.add(popup);
		
        setVisible(true);
               
    } 
    
    public void setDocument(Document doc)
    {

    	if (doc != null) 
    	{
    	    doc.putProperty("filterNewlines", Boolean.TRUE);
       	}
    	
    	textField.setDocument(doc);
        	
    }  
    
    public Dimension getPreferredSize()
    {
    	return new Dimension(textField.getPreferredSize().width + button.getPreferredSize().width, button.getPreferredSize().height);
    }
    
    public String getText()
    {
    	return textField.getText();
    }
      
    public void init(Vector<Vector> rowData, Vector<String> columnNames) 
	{
		    	
		if (rowData != null)
		{
			this.rowData = rowData;  	
		}
		
		if (columnNames != null)
		{
			this.columnNames = columnNames;  	
		}
				
		textField.setText(null);
		
		table.setModel(new DefaultTableModel(rowData, columnNames) 
		{
			public boolean isCellEditable(int x, int y) 
            {
            	return false;
            }
		});
			
	}
    
    public void setPopupVisible (Boolean visible)
    {
    	
     	if(visible)
    	{
      		popup.setInvoker(textField);
    		popup.setLocation(getLocationOnScreen().x, getLocationOnScreen().y + textField.getSize().height);
        	popup.setPreferredSize(new Dimension(getSize().width, getSize().height * 5));
        	popup.setVisible(true);
        }	
      	else
      	{
      		popup.setVisible(false);
      		popup.setInvoker(null);
      	}
     	
    }
    
    public Boolean isPopupVisible()
    {
    	return popup.isVisible();
    } 
    
    public void mousePressed(MouseEvent evt)
    {  	
    		  	
    	if (evt.getClickCount() == 2)
    	{
    		DefaultTableModel model = (DefaultTableModel) table.getModel();
    		textField.setText((String) model.getValueAt(table.getSelectedRow(),coltToSelect));
    		setPopupVisible(false);
    	}
    	
    	textField.requestFocus();
          	
    }    
      
    public void mouseDragged(MouseEvent evt)
    {  	
    	    	
    }
    
    public void mouseMoved (MouseEvent evt)
    {  	
    	table.setRowSelectionInterval(table.rowAtPoint(evt.getPoint()), table.rowAtPoint(evt.getPoint()));
    }
    
    
    public void mouseReleased(MouseEvent evt)
    {  	
    	    	
    }
    
    public void mouseEntered(MouseEvent evt)
    {  	
        	
    }
    
    public void mouseExited(MouseEvent evt)
    {  	
    
    }  
    
    public void mouseClicked(MouseEvent evt)
    {  	
    
    }  
    
    public void keyTyped(KeyEvent evt)
    {  	
        	
    }
    
    public void keyPressed(KeyEvent evt)
    {  	
    	    	
    }
    
    public void keyReleased(KeyEvent evt)
    {  		    	
    	refreshMatchBox();    	        		   	             	
    }
    
    private void refreshMatchBox()
    {
    	
    	Vector <Vector> tempData = new Vector<Vector>();    	
    	
    	for (int i = 0; i < rowData.size(); i++)
    	{ 		
    		for (int j = 0; j < rowData.get(i).size(); j++) 
    		{
    			
    			if (rowData.get(i).get(j).toString().toLowerCase().contains(textField.getText().toLowerCase()))
    			{
    				
    				tempData.add(rowData.get(i));
    				j = rowData.get(i).size();
    			}
    			
    		}	
    	}
    	
    	table.setModel(new DefaultTableModel(tempData, columnNames) 
    	{
    		public boolean isCellEditable(int x, int y) 
    		{
    			return false;
    		}
    	});
    	
    	textField.requestFocus();
    	
    }   
        
    public void actionPerformed(ActionEvent evt)
    {
    	
    	refreshMatchBox();
    	setPopupVisible(true);
    	textField.requestFocus();
    	
    }
    
    public void setFont(Font font)
    {
    	
    	textField.setFont(font);
    	
    	table.setFont(font);
    	table.setRowHeight(font.getSize());
    	    	
    	button.setFont(font);
		button.setPreferredSize(null);
		button.setPreferredSize(new Dimension(button.getPreferredSize().width, textField.getPreferredSize().height -1));		
		
    	revalidate();
    	
    }
    
	public void focusLost(FocusEvent evt) 
	{
			
		System.out.println(evt.getComponent());
		
		if ((evt.getOppositeComponent() != textField) && 
		    (evt.getOppositeComponent() != button) &&
		    (evt.getOppositeComponent() != popup) &&
		    (evt.getOppositeComponent() != table) &&
		    (evt.getOppositeComponent() != getRootPane()))
		{	
			evt.setSource(this);	
			super.processFocusEvent(evt);		
		}
		
		
			
	}
	
	public void focusGained(FocusEvent evt) 
	{
	
		if ((evt.getComponent() != textField) || 
			(evt.getComponent() != button) ||
			(evt.getComponent() != popup) ||
			(evt.getComponent() != table) ||
			(evt.getComponent() != getRootPane()))
		{
				if ((evt.getOppositeComponent() != textField) && 
					    (evt.getOppositeComponent() != button) &&
					    (evt.getOppositeComponent() != popup) &&
					    (evt.getOppositeComponent() != table) &&
					    (evt.getOppositeComponent() != getRootPane()))
				{
					evt.setSource(this);
					super.processFocusEvent(evt);
				}
		}
	}
   	
}

schonmal danke für die hilfe

greetz

ViDi

 

[vidi]

hmmz ich bekomms net hin

 

[Wildcard]

Warum addest du das Popup?
Einfach mit dieser Methode anzeigen:
http://java.sun.com/javase/6/docs/api/javax/swing/JPopupMenu.html#show(java.awt.Component, int, int)

 

[vidi]

hmm,

wenn ich das

public void setPopupVisible (Boolean visible)
    {
       
        if(visible)
       {
            popup.setInvoker(textField);
          popup.setLocation(getLocationOnScreen().x, getLocationOnScreen().y + textField.getSize().height);
           popup.setPreferredSize(new Dimension(getSize().width, getSize().height * 5));
           popup.setVisible(true);
        }   
         else
         {
            popup.setVisible(false);
            popup.setInvoker(null);
         }
        
    }

in das

public void setPopupVisible (Boolean visible)
    {
    	
     	if(visible)
    	{
     		popup.setPreferredSize(new Dimension(getSize().width, getSize().height * 5));
      		popup.show(textField, getLocationOnScreen().x, getLocationOnScreen().y + textField.getSize().height);    	
        }	
      	else
      	{
      		popup.setVisible(false);
      		popup.setInvoker(null);
      	}
     	
    }

änder ist das problem immer noch da..
ich klick drück auf den suchen button, das popup geht auf, ich klick in eins von den textfeldern und das popup geht zu (so wie es soll).
drücke wieder auf den suchen button, das popup geht wieder auf, ich klick in die combobox und es passiert garnichts..

verhällt sich das ding bei dir anders??!?

 

[vidi]

die combobox löst einfach kein popupCancel Event aus..
die textfields, oder auch ein button macht das..sehr merkwürdig

 

[vidi]

kann mir keiner helfen :-( ?

 

[mwildam]

Ich habe grade ein Problem damit, dass das FocusGained-Event nicht gefeuert wird.

Habe folgenden Link gefunden ()www.codeguru.com/forum/showthread.php?t=38542
Darin steht zu lesen, dass sich eine JComboBox in Wirklichkeit aus mehreren Sub-Komponenten zusammensetzt. Möglicherweise liegt darin das Problem begründet.