Problème class

**Anduriel** · 27/01/2006, 18h09

Salut,

J'ai une classe de connection et elle plante un peu lorsque j'ai deux connections d'ouvertes.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
$connect1 =& new connect;
   $connect1->query()
   while (list($id) = mysql_fetch_array($connect1->query) {
      $connect2 =& new connect();
      $connect2->query();
      $connect2->close();
   }
   $connect1->close();

Le problème c'est quand je ferme $connect2, alors je ne peux pas fermer $connect1 par la suite, il ne trouve pas le lien.

Mes deux fonctions de la classe:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
function connect() {
   global $db;
   $this->connect = @mysql_connect($db['host'], $db['name'], $db['pass']);
   mysql_select_db($db['db'], $this->connect);
}
 
function close() {
   mysql_close($this->connect);
}

**Anduriel** · 27/01/2006, 22h57

Aucune solution?

**ePoX** · 28/01/2006, 10h01

En faits ce qu'il se passe la c'est du à mysql_connect, et pas à ta classe.

En effet tu ouvres deux connexions sur la même entité (cf global $db), dans ce genre de cas mysql_connect detecte automatiquement la similarité et renvoie le même link pour les deux connections.

Si c'est pour piocher dans des bdd différentes sur le même serveur, utilise la syntaxe SQL,
SELECT [BDD].[TABLE].[CHAMP] FROM [BDD].[TABLE] WHERE ect

bye

**siddh** · 28/01/2006, 10h36

tes deux connections se font dans la même session ?

**Mr N.** · 28/01/2006, 10h46

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
$connect1 =& new connect;
   $connect1->query()
   while (list($id) = mysql_fetch_array($connect1->query) {
      $connect2 =& new connect();
      $connect2->query();
      $connect2->close();
   }
   $connect1->close();

pourquoi faire deux connect de suite ??? Pourquoi ne pas faire plutot ce code ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
$connect =& new connect;
$req1 = $connect->query()
while (list($id) = mysql_fetch_array($req1)) {
      $req2 = $connect->query();
}
$connect->close();

Sinon je trouve perturbant ce mysql_fetch_array au milieu de ton code...

**ePoX** · 28/01/2006, 10h48

pourquoi faire deux connect de suite ??? Pourquoi ne pas faire plutot ce code ?

C'est clair +1.

Sinon je trouve perturbant ce mysql_fetch_array au milieu de ton code...

Tu est bien reveillé

Sa fait quatre foisque je lis ce poste je ne l'avais toujours pas vu :o
+10

**siddh** · 28/01/2006, 10h50

moi surtout je voit pas pourquoi tu pourrais pas faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
$connect1 =& new connect;
   $connect1->query()
   while (list($id) = mysql_fetch_array($connect1->query) {
      $connect1->query();
      ......
   }
   $connect1->close();

essayes plutot de pouvoir gérer des requetes multiples .....

**Mr N.** · 28/01/2006, 10h52

Envoyé par siddh

moi surtout je voit pas pourquoi tu pourrais pas faire...

ca ressemble légèrement à ce que j'ai proposé

au moins on est d'accord

**siddh** · 28/01/2006, 10h54

voui (je suis pas encore au 4eme mug de café pas tres reveillé) , apres c est sa syntaxe qui est étrange efffectivement ....

**Anduriel** · 28/01/2006, 12h01

Pourquoi mysql_fetch_array est-il perturbant

?

Ce que je ne comprends pas c'est même si je fais un $req1 = $connect->query(blabla), alors ma variable de la requête dans ma classe sera ecrasée puis que je fais:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
function query($query) { 
   $this->query = mysql_query($query); 
}

Je suis perdu la...

**ePoX** · 28/01/2006, 12h04

Pq c'est bizarre tu as une partie de ta couche d'accession aux données qui est encapsulé dans une classe mais qu'à moitié :s

Sa rends ta couche d'accession aux données un peu nullle et non avenante :s

concernant la structure de taclasse il en été pas mal discuté sur ce thread http://www.developpez.net/forums/vie...283&highlight=

**Anduriel** · 28/01/2006, 12h09

Désolé j'ai mal écrit ma fonction queyr elle s'écrit comme ça:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
function query($query) { 
   $this->query = mysql_query($query); 
}

**siddh** · 28/01/2006, 12h22

deja, avoir une methode et un attribut qui portent le même nom, c est tres moyen ....

**Mr N.** · 28/01/2006, 14h49

Et pourquoi pas ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
function query($query) { 
   return mysql_query($query); 
}

:

**Anduriel** · 28/01/2006, 15h42

Oui mais dans ce cas ensuite je ne peux plus faire de bouche avec mysql_fetch_array car je n'aurais plus variable de requête.
Je vais essayer de bidouiller.

**ePoX** · 28/01/2006, 15h48

Benh si !?

ton $query tu le retrouves à la sortie de function query($sql) {return mysql_query($sql); }.

Mais c'est pas très pratique ton truc puisque dans ta conception une connection ne peut executer qu'un seul query à la fois :s

Le mieux serait qu'à la sortie tu retournes un objet encapsulant ton $query et qu'il possède une méthode fetch()

J'ai comme l'impressionde repeter un précédent thread : o

**Anduriel** · 28/01/2006, 17h40

Envoyé par ePoX

Mais c'est pas très pratique ton truc puisque dans ta conception une connection ne peut executer qu'un seul query à la fois :s

C'est mon problème, d'ou le topic

Le mieux serait qu'à la sortie tu retournes un objet encapsulant ton $query et qu'il possède une méthode fetch()

Ca ça m'interesse si tu sais comment faire...

**ePoX** · 28/01/2006, 18h07

Ca ça m'interesse si tu sais comment faire...

Oui j'ai une petite idée

je m'était deja attaqué au problème, même si se ne sera jamais aussi bon que PDO.

Et puis je ne l'ai jamais reellement testée,donc y'à peut être un truc ou deux à corrigé

Regardes cette classe et inspire t'en pour en faire ton implémentation.

n'hésites pas si tu as des questions.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
 
<?php
 
class MysqlQueryResults implements ISqlQueryResults
{
 
    private $Query_;
 
    public function __construct( $query )
    {
        $this->Query = false;
        if( is_resource($query) )
        {
            $this->Query_ = $query;
        }
        else
        {
            throw new Exception("L'argument fourni n'est pas une ressource !");
        }
    }
 
    public function Next($type = "ASSOC")
    {
        switch($type)
        {
            case "ASSOC":
                $res = mysql_fetch_array($this->Query_ ,MYSQL_ASSOC);
            break;
            case "NUM":
                $res = mysql_fetch_array($this->Query_ , MYSQL_NUM);
            break;
            case "OBJECT":
                $res = mysql_fetch_object($this->Query_);
            break;
            default:
                $res = mysql_fetch_array($this->Query_ , MYSQL_BOTH);
            break;
        }
        if( $res == false )
        {
            @mysql_free_result($this->Query_);
            return false;
        }
 
        return $res;
    }
 
    public function Count()
    {
        if($this->Query_ != false)
        {
            return mysql_num_rows($this->Query_);
        }
        return -1;
    }
 
    public function __destruct()
    {
        @mysql_free_result($this->Query_);
        unset($this->Query_);
    }
}
?>