Почему моя переменная класса перезаписывается после запуска несвязанного метода?

Итак, я пишу базовую игру MasterMind, которая ... в основном функциональна. Однако он демонстрирует странное поведение, и я не уверен, почему.

Идея состоит в том, что код и его поведение определяется одним файлом, игровой процесс - другим, а Main просто создает новую игру и начинает играть. Когда я инициализирую игру, компьютер создает новую случайную строку из 4 («секретный код»), как и ожидалось; но затем, когда я получаю данные для предположения пользователя, кажется, что секретный код переписывается во все, что я ввел. Кроме того, мои методы оценки совпадений вообще не работают, но, учитывая, что секретный код постоянно меняется, это означает, что он не устанавливается с самого начала, и я не уверен, почему.

Все три класса ниже. Почему моя переменная класса в игре не настроена должным образом и не доступна для других методов?

Main.java

class Main { 
  public static void main(String[] args) {
    Game newGame = new Game();
    newGame.play(); 
  }
}

Code.java

import java.util.Random;
import java.util.HashMap;
import java.util.Collection;
import java.util.ArrayList;
import java.util.Set;

import java.lang.Math;
import java.lang.StringBuilder;

class Code {
  private static HashMap<String,String> PEGS;
  private static ArrayList<String> pegStrings;
  protected static String secretCodeString;


  public static void main(String[] args) {

  }

  public Code(String input){
    this.secretCodeString = input;
  }

  public Code(){
    randomize();
  }


  //literally just creates the peghash
  public static void setPegs(){
    PEGS = new HashMap<String,String>();

    PEGS.put("C","c");
    PEGS.put("Y","y");
    PEGS.put("R","r");
    PEGS.put("P","p");
    PEGS.put("O","o");
    PEGS.put("G","g");
  }

  //turns the pegs ito something randomize can use
 public static ArrayList<String> makePegArray(){
    setPegs();

    pegStrings = new ArrayList<String>();

    Collection<String> pegValues = PEGS.values();
    Object[] pegObjects = pegValues.toArray();

      for (int i = 0; i < pegObjects.length; i++){
        pegStrings.add(pegObjects[i].toString());
      }

    return pegStrings;
  }

  // sets Class Variable secretCode to a four letter combination
  public static Code randomize(){
    secretCodeString = new String();

    Random rand = new Random();
    int randIndex = rand.nextInt(makePegArray().size());

    for (int i = 0; i < 4; i++){
      randIndex = rand.nextInt(makePegArray().size());
      secretCodeString = secretCodeString.concat(makePegArray().get(randIndex));
    }

      Code secretCode = parse(secretCodeString);
      return secretCode;
  }

  public static Code parse(String input) {
    setPegs();
    makePegArray();

    String[] letters = input.split("");
    StringBuilder sb = new StringBuilder();

    for (String letter : letters) {
      if (pegStrings.contains(letter)) {
        sb.append(letter);
      } else {
        System.out.println(letter);
        throw new RuntimeException();

      }
    }

    String pegListString = sb.toString();
    Code parsedCode = new Code(pegListString);
    //System.out.println(parsedCode);
    return parsedCode;

  }

  public int countExactMatches(Code guess){
    String guessString = guess.secretCodeString;

    int exactMatches = 0;

    String[] guessArray = guessString.split("");

    String[] winningCodeArray = (this.secretCodeString).split("");

    for(int i = 0; i < 4; i++){

      if(guessArray[i] == winningCodeArray[i]){
        exactMatches++;
      }
    }
    return exactMatches;
  }

  public int countNearMatches(Code guess) {

    String guessString= guess.secretCodeString;

    HashMap<String,Integer> guessCount = new HashMap<String,Integer>();
    HashMap<String,Integer> secretCodeCount = new HashMap<String,Integer>();

    Set<String> codeKeys = guessCount.keySet();

    int matches = 0;
    int keys = guessCount.keySet().size();


    String[] keyArray = new String[keys];



    for(int i = 0; i < guessString.length(); i++) {
      //removes character from string
      String codeCharacter = String.valueOf(guessString.charAt(i));
      String guessShort = guessString.replace(codeCharacter,"");

      //counts instances of said character
      int count = guessString.length() - guessShort.length();

      guessCount.put(codeCharacter, count);
    }

    for(int i = 0; i < secretCodeString.length(); i++) {
      //removes character from string
      String winningString = this.secretCodeString;

      String winningCodeCharacter = String.valueOf(winningString.charAt(i));
      String winningCodeShort = guessString.replace(winningCodeCharacter,"");

      //counts instances of said character
      int count = winningString.length() - winningCodeShort.length();

      secretCodeCount.put(winningCodeCharacter, count);
    }

    for (int i = 0; i < keys; i++) {
      codeKeys.toArray(keyArray);
      String keyString = keyArray[i];

      if (secretCodeCount.containsKey(keyString)) {
        matches += Math.min(secretCodeCount.get(keyString), guessCount.get(keyString));
      } 
    }

    int nearMatches = matches - countExactMatches(guess);

    return nearMatches;
  }
}

