在调试和运行期间由于某种原因跳过内部if语句

时间:2013-04-25 22:12:20

标签: java if-statement for-loop nested-if

当我单步执行代码时,内部if语句被完全跳过它永远不会进入updateGrade。它只是完全跳过了。为什么呢?

这是有问题的方法:

public void updateCourse(String courseID, String newLetterGrade){

    for(int i=0; i<classesTaken.size(); i++){

        if(classesTaken.contains(courseID)){
        classesTaken.get(i).updateGrade(newLetterGrade);
        }
    }
}

根据我的调试器,当i = 5而我的arrayList = 6,一旦我命中6,它就会无缘无故地立即突破循环。我不确定是什么问题,因为make或运行时没有显示错误。

这是在classesTaken:

中找到courseID的内容时调用的方法
public void updateGrade(String newGrade){
    letterGrade=newGrade;
    compute(newGrade);

我的测试文件如下所示:

public class TranscriptTester {

public static void main(String[] args){
    Transcript test = new Transcript("11234", "Cody");
    test.addCourse("COP2013","A+");
    test.addCourse("COP2014","B+");
    test.addCourse("COP2015","B+");
    test.addCourse("COP2016","D+");
    test.addCourse("COP2017","A");
    test.addCourse();
    test.updateCourse("COP2013", "B-");
    test.updateCourse("COP2014", "A+");
    test.getGPA();
    test.printArray();
    test.toString();
}
}

根据我的代码,它应该在classesTaken的第一次迭代中跳转到if语句,因为COP2013是0索引。

如果您需要更多信息以帮助我提问。谢谢!

这里是所有类文件,如果你想看到它们的全部内容:

public class Course {

//global variables
private String courseID;
private String letterGrade;
private Double numberGrade;

//CONSTRUCTORS
//param
public Course(String id, String letter){
courseID = id;
letterGrade=letter;
compute(letter);
}
//default
public Course(){
    courseID = "COP2053";
    numberGrade = 4.0;
    letterGrade = "A";
    compute(getLetterGrade());
}


//COMPUTING GRADES AND UPDATING
private void compute(String letter){


    if(letter == "A+"){numberGrade = 4.0;}
    else if(letter == "A"){numberGrade = 3.67;}
    else if(letter == "A-"){numberGrade = 3.33;}
    else if(letter == "B+"){numberGrade = 3.00;}
    else if(letter == "B"){numberGrade = 2.67;}
    else if(letter == "B-"){numberGrade = 2.33;}
    else if(letter == "C+"){numberGrade = 2.00;}
    else if(letter == "C"){numberGrade = 1.67;}
    else if(letter == "C-"){numberGrade = 1.33;}
    else if(letter == "D+"){numberGrade = 1.0;}
    else if(letter == "D"){numberGrade = .67;}
    else if(letter == "D-"){numberGrade = .33;}
    else if(letter == "F"){numberGrade = 0.0;}

}

public void updateGrade(String newGrade){
    letterGrade=newGrade;
    compute(newGrade);
}

@Override
public String toString(){
    return "\nCourse: "+getCourseID()+"\nLetter Grade: "+getLetterGrade()
          +"\nGPA in course: "+numberGrade;

}




//GETTERS
public String getCourseID(){
    return courseID;
}

public String getLetterGrade(){
    return letterGrade;
}

public Double getNumberGrade(){
    return numberGrade;
}



}

成绩单文件:

public class Transcript {

private ArrayList<Course> classesTaken = new ArrayList<Course>();

public Transcript(String studentID, String studentName){
     System.out.println("\nStudent ID: "+studentID+"\nStudent Name: "+studentName);
}

public void addCourse(String courseID, String letterGrade){
    Course myCourse = new Course(courseID,letterGrade);
    classesTaken.add(myCourse);
}

public void addCourse(){
    Course myCourse = new Course();
    classesTaken.add(myCourse);
}

public void updateCourse(String courseID, String newLetterGrade){

    for(int i= 0; i<classesTaken.size(); i++){

        if(classesTaken.get(i).equals(courseID)){
        classesTaken.get(i).updateGrade(newLetterGrade);
        }
    }
}

public void getGPA(){
    System.out.println("\nOverall GPA: "+calculateGPA());
}

private Double calculateGPA(){
    Double overallGPA = 0.0;
    for(int i=0; i<classesTaken.size(); i++){
    overallGPA = classesTaken.get(i).getNumberGrade()+overallGPA;
    }

    return overallGPA / classesTaken.size();
}

public void printArray(){
    for(int i=0; i<classesTaken.size(); i++){
    System.out.println("\n"+classesTaken.get(i));
    }
}



}

3 个答案:

答案 0 :(得分:1)

只是查看您的代码,我发现您误用了(除非您使用equals String contains方法:

classesTaken.get(i).updateGrade

您在对象(某种类型,而不是updateGrade)上调用String,这显然不是String类型。

但您尝试在列表String列表中找到classesTaken

classesTaken.contains(courseID)

显然此列表中没有String。否则代码甚至不会编译(因为String没有updateGrade方法)

您应该覆盖equals或提供其他方法来查明字符串是否位于自定义类的对象集合中

你应该做这样的事情(假设你的班级是ClassTaken):

for(int i=0; i<classesTaken.size(); i++){
    ClassTaken c = classesTaken.get(i);
    if(c.getCourseId().equals(courseID)){
        c.updateGrade(newLetterGrade);
    }
}

答案 1 :(得分:1)

在updateCourse方法中,您将Course对象(classesTaken ArrayList中的项)与String对象(courseID)进行比较。所以if条件永远不会满足。

如下所示,覆盖Course类中的equals方法,以便您可以使用“contains”方法来比较课程。

@Override
public boolean equals(Object other){
    if (other == null) return false;
    if (other == this) return true;
    if (!(other instanceof Course))return false;
    Course otherCourse = (Course)other;
    return this.courseID.equals(otherCourse.courseID);
}

然后你需要更新Transcript类中的updateCourse方法,以确保比较相同类型的对象:

public void updateCourse(String courseID, String newLetterGrade){   
    Course c = new Course(courseID, newLetterGrade);    
    for(int i= 0; i<classesTaken.size(); i++){
        if(classesTaken.get(i).equals(c)){
            classesTaken.get(i).updateGrade(newLetterGrade);
        }
    }
}

答案 2 :(得分:0)

似乎classesTaken存储了课程,而不是字符串。所以它永远不会包含ID本身。

看起来你可能想要的是classesTaken.get(i).getID().equals(courseID)的某些内容,假设有一个getID函数。