combobox auf leeres Feld prüfen?

[HanneSG]

Warum bekomme ich einen NullPointer?

oder wie prüfe ich ob bei einer editierbaren combobox kein Text oben eingegeben wurde sprich "" ist?

if(e.getSource().equals(Button))
        {     
        	if(ComboBox.getSelectedItem().equals(""))
        	{
        		JOptionPane.showMessageDialog(this,"No Text added!","Error",JOptionPane.ERROR_MESSAGE);	 
    	            return; 
        	}

}

 

[Marco13]

Wenn nichts ausgewählt ist, liefert ComboBox.getSelectedItem() eben 'null'. (BTW: Variablennamen klein schreiben!). Ggf. solltest du sicherstellen, dass das, was eingefügt werden soll, gültig ist, bevor es eingefügt wird. Andernfalls musst du irgendwie dafür sorgen, dass das "leere" Element ausgewählt ist, oder die COmboBox durchsuchen, ob sie ein leeres Element enthält. Wenn's nicht klappt, beschreib' vielleicht das Problem nochmal genauer (am besten mit einem kleinen, compilierbaren Beispiel)

 

[HanneSG]

der user soll bei buttondruck text der oben in der JCB steht der JCB-liste unten hinzufügen können. Ist der Text aber "" sprich leer, dann soll die msgbox aufgehen. nur das equals.("") funzt nicht :/

 

[L-ectron-X]

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

public class JComboBoxTest {
   public static void main(String[] args) {
       final JFrame frame = new JFrame("JComboBoxTest");
       frame.setLayout(new FlowLayout());
       frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
       
       final JComboBox box = new JComboBox(new String[] {
          "Eins", "Zwei", "Drei"
       });
       box.setEditable(true);
       
       JButton button = new JButton("Add");
       button.addActionListener(new ActionListener() {
          public void actionPerformed(ActionEvent e) {
             Object obj = box.getSelectedItem();
             if(obj.equals("")) {
                JOptionPane.showMessageDialog(frame, "No text added!", "Error", JOptionPane.ERROR_MESSAGE);
             }
             else {
                for(int i = 0; i < box.getItemCount(); i++) {
                   if(obj.equals(box.getItemAt(i))) {
                      return;
                   }
                }
                box.addItem(obj);
             }
          }
       });
       
       frame.add(box);
       frame.add(button);
       
       frame.pack();
       frame.setLocationRelativeTo(null);
       frame.setVisible(true);

   }
}

if(e.getSource().equals(Button))

Wenn du mit getSource() die Referenz auf das Objekt des Ereignisses holst, kannst du nicht mit equals() vergleichen/prüfen. Referenzen werden mit dem boolschen Operator == auf Gleichheit geprüft.
Also so:

if(e.getSource() == button)

 

[HanneSG]

Also so:

if(e.getSource() == button)

e.getSource().equals(myButton) mache ich seit Ewigkeit so, so funktionierte es immer und so habe ich es in der Ausbildung auch gelernt...

 

[SlaterB]

ich wollt gestern auch schon fast schreiben, dass das im Grunde egal ist,
für die meisten Objekte macht equals eh das gleiche wie ==

und wenn equals überschrieben ist, dann sollte es zumindest für dasselbe Objekt true liefern
ist nicht ganz verkehrt, klappt schon, nur etwas unnötig umständlich

 

[L-ectron-X]

Gut, ich hab genau die andere Variante gelernt. Dann kann ja jeder bei dem bleiben, was er gelernt hat...

 

[HanneSG]

:lol: :toll:

 

[HanneSG]

komisch ist deine Variante geht, doch meine hat GENAU deselben code und logik stimmt eigentlich auch...

wenn das feld in der jcb leer ist und ich adde etwas dann kommt die JOptionPane soweit so gut... wenn das feld der jcb leer ist und ich del was bekomme ich einen nullpointer, doch warum?

public void actionPerformed(ActionEvent e)
	{		
		Object movieFormatObject = movieFormatCB.getSelectedItem();
		
		 
		if(e.getSource().equals(movieFormatAddBT))
        {  				
			if(movieFormatObject.equals(""))
			{
				JOptionPane.showMessageDialog(this, "No text added!", "Error", JOptionPane.ERROR_MESSAGE);
	            return;
			}
			else
			{
				movieFormatCB.addItem(movieFormatCB.getSelectedItem());
			}	
        } 
	        
        if(e.getSource().equals(movieFormatDelBT))
        {    
        	if(movieFormatObject.equals(""))
			{
				JOptionPane.showMessageDialog(this, "No text to delete!", "Error", JOptionPane.ERROR_MESSAGE);
	            return;
			}
			else
			{
				movieFormatCB.removeItem(movieFormatCB.getSelectedItem());
			}
        }

}

 

[HanneSG]

weiß keiner warum?

 

[L-ectron-X]

Es wäre einfacher dein Problem nachzuvollziehen, wenn du ein KSKB posten würdest.
Also ein Kleines selbstständiges kompilierbares Beispiel (Begriff wurde hier von André Uhres geprägt), in dem du das Wesentliche einbaust, so dass man es auch mal kompilieren und ausprobieren kann. Dann lässt sich der Fehler schneller finden.

Guck einfach zuerst mal nach, was in der Zeile steht, in der die NPE geworfen wird.