1. The getAge method *must* return a String.  That means that for all paths through the method, it must return a String.  If the year variable does not fall into one of the four if statements (e.g. if year < 1 or year > 4), then the method will NOT return a value.  Even if you're guaranteed in other places that year hast to be between 1 and 4, you still need to make sure that it returns for all options, since Java won't be able to tell.

One fix to this is to add an else:

else{
	return "UNKNOWN YEAR!";
}

or something along those lines.

2. There are a couple of problems.  First, if we let year=1 denote a freshman, etc., then our indexing is wrong since arrays start counting at 0.  What we should be returning is years[year-1].

Second, there could be situations when year isn't in the range of 1-4 if we didn't restrict this in other places.  If this does happen, you would get an Exception.