JeuWeb Créez votre jeu par navigateur !

joshua · 13-12-2006, 08:17 AM

voici les fichiers en cause:

class planete_ressource_db.class.php

Code PHP :
<?php

class planete_ressource_db

{

protected $element = "";

protected $value = "";    

function __construct($db='')

{

    if ($db == '')

     {

         global $db_exploit;

         $db = $db_exploit;

     }

    $this->db = $db; 

    $this->id_planete = "NULL";

    $this->id_ressource = "NULL";

    $this->valeur = "NULL";

    $this->croissanceactuelle = "NULL";

    $this->timestamp = "NULL";

}    

function __get($element)

 {

     switch($element)

     {

         case 'id_planete':

         return $this->id_planete;

         case 'id_ressource':

         return $this->id_ressource;

         case 'valeur':

         return $this->valeur;

         case 'croissanceactuelle':

         return $this->croissanceactuelle;

         case 'timestamp':

         return $this->timestamp;

    }

 }

function check () //valide que tout est bon pour insérer en base

{

    if ($this->id_planete != "NULL")

  {

          if ($this->id_ressource != "NULL")

          {

              if ($this->valeur != "NULL")

              {

                  if ($this->croissanceactuelle != "NULL")

                  {

                      if ($this->timestamp != "NULL")

                      {

                          $response = "1";

                      }

                  }

              }

          }

  }

  return $response;

}

/**

* initialise les données en base

 */

protected function initialize ()

{

    $contenu="SELECT croissance FROM `tbl_ressource` WHERE `id_ressource`=".$this->id_ressource."  ";

    $res = $this->db->query($contenu);

    $table = $this->db->getObject($res);

    $this->valeur=$table->croissance;

    $this->croissanceactuelle=$table->croissance;

    $this->timestamp=time();

    $contenu="INSERT INTO `tbl_planete_ressource` VALUES ('".$this->id_planete."','".$this->id_ressource."','".$this->valeur."','".$this->croissanceactuelle."','".$this->timestamp."')";

    $res = $this->db->query($contenu); 

}

/**

 * charge un élement depuis la base en se reposant sur son "id"

 */  

 function load ($id_planete,$id_ressource)  

{ 

   $res = $this->db->query("SELECT * FROM tbl_planete_ressource WHERE id_planete = '".$this->id_planete."' AND id_ressource = '".$this->id_ressource."'"); 

    if ($this->db->sql_num_res[$res] != "0")

   {

           $table = $this->db->getObject($res);

           $this->id_planete = $table->id_planete;

        $this->id_ressource = $table->id_ressource;

        $this->valeur = $table->valeur;

        $this->croissanceactuelle = $table->croissanceactuelle;

        $this->timestamp = $table->timestamp;

   }

   else

   {

       return "false";

   } 

} 

?>

pascal · 13-12-2006, 09:45 AM

à mon avis, ça vient de la dernière ligne du test:

Code PHP :
<?php 

$planete_ressource = new planete_ressource($dbtest,"1","1"); 

joshua · 13-12-2006, 09:47 AM

Mais ca je le savais deja :'(
Ma base est bien settée un peu plus tot dans le SetUpContext ....
Par contre, visiblement pour lui il ne s'agit plus d'une base dans la fonction load alors qu'elle est bien définie précédemment.....
Si quelqu'un a une idée sur le pourquoi de la fonction load qui ne reconnait plus la db.....

Aramiil · 13-12-2006, 10:00 AM

Essai de faire

Code :
die(var_dump($this->db));

avant

Code :
$res = $this->db->query("SELECT * FROM tbl_planete_ressource WHERE id_planete = '".$this->id_planete."' AND id_ressource = '".$this->id_ressource."'");

pour voir ?

joshua · 13-12-2006, 11:32 AM

je teste ça ce soir... j'ai pas accès a mon pc avant :'(
Faut que je comprenne ce qui merde, ca me travaille!

X-ZoD · 13-12-2006, 12:13 PM

lol
mn code semble apocalyptique a cote de ca -_-

pascal · (Modification du message : 13-12-2006, 12:44 PM par pascal.)

mouais...

le code est lisible, mais je vais chipoter:
_ la methode check() peut être améliorée:
* la variable $response n'est pas initialisée, on a un risque d'erreur
* comme c'est une vérification, ça devrait renvoyer TRUE ou FALSE, pas la chaine "1"
_ la methode load():
* elle ne renvoie rien ou renvoie la chaine "false"; elle devrait renvoyer TRUE ou FALSE
* si elle ne renvoie rien, comment tester ça dans une variable?

pour identifier le bug, je rajouterai du code :
_ dans la classe dérivée, vérifier $this->db avant et après parent::__construct()
_ au debut de load(), verifier que $this->db existe et est un objet du type voulu ( SQL )

A+

Pascal

joshua · 13-12-2006, 02:42 PM

euuuuuh.... tu fais tout ca comment? propose, je suis toujours preneur pour améliorer mon code :p
Les tests c'est pas ma test de thé :p

pascal · 13-12-2006, 03:34 PM

je l'écrirai comme ça:

class planete_ressource_db.class.php

Code PHP :
<?php

class planete_ressource_db

{

function check () //valide que tout est bon pour insérer en base

{

    if ( $this->id_planete != "NULL" &&

             $this->id_ressource != "NULL" &&

             $this->valeur != "NULL" &&

             $this->croissanceactuelle != "NULL" &&

             $this->timestamp != "NULL" )

             {

                  return TRUE;

             }else{

                  return FALSE;

             }

}

/**

 * charge un élement depuis la base en se reposant sur son "id"

 */  

 function load ($id_planete,$id_ressource)  

{ 

   $res = $this->db->query("SELECT * FROM tbl_planete_ressource WHERE id_planete = '".$this->id_planete."' AND id_ressource = '".$this->id_ressource."'"); 

    if ($this->db->sql_num_res[$res] != "0")

   {

           $table = $this->db->getObject($res);

           $this->id_planete = $table->id_planete;

        $this->id_ressource = $table->id_ressource;

        $this->valeur = $table->valeur;

        $this->croissanceactuelle = $table->croissanceactuelle;

        $this->timestamp = $table->timestamp;

                return TRUE;

   }

   else

   {

       return FALSE;

   } 

} 

?>

joshua · 13-12-2006, 04:59 PM

l'ennui que jai découvert il y a peu c'est que quand tu sors avec return false, il sort en exit 1...... en gros ton programme s'arrete. C'est pour ca que j'ai mis volontairement des return "false" pour qu'il reconnaisse le nom et pas la fonctionnalité. Ou alors que j'ai mis return 1.......
Pour le reste je teste ca ce soir.
Je suis assez embeté d'avoir un debugger et qu'il ne me retourne pas l'etat de mes variables... visiblement je ne le maitrise pas encore assez!

Sujets apparemment similaires…
Sujet		Auteur	Réponses	Affichages	Dernier message
	[Résolu] Je ne comprends pas ce fragment de code	leni	8	3 301	24-02-2008, 10:46 PM Dernier message: leni