[Système] if() exécuté mais else() aussi

**Anduriel** · 18/05/2006, 19h41

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** · 18/05/2006, 19h46

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** · 18/05/2006, 20h17

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é · 18/05/2006, 20h27

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** · 19/05/2006, 01h52

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** · 19/05/2006, 17h08

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.

**Yogui** · 19/05/2006, 23h11

Si "if" n'est pas affiché, alors ton script ne passe pas dans le if. Si la requête est exécutée sans que "if" soit affiché, c'est assurément que tu l'as également mise ailleurs.

**francis m** · 20/05/2006, 01h13

je pense que si tu n'utilises pas d'accolades pour ton else, le code du else doit être écrit juste derrière le else et non dans la ligne en dessous

donc, à mon avis, ta session(erreur) n'est pas considérée comme faisant partie du else mais simplement une instruction qui suit le if/else

**Yogui** · 20/05/2006, 03h12

Non non, après un else, PHP exécute jusqu'au premier point virgule. Peu importent les sauts de ligne et autres espaces.

**Anduriel** · 20/05/2006, 13h01

Bon je crois que je vais craquer tout seul parce que je ne trouve pas du tout. Il doit y avoir un joli super méga bug...
J'ai deux pages liées et sur les deux je n'ai que 2 déclaration de la session. En commentant la seconde, je suis certain que la session erreur vient de ce que je vous ai montré.
J'ai tout regardé sans résultat... Je vais essayer autre chose

**francis m** · 20/05/2006, 22h49

es-tu sûr de ne pas avoir le même script deux fois dans ta page, et que tu passes une fois avec le if et l'autre fois avec le else ?

ou que c'est un include ?

ne peux-tu remettre ton dernier code (si tu l'as changé) et dans le cas contraire un peu plus de code si ce n'est toute la page (j'espère qu'elle ne fait pas 3 km)

**Anduriel** · 20/05/2006, 23h18

Actuellement j'ai ça mais je pense que je vais changer en + les explode sur chaine je trouve ça pas joli...

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
28
29
30
31
32
33
case 'EDIT_DELETE_CAT':
            global $connection, $str_cats;
            $categories = $connection->variable("cats");
            if ($_POST['to_do_cat'] != 'none') {
               if ($_POST['do_cat'] == "Supprimer") {
                  $categories_explode = explode(",", $categories);
                  for ($i = 0; $i < count($categories_explode); $i++) {
                     if ($categories_explode[$i] != $_POST['to_do_cat'] && !empty($categories_explode[$i]))
                        $str_cats .= $categories_explode[$i].',';
                  }
                  $str_cats = substr($str_cats, 0, strlen($str_cats) - 1);
                  $connection->query("UPDATE its_parametres SET valeur = '".$str_cats."' WHERE parametre = 'cats'");
                  $connection->query("UPDATE its_links SET cat = 'Divers' WHERE cat = '".$_POST['to_do_cat']."'");
               }
               elseif ($_POST['do_cat'] == "Editer") {
                  $cat = trim($_POST['name_new_cat']);                  
                  $categories_explode = explode(",", $categories);
                  for ($i = 0; $i < count($categories_explode); $i++) {
                     if ($categories_explode[$i] != $_POST['to_do_cat'] && !empty($categories_explode[$i]))
                        $str_cats .= $categories_explode[$i].',';
                  }
                  if (!preg_match('/'.$cat.'/', $str_cats)) {
                     $str_cats .= $cat;
                     echo 'ok';
                     $connection->query("UPDATE its_parametres SET valeur = ".quote_smart($str_cats)." WHERE parametre = 'cats'");
                  }
                  else {
                     $_SESSION['erreur'] = "Catégoris déjà existante";
                  }
              }
            }
            header("Location: account.php?op=liens");
            break;

**francis m** · 21/05/2006, 01h21

comme cela à vue de nez il n'y a rien de particulier

mais je vois que toute cette séquence fait partie d'un switch.
n'y aurait-il pas un autre case avant, sans break, qui exécute ta requête ?

sinon je ne vois qu'une seule façon pour toi de debugger, c'est de mettre des echo d'un numéro un peu partout, afin de voir quel part du script est exécutée

**Anduriel** · 21/05/2006, 12h35

Ouai j'ai regardé les break et y'a pas de problème. Je crois que après changé ma structure de bdd je vais revoir ça.
Merci

**Anduriel** · 30/05/2006, 20h38

Bon j'ai change ma structure de code et maintenant j'ai ça:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
elseif ($_POST['do_cat'] == "Editer") {
   $cat = quote_smart(trim($_POST['name_new_cat']));
   $old_cat = quote_smart($_POST['to_do_cat']);
   print_r($_SESSION);
   if ($connection->linesw("its_cat", "text=".$cat) == 0) {
      $connection->query("UPDATE its_cat SET text = ".$cat." WHERE text = ".$old_cat);
   }
   else {
      $_SESSION['erreur'] = "Catégorie déjà existante";
   }
   print_r($_SESSION);
}

C'est bien différent d'avant mais j'ai toujours le même problème: l'édition dans la table est bien faite, mais la session erreur est bien initialisée... (et c'est bien ici d'après les print_r()...

Gné
Je vois pas d'où ça vient

[Système] if() exécuté mais else() aussi

Langage PHP

Discussions similaires

Partager

Partager