Discussion :

[Seb2913]

Bonjour à tous ,

Voilà j'ai un projet de fin d'année en MVC et j'ai un soucis concernant la redirection de mes pages .

Le soucis n'est pas tant fonctionnel car tout fonctionne mais le fait que j'utilise des header pour me rediriger pour arriver sur l'action "choisirFiche" quand j'ai terminé une action comme "supprimer" donc j'aimerai savoir comment m'y prendre car on m'a dit que cela ne faisait pas d'utiliser un header pour retourner dans l'action initial donc j'aimerai un conseil technique pour le faire proprement .

Merci d'avance à tous

Voici mon code :

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
 
 
switch($action){        
    case 'choisirFiche':
        if(isset($_GET['maj'])){
            $info="Mise à jour effectuée";
            include('vues/v_info.php');
        }
        $lesMoisVisiteurs = $pdo->getFiches('CL');
        if($lesMoisVisiteurs == null){
            $info = "Aucune Fiches à traiter";
            include('vues/v_info.php');
        }
        else{
            $ucPasse = 'validFrais';
            $actionPasse = 'choisirFiche';
            if(!empty($_POST)){
                $value = $_POST['listeFiches'];
                $idVisiteurMois =  explode('-',$value);
                $id = $idVisiteurMois[0];
                $mois = $idVisiteurMois[1];
            }else{
                $lesMoisVisiteursLigne1 = $lesMoisVisiteurs[0];
                $mois = $lesMoisVisiteursLigne1['mois'];
                $id  = $lesMoisVisiteursLigne1['id'];
                $value = $id."-".$mois;
            }
            $listeFraisForfait = $pdo->getLesFraisForfait($id, $mois);
            $listeFraisHorsForfait = $pdo->getLesFraisHorsForfait($id, $mois);
            include 'vues/v_listeFiches.php';
            include 'vues/v_comptableValidation.php';
        }
    break;
    case 'mettreAJour':
        $fraisForfait = array(
                "ETP"=>intval($_POST["lesFrais"]["ETP"]),
                "KM"=>intval($_POST["lesFrais"]["KM"]),
                "NUI"=>intval($_POST["lesFrais"]["NUI"]),
                "REP"=>intval($_POST["lesFrais"]["REP"])
        );
        $pdo->majFraisForfait($_POST["id"], $_POST["mois"], $fraisForfait);
        header("Location:index.php?uc=validFrais&action=choisirFiche&maj=ok");
 
    break;
    case 'supprimer':
        $pdo->supprimerFraisHorsForfait($_GET['idFrais']);
        header("Location:index.php?uc=validFrais&action=choisirFiche");
    break;
    case 'reporter':
        $pdo->reporter1Mois($_GET['idFrais'], $_GET['date']);
        header("Location:index.php?uc=validFrais&action=choisirFiche");
    break;
    case 'valider':
        $pdo->modificationEtat("VA" , $_POST['id'] , $_POST['mois']);
        header("Location:index.php?uc=validFrais&action=choisirFiche");
    break;
 
 
 
}
?>

[rawsrc]

Salut,

c'est une manière de faire pas très coutumière faut avouer.
Pourquoi ne renvoies tu pas directement la vue correspondant à la bonne ou mauvaise fin de l'action ?
Exemple :

Code :

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

1
2
3
4
case 'supprimer':
        $pdo->supprimerFraisHorsForfait($_GET['idFrais']);
        header("Location:index.php?uc=validFrais&action=choisirFiche");
    break;

par quelque chose de cet acabit :

Code :

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

1
2
3
4
case 'supprimer':
        $pdo->supprimerFraisHorsForfait($_GET['idFrais']);
        include 'ta_vue';
    break;

D'ailleurs prends l'habitude de TOUJOURS mettre exit; après chaque redirection parce qu'une redirection n'arrête pas le script et si tu as du code après, il sera exécuté.

[Seb2913]

Par exemple quand j'ai supprimé une fiche de frais , je dois recharger les fiches restantes quand je reviens sur la page et c'est l'action "choisirFiche" qui fait ca , c'est pour ca que j'ai utilisé le header mais si cela ne se fait pas alors je vais modifier mais à part faire procédure reprenant le code de "choisirFiche" comme ca j'utilise pour les deux je vois pas trop comment faire autrement .

Merci du conseil pour le header

[rawsrc]

Ce n'est pas une question si ça se fait ou pas c'est juste que tu économises une grosse étape dans le traitement.
Au lieu de rediriger vers la page de traitement adéquate tu renvoies tout de suite la bonne page.
D'un côté tu vas faire : 2 allers-retours et de l'autre qu'un seul... Lequel est le mieux à ton avis ?

[Seb2913]

Un aller Merci des conseils