Ayuda con mini juego

[moyo18]

Gueno es dis q juegoe s sencillo se trata d lo siguiente


pide tu nombre
se tira el dado supuestamente con un random number y t da un numero ejemplo 5
luego t pregunta si deseas tirarlo otra vez y le das q si t da otro numero 4
t pregunta otrs vez si deseas seguir le das q no
t da la suma de los dos numeros

ahora va la compu y hace lo mismo

puedes seguir jugando hasta q alcenzes un numero mayor d 24 o si el numero q tiras es uno entonces su score se regesa a 0.

luego t dice quien gano, el q tenga mayor puntaje, ahora t pregunta si deseas volver a jugar,  y el problema esta q ya no muestra el score sino q sigue correidno por ella misma y nose en q podria estar el error.

aki les dejo el codigo.

Código: [Seleccionar]

import java.util.Random;
import java.util.Scanner;
import java.io.*;       
/**
 * Play one turn of Pig
 * @author (Moyo)
 * @version (1.0)
 *
 */
public class Pig
{
    static Random r = new Random();  // a random number generator
    static Scanner kb = new Scanner(System.in);
    static String playerName;
    public static void main(String[] args)
    {
      System.out.println("This game is a dice game to play against the computer \n" +
                         " to win the game you have to get the greater score \n" +
                         " if the computer gets a greater score than you, then you \n" +
                         " will lose the game.\n");
      System.out.println("you can still playing if you get a score less then 24, but \n" +
                         "if you get 1 your score will be equal to 0 \n" );
      System.out.println("What is your name? ");
      playerName=getName();
      System.out.println(playerName+", your total "+turn());
      System.out.println("Computer's total "+computerTurn());
     
     
       
     
     
//       winner();
      playAgain();
    }
/**
 * Get the player's name
 * @return the player's name
 *
 */
    public static String getName()
    {
        return kb.next();
    }   
/**
 * Get the winner
 * of the game
*/   
    public static void winner()
    {
        int computerScore = 0;
        int playerScore = 0;
       
        computerScore = computerTurn();
        playerScore = turn();
       
        if ( computerScore < playerScore )
            System.out.println("The Winner is: " + playerName);
        else
            System.out.println("The Winner is the Computer");               
    }     
       
/**
 * Manage a turn of the dice game Pig
 * @return the value of the player's turn
 *
 */
    public static int turn()
    {
        int turnTotal = 0;
        int roll;
        int totalScore = 0;
        Boolean again=true;
        // repeatedly roll the die until a 1 is rolled
        // or the sum exceeds 24
        // or the user says halt
        do
        {       
            roll = toss(6);
            turnTotal += roll ;
            totalScore += turnTotal;
            System.out.println(playerName+", you rolled "+roll);
           
            if (roll!=1)
            {  // ask user if they want to roll again
                System.out.println("roll again? (y/n) ");
                again = kb.next().equals("y");
            }
         
        } while (again && roll!=1 && turnTotal<25);
        // return 0 if a 1 was rolled last, sum of the throws otherwise
        return (roll != 1) ? turnTotal : 0 ;               
    }
/**
 * Computer Turn
 * @return the value of the computer's turn
 *
 */   
    public static int computerTurn()
    {
        int compTurnTotal = 0;
        int compRoll = 0;                 
        // repeatedly roll the die until a 1 is rolled
        // or the sum exceeds 20       
        while (compRoll!=1 && compTurnTotal<25)
        {       
            compRoll = toss(6);
            compTurnTotal += compRoll ;
           
           
            System.out.println("Computer rolled "+compRoll);
           
           
           
        }
        // return 0 if a 1 was rolled last, sum of the throws otherwise
        return (compRoll != 1) ? compTurnTotal : 0 ;                               
    }
/**
 * Toss a die
 * @param numberSides the number of sides on the die
 * @return a random value between 1 and numberSides
 *
 */
    public static int toss(int numberSides)
    {
        return r.nextInt(numberSides)+1;       
    }
/**
 * Ask the user to play again
 */
    public static void playAgain()
    {
        System.out.println("Do you want to play again? (y/n) ");
        while (kb.next().equals("y"))
        {
            turn();
            computerTurn();
            System.out.println("Do you want to play again? (y/n) ");
        }
    }
}
la primera vez q corre lo hace bien pero si le doy q siga jugando pasa el problema

