[SOLUCIONADO] Al volver a cargar el script se vuelven a iniciar las variables

Berman · #1 (**permalink**) 16/11/2017, 16:22

Buenas, mi problema es el siguiente:

Estoy intentando crear un juego en el que hay 4 jugadores y un dado con html y php. El problema es que al hacer clic en el dado (que es un input type=image) vuelve a cargar el script y vuelve a iniciar las variables, $contadorjugador, a 0. Da igual que el random del dado se sume al contador que se volverá a iniciar a 0. Os paso el archivo html y php para que me ayudéis un poco porque estoy perdido. Muchas gracias.

HTML:

Código HTML:

Ver original<html>
    <head>
        <TITLE>BochoGame</TITLE>
        <META charset="UTF-8">
    </head>
    <body bgcolor="#FE9A2E">
        <form action="BochoGame.php" method="post">
            <center><font size="7"><b><u>Menú</u></b></font><br></center>
            <br>
            <center>
                Jugador 1:<input type="text" name="jugador1" required>
                Jugador 2:<input type="text" name="jugador2" required>
                Jugador 3:<input type="text" name="jugador3" required>
                Jugador 4:<input type="text" name="jugador4" required><br>
                <br>
                <input type="submit" value="Enviar" name="send">
            </center>
        </form>
    </body
</html>

PHP:

Código PHP:

Ver original<html>
    <head>
        <meta charset="UTF-8">
        <title>BochoGame</title>
    </head>
    <form method="post">
        <body bgcolor="#FE9A2E">
            <table border="3" bordercolor="black" width="100%">
                <tr>
                    <td rowspan="2" width="20%" valign="top">
                <center><font size="7"><b><u>Jugadores</u></b></font><br></center>
                <?php
                session_start();
                if (isset($_POST["send"])) {
                    $_SESSION['primero'] = $_POST['jugador1'];
                    $jugador1 = $_POST['jugador1'];
                    $_SESSION['segundo'] = $_POST['jugador2'];
                    $jugador2 = $_POST['jugador2'];
                    $_SESSION['tercero'] = $_POST['jugador3'];
                    $jugador3 = $_POST['jugador3'];
                    $_SESSION['cuarto'] = $_POST['jugador4'];
                    $jugador4 = $_POST['jugador4'];
                } else {
                    $jugador1 = $_SESSION['primero'];
                    $jugador2 = $_SESSION['segundo'];
                    $jugador3 = $_SESSION['tercero'];
                    $jugador4 = $_SESSION['cuarto'];
                }
                $contadorjugador1 = 0;
                $contadorjugador2 = 0;
                $contadorjugador3 = 0;
                $contadorjugador4 = 0;
                echo"<center><h1><b>".$jugador1 . "= " . $contadorjugador1 . "</b></h1></center><br>";
                echo"<center><h1><b>". $jugador2 . "= " . $contadorjugador2 . "</b></h1></center><br>";
                echo"<center><h1><b>". $jugador3 . "= " . $contadorjugador3 . "</b></h1></center><br>";
                echo"<center><h1><b>". $jugador4 . "= " . $contadorjugador4."</b></h1></center>";
                ?>
                </td>
                <td rowspan="2" width="40%" valign="top">
                <center><font size="7"><b><u>Dado</u></b></font><br></center>
                <br>
                <center><input type="image" value="randomdado" name="dado" src="Fotos/dado.png" width="50%"></center>
                <br>
                <?php
                if (isset($_POST['dado'])) {
                    $randado = rand(1, 6);
                    switch ($randado) {
                        case 1:
                            echo "<center><img src='Fotos/1.png' width='30%'></center>";
                            $contadorjugador1++;
                            break;
                        case 2:
                            echo "<center><img src='Fotos/2.png' width='30%'></center>";
                            break;
                        case 3:
                            echo "<center><img src='Fotos/3.png' width='30%'></center>";
                            break;
                        case 4:
                            echo "<center><img src='Fotos/4.png' width='30%'></center>";
                            break;
                        case 5:
                            echo "<center><img src='Fotos/5.png' width='30%'></center>";
                            break;
                        case 6:
                            echo "<center><img src='Fotos/6.png' width='30%'></center>";
                            break;
                    }
                }
                ?>
                </td>
                <td>
 
                </td>
                </tr>
                <tr>
                    <td>
 
                    </td>
                </tr>
            </table>
    </form>      
