java GUI: while loop and if/else statement

Question

I am building a java GUI which asks the user to type in his user id and if I find a match in my text file list, then I display his info to the GUI panel. If his id is not found, then I display a prompt to ask him to type in his id again.

The program I am having right now is that every time after I type in a wrong id, even if the next id I type in is a correct one, I could not display the right info to the screen. My GUI stays at that "wrong id" sate forever. I spent hours trying to figure out what went wrong but I just could not find it. Any help would be appreciated!

The GUI screenshot

  private class okButtonListener implements ActionListener{
      public void actionPerformed(ActionEvent e){
          FileReader fr = null;
            try{
                fr = new FileReader("charList.txt");
            }
            catch (FileNotFoundException e1){
                e1.printStackTrace();
            }
            BufferedReader bf = new BufferedReader(fr);

            userID = Integer.parseInt(idField.getText());

            while(line != null){
                try{
                    line = bf.readLine();
                }
                catch (IOException e1){
                    e1.printStackTrace();
                }
                if (line != null){
                    fields = line.split("\t");
                    if (userID == Integer.parseInt(fields[0])){
                        System.out.println(fields[2]);
                        displayFieldsSelections();
                        resetFields();
                        break;
                    }
                }
            }

            if (userID != Integer.parseInt(fields[0])){
                mistakeResetFields();   
            }
      }
  }

First try to trace thru the code or put some System.err.println() to see where code may stuck. — PeterMmm
– PeterMmm, Commented Nov 9, 2013 at 17:10
try trim and then check if (userID == Integer.parseInt(fields[0].trim())){ — Ankit Rustagi
– Ankit Rustagi, Commented Nov 9, 2013 at 17:17
@PeterMmm Thank you so much! I declared line locally and it's working fine now! — user2350622
– user2350622, Commented Nov 9, 2013 at 17:17

APerson · Accepted Answer · 2013-11-09 17:58:34Z

1

I think the problem is that you are not declaring line locally. In the beginning of the method try declaring line.

answered Nov 9, 2013 at 17:58

APerson

7227 silver badges10 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

MadProgrammer Over a year ago

If you could explains why this is causing the OP an issue, I think it will go along way to helping them understand the problem, as I spent half an hour trying to figure out the problem and how your answer related to it ;)

Collectives™ on Stack Overflow

java GUI: while loop and if/else statement

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related