Vector

[Maya Naga]

hi am new to java collections...

in my program am doing calculations and adding those string result values to a vector...
i want to display those values one by one when i click the next and previous button..

when i code as follows only the last value is displayed in text field...

                             //this was given inside the calculation method

                        s	= lblnum1.getText();
				s1	= lblope.getText();
				s2	= lblnum2.getText();
				s3	= lbleql.getText();
				s4	= lblres.getText();
				string	= s+s1+s2+s3+s4;
				str	= string;
				
				v.addElement(str);
				objarray=new Object[v.size()];
				strobj=new String[v.size()];
									
						v.copyInto(objarray);
                              
                              // this i gave inside mouse listener method
                                       for(j=0;j<objarray.length;j++)
					{
						strobj[j]=objarray[j].toString();
						txtresult.setText(strobj[j]);
					}

by i want to display all values ..
what i want to do..
please help....

[Rob Spoor]

Check out JList.

[Pari Nanoti]

This is where i think the problem is

  # // this i gave inside mouse listener method  
#                                        for(j=0;j<objarray.length;j++)  
#                     {  
#                         strobj[j]=objarray[j].toString();  
#                         txtresult.setText(strobj[j]);  
#                     }  
# you'll  always get the last string here.

A) Logic:
1) You don't need MouseListener, you need ActionListener: one for each button (Next and Previous)

2) Don't loop through the collection, get the next or previous result strnig, based on what button was clicked, and set it in the txtresult

B) Optimization
1) No need of s,s1,s2,s3,s4, string and str variables, you can concatenate the result in the argument to v.addElement() itself.
2) There in no need to convert Vector into array, you can use the vector itself.
3) If you are using 1.5 or higher, go for generics, you can avoid the toString() call.

[Maya Naga]

thanks pari..
that worked