</body>
</html>

mortiprogramador · #2 (**permalink**) 16/11/2017, 17:05

Pues yo no veo que el dado como tal tenga una función de envío,
solo se muestra y ya.

Por otra parte, las variables siempre se fijan a 0,
es por eso que siempre se muestran esos valores.

Lo que habría que hacer es al momento de hacer el submit,
ver quien lanzó el dado, y afectar el contador correspondiente.

Y pues tal vez quieras usar ajax más adelante
para no tener que hacer el submit de toda la página.

xfxstudios · #3 (**permalink**) 16/11/2017, 18:17

prueba así:

Código PHP:

Ver original$contadorjugador1 = (isset($_SESSION['jugador1']) && $_SESSION['jugador1'] >0) ? $_SESSION['jugador1'] : 0;
$contadorjugador2 = (isset($_SESSION['jugador2']) && $_SESSION['jugador2'] >0) ? $_SESSION['jugador2'] : 0;
$contadorjugador3 = (isset($_SESSION['jugador3']) && $_SESSION['jugador3'] >0) ? $_SESSION['jugador3'] : 0;
$contadorjugador4 = (isset($_SESSION['jugador4']) && $_SESSION['jugador4'] >0) ? $_SESSION['jugador4'] : 0;

de esa manera validas si la variable de ese jugador existe y le asignas su valor, en caso contrario le asignas 0

Berman · #4 (**permalink**) 17/11/2017, 06:16

Gracias por responder! xfxstudios si lo pongo como dices obtengo el mismo resultado, lo que quiero que haga es que cuando salga un 1 en el random del dado sume 1 al "$contadorjugador1". El problema es que cada vez que cargo la pagina como la variable está inicializada a 0 al hacer el echo es siempre 0.

mortiprogramador de momento solo quiero hacer que la variable no sea destruida, cuando lo consiga entonces si tendré que pensar de quien es el turno. De momento si consigo que al salir un 1 se vaya sumando a "$contadorjugador1" estoy contento.

xfxstudios · #5 (**permalink**) 17/11/2017, 07:00

ok, en ese caso deberias de tener el codigo php en otro archivo y en este trabajar con variables de sesion, por ende al enviar las peticiones con ajax, actualizas las variables de sesion y al recargar la pagina las cantidades se mantienen hasta que seteees las variables a cero nuevamente.

mortiprogramador · #6 (**permalink**) 17/11/2017, 15:44

Pues creo que ya te esta funcionando,
el asunto es que estas incrementando la variable
pero no estas mostrando el incremento,
te falta imprimir la variable luego del incremento.

Berman · #7 (**permalink**) 19/11/2017, 11:14

mortiprogramador no funciona por que hace el incremento pero solo una vez, osea yo le sumo 1 y lo puedo sacar por pantalla, pero al refrescar la variable volverá a 0 y si vuelve a caer el random en 1, sumara 0+1 otra vez. Lo ideal es que cada vez que entrara en el random 1 sumara uno en plan 0+1+2+3...etc

xfxstudios soy muy nuevo en esto y nunca he tocado ajax, pero he pensado en lo que me has dicho por eso he creado un php que se llama contador y he empezado a indagar con javaescript, como mi problema es que se vuelve a cargar la pagina, javascript me deja hacer un random de 1 a 6 sin cargar la pagina.

el problema es que he intentado hacer que a una variable se le sume todo el rato un random, pero no funciona... algo he hecho mal

Código PHP:

Ver original<html>
    <head>
        <meta charset="UTF-8">
        <title>BochoGame</title>
    </head>
    <body bgcolor="#FE9A2E">
    <center><button onclick="dado()">Dado</button></center>
    <p id="dado"></p>
    <p id="contador"></p>
    <script>
        function dado() {
            var dadorand = Math.floor((Math.random() * 6) + 1);
            document.getElementById("dado").innerHTML = dadorand;
        }
        var contador = 0;
        contador = contador + dadorand;
        document.getElementById("contador").innerHTML = contador;
    </script>
