Discussion :

[okoweb]

Code :

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

1
2
3
4
5
6
7
 
if(isset($_GET['id'])){
  $id =(int)$_GET['id'];
  unset($_GET['id']);
  //Suite du code avec $id
  ...
}

A quoi peut vraiment servir ce bout de code ? Pourquoi détruire $_GET['id']

Code :

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

unset($_GET['id']);

et pour le reste du code bosser avec $id ?

Merci d'avance...

[transgohan]

Il est vrai qu'à part être sûr que la valeur ne pourra être réutilisé sans être épurée et testée il n'y a aucun intérêt.
Mais c'est tout de même un très bon point que d'interdire d'utiliser une variable brute.

[ThomasR]

Ya quand même plus simple :

Code :

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

1
2
3
 
if(isset($_GET['id']))
    $_GET['id'] = intval($_GET['id']);

comme çà quelque-soit le code qui utilise la variable passée en paramètre elle est bien nettoyée. De plus cela permet de conserver un fonctionnement normal d'application sans se casser la tête.

[ABCIWEB]

Comme l'a dit transgohan on évite d'utiliser directement des variables externes comme $_GET, $_POST, $_COOKIE, puisque celles-ci peuvent être définies directement pas l'utilisateur.
Dans ton exemple $_GET['id'] est transtypé en entier, et donc "id" sera nécessairement un entier.

On peut faire aussi comme dit ThomasR, chacun fait suivant ses préférences et suivant ses besoins, mais l'important est de prendre l'habitude de vérifier ses variables avant de les utiliser.

Souvent on profite de l'occasion pour attribuer une valeur par défaut, par exemple

Code :

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

$id = isset($_GET['id']) && is_numeric($_GET['id']) ? intval($_GET['id']) : 0;

[okoweb]

Bjr,
Pour les données provenant d'un formulaire

Code :

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

1
2
3
 
$id =$_POST['var'];
unset($_POST['var']);

Ceci ne résoudrait pas le problème de rafraichissement d'une page recevant les données d'un formulaire ?

[Benjamin Delespierre]

Tant que le formulaire est soumis en HTTP classique (et non pas avec un XHR) le rafraichissement de page se produit car le navigateur fait partir une nouvelle requête POST.

[RunCodePhp]

Salut

Ceci ne résoudrait pas le problème de rafraichissement d'une page recevant les données d'un formulaire ?

Théoriquement non.
Quand on rafraichi (ou actualise) une page contenant un formulaire, les données sont à nouveaux transmises, donc ensuite à nouveau récupérées par le serveur et le même code sera alors exécuté.

Ce que ThomasR à suggéré me semble plus judicieux : filtrer la donnée plutôt que la détruire.


Ceci dit, quand on regarde comment procède certains FrameWork, ils procèdent en gros de la même manière que tu fais.
En gros ils stockent les données GET/POST entre autre dans un Objet, les filtre par la même occasion.
Les tableaux $_GET $_POST sont détruits/vidés, et c'est via l'Objet dédié qu'il faut exploiter coté application.

Ce n'est pas tous les FrameWork qui le font, mais n'empêche ça se fait.

[ThomasR]

Ceci ne résoudrait pas le problème de rafraichissement d'une page recevant les données d'un formulaire ?

Non car le navigateur va exécuter la requête HTTP précédente, et pas à partir de la réponse HTTP.