2013-09-27 128 views
0

循环将只运行两次 - 如果输入一个无效的id两次,它只会在第二次之后结束。没有错误,它就像它认为它应该结束一样。我没有看到validID变成true,因为它仍然不是id阵列中的数字之一。循环仅运行两次

import javax.swing.*; 
public class StudentIDArray { 
    public static void main(String[] args) { 
     int[] id = {1121, 1234, 2864, 3141, 4650, 5167, 5678, 6044, 7737, 9101}; 
     String[] name = {"Bryan", "Colleen", "David", "Frank", "Glennis", "Jerome", "Jessie", "Larry", "Steve", "Tina"}; 
     double[] gpa = {3.7, 3.2, 2.9, 3.5, 2.4, 3.8, 3.9, 3.9, 2.6, 2.2}; 

     final int STUDENTS = 10; 
     String idNumber; 
     int studentID; 
     double studentGPA = 0.0; 
     boolean validID = false; 
     String studentName = ""; 
     int x; 

     do{ 
      idNumber = JOptionPane.showInputDialog(null, "Enter the student ID number."); 
      studentID = Integer.parseInt(idNumber); 
      for(x = 0; x < STUDENTS; x++){ 
       if(studentID == id[x]){ 
        validID = true; 
        studentName = name[x]; 
        studentGPA = gpa[x]; 
       } 
      } 

      if(validID) { 
       JOptionPane.showMessageDialog(null, "ID number " + studentID + " belongs to " + studentName + " who has a GPA of " + studentGPA + "."); 
      } else { 
       JOptionPane.showMessageDialog(null, studentID + " is an invalid ID. Please try again."); 
       idNumber = JOptionPane.showInputDialog(null, "Enter the student ID number."); 
      } 
     } 
     while(validID = false); 
    } 
} 

当我尝试while(studentID != id[x])为做循环的结束,但随后它提供了一个数组索引越界异常。

回答

2

你犯了一个典型的错误:

while(validID = false); 

这应该使用等号==,而不是赋值运算符。

1

使用

while(!validID); 

你有什么现在

while(validID = false); 

是价值false分配给validID,并将它作为条件表达式。换句话说,它将始终评估为false,并且只会循环一次。

0

应该

while(validID == false); 
1

那是因为你曾经说过

while (validId = false)

将分配假以validId并在while循环计算错误,而是使用

while (validId == false)

,以便它检查的条件而不是分配一个值

0

首先问自己,“我应该创造一个对象?“

在面向对象的语言中,并行数组从不是一个好主意。

此:

int[] id = {1121, 1234, 2864, 3141, 4650, 5167, 5678, 6044, 7737, 9101}; 
String[] name = {"Bryan", "Colleen", "David", "Frank", "Glennis", "Jerome", "Jessie", "Larry", "Steve", "Tina"}; 
double[] gpa = {3.7, 3.2, 2.9, 3.5, 2.4, 3.8, 3.9, 3.9, 2.6, 2.2}; 

应该是:

public class Student { 
    private int id;  // Generate getter/setter 
    private String name; // Generate getter/setter 
    private double gpa; // Generate getter/setter 

    public Student(int id, String name, double gpa) { 
    this.id = id; 
    this.name = name; 
    this.gpa = gpa; 
    } 
} 

Student[] students = { new Student(1121, "Bryan", 3.7), ... }; 

要回答你的问题,只需添加一个!validId

if (!validID) 
+0

该分配需要平行阵列。 –

1

使用while(validID == false);,而不是while(validID = false);即使用双等于(====)而不是单等于(=),这是分配运算符。

后来将false指定为validID而不是比较,因此总是false