Extraire des informations d'une requête SQL

[XtofRoland]

bonjour je cherche a extraire le nom d'une table d'un string sql.

pour le moment je pratique ainsi:

Code :

1
2
3
4
5
 
$SQL = strtoupper($SQL);
$SQL = str_replace(" FROM ", "[*%&]", $SQL );
$SQL = str_replace(" WHERE ", "[*%&]", $SQL);
list($fields,$this->table,$whereclause) = explode("[*%&]",$SQL);

mais qd il n'y a pas de where ca plante. jusqu'a present je rajoutais where 1=1 et ca ne me posait pas de problemes mais un nouvelle ecran ou je fais un select ds le dual m'impose de ne pas mettre de clause where.

j'en profite pour essayer de mettre une expression reguliere mais je cale completement... entre le regex le posix.. je patauge
quel est le nom de la fonction qui me retournerai le nom de la table? (si je lui donne le bon pattern)

merci

[Yogui]

Salut

Si tu cherches un truc vraiment béton, tu vas te prendre la tête, je te le promets...

Voici une piste :

Code :

/(?:SELECT|UPDATE|INSERT)[[:space:]](?:FROM|INTO)?([[:space:]])?`?([^` ]+)`?/i

PS : Je n'ai pas testé...

[Maxoo]

Code :

preg_match(/^.*(FROM|INTO)\s+(\w)+/,$SQL,$resultat)

le tout etant de faire

Code :

print_r($Resultat);

et de voir si ca te donne ce qu'il faut.

Si ca marche pas, donne moi un exemple de requete SQL que tu peux avoir et je te dirai ou le regexp merde.

[Yogui]

maxoo : Ton expression semble passer mais il mettre le + dans la parenthèse.

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
$sql = 'SELECT truc FROM table1 WHERE INSERT INTO table2() VALUES () UPDATE table3 SET ';
 
preg_match_all('/(?:SELECT|UPDATE|INSERT)\s([a-z0-9_]+)?\s?(?:FROM\s|INTO\s|SET\s)`?([a-z0-9_]*\s?)/i', $sql, $matches, PREG_SET_ORDER);
echo '<pre>';
print_r($matches);
echo '</pre>';
 
foreach($matches as $match){
   if(!empty($match[2])){
      $tablename = $match[2];
   }
   else{
      $tablename = $match[1];
   }
 
   echo $tablename.'<br />';
}

J'ai testé mon code, cette fois, et ça fonctionne.
Parcours $matches : pour chaque élément, ton nom de table est en première position. Tu peux enlever des ?: pour avoir plus d'infos.

[Edit] Correction : ça ne fonctionnait pas correctement pour les UPDATE.
Pour avoir les résultats, tu dois tester chaque $match : si $match[2] est non vide, alors c'est ton nom de table, sinon tu le trouveras dans $match[1].

[Maxoo]

Envoyé par Kirkis

maxoo : Ton expression semble passer mais il mettre le + dans la parenthèse.

exact !!!

une question : pourquoi tu t'embete a définir des trucs avant le FROM et apres le nom de la TABLE ... ca ne sert a rien ici.

il faut optimiser (le code et la comprehension !!)

[Yogui]

C'est pour les requêtes UPDATE. Comment les gères-tu si tu ne mets rien ?

De plus, il est plus performant d'ancrer les regex et de les faire commencer par une chaîne littérale. Dans mon exemple, il n'y a pas d'ancrage car on peut parser une chaîne complète pour plusieurs requêtes.

[Edit] Tiens, pour la compréhension :

Code :

1
2
3
4
5
6
7
preg_match_all('/'
   .'(?:SELECT|UPDATE|INSERT)\s'
   .'([a-z0-9_,` ]+)?'        // Maybe a tablename, maybe a list of fields
   .'(?:FROM\s|INTO\s|SET\s)'
   .'`?'
   .'([a-z0-9_]*\s?)'         // If present, assuredly a tablename
   .'/i', $sql, $matches, PREG_SET_ORDER);

[Maxoo]

Code :

preg_match(/(FROM|INTO|UPDATE)\s+(\w+)/i,$SQL,$resultat)

mais avec ca, ca marche pour tout les SQL aussi

[Yogui]

