[Resolvido]Sink a DotCom não compila, e não compreendo o erro

Programação Java
#1

O que tem de errado aqui? não compreendo o por que.
Existe algum guia algum modo de compreender melhor os erros do compilador? as vezes fica muito subjetivo o erro.

import java.util.ArrayList;

	public class DotCom  {
		private ArrayList<String> locationCells;

		public void setLocationCells(ArrayList<String> loc)  {
			locationCells = loc;

	 public String checkYourself(String userInput)  {

		String result = "miss";
		int index = locationCells.indexOf(userInput);
		
		if (index >= 0)  {
			locationCells.remove(index);
			
			if (locationCells.isEmpty())  {
				result = "kill";
			}  else  {
				result = "hit";
			}
		}
	}
}

Quando compilo aparece o seguinte erro:

[quote]javac DotCom.java
DotCom.java:9: error: illegal start of expression
public String checkYourself(String userInput) {
^
DotCom.java:9: error: ‘;’ expected
public String checkYourself(String userInput) {
^
DotCom.java:9: error: ‘;’ expected
public String checkYourself(String userInput) {
^
DotCom.java:24: error: reached end of file while parsing
}
^
4 errors
[/quote]

#2

Você colocou um método (checkYourself(String userInput) ) dentro de outro método (setLocationCells(ArrayList loc)), o que não é válido. De fato, nesse caso, a mensagem não ajudou muito, mas não creio que haja um modo de tornar esta mensagem em particular mais objetiva.

Tente:[code]import java.util.ArrayList;

public class DotCom {
private ArrayList locationCells;

public void setLocationCells(ArrayList loc) {
locationCells = loc;
}
// veja que a função “checkYourself” está fora da função setLocationCells
public String checkYourself(String userInput) {
String result = “miss”;
int index = locationCells.indexOf(userInput);

if (index >= 0)  {  
  locationCells.remove(index);  

  if (locationCells.isEmpty()) { 
    result = "kill";  
  }  else  {
    result = "hit";  
  }
}

}
}[/code]
Abraço.

#3

Obrigado pela ajuda , deu certo agora, também havia esquecido o return result;
vlws, abs

#4