Swing JList und veränderte CellRenderer

[KingOfExceptions]

Ich bin gerade dabei, mich mit JLists zu befassen. Ich habe nun eine JList mit einem JButton und einer JCheckBox.
Damit die beiden auch angezeigt werden, habe ich die CellRenderer verändert. Nun ist mein Problem, dass
1. Die Farben ohne Selection nicht passen (siehe Bild 1).
2. Die Farben mit Selection nicht passen (siehe Bild 2).
3. Der Text auf dem JButton erst nach einem ersten Klick erscheint (siehe CellEditor).

Ich hoffe, jemand hat die Lust und das nötige Handwerkszeug hier mal drüber zu schauen.
BTW: ich benutze NICHT das null-Layout

1.Teil: Table wird zum Teil erstellt

table.getColumn("Supprimer").setCellRenderer(
						new DefaultTableCellRenderer()
						{

							private static final long serialVersionUID = 1L;

							public Component getTableCellRendererComponent(
									JTable table, Object value, boolean isSelected,
									boolean hasFocus, int row, int column)
							{
								JCheckBox checkBox = new JCheckBox();
								checkBox.setSelected(false);
								
								isSelected=true;
								if (isSelected)
								{
									setForeground(table.getSelectionForeground());
									setBackground(table.getSelectionBackground());
								} else
								{
									setForeground(table.getForeground());
									setBackground(UIManager
											.getColor("JCheckBox.background"));
								}

								return checkBox;
							}
						});

3. Teil: Button Renderer

class ButtonRenderer extends JButton implements TableCellRenderer
	{

		private static final long serialVersionUID = 1L;

		public ButtonRenderer()
		{
			setOpaque(true);
		}

		public Component getTableCellRendererComponent(JTable table, Object value,
				boolean isSelected, boolean hasFocus, int row, int column)
		{
			if (isSelected)
			{
				setForeground(table.getSelectionForeground());
				setBackground(table.getSelectionBackground());
			} else
			{
				setForeground(table.getForeground());
				setBackground(UIManager.getColor("Button.background"));
			}
			setText((value == null) ? "" : value.toString());
			return this;
		}
	}

4. Teil: ButtonEditor

class ButtonEditor extends DefaultCellEditor
	{

		private static final long serialVersionUID = 1L;

		protected JButton button;
		
		private boolean isPushed;

		public ButtonEditor(JCheckBox checkBox)
		{
			super(checkBox);
			button = new JButton("Plus d'infos");
			button.setOpaque(true);
			button.addActionListener(new ActionListener()
			{
				public void actionPerformed(ActionEvent e)
				{
					fireEditingStopped();
				}
			});
		}

		public Component getTableCellEditorComponent(JTable table, Object value,
				boolean isSelected, int row, int column)
		{
			if (isSelected)
			{
				button.setForeground(table.getSelectionForeground());
				button.setBackground(table.getSelectionBackground());
			} else
			{
				button.setForeground(table.getForeground());
				button.setBackground(table.getBackground());
			}
			button.setText("Plus d'infos");
			isPushed = true;
			return button;
		}

		public Object getCellEditorValue()
		{
			if (isPushed)
			{
				// HIER MUSS DAS REIN; WAS NACH EINEM KLICK AUF DEN "PLUS D'INFO" BUTTON PASSIERT
				// ZB EIN NEUES FENSTER MIT EINEM JTABLE MIT ALLEN ABHOLUNGEN UND EINEM BUTTON FÜR
				// NEUE ABHOLUNGEN !!!
			}
			isPushed = false;
			return new String("Plus d'infos");
		}

		public boolean stopCellEditing()
		{
			isPushed = false;
			return super.stopCellEditing();
		}

		protected void fireEditingStopped()
		{
			super.fireEditingStopped();
		}
	}

5. Teil: CheckBoxEditor

class CheckBoxEditor extends DefaultCellEditor
	{

		private static final long serialVersionUID = 1L;

		private JCheckBox check;
		private boolean isPushed;

		public CheckBoxEditor(JCheckBox checkBox)
		{
			super(checkBox);
			check = new JCheckBox();
			check.setOpaque(true);
			check.addActionListener(new ActionListener()
			{
				public void actionPerformed(ActionEvent e)
				{
					fireEditingStopped();
				}
			});
		}

		public Component getTableCellEditorComponent(JTable table, Object value,
				boolean isSelected, int row, int column)
		{
			isSelected=true;
			if (isSelected)
			{
				check.setForeground(table.getSelectionForeground());
				check.setBackground(table.getSelectionBackground());
			} else
			{
				check.setForeground(table.getForeground());
				check.setBackground(table.getBackground());
			}
			isPushed = true;
			return check;
		}

		public Object getCellEditorValue()
		{
			if (isPushed)
			{
				JOptionPane.showMessageDialog(null, "Client supprimé, re-ouvrez 'tous les clients'");
				//ES WIRD NOCH NIEMAND GELÖSCHT!!!
				System.out.println(table.getSelectedRow()) ;
				table.remove(table.getSelectedRow());
				creator.getClients().remove(table.getSelectedRow());
				table.validate();
				table.repaint();
			}
			isPushed = false;
			return Boolean.TRUE;
		}

		public boolean stopCellEditing()
		{
			isPushed = false;
			return super.stopCellEditing();
		}

		protected void fireEditingStopped()
		{
			super.fireEditingStopped();
		}
	}

