Thread: Logical Problem

Hi,

The code below compiles without any errors. It's rather a logical problem. The actionPerformed method below loops through a list of Question objects ,While there is an element ahead using the hasNext() method where then if the button 'hitMe' is clicked will set text on a label, which all works fine. But however it loops through the entire list of before I can hit the button a second time. I am trying to list a question every time i click the 'hitMe' button?

Thanks in advanced

public void actionPerformed(ActionEvent e)
{
questionIterator = questions.listIterator();

for(Question currQuestion: questions)
{
while(questionIterator.hasNext())
{
if(e.getSource() == hitMe)
{
Question currQ = (Question) questionIterator.next();
questionLabel.setText(currQ.getName());
break;
}
}
}

Thanks for the reply. Sorry bad grammar.

After I hit the JButton 'hitMe', I am trying to print each Question object on the 'questionLabel' JLabel during the for loop. But however it loops through the entire 'questions' List before I can click the 'hitMe' button again, as I want each individual current Question on the label?

public void actionPerformed(ActionEvent e)
{
questionIterator = questions.listIterator();
 
for(Question currQuestion: questions)
{
while(questionIterator.hasNext())
{
if(e.getSource() == hitMe)
{
Question currQ = (Question) questionIterator.next();
questionLabel.setText(currQ.getName());
break;
}
}
}

After I hit the JButton 'hitMe',

The program should print a Question object within the for loop on the JLabel 'questionLabel',
Then I should be able to hit the JButton again which then the program will print the next Question object on the JLabel within the for loop,
until the end of Question List.

The program instead prints the last Question object on the List on the JLabel after I hit the 'hitMe' button. (It loops till the end of the list, rather waiting for me to hit the JButton 'hitMe' again.

public void actionPerformed(ActionEvent e)
{
	questionIterator = questions.listIterator();
 
	for(Question currQuestion: questions)
	{
		while(questionIterator.hasNext())
		{
		        if(e.getSource().equals(hitMe))
		        {
			        Question currQ = (Question) questionIterator.next();
			        questionLabel.setText(currQuestion.getName());
			        break ;
			}
		}
	}
}

Thanks for the reply.

So I would declare a variable where I instantiate the Listener class, and within in an if clause? Or how would I go about doing so?

Thanks

OK thanks.

I understand on preserving the value outside the method, but how could I increment an object (Question in this case) where it is not an integer?

Iv'e tried to modify the code to apply your context. Am I on the right track?

Thanks again

//ListenerClass
class ListenerClass implements ActionListener
{
	//Preserving value for next method call
	Question temp;
 
	public void actionPerformed(ActionEvent e)
	{
		questionIterator = questions.listIterator();
 
		if(e.getSource().equals(hitMe))
		{
			//iterate though Question objects
			while(questionIterator.hasNext())
			{
				Question currQ = (Question) questionIterator.next();
				//Set value on JLabel
				questionLabel.setText(currQ.getName());
 
				//Assigning current value to temp variable
				temp = currQ;
				//Wont compile?
				temp++;
				break;
			}
		}
	}
}

OK thanks. Hopefully this will be my last question.

Yea I could increment index value for the next method call.
Could I use a static variable, or maybe I could declare my Question List static? In the case of holding the data in memory during runtime?

I still do not have an idea on applying the preserved value to the Question List?

Thanks

OK. I have now defined the 'questions' next to 'index' in the ListenerClass.

I just now have created a getIndex(int index) method in the List class below:

public int getIndex(int index)
{
	int var;
	return var = listOfQuestions.indexOf(index);
}

Would that be correct in determining the index of the current object in the list?

Thanks

I'm not to sure, I maybe thought that the index could be returned from the current object's index location, but I realized that wont work.

See I'm pretty confused in my code.

I create my LinkedList<Question> questions reference,
Then I create an instance of the class where my questions are stored: LayoutOfQuestion, which is then passed in to the questions reference calling a specified method,
Then I create a ListIterator reference which assigns the questions reference using listIterator() method.

These are defined in my GUI class

LinkedList<Question> questions = new LinkedList<Question>();
LayoutOfQuestions layQuestions = new LayoutOfQuestions();
ListIterator<Question> questionIterator;

In the GUI constructor(), I assign the list of questions in the LinkedList :

questions = layQuestions.getMildQuestionList();

The actionPerformed() method then was previously listed above.

Thanks

Yes I've used that method many of times even in other various classes, but I still don't know how to apply that to any of my code

Ahh That worked perfectly, couldn't get my head around it, thanks alot for your patients.

Appreciated