[moyo18]

el problema esta en quien es el ganador

Código: [Seleccionar]

// Find out who wins... the one with the greater score will win the game
      int computerScore = computerTurn();
      int playerScore = turn();
       
      if ( computerScore < playerScore )
          System.out.println("The Winner is: " + playerName);
      else
          System.out.println("The Winner is the Computer"); 
no se q hago mal ahi pero eso es lo q arruina todo....

[g00mba]

hmm lo q veo es q la funcion playagain esta separada del main, te sugeriria mover el bucle de juego al main en lugar de tenerlo afuera. lo demas esta bien encapsulado asi a ojo de águila

[moyo18]

medio lo tengo arreglado pero aun sigue el problema por el ganador, ya q si emito ese codigo el programa trabaja bien de lo contrario pasa el mismo problema...

asi lo tengo ahorita

Código: [Seleccionar]

import java.util.Random;
import java.util.Scanner;
import java.io.*;       
/**
 * Play one turn of Pig
 * @author (Moyo)
 * @version (1.0)
 *
 */
public class Pig
{
    static Random r = new Random();  // a random number generator
    static Scanner kb = new Scanner(System.in);
    static String playerName;
    public static void main(String[] args)
    {
      System.out.println("This game is a dice game to play against the computer \n" +
                         " to win the game you have to get the greater score \n" +
                         " if the computer gets a greater score than you, then you \n" +
                         " will lose the game.\n");
      System.out.println("you can still playing if you get a score less then 24, but \n" +
                         "if you get 1 your score will be equal to 0 \n" );
      System.out.println("What is your name? ");
      playerName=getName();
      System.out.println(playerName+", your total "+turn());
      System.out.println("Computer's total "+computerTurn());     
     
      [b]// Find out who wins... the one with the greater score will win the game
      int computerScore = computerTurn();
      int playerScore = turn();
       
      if ( computerScore < playerScore )
          System.out.println("The Winner is: " + playerName);
      else
          System.out.println("The Winner is the Computer");  [/b]
     
           
      // Ask the user if they want to continue playing     
      System.out.println("Do you want to play again? (y/n) ");
        while (kb.next().equals("y"))
        {
            System.out.println(playerName+", your total "+turn());
            System.out.println("Computer's total "+computerTurn());     

            System.out.println("Do you want to play again? (y/n) ");
        }
    }
/**
 * Get the player's name
 * @return the player's name
 *
 */
    public static String getName()
    {
        return kb.next();
    }   
/**
 * Manage a turn of the dice game Pig
 * @return the value of the player's turn
 *
 */
    public static int turn()
    {
        int turnTotal = 0;
        int roll;
        int totalScore = 0;
        Boolean again=true;
        // repeatedly roll the die until a 1 is rolled
        // or the sum exceeds 24
        // or the user says halt
        do
        {       
            roll = toss(6);
            turnTotal += roll ;           
            System.out.println(playerName+", you rolled "+roll);           
            if (roll!=1)
            {  // ask user if they want to roll again
                System.out.println("roll again? (y/n) ");
                again = kb.next().equals("y");
            }
         
        } while (again && roll!=1 && turnTotal<25);
        // return 0 if a 1 was rolled last, sum of the throws otherwise
        return (roll != 1) ? turnTotal : 0 ;               
    }
/**
 * Computer Turn
 * @return the value of the computer's turn
 *
 */   
    public static int computerTurn()
    {
        int compTurnTotal = 0;
        int compRoll = 0;                 
        // repeatedly roll the die until a 1 is rolled
        // or the sum exceeds 24       
        while (compRoll!=1 && compTurnTotal<25)
        {       
            compRoll = toss(6);
            compTurnTotal += compRoll ;                       
            System.out.println("Computer rolled "+compRoll);                                   
        }
        // return 0 if a 1 was rolled last, sum of the throws otherwise
        return (compRoll != 1) ? compTurnTotal : 0 ;                               
    }
/**
 * Toss a die
 * @param numberSides the number of sides on the die
 * @return a random value between 1 and numberSides
 *
 */
    public static int toss(int numberSides)
    {
        return r.nextInt(numberSides)+1;       
    }           
}
pero no se como arreglar eso del ganador porq eso vuelve loco los resultados

