Discussion :

[thebarbarius]

Bonjour,

Maintenant que je commence a mieux gérer ODBC, je dois créer une classe de connexion pour mon formateur qui est payé par mon entreprise.

Sans aller dans les détails et pour une meilleur compréhension voici ma classe très réduite

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
 
// bdd.class.php
 
class bdd {
	public $odbc;
 
 function __construct($odbc) {
	$this->odbc = $odbc; 
 }
 
 function connect() {
	 odbc_connect("$this->odbc", "xx", "xx");
	 }
 
 function close() {
	 odbc_close();
	 }
}
 
 
//appel de la clase
$conn = new bdd('atum2_db_1');
$conn->connect();
 
				$sql = "SELECT * FROM ma_table";
				$query = odbc_exec($conn,$sql);
				$result = odbc_fetch_array($query);

Et voici mon message d'erreur.

Code :

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

1
2
3
4
 
Warning: odbc_exec() expects parameter 1 to be resource, object given in x on line 67
 
Warning: odbc_fetch_array() expects parameter 1 to be resource, null given in x on line 68

[gene69]

poufpouf

Je te conseille de faire une classe abstraite avec toutes les methodes que tu veux offrir sur ton objet "bdd" et ensuite de la dériver en "bdd_odbc", "bdd_mysql" "bdd_oracle".
L'interet s'est de pouvoir uniformiser l'appel à préprare() et execute() pour toutes les bdd, de placer un bout de code qui logue les requetes qui passent.... Ensuite il y a des classes d'abstraction de base de données toute faite en php. fin de la digression.

Code :

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

odbc_connect("$this->odbc", "xx", "xx");

forcer une conversion en chaine ici est un luxe que tu peux éviter.

Code :

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

$query = odbc_exec($conn,$sql);

ça c'est la ligne 67. Ce qui est terrible c'est que le message est explicite. Ce tu tu n'as pas compris c'est qu'il faut que tu stockes le résultat de odbc_connect() dans ta classe. ensuite au moment de l'appel de odbc_exec() tu ne seras pas embété de savoir quoi passer à la fonction et tu lui donneras la ressource stockée dans l'objet et non l'objet lui-même.


EDIT: L'autre erreur découle de la première.

[thebarbarius]

Code :

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

ça c'est la ligne 67. Ce qui est terrible c'est que le message est explicite. Ce tu tu n'as pas compris c'est qu'il faut que tu stockes le résultat de odbc_connect() dans ta classe. ensuite au moment de l'appel de odbc_exec() tu ne seras pas embété de savoir quoi passer à la fonction et tu lui donneras la ressource stockée dans l'objet et non l'objet lui-même.

Concrètement sa donne quoi ?

Je ne comprends pas.


PS : merci de votre réponse !

[gene69]

ok je passe en mode débutant.
lisons la doc

resource odbc_connect ( string $dsn , string $user , string $password [, int $cursor_type ] )
resource odbc_prepare ( resource $connection_id , string $query_string )

tu ne ferais pas une classe, la variable en rouge passé à prepare() c'est le résultat de connect().

je verrai bien un

Code :

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

$this->link = odbc_connect()

puis un

Code :

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

 $this->statment = odbc_prepare($this->link, ... )

sachant que tu dois réserver le résultat du prépare pour l'execute... mais c'est l'étape d'apres. comme le dit la doc, d'ailleurs...

$stmt = odbc_prepare($conn, 'CALL myproc(?,?,?)');
$res = odbc_execute($stmt, array($a, $b, $c));

m'enfin concretement je ne suis pas là pour écrite ta classe. C'est un peut délicat de faire un seul bel objet sur odbc, pour moi l'idéal est d'en avoir deux ou d'être rusé.


EDIT: le probleme des ressources c'est que les trois ressources (rouge, rose, indigo) sont trois ressources is_ressource() réponds oui mais représentent des objets VRAIMENT différents.

[thebarbarius]

Ce qui me parait bizarre c'est que ma classe marche correctement avec mysql.


Et la rien n'est compatible avec ODBC....

Je te remercie sa marche.

[gene69]

je vais m'autociter.

C'est un peut délicat de faire un seul bel objet sur odbc, pour moi l'idéal est d'en avoir deux ou d'être rusé.

En fait ça semble marcher avec mysql parce que le pilote mysql gère l'équivalent d'un $this->link en interne. Il m'est avis que tu t'appuies sur le link implicite de mysql, donc tu ne peux pas te connecter à deux bases mysql différentes simultanément (ou avoir deux connexions distinctes à la même base). Je sais c'est assez rare d'en avoir l'usage, mais c'est sympa de pouvoir le faire. (j'ai déjà eu à le faire pour lire dans des tables de GLPI et OCS simultanément, du coup... )