[Fonction Explode] Problème

Version imprimable

Bonjour,

Je fait appel à vous car j'ai un soucis avec la fonction explode.

J'ai une liste déroulante HTML comme ceci :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
<select name="categorie">
 
<option value="Equipe2">S&eacute;niors 1</option>
<option value="Equipe3">S&eacute;niors 2</option>
<option value="Equipe4">S&eacute;niors 3</option>
<option value="Equipe5">U19 (18 ans)</option>
<option value="Equipe6">U17 (15 ans)</option>
<option value="Equipe7">U15 (13 ans)</option>
<option value="Equipe8">U13 (Benjamins)</option>
<option value="Equipe9">U11 (Poussins)</option>
<option value="Equipe10">U9 (Debutants)</option>
<option value="Equipe11">U7 (Ecole de foot)</option>
<option value="Equipe12">Loisirs</option>
 
 
<option value="Evenement2">Le Loto</option>
<option value="Evenement3">Le Bal</option>
<option value="Evenement4">Le Stage de fin d ann&eacute;e</option>
 
 
</select>

Lors de la validation, je veut savoir si "categorie" est un evenement ou une equipe, et quel est le 'suffixe' (1, 2, 3, 4 ...)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
// Decoupage de $categorie pour savoir si c'est une evenement ou une equipe
$Trouve = contientChaine( $categorie , "Evenement");
 
// Puis requete adaptée	
if ($Trouve == true){	// Les photos sont en rapport avec un evenement
 
     $tab = explode("Evenement", $categorie);
     $idE = $tab[1];
 
else{	// Les photos sont en rapport avec une équipe
 
     $tab = explode("Equipe", $categorie);
     $idE = $tab[1];
 
 
}

Voici la fonction contientChaine utilisée :

Code:

1
2
3
4
5
6
7
8
9
10
function contientChaine($cherche , $chaine) {
 
	if(strpos($chaine, $cherche) <> ''){
		return true;
	}
	else{
		return false;
	}
 
}

Le probleme est que 'trouve' ne vaut jamais true, il va systematiquement dans le else :(

Ou-est mon erreur ? je ne vois pas ...

Merci pour votre aide précieuse :ccool:

Bonjour,

je veux pas être méchant car on est vendredi mais quand même un petit tour dans le manuel t'aurais épargné ce post à rallonge.

La fonction strpos() ne renvoie pas une chaine mais un index de position dans d'une chaine au sein d'une autre chaine

De plus, je ne vois pas l'interet de ta fonction contientChaine si celle-ci ne fait que ca. Pourquoi réinventer la roue et perdre ainsi la maintenabilité du code.

En faisant, simplement ceci :
Code:

1 2 3 4 5 6 7 if ( strpos($categorie, 'Evenement') !== false ){ $idE = str_replace('Evenement','',$categorie); } elseif (strpos($categorie, '') !== false) { $idE = str_replace('Equipe','',$categorie); }
Ensuite, suivant ce que tu fais de $idE, il y aura surement moyen de réorganiser le code.
Que se passe t-il si tu rajoute un élément dans la combo ?
Pour ma part, j'aurais inséré un caractère entre le type d'élément et son numéro, de cette façon l'explode() deviendrait utile

18/09/2009, 14h54
Xenon03

Je peut sans problème rajouter un '-' entre evenement (ou equipe) et le chiffre. :ccool:

Merci de ton aide

Sans prendre en compte la validité de ton code dans ce contexte, tu peux simplifier:

Code:

1
2
3
4
5
6
7
8
<?php
if(strpos($chaine, $cherche) <> ''){
		return true;
	}
	else{
		return false;
	}
?>

Code:

1
2
3
<?php
return ( strpos($chaine, $cherche) <> '' ); //Ceci dit cette condition n'est pas sémantiquement correcte.
?>

-----

Code:

1
2
3
4
<?php
$tab = explode("Evenement", $categorie);
$idE = $tab[1];
?>

Code:

1
2
3
<?php
list(, $idE) = explode("Evenement", $categorie);
?>

-----

Code:

1
2
3
<?php
if ($Trouve == true){
?>

Code:

1
2
3
<?php
if ( $Trouve ){ //Avec ou sans == true c'est EXACTEMENT pareil.
?>

J'espère que ces quelques astuces te serviront...

21/09/2009, 09h12
Xenon03

OK merci de toutes ces astuces.
Je débute en PHP, je suis plus ASP mais je dois changer de langage donc j'apprend, et tous les conseils sont bons à prendre ^^
21/09/2009, 09h29
blueice

Euh... une simple regex suffit avec preg_match ! 8O

et pour le suffixe, suffit d'utiliser le filtre de conversion FILTER_SANITIZE_NUMBER_INT