[g00mba]

y cual es exactamente el problema con el ganador? traba el programa, se equivoca no muestra nada?

[g00mba]

ah olvidalo ya lo entendi...

por alguna razon no limpia la variable y te sigue sumando.. creo q eso es el problema verdad

[g00mba]

ahhh... de hecho vos le estas diciendo q siga sumando...

Código: [Seleccionar]

       while (kb.next().equals("y"))
        {
            System.out.println(playerName+", your total "+turn());
            System.out.println("Computer's total "+computerTurn());     

            System.out.println("Do you want to play again? (y/n) ");
        }crea una nueva instancia o mata la anterior y volve a usarla. ahi solo estas añadiendo resultados a las mismas instancias.

ahhhh ya te entendi con lo q pusiste abajo

[moyo18]

mira el problema es este ejemplo d como corre

What is your name?
ron
ron, you rolled 6
roll again? (y/n)
y
ron, you rolled 4
roll again? (y/n)
y
ron, you rolled 1
ron, your total 0





Computer rolled 3
Computer rolled 6
Computer rolled 3
Computer rolled 3
Computer rolled 4
Computer rolled 1
Computer's total 0

-------- hasta aki todo bien

---- aki empieza el devergue porq tendria q decir kien gano y no correr la comp y leugo el user.. y asi hasta el final
no hace lo q supuestamente tiene q hacer.

Computer rolled 1
ron, you rolled 4
roll again? (y/n)
n
The Winner is: ron
Do you want to play again? (y/n)
y
ron, you rolled 1
ron, your total 0
Computer rolled 6
Computer rolled 1
Computer's total 0
Do you want to play again? (y/n)
y
ron, you rolled 2
roll again? (y/n)
n
ron, your total 2
Computer rolled 5
Computer rolled 4
Computer rolled 3
Computer rolled 2
Computer rolled 3
Computer rolled 6
Computer rolled 3
Computer's total 26
Do you want to play again? (y/n)
n


para q corra bien tendria q ser algo asi

What is your name?
ron
ron, you rolled 6
roll again? (y/n)
y
ron, you rolled 4
roll again? (y/n)
y
ron, you rolled 6
roll again? (y/n)
n
ron, your total 16
Computer rolled 3
Computer rolled 6
Computer rolled 6
Computer rolled 4
Computer rolled 4
Computer rolled 3
Computer's total 26
Do you want to play again? (y/n)
y
ron, you rolled 3
roll again? (y/n)

y
ron, you rolled 3
roll again? (y/n)
y
ron, you rolled 1
ron, your total 0
Computer rolled 2
Computer rolled 2
Computer rolled 2
Computer rolled 3
Computer rolled 5
Computer rolled 2
Computer rolled 2
Computer rolled 1
Computer's total 0
Do you want to play again? (y/n)
n


ahi funciona bien sin el pedazo d codigo de

// Find out who wins... the one with the greater score will win the game
       int computerScore = computerTurn();
      int playerScore = turn();
        
       if ( computerScore < playerScore )
           System.out.println("The Winner is: " + playerName);
      else
          System.out.println("The Winner is the Computer");  

[g00mba]

Código: [Seleccionar]

       int computerScore = computerTurn();
      int playerScore = turn();
       
       if ( computerScore < playerScore )
           System.out.println("The Winner is: " + playerName);
      else
          System.out.println("The Winner is the Computer");  ese pedazo de código debería de ser inofensivo 0_o
y si probas asi:

Código: [Seleccionar]

     
       if ( computerTurn() < turn() )
           System.out.println("The Winner is: " , playername);
      else
          System.out.println("The Winner is the Computer"); 
no lo he compilado, pero es lo q se me viene a la cabeza...
por cierto... instancia los juegos completos no solo los turnos.

te explico, el programa como lo tenes vos, es un solo juego q puede repetir las tiradas varias veces y volver a empezar, yo lo veria como un programa q puede tener multiples instancias y dentro de cada instancia multiples tiradas...

[moyo18]

siempre da el mismo problema y cambiando el codigo como dices aunq asi no lo miro bien, prefiero darle el valod de turn() a una variable ya q asi me ense;aron ...

