Given the following class for a student:

public class Student{
	private String name;
	private int id;
	private int year;

	public Student(String tempName, int tempId, int tempYear){
		name = tempName;
		id = tempId;
		year = tempYear;
	}

	// some more code in here

	public String getYear(){
		if( year == 1 ){
			return "Freshman";
		}else if( year == 2 ){
			return "Sophomore";
		}else if( year == 3 ){
			return "Junior";
		}else if( year == 4 ){
			return "Senior";
		}
	}
}

1. If you try to compile the code above (i.e. open it in Eclipse) Java/Eclipse will complain about the getYear method.  What's wrong with the method and why does Java complain?

2. You decide to rewrite the method using your new found knowledge of arrays:

public String getYear(){
	String[] years = {"Freshman", "Sophomore", "Junior", "Senior"};
	return years[year];
}

This is much more compact and avoid the problem of the other version.  However, in testing it out, you realize that it doesn't work right.  What's the problem?