Discussion :

[Msieurduss]

Bonjour à tous,

Voilà mon souci... J'ai une entrée dans un formulaire dont le premier caractère est très souvent "," (exemple : ",Machin,Bidule,Truc,chose"). Cette entrée est récupérée dans un formulaire suivant et traitée comme suit :

Code :

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

1
2
3
4
5
 
$entree= addslashes($_POST["entree"]);
if(substr($entree,0,1)==",") { // On enleve la premiere "," si elle y est.
	$entree[0]="";
}

En gros, j'enlève la première virgule pour éviter diverses erreurs par la suite. Cela est alors mis en base de données : aucun problème !
Mais je récupère ceci dans un deuxième formulaire...
Dans un input dont la valeur est mon entrée, sans virgule toujours.
Et voilà que sous firefox, j'ai un bug ! Si je soumets le deuxième formulaire, il me dit que $_POST["entree"] n'est pas définie et il ne la met donc pas en base !
Si j'essaye de modifier le contenu de mon formulaire, je m'aperçois qu'il y a un petit carré présent tout au début du texte de $entree...
Si je l'enlève, ça marche.

Ce problème ne m'arrive pas avec IE...

Si quelqu'un a une idée... là pour moi c'est mystère et boule de gomme !

[KiLVaiDeN]

Salut,

Je ne pense pas que tu puisses manipuler ta chaine ainsi, en mettant la première valeur à "" pour espérer supprimer le premier caractère.

Il faut que tu recrées une chaine, par exemple :

Code :

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

1
2
3
4
// $entree= addslashes($_POST["entree"]);
if(substr($entree,0,1)==",") { // On enleve la premiere "," si elle y est.
	$entree= substr($entree,1);
}

Question : pourquoi faire le addslashes à ce moment là ? Je te conseille de le faire au moment de l'insertion en base

A+

[Eusebius]

change de méthode pour supprimer le premier caractère. Ce que tu fais... c'est mal !

Code :

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

$str = subst($str, 1);

Si tes chaînes sont modifiées de cette manière, je mets le fil de mon clavier à couper qu'elles ne te poseront plus de problème.

[Msieurduss]

Merci à vous deux pour vos réponses rapides (ce forum est vraiment exceptionnel).

change de méthode pour supprimer le premier caractère. Ce que tu fais... c'est mal !

Pourquoi c'est si mal ? Je croyais qu'on pouvait considérer les caractères d'une chaîne de la même manière qu'un tableau...

Si tes chaînes sont modifiées de cette manière, je mets le fil de mon clavier à couper qu'elles ne te poseront plus de problème.

Bon ben, tu seras ravi de savoir que tu peux encore garder ton clavier quelques temps .

Question : pourquoi faire le addslashes à ce moment là ? Je te conseille de le faire au moment de l'insertion en base

Déjà, merci de ta réponse .
Il y a une insertion en base juste après ça...
En version simplifiée, on a :

Code :

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

1
2
3
4
 
$entree=addslashes($_POST["entree"]);
 
$sql=mysql_query("INSERT INTO .......")

directement après donc... Tu verrais ça autrement ?

[Eusebius]

Pourquoi c'est si mal ? Je croyais qu'on pouvait considérer les caractères d'une chaîne de la même manière qu'un tableau...

En C oui, mais pas en PHP ! En PHP il faut utiliser des accolades et pas des crochets. Et encore, même si on considère que ça pourrait être le cas, tu veux mettre "" dans la première case du tableau, comment ça pourrait être codé en interne ? Même en C on ne peut pas faire comme ça, il faut décaler tout le tableau d'une case. D'où la reconstruction d'une nouvelle chaîne avec substr.

Bon ben, tu seras ravi de savoir que tu peux encore garder ton clavier quelques temps .

T'inquiète, j'avais pas pris de risques, c'est un clavier infrarouge

[Msieurduss]

En C oui, mais pas en PHP ! En PHP il faut utiliser des accolades et pas des crochets. Et encore, même si on considère que ça pourrait être le cas, tu veux mettre "" dans la première case du tableau, comment ça pourrait être codé en interne ? Même en C on ne peut pas faire comme ça, il faut décaler tout le tableau d'une case. D'où la reconstruction d'une nouvelle chaîne avec substr.

Maintenant que tu le dis, oui, je me rends compte de mon (énorme) erreur . Merci pour les infos

Code :

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

1
2
 
Etant donné que addslashes ne sert que dans la création de la requête, tu n'as pas vraiment besoin de l'utiliser ailleurs qu'au moment de la création de celle-ci justement ! ;)

Je vois . J'avoue que j'ai pas l'habitude de procéder comme ça... Mais tu as raison, ta version permet quelque chose de plus souple .

[KiLVaiDeN]

Non non, c'est OK alors C'est une bonne idée de récupérer les valeurs de ton formulaire dans des variables au lieu d'utiliser directement le tableau _POST, ça permet une meilleure lisibilité + une meilleure maintenabilité.

Mais Je verrais plutot quelque chose comme ça :

Code :

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

1
2
3
$entree = $_POST['entree'];
 
$sql = "SELECT * FROM table WHERE entree = ' " . addslashes($entree) . "' ";

Etant donné que addslashes ne sert que dans la création de la requête, tu n'as pas vraiment besoin de l'utiliser ailleurs qu'au moment de la création de celle-ci justement !

C'est un tout petit peut plus propre quand même !

A+

PS : il vaut mieux utiliser mysql_real_escape_string() que addslashes