ahora no entiendo lo q dices aki

te explico, el programa como lo tenes vos, es un solo juego q puede repetir las tiradas varias veces y volver a empezar, yo lo veria como un programa q puede tener multiples instancias y dentro de cada instancia multiples tiradas...

mira aki esta mi tarea, aunq yo se q me falta pero esto es lo primero q tengo

http://io.uwinnipeg.ca/~rmcfadye/1903/1903Assignment06.htm

aki esta la respuesta a la parte A

http://io.uwinnipeg.ca/~rmcfadye/1903/Pig5.java

[Camus de Acuario]

el problema es porque a la segunda, tercera o x corrida ya no te dice quien es el ganador ni el puntaje???

es porque cuando haces el if de querer seguir jugando:

Código: [Seleccionar]

// Ask the user if they want to continue playing     
      System.out.println("Do you want to play again? (y/n) ");
        while (kb.next().equals("y"))
        {
            System.out.println(playerName+", your total "+turn());
            System.out.println("Computer's total "+computerTurn());     

            System.out.println("Do you want to play again? (y/n) ");
        }

aca adentro se te olvido poner lo de:

Código: [Seleccionar]

int computerScore = computerTurn();
      int playerScore = turn();

      if ( computerScore < playerScore )
          System.out.println("The Winner is: " + playerName);
      else
          System.out.println("The Winner is the Computer");

con eso ya te tira el resultado, ahora tu programa tiene varios errores, como que a veces la computadora tira 1, y luego vuelve a tirar y luego volves a tirar vos, bien raro

[moyo18]

eso mismo q dices camus me escribio un man q es d los q ayuda en la universidad a los alumnos

Código: [Seleccionar]

I find your program only get scores but never compare them. You can change
to see if it follow your rule.

