Discussion :

[Anduriel]

Salut,

Dans mon code j'ai ça:

Code :

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

1
2
3
4
5
6
if (!ereg("(.*)".$cat."(.*)", $str_cats)) {
   $str_cats .= $cat;
   $connection->query("UPDATE its_parametres SET valeur = ".quote_smart($str_cats)." WHERE parametre = 'cats'");
}
else 
   $_SESSION['erreur'] = "Catégorie déjà existante";

Je problème c'est que l'édition est faite dans ma base de donnée, mais que j'ai la session erreur qui est définie
J'ai fais des print_r($_SESSION) avec et après cette partie et la session erreur est bien définie ici.

Comme ça se fait?
Merci

[Eusebius]

Tu es sûr que la variable de session n'est pas définie avant ? Genre, ça te dirait pas de faire un petit unset dessus avant ton if/else ?

[Anduriel]

Merci mais ça ne change rien.
Si je commente la ligne de la requête plus d'édition, mais plus de message d'erreur.

[Invité]

mets des accolades dans le pire des cas (c plus propre)
sinon je pense pareil ta session doit etre defini quelque part avant.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
if (!ereg("(.*)".$cat."(.*)", $str_cats))
{
   $str_cats .= $cat;
   $connection->query("UPDATE its_parametres SET valeur = ".quote_smart($str_cats)." WHERE parametre = 'cats'");
}
else 
{
   $_SESSION['erreur'] = "Catégorie déjà existante";
}

[Yogui]

Au passage, ta regex est très peu optimisée.
Voici ce que je te propose :

Code :

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

preg_match('/'.$cat.'/', $str_cats)

J'ai changé de POSIX avec ereg() à PCRE avec preg_match() : plus rapide, plus intéressant à tous les niveaux.
J'ai également supprimé les .* qui ne servaient à rien. Cela devrait largement améliorer les perfs de la regex, même si tu ne le remarqueras probablement pas.

Pour terminer, une structure if/else ne passera jamais dans les deux branches à la fois. Pour t'en assurer, tu peux exécuter ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
if (!preg_match('/'.$cat.'/', $str_cats))
{
   echo 'if';
   $str_cats .= $cat;
   $connection->query("UPDATE its_parametres SET valeur = ".quote_smart($str_cats)." WHERE parametre = 'cats'");
}
else 
{
   echo 'else';
   $_SESSION['erreur'] = "Catégorie déjà existante";
}

[Anduriel]

Merci pour le preg_match c'était un peu trivial
Sinon c'est toujours aussi louche: le else est affiché, mais pourtant l'update est faite. Le script doit repasser par la mais le if ne s'affiche pas.