Ouaip...
/me va se pendre

[XtofRoland]

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
<?
$SQL = "select * from table where qqq = 111";
preg_match(/(FROM|INTO|UPDATE)\s+(\w+)/i,$SQL,$resultat); //line 3
foreach($resultat as $brol) echo $brol.'<br />'; 
 
//Parse error: parse error, expecting `')'' in h:\easyphp1-8\www\index.php on line 3
 
 
$sql = 'SELECT truc FROM table1  '; 
 
preg_match_all('/(?:SELECT)\s([a-z0-9_]+)?\s?(?:FROM\s)`?([a-z0-9_]*\s?)/i', $sql, $matches, PREG_SET_ORDER); 
 
echo $matches[0][2].'<br />'; 
 
?>

la deuxième solution marche impecc ;-)

[XtofRoland]

Code :

1
2
3
4
5
//$msSQL = "SELECT top 20 WO_NUM, TASK, TYPE, STATUS ,'detail.php?num=' + cast( wo_num as char ) from TASKS WHERE TYPE IS NOT NULL AND STATUS = 'TERMINÉ' ";
 
$msSQL = strtoupper($msSQL);
 
preg_match_all('/(?:SELECT)\s([a-z0-9_]+)?\s?(?:ORDER BY\s)`?([a-z0-9_]*\s?)/i', $msSQL, $matches, PREG_SET_ORDER);

me retourne un tableau vide, vous savez pourquoi?

[Yogui]

Arf, tu voudrais pas utiliser la balise code stp ?

Sinon, ne serait-ce pas parce que tu as mis ta chaîne en commentaires ?


Tu n'aimes pas la solution de maxoo ?

[Edit] La vraie réponse : la classe qui matche ce qui suit SELECT est "[a-z0-9_]", or tu utilises également d'autres signes comme "?", "+", etc.
Modifie la classe ou utilise la solution de maxoo (plus simple, c'est évident).

[Edit 2] Tu mets également une obligation de trouver "ORDER BY" alors qu'il n'y en a pas...

[XtofRoland]

mon select est :

Code :

SELECT WO_NUM, TASK, TYPE, STATUS ,'DETAIL.PHP?NUM=' + CAST( WO_NUM AS CHAR ) FROM TASKS WHERE TYPE IS NOT NULL AND STATUS <> 'TERMINÉ' ORDER BY WO_NUM DESC

j'ai rajouté les caracteres manquants:

Code :

preg_match_all('/(?:SELECT)\s([a-z0-9_\'\?\+\.\=\(\)]+)?\s?(?:ORDER BY\s)`?([a-z0-9_]*\s?)/i', $msSQL, $matches, PREG_SET_ORDER);

mais ca ne va pas..

la solution de maxoo me donne tjrs :

Parse error: parse error, expecting `')'' in h:\easyphp1-8\www\tfe\OO\msmaster.inc on line 16

l'adaptation qui me donne le resultat le plus proche est :

preg_match_all('/(ORDER BY)\s+?([a-z0-9_]*\s?)/i',$msSQL,$resultat);

foreach($resultat as $brol) foreach( $brol as $r)echo $r."<br />";

elle m'affiche:

ORDER BY WO_NUM
ORDER BY
WO_NUM

le DESC manque...

[Maxoo]

Code :

1
2
3
4
<?
$SQL = "select * from table where qqq = 111";
preg_match(/(FROM|INTO|UPDATE)\s+(\w+)/i,$SQL,$resultat); //line 3
foreach($resultat as $brol) echo $brol.'<br />';

//Parse error: parse error, expecting `')'' in h:\easyphp1-8\www\index.php on line 3

preg_match(/'(FROM|INTO|UPDATE)\s+(\w+)/i',$SQL,$resultat);

une parse error ca se trouve facilement quand même ... t'aurais pu chercher un peu

[XtofRoland]

j'ai cherché! la preuve .... dans mon post precedent en mélangeant les deux preg j'ai essayé de faire ca:

l'adaptation qui me donne le resultat le plus proche est :

preg_match_all('/(ORDER BY)\s+?([a-z0-9_]*\s?)/i',$msSQL,$resultat);

foreach($resultat as $brol) foreach( $brol as $r)echo $r."<br />";