Отображение выигрышей-проигрышей

Хотя это не прямой ответ на ваш вопрос «где ошибка», я думаю, что все ваше решение можно было бы улучшить. Имея это в виду, я использовал перечисление для игр и простую функцию, которая проверяет все правила игры. Я думаю, что так быстрее отлаживать/понимать. Посмотрите, поможет ли это вам:

ENUM:

public enum Play{
    ROCK,PAPER,SCISSORS;
}

И функция, которая возвращает false, если player1 проигрывает или true, если player1 выигрывает:

public boolean play(Play player1, Play player2) throws UnsuportedPlayException{
    // rock wins scissors
    if(player1 == Play.ROCK && player2 == Play.SCISSORS)
        return true;
    if(player2 == Play.ROCK && player1 == Play.SCISSORS)
        return false;

    //rock loses to paper
    if(player2 == Play.ROCK && player1 == Play.PAPER)
        return true;
    if(player1 == Play.ROCK && player2 == Play.PAPER)
        return false;

    //paper loses to scissors
    if(player1 == Play.PAPER && player2 == Play.SCISSORS)
        return false;
    if(player2 == Play.PAPER && player1 == Play.SCISSORS)
        return true;
    throw new UnsuportedPlayException("That play is not yet available.");
}

Исключение было создано мной только для обработки нулей или если вы решите добавить игры и забудете добавить правила (вы можете добавить четвертую возможную игру, просто для удовольствия).

Класс исключения:

public class UnsuportedPlayException extends Exception {

    private static final long serialVersionUID = 1L;

    public UnsuportedPlayException() {
        super();
    }

    public UnsuportedPlayException(String message, Throwable cause) {
        super(message, cause);
    }

    public UnsuportedPlayException(String message) {
        super(message);
    }

    public UnsuportedPlayException(Throwable cause) {
        super(cause);
    }
}

Ваша ошибка в вашем методеwinRet. Этот код:

if(user == 'R')
        {
            if(compGuess != 'P')
            {
                if(compGuess != 'R')
                {
                    ret = 1;
                }
                ret = 2;
            }
            ret = 0;
        }

ВСЕГДА будет устанавливать ret в 0. Если логика погружается в один из внутренних операторов if, переменная ret все равно устанавливается в 0 в конце. Надеюсь, это поможет.

В вашем коде ret=0 является последним оператором условия if, поэтому он всегда будет присваивать ret значение 0

Вместо

if(user == 'R'){
            if(compGuess != 'P')
            {
                if(compGuess != 'R')
                {
                    ret = 1;
                }
                ret = 2;
            }
            ret = 0;
}

Используйте приведенный ниже код

if (user == 'R') {
            ret = 0;
            if (compGuess != 'P') {
                ret = 2;
                if (compGuess != 'R') {
                    ret = 1;
                }
        }
}

Используйте этот тип присваивания и для двух других условий if.

import java.util.Random;

public class RockPaperScissor {

    static String ret;

    public static void main(String args[]) {

        String computer = compChoice();
        winnerRet(args[0], computer);

    }

    public static void winnerRet(String user, String compGuess) {

        if (user.equals("R")) {
            if (!compGuess.equals("P")) {
                if (!compGuess.equals("R")) {
                    ret = "Win";
                } else {
                    ret = "Draw";
                }
            } else {
                ret = "Lose";
            }
        } else if (user.equals("S")) {
            if (!compGuess.equals("R")) {
                if (!compGuess.equals("S")) {
                    ret = "Win";
                } else {
                    ret = "Draw";
                }
            } else {
                ret = "Lose";
            }
        } else if (user.equals("P")) {
            if (!compGuess.equals("S")) {
                if (!compGuess.equals("P")) {
                    ret = "Win";
                } else {
                    ret = "Draw";
                }
            } else {
                ret = "Lose";
            }
        }

        System.out.println(compGuess);
        System.out.println(ret);

    }

    public static String compChoice() {

        String compGuess;

        Random random = new Random();
        int compNum = random.nextInt(3) + 1;

        if (compNum == 1) {
            compGuess = "R";
        } else if (compNum == 2) {
            compGuess = "S";
        } else {
            compGuess = "P";
        }

        return compGuess;

    }

}

Что ж, я быстро создал игру «камень-ножницы-бумага» без счетной части, основываясь на вашем коде. Кажется, это работает. Что ж, думаю, вы можете посмотреть мой код и отредактировать на его основе свой. Я думаю, проблема была в том, что вы сравнивали символы с ==. Я использовал струны, так что вы можете поменять их местами.

Создайте перечисление Move и позвольте ему сделать всю работу за вас. Это гораздо более читабельно, чем все операторы if. Кроме того, причина, по которой вам нужны initMoves, заключается в том, что вы не можете ссылаться на перечисления до их инициализации (однако они ссылаются друг на друга).

import java.util.Random;

public class RockPaperScissors {

  public static void main(String[] args) {
    Move.initMoves();
    Move computerMove = Move.randomMove();
    Move.ROCK.printWin(computerMove);
    Move.PAPER.printWin(computerMove);
    Move.SCISSORS.printWin(computerMove);
  }

  private static enum Move {

    ROCK,
    PAPER,
    SCISSORS;

    private Move beats;

    private Move() {}

    private static void initMoves() {
      ROCK.beats = SCISSORS;
      PAPER.beats = ROCK;
      SCISSORS.beats = PAPER;
    }

    public void printWin(Move computer) {
      if (this == computer) {
        System.out.println("It was a tie.");
      } else if (this.beats == computer) {
        System.out.println("Player wins.");
      } else {
        System.out.println("Computer wins.");
      }
    }

    public static Move randomMove() {
      int move = new Random().nextInt(3);
      switch(move) {
      case 0:
        return ROCK;
      case 1:
        return PAPER;
      default:
        return SCISSORS;
      }
    }
  }
}