Won't repaint!

[William Kohus]

I have to enter text into my jtextfield and have it show in the lbChoices jlist WHEN the focus is lost. It acts like it is there because I have a no duplicate line in there that appears when I try to type the same thing twice! Driving me crazy! Any help would be greatly appreciated. Thank you.

UserEntryBox.addFocusListener(new FocusAdapter() {
			@Override
				public void focusLost(FocusEvent arg0) {
				// Initialize model
				DefaultListModel model = (DefaultListModel) lbChoices.getModel();   
				
				// If model already contains entered text 
				// This will PREVENT duplicates
				if (model.contains(UserEntryBox.getText()))
					  
				{  
					  // Print Statement to tell user no duplicates can be allowed
					  JOptionPane.showMessageDialog(btnReport, "No duplicates please!");  
				    
				}  
				  // Else add value to lbChoices
				else model.addElement(UserEntryBox.getText()); 
				
				UserEntryBox.setText("");
				 
				lbChoices.repaint();
				  
			}  		
			
		});

HERE IS MY FULL CODE

 * ListboxMania
package ListBoxManiaPackage;

/**
 * Java Imports
 */
import java.awt.BorderLayout;

/**
 * This is an public method that extends JFrame.
 * @param Listed buttons, lists, text fields and various other parameters declared below that comprise the NFL Betting Program
 */
public class ListBoxMania extends JFrame {

	// Declare contentPane as main canvas
	private JPanel contentPane;
	// *** Initialize the Selections list box to use a Model that supports adding and removing ***
	private final JList lbSelection = new JList(new DefaultListModel());
	// *** Initialize the Single add button that supports adding to Selected***
	private final JButton btnAddToSelected = new JButton(">");
	// *** Initialize the Single add button that supports removing from Selected***
	private final JButton btnRemoveFromSelected = new JButton("<");
	// *** Initialize the multi add button that supports adding ALL to Selected***
	private final JButton btnAddAll = new JButton(">>");
	// *** Initialize the multi remove button that supports removing from Selected***
	private final JButton btnRemoveAll = new JButton("<<");
	// *** Initialize the report button that will report selected teams in Selected list***
	private final JButton btnReport = new JButton("Report");
	// *** Initialize label for the count of selected teams in Selected list***
	private final JLabel lblSelectedCount = new JLabel("Count of Selected:");
	// *** Initialize the Choices list***
	private final JList lbChoices = new JList();
	// *** Initialize the user text box for entering data into the Choices list***
	private final JTextField UserEntryBox = new JTextField();
	// *** Initialize the label for names selected after the report button is clicked***
	private final JLabel lblNamesSelected = new JLabel("Teams Selected:");
	// *** Initialize the jl label***
	private final JLabel jl = new JLabel();
	// *** Initialize the lable for the NFL Icon***
	private final JLabel lblIcon = new JLabel("");
	// *** Initialize and set the location for the NFL icon***
	ImageIcon image = new ImageIcon("resources/nfl.png");
	// *** Initialize a spinner for the selection of total points***
	private final JSpinner sTotalPoints = new JSpinner();
	// *** Initialize radio buttons for winning team selection***
	private final JRadioButton rdbtnTeam1 = new JRadioButton("Team 1");
	private final JRadioButton rdbtnTeam2 = new JRadioButton("Team 2");
	// *** Initialize label for total points***
	private final JLabel lblTotalPoints = new JLabel("Total Points");
	// *** Initialize the label for choose winner***
	private final JLabel lblChooseWinner = new JLabel("Choose Winner");
	// *** Initialize the Single button for submit***
	private final JButton btnSubmit = new JButton("Submit");
	// *** Initialize the label for select two teams***
	private final JLabel lblSelectTwoTeams = new JLabel("Select Two Teams!");
	// *** Initialize the Single button for exit***
	private final JButton btnExit = new JButton("Quit");
	private final JButton btnAdd = new JButton("add");
	
