best way to update a JList

[Edmund Castermund]

Hi,
I have an app I'm working on that reads an xml file, creates a map of objects, I then use the map to populate the list.

I have a search window that I want to use to narrow the list; when a user enters a character, I have a method that loops through the list, looks for that string, and only places matching objects in the new list.

So far it works in terms of what is viewable on the panel, but in the narrowed list I can't select anything....

What's the best way to accomplish this task?

   	public DefaultListModel getJobListData(String text){
		session.notifySubscribers(new Message("LOG", "getting joblist data with search term"));
		DefaultListModel model = new DefaultListModel();

		for(Iterator i = jobMap.keySet().iterator(); i.hasNext(); ){
			String key = (String)i.next();
			JobDescription jd = jobMap.get(key);
			if(jd.getTitle().contains(text) || jd.getDescription().contains(text)){
				model.addElement(jd.getTitle());	
			}
				
						
			
		}
		
		session.notifySubscribers(new Message("LOG", "done getting job list data with search term"));
		return model;
	}
	
	 

I notice that I'm not removing the list that exists on the panel....maybe I need to do that. Any other ideas?
bp

[Miklos Szeles]

Hi Edmund,

Please post an Short, Self Contained, Correct (Compilable), Example.
From this code I can't see anything. You create a new DefaultListModel and that's all. How do you change your list's model? The content of the listmodel shouldn't prevent you to select anything, so my guessing is that there's some problem in other part of your code.

[Edmund Castermund]

sorry, here's the method that adds the new list to the JPanel:

   	public class SearchAreaListener implements KeyListener{

		public void keyPressed(KeyEvent ev) {
			// TODO Auto-generated method stub
			//System.out.println("key pressed: " + searchArea.getText());
		}

		public void keyReleased(KeyEvent arg0) {
			// TODO Auto-generated method stub
			//System.out.println("key released: " + searchArea.getText());
		}

		public void keyTyped(KeyEvent ev) {
			// TODO Auto-generated method stub
			System.out.println("key typed: " + searchArea.getText());
			jobList = new JList(getJobListData(searchArea.getText()));
			jobList.setBorder(BorderFactory.createEmptyBorder(10,25,0,0));
			jobList.setFont(new Font("courier", Font.PLAIN, 18));
			
			JScrollPane sp = new JScrollPane(jobList);
			sp.setBorder(BorderFactory.createBevelBorder(1));
			mainPanel.add(sp, BorderLayout.CENTER);
			
		}
	
	
	}
 

bp

[Rob Spoor]

Try reusing the same JList but changing the model instead. So instead of

jobList = new JList(getJobListData(searchArea.getText()));   
jobList.setBorder(BorderFactory.createEmptyBorder(10,25,0,0));   
jobList.setFont(new Font("courier", Font.PLAIN, 18));   
               
JScrollPane sp = new JScrollPane(jobList);   
sp.setBorder(BorderFactory.createBevelBorder(1));   
mainPanel.add(sp, BorderLayout.CENTER);   

you simply call

jobList.setModel(getJobListData(searchArea.getText())); 

That way, all the other stuff like listeners, fonts, borders etc will stay.

[Edmund Castermund]

that worked perfectly,thanks!
bp