erreur php dans fonction

Version imprimable

Salut à tous,
Je suis entrain de m'essayer à l'optimisation de mon code php sql-server 2005.
Pour ce faire j'essaye de mettre les parties de requetes sql commune dans une fonction et à partir d'une codition d'y ajouter la condition where qui vas bien. Cependnant j'ai une erreur de syntaxe et je n'arrive pas à la trouver quelqu'un pourrait-il m'aider?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
function recherche_contact($nom=null, $entreprise=null){
		$sql = "SELECT 	CONTACTS.[ID_CONTACT]
							,CONTACTS.[NOMCONTACT]
							,CONTACTS.[PRENOMCONTACT]
							,CONTACTS.[MAILPRO]
							,CONTACTS.[MAILPERSO]
							,CONTACTS.[TELPORT]
							,CONTACTS.[TELFIXE]
							,CONTACTS.[FAX]
							,STATUTS.[STATUT]
							,ENTREPRISE.[NOMENTREPRISE]
						FROM ([BDD_TOOL_V2].[dbo].[STATUTS] JOIN [BDD_TOOL_V2].[dbo].[CONTACTS] ON CONTACTS.[ID_STATUT] = STATUTS.[ID_STATUT])
															JOIN [BDD_TOOL_V2].[dbo].[ENTREPRISE] ON CONTACTS.[ID_ENTREPRISE] = ENTREPRISE.[ID_ENTREPRISE]'";
						if($entreprise != null){
							$sql = $sql."' WHERE ENTREPRISE.[NOMENTREPRISE] = '".$entreprise."'";
						}
						else{
							$sql = $sql."'WHERE CONTACTS.[NOMCONTACT] LIKE '".$nom."%'";
						}
return $sql;
}

erreur generé:

Code:

1
2
3
 
Warning: odbc_do() [function.odbc-do]: SQL error: [Microsoft][ODBC SQL Server Driver][SQL Server]Syntaxe incorrecte vers ''., SQL state 37000 in SQLExecDirect in C:\wamp\www\test_v3.0\Controleur\controleur_recherche_contact.php on line 34
37000

Merci de votre aide

12/05/2009, 11h38
jean-ba

Salut,

Pourquoi mettez les variables $nom et $entreprise a 'null' dans votre fonction.
Elles seront toujours nulles...

Ecrivez plutot:
function recherche_contact($nom, $entreprise){...}

enfin je pense...

byby
JB
12/05/2009, 11h53
fasyr

suite

ben en fait j'ai des condisions avant cette fonction et lors de l'appel de le fonction, une des deux valeur est vide. le fait de mettre recherche_contact($nom=null, $entreprise=null){...} inflige une valeur par defaut si la variable est vide. Cependnant, si une des deux variables n'est pas vide le null ne lui sera pas affecté.
12/05/2009, 12h03
jean-ba

Ok , autant pour moi.

Sinon il y a une quote qui se balade a la fin de ton select a la fin de la ligne...
Peut tu nous specifier quel est la ligne 34?
12/05/2009, 12h10
fasyr

suite

la ligne 34 correspond à l'execution de la requete soit:

Code:

$resultat = odbc_do($connexion, $sql) or die(odbc_error());
12/05/2009, 17h22
Antherak

Peut-être

remplacer

Code:

$sql = $sql."' WHERE ENTREPRISE.[NOMENTREPRISE] = '".$entreprise."'";

par

Code:

$sql .= "' WHERE ENTREPRISE.[NOMENTREPRISE] = '".$entreprise."'";

merci

merci de votre aide mais la solution résidé dans les " et les ' je poste la bonne syntaxe pour ce que ça intéresse. Merci de otre aide.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
$sql = "SELECT 	CONTACTS.[ID_CONTACT]
	            ,CONTACTS.[NOMCONTACT]
		,CONTACTS.[PRENOMCONTACT]
		,CONTACTS.[MAILPRO]
		,CONTACTS.[MAILPERSO]
		,CONTACTS.[TELPORT]
		,CONTACTS.[TELFIXE]
		,CONTACTS.[FAX]
		,STATUTS.[STATUT]
		,ENTREPRISE.[NOMENTREPRISE]
	FROM ([BDD_TOOL_V2].[dbo].[STATUTS] JOIN [BDD_TOOL_V2].[dbo].[CONTACTS] ON CONTACTS.[ID_STATUT] = STATUTS.[ID_STATUT])
					JOIN [BDD_TOOL_V2].[dbo].[ENTREPRISE] ON CONTACTS.[ID_ENTREPRISE] = ENTREPRISE.[ID_ENTREPRISE]";
						if($entreprise != null){
							$sql = $sql."WHERE ENTREPRISE.[NOMENTREPRISE] = '".$entreprise."'";
						}
						else{
							$sql = $sql."WHERE CONTACTS.[NOMCONTACT] LIKE '".$nom."%'";
						}

19/05/2009, 13h49
amateur_dev

bonjour,
Pour optimiser, il faut déjà :
1-ne pas écrire de SQL embarqué mais des procédures stockées (c'est mieux pour la rapidité, la sécurité),
2-ne pas utiliser ODBC mais un pilote natif (Download the SQL Server 2005 Driver for PHP chez www.microsoft.com).
3-écrire du SQL optimisé (donc tester les requêtes sous SQL Server: see SQL Server Query Execution Plan Analysis)
Bonne journée