public static void playAgain() {
        System.out.println("Do you want to play again? (y/n) ");
        while (kb.next().equals("y")) {
            //turn();
            //computerTurn();
            winner();
            System.out.println("Do you want to play again? (y/n) ");
        }
lo mismo me dice y si los mismos errores pasan otra vez, osea q sin eso funciona como deberia pero con eso funciona a lo loco, ya no muestra el total ni quien gano ni nada, nada mas agarra puros numeros random y los muestra.

aki si funciona bien

Código: [Seleccionar]

import java.util.Random;
import java.util.Scanner;
import java.io.*;       
/**
 * Play one turn of Pig
 * @author (Moy)
 * @version (1.0)
 *
 */
public class Pig
{
    static Random r = new Random();  // a random number generator
    static Scanner kb = new Scanner(System.in);
    static String playerName;
    public static void main(String[] args)
    {
      System.out.println("This game is a dice game to play against the computer \n" +
                         " to win the game you have to get the greater score \n" +
                         " if the computer gets a greater score than you, then you \n" +
                         " will lose the game.\n");
      System.out.println("you can still playing if you get a score less then 24, but \n" +
                         "if you get 1 your score will be equal to 0 \n" );
      System.out.println("What is your name? ");
      playerName=getName();
      System.out.println(playerName+", your total "+turn());
      System.out.println("Computer's total "+computerTurn());     
     
     
      System.out.println("Do you want to play again? (y/n) ");
      while (kb.next().equals("y"))
      {
            System.out.println(playerName+", your total "+turn());
            System.out.println("Computer's total "+computerTurn());           
            System.out.println("Do you want to play again? (y/n) ");
      }
    }
/**
 * Get the player's name
 * @return the player's name
 *
 */
    public static String getName()
    {
        return kb.next();
    }   
/**
 * Manage a turn of the dice game Pig
 * @return the value of the player's turn
 *
 */
    public static int turn()
    {
        int turnTotal = 0;
        int roll;
        int totalScore = 0;
        Boolean again=true;
        // repeatedly roll the die until a 1 is rolled
        // or the sum exceeds 24
        // or the user says halt
        do
        {       
            roll = toss(6);
            turnTotal += roll ;           
            System.out.println(playerName+", you rolled "+roll);           
            if (roll!=1)
            {  // ask user if they want to roll again
                System.out.println("roll again? (y/n) ");
                again = kb.next().equals("y");
            }
         
        } while (again && roll!=1 && turnTotal<25);
        // return 0 if a 1 was rolled last, sum of the throws otherwise
        return (roll != 1) ? turnTotal : 0 ;               
    }
/**
 * Computer Turn
 * @return the value of the computer's turn
 *
 */   
    public static int computerTurn()
    {
        int compTurnTotal = 0;
        int compRoll = 0;                 
        // repeatedly roll the die until a 1 is rolled
        // or the sum exceeds 24       
        while (compRoll!=1 && compTurnTotal<25)
        {       
            compRoll = toss(6);
            compTurnTotal += compRoll ;                       
            System.out.println("Computer rolled "+compRoll);                                   
        }
        // return 0 if a 1 was rolled last, sum of the throws otherwise
        return (compRoll != 1) ? compTurnTotal : 0 ;                               
    }
/**
 * Toss a die
 * @param numberSides the number of sides on the die
 * @return a random value between 1 and numberSides
 *
 */
    public static int toss(int numberSides)
    {
        return r.nextInt(numberSides)+1;       
    }
}
pero parto d ese pero naxa al agregar el codigo de winner todo se jode...

[g00mba]

vaya lo q te digo es q encapsules todo el proceso central del juego para q el main solo lo ocupes para llamar todas las instancias del juego q necesites, el main te quedaría bien chiquito por solo necesitar la llamada a la instancia y un while para preguntar si desea iniciar otro juego, al rato te pongo un ejemplo sencillito

[Camus de Acuario]

ya vi cual es el error ese que te digo, que computadora y tu juegan 2 veces, como te digo, primero jugas vos, luego la compu, luego la compu otra vez y luego tu, y es por esto:

Código: [Seleccionar]

     System.out.println(playerName+", your total "+turn());
      System.out.println("Computer's total "+computerTurn());      
      
      // Find out who wins... the one with the greater score will win the game
      int computerScore = computerTurn();
      int playerScore = turn();

sea como sea, imprimiendo turn() o asignandole el valor a una variable int, ambas veces llamas la misma funcion y cada una se ejecuta con tiradas diferentes, por eso cada uno juega 2 veces

System.out.println(playerName+", your total "+turn());
aca jugas la primera vez

System.out.println("Computer's total "+computerTurn());  
aca la compu juega la primera vez

int computerScore = computerTurn();
aca la compu juega la segunda vez

int playerScore = turn();
aca jugas la segunda vez

para mi que quites esas todo eso y pongas:

Código: [Seleccionar]

     // Find out who wins... the one with the greater score will win the game
      int computerScore = computerTurn();
      int playerScore = turn();
      
      System.out.println(playerName+", your total "+playerScore);
      System.out.println("Computer's total "+computerScore);
      
      if ( computerScore < playerScore )
          System.out.println("The Winner is: " + playerName);
      else
          System.out.println("The Winner is the Computer");  
      
            
      // Ask the user if they want to continue playing      
      System.out.println("Do you want to play again? (y/n) ");
        while (kb.next().equals("y"))
        {
         computerScore = computerTurn();
            playerScore = turn();
            
            System.out.println(playerName+", your total "+playerScore);
            System.out.println("Computer's total "+computerScore);
            
            if ( computerScore < playerScore )
                System.out.println("The Winner is: " + playerName);
            else
                System.out.println("The Winner is the Computer");
            
            System.out.println("Do you want to play again? (y/n) ");
        }

EDIT: ahora que ya encontramos el error, concuerdo con goomba para que encapsules la main class para que solo haga las llamadas de lo que necesitas, facil y tranquilo, ahorita lo hago y lo pego por aca

[moyo18]

System.out.println(playerName+", your total "+turn());
aca jugas la primera vez

System.out.println("Computer's total "+computerTurn()); 
aca la compu juega la primera vez

int computerScore = computerTurn();
aca la compu juega la segunda vez

int playerScore = turn();
aca jugas la segunda vez

eso pense hace un buen rato, pero no encontraba como guardar los valores, porq si me imagine que al llamar las los metodos y guardarlos en esas variables serian otro valores no se si me entiendes bueno se q si lol.. pero por ahi andaba, no taba tan perdido tampoco.