Vielen Dank!

 

[bERt0r]

Also erstens mal ist das ein JTable und keine JList. Ein JTable hat neben einem Renderer auch noch einen Editor: Der Renderer zeigt nur an, im Editor kann man auch was ändern.
Zweitens, was heißt die Farben passen nicht? Du setzt bei deinen Buttons ja explizit die Farben um? Wenn du die normale Hintergrundfarbe haben willst, dann lass das umsetzen doch einfach... table.getBackground ist eben Weiß und nicht Button-Grau.
Ich werd nacher, nochmal drübergucken - jetzt gibts Fußball

 

[KingOfExceptions]

So ich hab jetzt auch zu Ende geguckt... Schade das mit Portugal, aber egal.

Ich war brav, hab alles farbige aus meinem Code gestrichen und jetzt sieht er so aus:

class ButtonRenderer extends JButton implements TableCellRenderer
	{

		private static final long serialVersionUID = 1L;

		public ButtonRenderer()
		{
		}

		public Component getTableCellRendererComponent(JTable table, Object value,
				boolean isSelected, boolean hasFocus, int row, int column)
		{
			setText((value == null) ? "" : value.toString());
			return this;
		}
	}

class ButtonEditor extends DefaultCellEditor
	{

		private static final long serialVersionUID = 1L;

		protected JButton button;

		private boolean isPushed;

		public ButtonEditor(JCheckBox checkBox)
		{
			super(checkBox);
			button = new JButton("Plus d'infos");
			button.addActionListener(new ActionListener()
			{
				public void actionPerformed(ActionEvent e)
				{
					fireEditingStopped();
				}
			});
		}

		public Component getTableCellEditorComponent(JTable table, Object value,
				boolean isSelected, int row, int column)
		{
			
			button.setText("Plus d'infos");
			isPushed = false;
			isSelected=false;
			return button;
		}

		public Object getCellEditorValue()
		{
			if (isPushed)
			{
				// HIER MUSS DAS REIN; WAS NACH EINEM KLICK AUF DEN "PLUS D'INFO" BUTTON PASSIERT
				// ZB EIN NEUES FENSTER MIT EINEM JTABLE MIT ALLEN ABHOLUNGEN UND EINEM BUTTON FÜR
				// NEUE ABHOLUNGEN !!!
			}
			isPushed = false;
			return new String("Plus d'infos");
		}

		public boolean stopCellEditing()
		{
			isPushed = false;
			return super.stopCellEditing();
		}

		protected void fireEditingStopped()
		{
			super.fireEditingStopped();
		}
	}

class CheckBoxEditor extends DefaultCellEditor
	{

		private static final long serialVersionUID = 1L;

		private JCheckBox check;
		private boolean isPushed;

		public CheckBoxEditor(JCheckBox checkBox)
		{
			super(checkBox);
			check = new JCheckBox();
			check.addActionListener(new ActionListener()
			{
				public void actionPerformed(ActionEvent e)
				{
					fireEditingStopped();
				}
			});
		}

		public Component getTableCellEditorComponent(JTable table, Object value,
				boolean isSelected, int row, int column)
		{
			isSelected = true;
			
			isPushed = true;
			return check;
		}

		public Object getCellEditorValue()
		{
			if (isPushed)
			{
				JOptionPane.showMessageDialog(null,
						"Client supprimé, re-ouvrez 'tous les clients'");
				System.out.println(table.getSelectedRow());
				table.remove(table.getSelectedRow());
				creator.getClients().remove(table.getSelectedRow()); 
				table.validate();
				table.repaint();
				//LÖSCHUNG KANN NUR FÜR EINEN CLIENT AUF EINMAL GEHEN!!! WIESO?	
			}
			isPushed = false;
			return Boolean.FALSE;
		}

		public boolean stopCellEditing()
		{
			isPushed = false;
			return super.stopCellEditing();
		}

		protected void fireEditingStopped()
		{
			super.fireEditingStopped();
		}
	}

table.getColumn("Supprimer").setCellRenderer(
						new DefaultTableCellRenderer()
						{

							private static final long serialVersionUID = 1L;

							public Component getTableCellRendererComponent(
									JTable table, Object value, boolean isSelected,
									boolean hasFocus, int row, int column)
							{
								JCheckBox checkBox = new JCheckBox();
								checkBox.setSelected(false);

								isSelected = true;
						
								return checkBox;
							}
						});

