Discussion :

[ledisciple]

Bonjour et désolé pour le titre pourri.

J'aimerai juste un conseil sur le code suivant :

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
 
// $categorie est la résultante d'un $_POST, analysé et contrôlé pas forcément obligatoirement rempli.
 
if ( !empty($categorie) && isset ($categorie) )
{
$cat = 'AND UnChamp like "qqchose"';
}
else
{
$cat = "";
}
 
//$imaginaire  est la résultante d'un $_POST, analysé et contrôlé pas forcément obligatoirement rempli.
 
if ( !empty($imaginaire) && isset ($imaginaire) )
{
$dog = 'AND DeuxChamp like "autre chose"';
}
else
{
$dog = "";
}
 
$sql ='SELECT plein de choses
FROM plein de tables
WHERE Table1 LEFT JOIN table2  ON [...]
'.$saison.' '.$cat.' ;

Il y a peut-être des erreurs dans le code, mais j'aimerai surtout votre avis sur le structure

Merci beaucoup d'avance

PS: je viens de penser un faire un array pour y inclure tous les champs du formualire

[sabotage]

Ta requête est fausse.
isset est redondant sur ton empty.

[andry.aime]

Bonsoir,

1- Met un espace avant le AND de ton requête.
2- Un sigle quote dans une requête mais pas une double quote.

[ledisciple]

Ta requête est fausse.
isset est redondant sur ton empty.

oui, j'ai vraiment écris n'importe quoi. Désolé ...

Code :

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

1
2
3
4
5
 
$sql ="SELECT plein de choses
FROM table1 LEFT JOIN table2 ON table1.champ1 = table2.champ1
WHERE [...] //condition obligatoire
".$saison." ".$cat." ;

Bonsoir,

1- Met un espace avant le AND de ton requête.
2- Un sigle quote dans une requête mais pas une double quote.

ok pour les quotes.

Sinon, c'est pas trop mal?