Game.java

import java.util.Scanner;

class Game {

  protected static Code winningCode;

  public static void main(String[] args){

  }

  public Game(){
    winningCode = new Code();
  }

  protected static Code getGuess() {

    Scanner userInput = new Scanner(System.in);

    int count = 0;
    int maxTries = 5;
    while(true){
      try {
        String codeToParse = userInput.next();
        Code guess = Code.parse(codeToParse);
        return guess;

      } catch(RuntimeException notACode) {
        System.out.println("That's not a valid peg. You have " + (maxTries - count) + " tries left.");
        if (++count == maxTries) throw notACode;
      }
    }


  }

  protected static void displayMatches(Code guess){

    int nearMatches = winningCode.countNearMatches(guess);
    int exactMatches = winningCode.countExactMatches(guess);

    System.out.println("You have " + exactMatches + " exact matches and " + nearMatches + " near matches.");
  }

  protected static void play(){
    int turnCount = 0;
    int maxTurns = 10;

    System.out.println("Greetings. Pick your code of four from Y,O,G,P,C,R.");

    while(true){
      Code guess = getGuess();
      displayMatches(guess);

      if (guess == winningCode) {
        System.out.print("You win!!");
        break;
      } else if (++turnCount == maxTurns) {
        System.out.print("You lose!!");
        break;
      }
    }
  }
}

person Andrea McKenzie    schedule 30.12.2016    source источник
comment
Почему вы все объявляете статичным ?? Плохой. stackoverflow.com/questions/2671496/   -  person OldProgrammer    schedule 30.12.2016
comment
Я очень новый программист, так что, наверное, делаю это неправильно. Но многие из моих методов выскакивают с ошибкой, в которой говорится, что вы не можете вызывать static из нестатического контекста или наоборот. Я меняю их по мере необходимости, чтобы ошибка перестала отображаться   -  person Andrea McKenzie    schedule 30.12.2016
comment
Нет, нет, нет. Вы делаете это, потому что Main () объявлен статическим. Вам нужно создать экземпляр класса (объекта) в Main (), а затем ссылаться на методы через объект. Прочтите ссылку в комментарии.   -  person OldProgrammer    schedule 30.12.2016
comment
... В этом гораздо больше смысла, чем то, что я делал. Так могло бы это объяснить, почему winCode продолжает переписывать себя? Потому что это не однозначное установление объекта?   -  person Andrea McKenzie    schedule 30.12.2016


Ответы (2)


При каждом предположении вы вызываете Code.parse, Code.parse создает новый Code (new Code(pegListString);), и этот конструктор устанавливает secretCodeString, и, поскольку он статичен, все экземпляры Code используют одну и ту же переменную. Вам нужно избегать static изменяемых членов.

Другой совет: либо метод должен возвращать значение, либо изменять состояние (либо его ввода, либо его собственного экземпляра, this), но избегайте того и другого.

person weston    schedule 30.12.2016
comment
Я понимаю, что вы имеете в виду, но у меня есть второй конструктор, который принимает входные данные и должен устанавливать значение String кода равным предоставленному аргументу this.secretCodeString = input. Это недействительно теперь, когда есть конструктор randomize / no param? - person Andrea McKenzie; 30.12.2016
comment
Понятно, я пропустил это, ну, это все еще похоже на то, что я все еще думал, поскольку оба конструктора изменяют общее статическое поле secretCodeString. - person weston; 30.12.2016

«Почему моя переменная класса перезаписывается после запуска несвязанного метода?»

Потому что, на самом деле, это не безразлично. «Беспорядок», который вы создали, объявив переменные и методы как static, привел к нежелательному связыванию между различными частями вашего кода.

Трудно сказать, какое здесь правильное решение, потому что ваш код настолько запутался из-за переписывания, что трудно различить исходный «замысел дизайна».

Я бы посоветовал начать все сначала. Теперь у вас должно быть более четкое представление о том, какие функции требуются. Что вам нужно сделать, так это переделать дизайн объекта, чтобы у каждого класса была четкая цель. (Классы Main и Game имеют смысл, но Code, похоже, представляет собой гибрид функциональности и состояния, не имеющего четкой цели.)

person Stephen C    schedule 30.12.2016