(Ich weiss auch nicht, wie ich auf JList gekommen bin, vielleicht kann ja ein Mod den Titel des Threads hier löschen, Danke!)

Jetzt funktioniert leider auch nicht alles so, wie es sollte. Wenn es jemand ausprobieren möchte, um sich ein besseres Bild zu verschaffen: siehe Anhang.
Ansonsten mal eine kurze Beschreibung: Der Button in der ersten Reihe ist grau, wenn man einmal drauf klickt, wird er weiß. Der Button in der zweiten Reihe ist schon weiß.
Die erste Reihe ist mit Ausnahme vom Button weiß, wenn man sie anklickt, wird die JCheckBox ausgelassen, erst wenn man auf diese klickt, wird sie für kurze Zeit blau und dann wieder weiss.
Das gleiche Problem mit der JCheckBox in Zeile 2.
Zeile 2 ist leicht gräulich gefärbt, sie wird durch nichts weiß, der Button in Zeile 2 ist weiß und bleibt auch so.

 

[KrokoDiehl]

Das mit den Farben war größtenteils schon ok, nur hast du nicht die Hintergrundfarbe von der JTable verwendet, also folgender Code hilft meistens:

if (isSelected) {
    comp.setForeground( table.getSelectionForeground() );
    comp.setBackground( table.getSelectionBackground() );
}
else {
    comp.setForeground( table.getForeground() );
    comp.setBackground( table.getBackground() );
}

Bei Checkbox, Buttons etc. kann es noch sein, dass man weitere Eigenschaften setzen muss, sowas wie

setOpaque()

,

setBorderPainted()

oder was es ggfs. noch gibt. Ansonsten sollte das passen.

Nebenbei bemerkt halte ich es beim Renderer für eine schlechte Idee immer wieder eine neue JCheckBox zu erzeugen. Mach mal ein System.out in den Renderer, dann siehst du wie oft der aufgerufen wird ;-)

 

[bERt0r]

Um eine Checkbox im Table darzustellen würde es schon genügen die Column Identifiers im Model richtig zu setzen. Wenn du da die Spalte auf Boolean setzt macht dir der DefaultRenderer eine Checkbox draus, ohne dass du irgendwas überschreiben musst.
Bzgl. Button: Ich denke nicht dass du beim Button irgendwelche Hintergrund/Vordergrundfarben ändern solltest. Ich hab mal gegraben und das hier gefunden, guck dirs an:Anhang anzeigen SaveButtonTable.jar

 

[KingOfExceptions]

@KrokoDiehl:
Genau das hatte ich ja vorher, aber damit hatte ich ja leider keinen Erfolg. Somit liegt es glaube ich nicht daran (?)
Auch mit setOpaque() usw konnte ich keinen Erfolg verbuchen.

@bERt0r:
Könntest du mir das mit der CheckBox nochmal bitte erläutern, was du damit meinst? Ich hab das nicht ganz verstanden.
Wegen des Buttons: Hättest du da auch noch Quellcode von?

BTW: Ich habe das ganze jetzt mal mit dem Standart-Look-And-Feel ausprobiert, und da gibt es gar keine Probleme mit dem Button oder der JCheckBox.
Könnte es sein, dass das Nimbus-Look-And-Feel nicht ganz Bug-frei ist?
Dabei hat es mir doch so gut gefallen ;(
Mit dem Windows-Look-And-Feel läuft alles ohne Probleme...

Ausserdem scheint es mir, dass das Numbus-Look-And-Feel die Zeilennummern vertauscht. Will ich denn dann auf die Objekte in den Zeilen zugreifen, ist es beim ersten Zugriff noch gar kein Problem, aber dann beim zweiten Zugriff, wirft er mir eine OutOfBoundsException.
Das kommt beim Windows-Look-And-Feel nicht vor.

 

[KingOfExceptions]

[EDIT]Ich korrigiere: NimbusLookAndFeel vertauscht die Reihen nicht, ich hatte einen Fehler in meinem restlichen Programm.[/EDIT]

 

[bERt0r]

Der ButtonRenderer/Editor sollte in dem SaveButtonTable jar dabei sein.
Das DefaultTableModel hat die Funktion getColumnClass. Wenn man diese Funktion überschreibt, kann man bestimmen, von welchem Datentyp die jeweiligen Spalten sind. Der JTable bzw. der Renderer (weiß nicht genau, ist intern) fragt diese Informationen ab und wählt dann die entsprechende Darstellung. Z.B ein Textfeld bei String oder eine Checkbox bei Boolean.

Beim JTable kann man infolge auch noch Renderer für Datentypen festlegen, wenn sie in der getColumnClass definiert sind. Checkboxen kann er aber von Haus aus

 

[KingOfExceptions]

Kennt sich ansonsten noch jemand mit den LookAndFeels aus? Die Farben passen nämlich immernoch nicht und das von Windows ist so hässlich