	/**
	 * Launch NFL picks application.
	 */
	public static void main(String[] args) {
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					ListBoxMania frame = new ListBoxMania();
					frame.setVisible(true);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}
		});
	} 

	/**
	 * Create the frame.
	 */
	public ListBoxMania() {
		// Set Title for JFrame
		setTitle("AFC Team Challenge");

		// Set size and details for User Entry Box
		UserEntryBox.setBounds(31, 13, 136, 20);
		UserEntryBox.setColumns(10);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(100, 100, 471, 467);

		// Create Content Pane
		contentPane = new JPanel();
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		contentPane.setBackground(Color.red);
		setContentPane(contentPane);
		contentPane.setLayout(null);
		
		/**
		 * Create array of values for lbChoices
		 * Supports a model that sets values
		 * These are the initial values for the choices list
		 * @return values.length/values
		 */		

		lbChoices.setModel(new DefaultListModel() {
			String[] values = new String[] {"Bengals", "Steelers", "Browns", "Ravens"};
			// Get size
			public int getSize() {
				// Return value length
				return values.length;
			
			}
			
			public Object getElementAt(int index) {
			
				return values[index];
			
			}
		
		});
		
		// Set sizes/colors for lbChoices
		lbChoices.setBounds(31, 44, 136, 133);
		lbChoices.setBackground(Color.blue);		
		lbChoices.setForeground(Color.white);			
		contentPane.add(lbChoices);
		
		/**
		 * Create focus listener for user entry box
		 * Supports a model that allows entered text
		 * When Text is entered and focus is lost the data will be entered into Choices label
		 */		
		UserEntryBox.addFocusListener(new FocusAdapter() {
			@Override
				public void focusLost(FocusEvent arg0) {
				
				// Initialize model
				DefaultListModel model = (DefaultListModel) lbChoices.getModel();   
				
				// If model already contains entered text 
				// This will PREVENT duplicates
				if (model.contains(UserEntryBox.getText()))
					  
				{  
					  // Print Statement to tell user no duplicates can be allowed
					  JOptionPane.showMessageDialog(btnReport, "No duplicates please!");  
				    
				}  
				  // Else add value to lbChoices
				else model.addElement(UserEntryBox.getText()); 
				
				UserEntryBox.setText("");
				
				lbChoices.setModel(model);
				
				lbChoices.repaint();
				
			}  		
			
		});

		// Set sizes/colors for lbSelection
		lbSelection.setBounds(284, 44, 136, 133);
		lbSelection.setBackground(Color.blue);		
		lbSelection.setForeground(Color.white);			
		contentPane.add(lbSelection);
		
		/**
		 * Create action single add button
		 * When value is selected from Choices and this button is pressed it adds the selected value to Selected values
		 * This Listener PREVENTS duplicates
		 */		
		btnAddToSelected.addMouseListener(new MouseAdapter() {
			@Override
			public void mouseClicked(MouseEvent e) {
			
				// Append an item from the choices ListBox to the selected ListBox.
				// This does not check for an item that has already been selected: it will be added again.
				  DefaultListModel model = (DefaultListModel) lbSelection.getModel();  
				  // If duplicate value exists
				  if (model.contains(lbChoices.getSelectedValue()))  
				  
				  {  
					  // Print Statement
					  JOptionPane.showMessageDialog(btnReport, "No duplicates please!");  
				    
				  }  
				  // Else add value
				  else model.addElement(lbChoices.getSelectedValue());  
				  
			}  		

		});
		
		// Set sizes for add button
		btnAddToSelected.setBounds(179, 80, 95, 25);
		
		// Add button to content pane
		contentPane.add(btnAddToSelected);
		
		// Create event for single button remove
		btnRemoveFromSelected.addMouseListener(new MouseAdapter() {
		@Override
			public void mouseClicked(MouseEvent e) {
				
				// Create model 
				DefaultListModel model = (DefaultListModel) lbSelection.getModel();

				// remove selected value from lbSelection
				model.removeElementAt(lbSelection.getSelectedIndex());			
			}
	
		});
		
		// Set Size position for button
		btnRemoveFromSelected.setBounds(179, 116, 95, 25);
		// Add button to contentPane
		contentPane.add(btnRemoveFromSelected);

		/**
		 * Create action for add all button
		 * When this button is pressed it adds all values to Selected values
		 * This Listener PREVENTS duplicates
		 */		
		btnAddAll.addMouseListener(new MouseAdapter() {
			@Override
			public void mouseClicked(MouseEvent e) {
			
				// Create models
				DefaultListModel choicesModel  = (DefaultListModel) lbChoices.getModel();
								
				DefaultListModel selectionModel  = (DefaultListModel) lbSelection.getModel();
				
				// Declare max
				int max = choicesModel.getSize();
				
				// Create loop for add all
				for (int i = 0; i < max; i++) {
						// If duplicate exists
					    if (selectionModel.contains(lbChoices.getSelectedValue()))  
					    
					    {  
					    	// Print statement
					    	JOptionPane.showMessageDialog(btnReport, "No duplicates please!");  
					    
					    }  
					    // Else add selection
					    else selectionModel.addElement(choicesModel.getElementAt(i));
						  
					  }  
			}
		});
		// Set Size position for add all button
		btnAddAll.setBounds(179, 44, 95, 25);
		
		// Add button to contentPane
		contentPane.add(btnAddAll);
		
		// Create action for remove all button
		btnRemoveAll.addMouseListener(new MouseAdapter() {
			@Override
			public void mouseClicked(MouseEvent e) {
				// Clear all 
				ClearSelection();
			}
		});
		// set size and position for remove all button
		btnRemoveAll.setBounds(179, 152, 95, 25);
		
		// Add remove all button
		contentPane.add(btnRemoveAll);
		
		/**
		 * Create action for report button
		 * When this button is pressed it counts selected values and displays count in lbSelectedCount and displays array of teams selected
		 * This Listener will display error message when NO teams are selected to report
		 */		
		btnReport.addActionListener(new ActionListener() {
		
			public void actionPerformed(ActionEvent arg0) {
				
				// Create selected array
				int selected[] = lbSelection.getSelectedIndices();
				
				// If nothing is selected
				if (selected.length <= 0){
					// Print statement
					JOptionPane.showMessageDialog(btnReport, "Must select at least one teams to report!");
					
				}
				// For loop for length of selected
				for (int i = 0; i < selected.length; i++) {
					
					// Declare item
					Object item = lbSelection.getModel();
				
					// Print item
					lblNamesSelected.setText("The Teams Selected are: " + item);
					
				}
				
				// Declare size
				int size = lbSelection.getModel().getSize();
				
				// For loop for size
				for (int i = 0; i < size; i++){						
					
					// Print count of teams
					lblSelectedCount.setText("The Count of Teams is " + lbSelection.getModel().getSize());
				
				}
			}
		});
		
		// Set size and position for report button
		btnReport.setBounds(179, 186, 95, 23);

		// Add button report button to content Pane
		contentPane.add(btnReport);
		
		// Set size, color and position
		lblSelectedCount.setBounds(118, 222, 215, 14);
		lblSelectedCount.setForeground(Color.white);		
		contentPane.add(lblSelectedCount);
		
		
		contentPane.add(UserEntryBox);
		// Set size and position
		lblNamesSelected.setBounds(10, 237, 410, 14);
		
		contentPane.add(lblNamesSelected);
		contentPane.add(jl);

		lblNamesSelected.setForeground(Color.white);		
		// Set size and position
		lblIcon.setBounds(40, 288, 186, 141);
		
		// Set Icon as image
		lblIcon.setIcon(image);
		// Add icon to contentPane
		contentPane.add(lblIcon);
		
		// Set size color of spinner
		sTotalPoints.setModel(new SpinnerNumberModel(2, 2, 200, 1));
		sTotalPoints.setForeground(Color.white);
		sTotalPoints.setBounds(389, 324, 47, 20);
		
		// Add totalPoints to content pane/ set colors
		contentPane.add(sTotalPoints);
		sTotalPoints.setBackground(Color.red);
		sTotalPoints.setForeground(Color.white);

		// Set size and color of buttons for team1 and 2
		rdbtnTeam1.setBounds(327, 253, 109, 23);
		rdbtnTeam1.setForeground(Color.white);		
		rdbtnTeam1.setBackground(Color.red);		
		rdbtnTeam2.setForeground(Color.white);		
		rdbtnTeam2.setBackground(Color.red);						
		rdbtnTeam2.setBounds(327, 288, 109, 23);
		rdbtnTeam2.setForeground(Color.white);		
		rdbtnTeam2.setBackground(Color.red);		

		// Add radio buttons to contentPane
		contentPane.add(rdbtnTeam1);

		contentPane.add(rdbtnTeam2);

		// Set size and color of total points label
		lblTotalPoints.setBounds(302, 326, 79, 14);
		lblTotalPoints.setForeground(Color.white);
		
		// Add total points label to content pane
		contentPane.add(lblTotalPoints);

		// Add choose winner label to content pane and set sizes/colors
		contentPane.add(lblChooseWinner);
		lblChooseWinner.setBounds(211, 276, 117, 14);
		lblChooseWinner.setForeground(Color.white);		
		lblChooseWinner.setBackground(Color.red);	
		
		/**
		 * Create action for submit button
		 * When this button is pressed a verification window pops up
		 * At this time no database has been set to collect information
		 */		
		btnSubmit.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) {
				
				// Print statement
				JOptionPane.showMessageDialog(btnReport, "Thank you!  Your Picks Have Been Submitted!");				
			}
		});
		
		// Set size and position		
		btnSubmit.setBounds(238, 391, 95, 25);
		
		contentPane.add(btnSubmit);
		lblSelectTwoTeams.setForeground(Color.WHITE);
		lblSelectTwoTeams.setFont(new Font("Rockwell Extra Bold", Font.PLAIN, 20));
		// Set size and position
		lblSelectTwoTeams.setBounds(199, 17, 244, 14);
		
		contentPane.add(lblSelectTwoTeams);

		/**
		 * Create action for exit button
		 * When this button is clicked the program will exit 
		 */		

		btnExit.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				
				System.exit(0);
			}
		});
		
		// Set size and location of exit button
		btnExit.setBounds(348, 391, 95, 25);
		
		// Add exit button to content pane
		contentPane.add(btnExit);
		
		}
	
	/**
	 * Method for clearing selection
	 * Clear all the selected items by erasing the lbSelected listBox
	 * @param selectionModel
	 */		
	private void ClearSelection() {
		DefaultListModel selectionModel  = (DefaultListModel) lbSelection.getModel();
		selectionModel.clear();
	}
}

[Tina Smith]

Do you mean that the text isn't in the JList before you enter the new list item, but you get a no duplicates message and then the text still isn't in the JList?

The JList should repaint when the entry is added, and there should be no need to repaint when the list contents are not modified. Repainting a JList will repaint each element in the list. To ensure the list is repainted (equivalent to repaint() call) is cover up the JList and uncover it. This will force all elements in the list to rerender themselves.

[Michael Dunn]

it's likely to be the way you've created the initial model

change

/*
lbChoices.setModel(new DefaultListModel() {
  String[] values = new String[] {"Bengals", "Steelers", "Browns", "Ravens"};
  // Get size
  public int getSize() {
    // Return value length
    return values.length;
  }
  public Object getElementAt(int index) {
    return values[index];
  }
});
*/
to this

DefaultListModel dlm = new DefaultListModel();
String[] values = new String[] {"Bengals", "Steelers", "Browns", "Ravens"};
for(int x = 0, y = values.length; x < y; x++) dlm.addElement(values[x]);
lbChoices.setModel(dlm);

[William Kohus]

Michael Dunn wrote:it's likely to be the way you've created the initial model

change

Michael Dunn, you are the man. Thanks!