Вместо того, чтобы изначально установить для этого параметра значение null, а затем присвоить ему значение, лучше просто определить его значение как N, чтобы другой кодер, который приходит, не подразумевал поведения, связанного с нулем, и действовал соответствующим образом.Это значение должно быть 1 из 2 вещей ... Y или N, не инициализируйте его чем-то отличным от двух предполагаемых значений.
Вы также можете изменить это, хотя условие будет немного чище, используяМетод есть.Это более наглядное описание того, что вы хотите сделать, и вы можете выполнить модульное тестирование условия.
С изменениями, которые составляют:
String proceed="N";
while (this.playAgain(proceed)) { //playAgain performs the same test as you have in your while
//your code
}
Во-вторых, для вашего кода, который намеренно оторван, убедитесь, чтоон состоит из чего-то вроде:
Game game = new Game();
game.play();
Если у вас гораздо больше кода, это слишком много в одном месте.