</body>
</html>

muchas gracias!!

xfxstudios · #8 (**permalink**) 19/11/2017, 19:58

Esta mal, prueba así:

Código HTML:

Ver original<div>DADO: <span id="dado"></span></div>
<div>Contador: <span id="contador">0</span></div>
<center><button onclick="dado()">Dado</button></center>

Código Javascript:

Ver originalfunction dado() {
    var contador = $("#contador").html();
  var dadorand = Math.floor((Math.random() * 6) + 1);
  $("#dado").html(dadorand);
  $("#contador").html(parseInt(contador)+parseInt(dadorand));
  
}

MUESTRA FUNCIONANDO

de igual manera puedes enviar el resultado o los resultados del contador via ajax a un php para ir actualizando variable sde sesion si las necesitas

Berman · #9 (**permalink**) 20/11/2017, 07:12

Perdón por mi incompetencia, pero al poner el código en netbeans no me funciona.

lo he puesto en un archivo php así:

Código PHP:

Ver original<html>
    <head>
        <meta charset="UTF-8">
        <title>Contador</title>
    </head>
    <body bgcolor="#FE9A2E">
        <div>DADO: <span id="dado"></span></div>
        <div>Contador: <span id="contador">0</span></div>
    <center><button onclick="dado()">Dado</button></center>
    <script>
        function dado() {
            var contador = $("#contador").html();
            var dadorand = Math.floor((Math.random() * 6) + 1);
            $("#dado").html(dadorand);
            $("#contador").html(parseInt(contador) + parseInt(dadorand));
 
    </script>
</body>
</html>

es así como se hace, hoy que hacerlo en un archivo JS? Muchas gracias

mortiprogramador · #10 (**permalink**) 20/11/2017, 09:46

Uhm, y si lo ejecutas el navegador te muestra algún error?
fijate en el inspector del navegador

Tal vez sea porque te falta agregar jquery (a menos que lo agregues antes),
que es la librería que se está usando para la interacción,
según muestra el jsfiddle es jquery 3.1.1, así que si no la tienes incluida
buscala y la agregas como si agregaras un javascript externo.

xfxstudios · #11 (**permalink**) 21/11/2017, 15:25

si ves tu codigo no estas cargando la libreria de jquery para que este funcione,

debes cargarla antes para que el codigo te funcione

Código HTML:

Ver original<script src="https://code.jquery.com/jquery-3.2.1.js"></script>
</head>

Berman · #12 (**permalink**) 23/11/2017, 06:29

Después de mirar un tutorial he creado una carpeta javascript y dentro he puesto el archivo jquery-3.2.1.min.js, descargado de la pagina oficial.

Lo he importado, pero debe estar mal porque no hace nada :(, contador siempre vale 0

Código PHP:

Ver original<html>
    <head>
        <meta charset="UTF-8">
        <title>Contador</title>  
    </head>
    <body bgcolor="#FE9A2E">
        <div>DADO: <span id="dado"></span></div>
        <div>Contador: <span id="contador">0</span></div>
    <center><button onclick="dado()">Dado</button></center>
    <script type="text/javascript" src="javascript/jquery-3.2.1.min.js"></script>
    <script>
        function dado() {
            var contador = $("#contador").html();
            var dadorand = Math.floor((Math.random() * 6) + 1);
            $("#dado").html(dadorand);
            $("#contador").html(parseInt(contador) + parseInt(dadorand));
 
    </script>
</body>
</html>

xfxstudios · #13 (**permalink**) 23/11/2017, 07:26

1.- La importacion del escript Jquery, colocala en el heade de tu codigo como te explique:

Código HTML:

Ver original<script src="https://code.jquery.com/jquery-3.2.1.js"></script>
</head>

2.- No estas cerrando la llave de la funcion dado(), revisa eso y prueba de nuevo

MUESTRA Con tu codigo exacto

Berman · #14 (**permalink**) 26/11/2017, 16:25

Si, el problema era la llave de la funcion dado(), ahora si hace lo que quiero, muchas gracias por la ayuda, doy el tema por solucionado