Thread: "While" Problem

I do noT undErstaNd why my pRogram inSists on continUing to aSk for a new Student, eveN after I have told my while Loop "no".

Just wondeRing if Anyone can helP me.

-Ki

 
public class MainClass {
 
	static private int nextStudent = 0;
	static StudentInformation Student[] = new StudentInformation[100];
	static BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
	static boolean addAnotherStudent = true;
 
	public static void main(String[] args) {		
 
		while (addAnotherStudent == true) {
			addStudent();
			System.out.println("Would you like to add another Student?");
			try {
				String temp = br.readLine();
				if (temp == "no") {
					System.out.println("No more students to enter, Good bye");
					addAnotherStudent = false;
					}
				else {
						addAnotherStudent = true;
					}
				} 
		catch (IOException ioExcption) {
			System.out.println("IO exception occurred!");
			System.exit(1);
			}
		}
	}	
 
	public static void addStudent(){
		StudentInformation new1 = new StudentInformation();		
 
		System.out.println("Enter your Student Information: ");
 
		System.out.println("Enter student's full name: ");
		 try {
			 new1.setName(br.readLine());
		 } 
		 catch (IOException ioExcption) {
			 System.out.println("IO exception occurred!");
			 System.exit(1);
		 }
 
		 System.out.println("What is your address? ");
		 try {
			 new1.setAddress(br.readLine());
		 } 	
		 catch (IOException ioExcption) {
			 System.out.println("IO exception occurred!");
			 System.exit(1);
		 }
 
		 System.out.println("What is your city? ");
		 try {
			 new1.setCity(br.readLine());
		 } 
		 catch (IOException ioExcption) {
			 System.out.println("A double was not entered!");
			 System.exit(1);
		 }
 
		 System.out.println("What is your state? ");
		 try {
			 new1.setState(br.readLine());
		 } 
		 catch (IOException ioExcption) {
			 System.out.println("IO exception occurred!");
			 System.exit(1);
		 }
 
		 System.out.println("What is your zipcode? ");
		 try {
			 new1.setZipcode(br.readLine());
		 } 
		 catch (IOException ioExcption) {
			 System.out.println("IO exception occurred!");
			 System.exit(1);
		 }
 
		 System.out.println("What is your current GPA? ");
		 try {
			 new1.setCurrentGPA(Double.parseDouble(br.readLine()));
		 } 
		 catch (IOException ioExcption) {
			 System.out.println("A double was not entered!");
			 System.exit(1);
		 }
 
		System.out.println("Student's Information: "); 
		System.out.println(new1.getName());
		System.out.println(new1.getAddress());
		System.out.println(new1.getCity());
		System.out.println(new1.getState());
		System.out.println(new1.getCurrentGPA());
 
		Student[nextStudent] = new1;
		++ nextStudent;
	}
 
 
}

try using temp.equals("no") and would you please stop with the random capitals

Thanks,
Chris

Thanks foR everythinG!! I was goinG throgh Hell!

Originally Posted by JavaPF

I think Kelly may be disabled Chris? When I asked about the caps:



No idea?!

He plas solely FPS games, but he dabbles with other things / different FPS's if that makes sense lol. What he said makes perfect sense in my eyes at least.

@op

The == operator only tests if the references to objects held by two variables is pointing to the same object, rather than looking at two object to see if they are equal. To compare two objects, you need to use the equals method.

String s1 = new String("string");
String s2 = new String("string");
 
if (s1 == s2){
	//This check will fail
}

Thanks for sharing that for anybody who wishing to read more into it, maybe those who didn't know this here's a link, but read carefully otherwise you may think there is a === operator

Java: ==, .equals(), compareTo(), and compare()

Chris

However this will be true.

        String s1 = "string";
        String s2 = "string";
 
        if (s1 == s2) {
            // This is true
        }

Because of the way strings work in Java.

Another one to watch out for is the use of wrapper objects and ==

This will be true.

        Integer integer1 = 127;
        Integer integer2 = 127;
 
        if (integer1 == integer2) {
            // True
        }

But this will be false.

        Integer integer1 = 128;
        Integer integer2 = 128;
 
        if (integer1 == integer2) {
            // False
        }

Scary stuff. Like the other posts in this thread, refrain from using == unless you know what you are doing or you are using primitives. When comparing a wrapper object with a primitive its also alright to use the == because the JVM will automagically unbox the wrapper for you and do a pure primitive comparison.

// Json