Thread: Need Urgent Help for Java assignment..

I am suppose to read a text file which contains info on cars and i have to display the filtered result on a defaultlistmodel....my code works fine when one combo box is chosen....but wen i give the condition if two combo boxes are chose it displays the || result on the list....how do i solve it....i need display only the && result on list(but the filter works just fine in the console)


Here is the code:-

public void actionPerformed(ActionEvent a) {
		// TODO Auto-generated method stub
			String result[];
       	 String combobox[]={combo[0].getSelectedItem().toString(),combo[1].getSelectedItem().toString(),
    			 combo[2].getSelectedItem().toString(),combo[3].getSelectedItem().toString(),
    			 combo[4].getSelectedItem().toString(),combo[5].getSelectedItem().toString()};
// search is a JButton..
			if(a.getSource()==search){
				list.clear();		
				try{
				FileInputStream fin=new FileInputStream ("carList.txt");
				 Scanner scanner = new Scanner(fin);
				 scanner.useDelimiter(";");
				 scanner.skip("// List of cars to use:");
				 char d='"';
				 while (scanner.hasNext()){
		        	 String s1=scanner.next().replace(d, ' ');
		        	 String s2=s1.replace("(", " ");
		        	 String s3=s2.replace(")", " ");
		        	 String s4=s3.replace("new","");
		        	 String s5=s4.replace(" ", "");
		        	 String s6=s5.replace("int", "*");
		        	 String s7=s6.replace("float", "");
		        	 result=scan.split(",");
 
 
//		        	When only one combobox option is chosen
		        	 String scan=s7.replace("CarSpec", "");
		           	   if(scan.contains(combobox[0])||scan.contains(combobox[1])||scan.contains(combobox[2])||scan.contains(combobox[3])
		           			 ||scan.contains(combobox[4])||scan.contains(combobox[5])){
 
		               	 list.addElement(result[2]+" "+result[3]+","+result[0]);
		        	 }
 
		        	 // when all the option are any
		           	if(combobox[0]=="Any"&&combobox[1]=="Any"&&combobox[2]=="Any"&&combobox[3]=="Any"&&
                 			combobox[4]=="Any"&&combobox[5]=="Any"){
		           		list.addElement(result[2]+" "+result[3]+","+result[0]);
		           	}
                                //when options from combobox1 and combobox2 are chosen...combobox[0] =combo[1].getSelectedItem().toString
		           	if(scan.contains(combobox[0])&&scan.contains(combobox[1])){
 
		           		list.addElement(result[2]+" "+result[3]+","+result[0]);
		           		System.out.println(result[2]+" "+result[3]+","+result[0]);
		            }
		            if(scan.contains(combobox[0])&&scan.contains(combobox[2])){
 
		           		list.addElement(result[2]+" "+result[3]+","+result[0]);
		           		System.out.println(result[2]+" "+result[3]+","+result[0]);
		            }
 
 
		         }

Please Help

Please Edit your post and wrap your code with[code=java]<YOUR CODE HERE>[/code] to get highlighting

wen i give the condition if two combo boxes are chose it displays the || result on the list

Can you explain what the code is supposed to do?
What is it supposed to display in the list?

One problem I see is the use of == to compare Strings. You should use the equals() method for comparing the contents of String objects.

Sorry for not being informative.....this code is suppose to read a text file line by line and add to the DefaultlistModel when a an item from a combo box is chosen.....my assignment is to filter the no. of cars When an option is chosen from the combobox.....its works fine wen one combobox is chosen but when two combo boxes are selected it displays all the items selected from 1st combo and 2nd combo.
below code is the code that is having issues:-

list.addElement(result[2]+" "+result[3]+","+result[0]);
System.out.println(result[2]+" "+result[3]+","+result[0]);
}
if(scan.contains(combobox[0])&&scan.contains(combobox[2])){
 
list.addElement(result[2]+" "+result[3]+","+result[0]);
System.out.println(result[2]+" "+result[3]+","+result[0]);
}
>

the && method doesnt work for the list but it works fine for the console are..
i think it calls the first if statement as well....please tell me if you need the whole code ...

&& is an operator, not a method.
Are you having trouble with the if conditions not being true (or false) when you think they should be?

Try debugging the code by adding a println before the if statement that prints out the values of all the variables used in the if statement so you can see what the computer sees. Be sure to add id Strings and delimiters so you the printed out put shows you values:
println("var=" + var + "<");
where you should replace var with the names of the variables in the program.

Edit your post and wrap your code with[code=java]<YOUR CODE HERE>[/code] to get highlighting

I found the problem

Please explain what the problem is.

What do the printlns show for the values used in those if statements?

Perhaps you want to use else if instead of if for all of your if statements after the first one. An else if statement ONLY executes if the preceeding if (or preceeding else if) evaluated to false.

Please show the code that made the print out you show in post#7.

The printout does not include the suggestions I made in post#4. How do you know what was in combobox[0] etc?

Also print out the contents of scan.

Moving this from the Member Introduction forum. Please read this: http://www.javaprogrammingforums.com...e-posting.html

You might also consider the link in my signature on asking questions the smart way, especially the bit about using informative titles.