correction d'une foreach

[apt]

Bonjour,

J'aimerais changé ce bout de code :

Code :

1
2
3
4
5
6
7
8
 
$title = ($_POST['title'];
$post = $_POST['post'];
$titleimg = $_POST['titleimg'];
	if(verifiesimage($_POST['linkupload'], 150) == TRUE)
	{
		$linkupload = $_POST['linkupload'];
	}

par ceci :

Code :

1
2
3
4
5
6
7
8
9
10
foreach ($_POST as $key => $value) {
		if ( $value == $_POST['linkupload']) {
			if(verifiesimage($_POST['linkupload'], 150) == TRUE)
			{
				$linkupload = $_POST['linkupload'];
			}
 
			$key = $_POST[$key];
		}
	}

Mais ça ne marche pas !!

Merci d'avance.

[sabotage]

Tu devrais effectivement revoir le principe d'une boucle foreach().

En dehors de ça, pourquoi vouloir mettre une boucle ici ?

[apt]

Seulement pour ne pas réécrire toutes les variables

[vorace]

Code :

1
2
3
4
5
6
			if(verifiesimage($_POST['linkupload'], 150) == TRUE)
			{
				$linkupload = $_POST['linkupload'];
			}
 
			$key = $_POST[$key];

quand tu fais ça tu récupère exactement la même valeur dans $key et $linkupload parce que $_POST[$key] = $_POST['linkupload'], si c'est la clé que tu veux récupérer, elle est déjà dans $key, tu n'as pas besoin de lui affecter $_POST[$key]...

[apt]

Parmi tout les POST, je devrais vérifier la valeur de $_POST['linupload'] (lien complet d'une image).

C'est pour ça que j'ai fais ce test.

[vorace]

ce que je veux dire c'est que $key contient le name de l'input et $valeur sa value, quand ton input s'appelle linkupload $linkupload et $key contiendront tout les deux la même valeur et dans ton foreach comme tu utilises toujours $key tu écrases les valeurs de tes champs et tu ne récupère que la valeur du dernier...
ce que tu peux faire dans ton foreach c'est :

Code :

${$key} = CleanPost($valeur);

tu as ainsi des variables qui ont le nom de tes input avec pour valeur les values nettoyé avec ta fonction cleanpost que tu as du corriger...

[sabotage]

Tu as la fonction extract() qui fait déjà ça :

Code :

1
2
3
 
extract($_POST);
$linkupload = (verifiesimage($linkupload, 150)) ? $linkupload : NULL;

Il n'est cependant pas recommandé de créer automatiquement les variables, quelque soit la méthode.

[vorace]

Code :

extract($_POST)

crées les variables mais ne les nettoies pas.

[apt]

Envoyé par vorace

ce que tu peux faire dans ton foreach c'est :

Code :

${$key} = CleanPost($valeur);

tu as ainsi des variables qui ont le nom de tes input avec pour valeur les values nettoyé avec ta fonction cleanpost que tu as du corriger...

par exemple si je veux utiliser la variable $title ou $linkupload, comment le faire ?

parce que j'ai essayé d'afficher cette variable, mais elle est toujours vide ?

Code :

<p><?php echo $title;?></p>

[vorace]

si tu veux à la base si $title est vide c'est parceque tu as une faute dans ton code :

Code :

$title = ($_POST['title'];

la parenthèse est en trop et si tu veux une variable du nom de l'input la ligne de code suivante :

Code :

${$key} = CleanPost($valeur);

le fait.
exemple avec ton input title
dans le foreach, quand $key ="title" tu vas avoir $title qui sera égale à sa valeur $valeur.

[apt]

L'erreur était dans le manque d'un $ entre {}:

${$key} = CleanPost($valeur);

mais pourquoi écrire ${$key} au lieu $key seulement ?

[sabotage]

$key est la variable "key"
${$key} est la variable dont le nom est la valeur de $key

[vorace]

$key est la variable "key"
${$key} est la variable dont le nom est la valeur de $key

je ne l'aurais pas mieux expliqué, c'est vrai que c'est assez difficile à appréhender la première fois...surtout l'utilité !

[apt]

Envoyé par sabotage

$key est la variable "key"
${$key} est la variable dont le nom est la valeur de $key

Merci.