[Réglé]Système de déplacement

[pascal]

quelques remarques en vrac :
_ essaie la structure switch/case pour allèger le nombre d'accolades
_ fais une jointure au début, pour récupérer la case du membre et sa position en une seule requête
_ essaie de regrouper toutes les requête similaires pour chaque direction : ça allègera le code et simplifiera sa lecture
_ regarde du coté des tutoriaux sur ce forum

A+

Pascal

[Joojo]

- switch ne test que l'égalité et donc je ne peux pas utiliser <= >= < > alors qu'il faut que je mette des "barriere" pour pas que le joueur sorte de la map


Bon voilà ma première jointure mais qui ne fonctionne pas.

Code PHP :

<?php 

$test = mysql_query('SELECT posX, posY FROM map, id_case WHERE map.id_case = map_membre.id_case AND map_membre.login_membre = "'. $_COOKIE['pseudo'] .'"');

        $_test = mysql_fetch_array($test);

        $posX = $_test['posX'];

        $posY = $_test['posY'];    



        echo $posX;

        echo $posY; 

[pascal]

il y a une erreur dans le nom des tables, au niveau du FROM
( ce genre d'erreur, tu dois le détecter tout seul en utilisant par exemple ce tutorial )

pour le switch, c'était sur le mouvement. ensuite dans chaque case du switch, tu dois vérifier que les nouvelles coordonnées ne débordent pas. à la fin, tu peux exécuter une requête, avec $nouvelle_position_X et $nouvelle_position_Y puis une autre sur l'id_case.

A+

Pascal

[Joojo]

Je viens d'utiliser la méthode du cours sur les jointures de ce forum et ça fonctionne, voilà ce que ça donne pour les interéssés:

Code PHP :

<?php 

$test = mysql_query('SELECT posX, posY FROM map INNER JOIN map_membre ON map.id_case = map_membre.id_case WHERE map_membre.login_membre = "'. $_COOKIE['pseudo'] .'"'); 

[Joojo]

Dsl du double post.

pour le switch, c'était sur le mouvement. ensuite dans chaque case du switch, tu dois vérifier que les nouvelles coordonnées ne débordent pas. à la fin, tu peux exécuter une requête, avec $nouvelle_position_X et $nouvelle_position_Y puis une autre sur l'id_case.

Voilà le début de ce que j'ai fait, c'est bien? :heuuu:

Code PHP :

<?php 

if($mouvement == "haut" OR $mouvement == "bas")

            {

            $nouvelle_position_Y = $posY - 1;

            switch($nouvellee_position_Y)

                {

                case 1:

                $a = mysql_query('SELECT id_case FROM map WHERE posX = "'. $posX .'" AND posY = "'. $nouvelle_position_Y .'"')or die(mysql_error());

                $b = mysql_fetch_array($a);

                $nouvelle_id_case = $b['id_case'];

                mysql_query('UPDATE map_membre SET id_case = "'. $nouvelle_id_case .'" WHERE login_membre = "'. $_COOKIE['pseudo'] .'"')or die(mysql_error());

                break;

                

                case 2:

                //blablabla

                

                case 3:

                //blablabla

                }

            } 

[pascal]

non, plutot comme ça pour le switch :

Code PHP :

<?php 

switch($mouvement){

    case 'haut':

        $nouvelle_position_Y = $posY - 1;

        // verification que le Y ne déborde pas

        // [à coder, cf min() et max() dans la doc PHP]

        $nouvelle_position_X = $posX;

        break;

    case 'bas':// à coder

        break;

    case 'gauche': // à coder

        break;

    case 'droite': // à coder

        break;    

}        

$a = mysql_query('SELECT id_case FROM map WHERE posX = "'. $nouvelle_position_X .'" AND posY = "'. $nouvelle_position_Y .'"')or die(mysql_error());

$b = mysql_fetch_array($a);

$nouvelle_id_case = $b['id_case'];

mysql_query('UPDATE map_membre SET id_case = "'. $nouvelle_id_case .'" WHERE login_membre = "'. $_COOKIE['pseudo'] .'"')or die(mysql_error()); 

[Joojo]

Je comprend pas l'utilité de min() et de max() parcequ' il retournent le chiffre le plus petit et le plus grand d'une chaîne de charactére. :heuuu:

Je deviens pas embêtant j'espére.iffle:


Edit: je ne sais pas pourquoi, je ne sais pas comment mais mon ancien script fonctionne, si ça peut aider quelqu'un voici ce que cela donne:

Code PHP :

<?php 

    //si les variables existent

    if(isset($_POST['mouvement']))

        {

        $mouvement = htmlentities($_POST['mouvement']);

        

        $test = mysql_query('SELECT posX, posY FROM map INNER JOIN map_membre ON map.id_case = map_membre.id_case WHERE map_membre.login_membre = "'. $_COOKIE['pseudo'] .'"');

        $_test = mysql_fetch_array($test);

        $posX = $_test['posX'];

        $posY = $_test['posY'];

        

        

        if($mouvement == "haut")

            {

            $nouvelle_position_Y = $posY - 1;

            if($nouvelle_position_Y >= 1)

                {

                $a = mysql_query('SELECT id_case FROM map WHERE posX = "'. $posX .'" AND posY = "'. $nouvelle_position_Y .'"')or die(mysql_error());

                $b = mysql_fetch_array($a);

                $nouvelle_id_case = $b['id_case'];

                mysql_query('UPDATE map_membre SET id_case = "'. $nouvelle_id_case .'" WHERE login_membre = "'. $_COOKIE['pseudo'] .'"')or die(mysql_error());

                }

            else

                {    

                }    

            }

        elseif($mouvement == "bas")

            {

            $nouvelle_position_Y = $posY + 1;

            if($nouvelle_position_Y <= 2)

                {

                $a = mysql_query('SELECT id_case FROM map WHERE posX = "'. $posX .'" AND posY = "'. $nouvelle_position_Y .'"')or die(mysql_error());

                $b = mysql_fetch_array($a);

                $nouvelle_id_case = $b['id_case'];

                mysql_query('UPDATE map_membre SET id_case = "'. $nouvelle_id_case .'" WHERE login_membre = "'. $_COOKIE['pseudo'] .'"')or die(mysql_error());

                }

            else

                {

                }

            }

        elseif($mouvement == "gauche")

            {

            $nouvelle_position_X = $posX - 1;

            if($nouvelle_position_X >= 1)

                {

                $a = mysql_query('SELECT id_case FROM map WHERE posX = "'. $nouvelle_position_X .'" AND posY = "'. $posY .'"')or die(mysql_error());

                $b = mysql_fetch_array($a);

                $nouvelle_id_case = $b['id_case'];

                mysql_query('UPDATE map_membre SET id_case = "'. $nouvelle_id_case .'" WHERE login_membre = "'. $_COOKIE['pseudo'] .'"')or die(mysql_error());    

                }

            else

                {    

                }

            }

        elseif($mouvement == "droite")

            {

            $nouvelle_position_X = $posX + 1;

            if($nouvelle_position_X <= 9)

                {

                $a = mysql_query('SELECT id_case FROM map WHERE posX = "'. $nouvelle_position_X .'" AND posY = "'. $posY .'"')or die(mysql_error());

                $b = mysql_fetch_array($a);

                $nouvelle_id_case = $b['id_case'];

                mysql_query('UPDATE map_membre SET id_case = "'. $nouvelle_id_case .'" WHERE login_membre = "'. $_COOKIE['pseudo'] .'"')or die(mysql_error());    

                }

            else

                {    

                }

            }

        else

            {

            }

        } 

[pascal]

par exemple pour le haut, on fait Y = Y - 1; mais si on arrive à Y = -1, ça va pas, car la valeur minimum c'est zero.

Code PHP :

<?php 

$nouvelle_position_Y = max($posY - 1, 0); 

[Byleth]

Bien vu d'utiliser ça Pascaljte...

Oui, évidemment, tu vas devoir utiliser des "barrières" car dans l'absolu, ton personnage se promène à l'infini. Donc tu as un script général qui va fonctionner jusqu'à ce que tu arrive au bout de la carte que tu vas définir.

Parcontre, je trouve que ton formulaire est tiré par les cheveux Joojo, avec tes hidden et tout. Ce n'est pas la peine : à l'aide de l'attribut name, tu donnes un nom à tes 6 submit (genre N,NE,E,SE,S,SO,O,NO) et tout ça dans un seul <form> si je ne me trompe pas. Une fois que tu as ça, tu utilise switch pour savoir si par exemple $_POST['N'] existe, etc...

Je pense que la suite est pas trop mal, j'ai juste regardé vite fait. Je ne comprends juste pas le but du htmlentities...

Donc ce que je te dis n'est pas vitale, mais allège et simplifie beaucoup ton code...

[Joojo]

Merci pascaltje ça fonctionne parfaitement. Voici la version 2 du script:

Code PHP :

<?php 

//si les variables existent

    if(isset($_POST['mouvement']))

        {

        $mouvement = htmlentities($_POST['mouvement']);

        

        $test = mysql_query('SELECT posX, posY FROM map INNER JOIN map_membre ON map.id_case = map_membre.id_case WHERE map_membre.login_membre = "'. $_COOKIE['pseudo'] .'"');

        $_test = mysql_fetch_array($test);

        $posX = $_test['posX'];

        $posY = $_test['posY'];

        

        switch($mouvement)

        {

            case 'haut':

            $nouvelle_position_Y = $posY - 1;

            $nouvelle_position_Y = max($posY - 1, 1); 

            $nouvelle_position_X = $posX;

                break;

            case 'bas':

            $nouvelle_position_Y = $posY + 1;

            $nouvelle_position_Y = min($posY + 1, 2); 

            $nouvelle_position_X = $posX;

                break;

            case 'gauche':

            $nouvelle_position_X = $posX - 1;

            $nouvelle_position_X = max($posX - 1, 1); 

            $nouvelle_position_Y = $posY;

                break;

            case 'droite':

            $nouvelle_position_X = $posX + 1;

            $nouvelle_position_X = min($posX + 1, 9); 

            $nouvelle_position_Y = $posY;

                break;    

        }        

        $a = mysql_query('SELECT id_case FROM map WHERE posX = "'. $nouvelle_position_X .'" AND posY = "'. $nouvelle_position_Y .'"')or die(mysql_error());

        $b = mysql_fetch_array($a);

        $nouvelle_id_case = $b['id_case'];

        mysql_query('UPDATE map_membre SET id_case = "'. $nouvelle_id_case .'" WHERE login_membre = "'. $_COOKIE['pseudo'] .'"')or die(mysql_error()); 





    

        }