Optimiser ce code

Version imprimable

Optimiser ce code

Bonjour,

J'aimerais optimiser une fonction d'une de mes classe car je trouve qu'elle prend pas mal de temps.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 // Transforme un resultSet en tableau associatif public function makeTab($result){ $arrRes = array(); $fonction = ($this->typeBase == "MySQL") ? "mysql_fetch_assoc" : "odbc_fetch_array"; if ($this->nbResultats($result) > 0){ while($res = $fonction($result)){ array_push($arrRes , $res); } } return $arrRes; }
En fait, selon l'attribut "typeBase" , la fonction a utiliser est "mysql_fetch_assoc" ou "odbc_fetch_array".

Je pense que le principal à optimiser est la partie

Code:

while($res = $fonction($result))

car à chaque passage dans la boucle on utilise la fonction, et j'imagine que c'est pas vraiment performant :?

Vous auriez une idée pour optimiser ce code ?

Bonne journée à tous :mrgreen:

14/12/2010, 11h25
stealth35

hello,
array_push c'est pas fait pour insérer 1 valeur
fait direct

Code:

$arrRes[] = $res;

après t'as pas trop le choix, c'est comme ça que sa marche, utilise PDO sinon
14/12/2010, 11h30
Xenon03

:cry: Merci quand même ...
14/12/2010, 11h41
stealth35

Citation:

Envoyé par Xenon03

:cry: Merci quand même ...

t'as pas PDO ?
14/12/2010, 13h55
Pierrot0459

c'est vrai qu'avec PDO, ce serai top :D

Code:

return $res->fetchAll(PDO::FETCH_ASSOC);

a++
21/12/2010, 09h55
Xenon03

En fait le soucis est que je me connecte à une base Hyperfile, et que je n'arrive pas à utiliser PDO pour ce type de base :(

Normalement je fait :

Code:

$connect = odbc_connect($nomDriver,"","")

Avec PDO :

Code:

$connect = new PDO('odbc:'.$nomDriver, '', '');

Mais l'objet $connect est vide :

Code:

print_r($connect);

renvoit PDO Object ( ) :cry:
21/12/2010, 14h01
stealth35

normal ca ;)

ca devrai aider ceci, j'ai extrait ca d'un de mes codes

Code:

1
2
3
define('DB_NAME', 'C:\temp\monfichier.mdb');
define('DB_USER', 'root');
define('DB_PASS', '');

Code:

1
2
3
4
5
6
7
8
9
10
11
	/**
	 * @param string $db_name
	 * @param string $db_user
	 * @param string $db_pass
	 */
	protected function odbcConnect($db_name, $db_user, $db_pass){
		if(is_null($db_name) || is_null($db_user) || is_null($db_pass))
			die('intance DAO impossible! parametre manquant verifier fichier conf.php');
		$server = 'odbc:Driver={Microsoft Access Driver (*.mdb)};Dbq='.$db_name;
		$this->db = new PDO($server,$db_user,$db_pass);
	}

21/12/2010, 19h16
mathieu44800

Salut,

Ca fait mal de travailler sur ACCESS pour un projet web ??

A+

Mathieu
22/12/2010, 02h42
Helfima

Citation:

Envoyé par mathieu44800

Salut,

Ca fait mal de travailler sur ACCESS pour un projet web ??

A+

Mathieu

clair mais cela dépend de la source qu'on te donne, dans mon cas l'outil de base utilise de l'access et je devais le porter sur une page web.....
en plus si la source n'est pas sur le serveur web tu rigole déjà moins :P

Citation:
Envoyé par Helfima
ca devrai aider ceci, j'ai extrait ca d'un de mes codes
Code:

1 2 3 define('DB_NAME', 'C:\temp\monfichier.mdb'); define('DB_USER', 'root'); define('DB_PASS', '');
Code:

1 2 3 4 5 6 7 8 9 10 11 /** * @param string $db_name * @param string $db_user * @param string $db_pass */ protected function odbcConnect($db_name, $db_user, $db_pass){ if(is_null($db_name) || is_null($db_user) || is_null($db_pass)) die('intance DAO impossible! parametre manquant verifier fichier conf.php'); $server = 'odbc:Driver={Microsoft Access Driver (*.mdb)};Dbq='.$db_name; $this->db = new PDO($server,$db_user,$db_pass); }
évite de faire des die, y'a aucun controle de l'erreur derriere et en plus c'est un alias, sois utuiliser trigger_error, sois les exceptions, il faut aussi rajouter dans code la gestion des erreurs pour PDO, et faire un bloque try...catch autour de la connexion