Discussion :

[ingeniomatique]

Bonsoir à tous,

J'ai le code suivant :

Code :

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

1
2
3
4
$r4=mysql_query("SELECT pseudo FROM users WHERE mort=1 AND timemort<=0");
while ($row2 = mysql_fetch_array($r4)){
$pseudo3=$row2['pseudo'];
$r5=mysql_query("DELETE FROM users WHERE pseudo='".$pseudo3."'");

Pour effacer les utilisateurs trop anciens de ma BDD.
Or, cela ne fonctionne pas pour les utilisateurs dont le pseudo contient un guillement simple : '

Comment faire pour contourner le souci ?

Merci par avance,
Antoine

[stealth35]

mysql_real_escape_string

[sabotage]

Et pourquoi pas

Code sql :

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

DELETE FROM users WHERE mort=1 AND timemort<=0

?

[ingeniomatique]

Sabotage, impossible car j'ai toute une liste de requêtes qui suivent Mais ça aurait été trop beau !!!

J'ai donc testé avec :

Code :

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

$r5=mysql_query("DELETE FROM users WHERE pseudo=".$pseudo3.",mysql_real_escape_string($pseudo3)");

Sans succès.

Merci par avance de votre aide.

[sabotage]

ça aurait été trop beau

Ba non ce n'est pas trop beau, même si as d'autres traitement qui nécessitent d'être fait utilisateur par utilisateur (et encore il faut encore une fois être sur que c'est pertinent), rien ne t'empêche d'au moins faire le DELETE d'un seul coup.

Pour le mysql_real_escape_string(), je ne sais pas d'ou tu nous a sorti cette idée de le placer à la fin séparé par une virgule.
Il est de toute façon plus simple et efficace de prendre comme référence l'id de l'utilisateur et non son pseudo.

[ABCIWEB]

Bah y'a peut-être pas d'id dans sa table. Si pseudo est unique il peut servir d'identifiant.

@ingeniomatique
Mais bon c'est certain que plutôt que de sélectionner une condition "WHERE mort=1 AND timemort<=0" pour rechercher ensuite le pseudo et pour enfin effacer les lignes correspondantes, autant effacer directement les lignes sur la condition et donc DELETE FROM users WHERE mort=1 AND timemort<=0 devrait suffire pour effacer les lignes ciblées.

Sinon avec mysql_real_escapre_string on devrait écrire :

Code :

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

$r5=mysql_query("DELETE FROM users WHERE pseudo='".mysql_real_escape_string($pseudo3)."'");

[ingeniomatique]

Merci beaucoup. Cela fonctionne. Une bonne astuce à garder au chaud !!!


Non non non, je vous assure, la requête proposée avant ne me servait à rien toute seule. Car il y avait ensuite toute une liste de DELETE sur d'autres tables, qui ne contiennent pas le champ MORT ou TIMEMORT. Donc la question restait entière.

Merci de votre aide et très bonne fin de soirée (début pour d'autres).