Mantenere parametri di filtro form

[Fractals87]

Ciao a tutti,

Tempo fa avevo aperto una discussione in tal proposito ed era emerso che uno dei modi per farlo era salvare tutto in una variabile di sessione...
Dato che mi sto lanciando nella creazione di classi a caso vorrei sottoporre quanto partorito e sapere se è qualcosa di sensato....
SIATE BRUTALI...

<?php
class form_form {
    private $form_name;
    
    public function getFormName(){        
        return $this->form_name;   
    } 
    
    public function setFormName($str){
        $this->form_name=$str;   
    }
    

    //Crea una variabile di sessione nominata come il campo "form_name" contenuto nel post
    public function createSessionFilter(){
        $arr = array();
        foreach($_POST as $key => $value) {
            if($key == "form_name"){
                $name = $value;
                continue;
            }
            $arr[$key] = $value ;
        }
        $_SESSION[$name] = $arr;
    }
    
    public function getFilter($param){
        foreach($_SESSION[$this->form_name] as $key => $value) {
            if($key == $param)
                return $value;
        }
        return "";
    }
}

//CODICE DI QUANDO OTTENGO IL POST

$frm_mod_int = new form_form($_POST);
$frm_mod_int->setFormName($_POST['form_name']);
$frm_mod_int->createSessionFilter();
unset($frm_mod_int);

//se l'utente rientra nella pagina posto un esempio di rivalorizzazione input
$frm_mod_int = new form_form(); 
$frm_mod_int->setFormName('FrmFilterModInt');
?>
<input name="pdr" type="text" size="14" value="<?php if(isset($_SESSION['FrmFilterModInt'])){echo $frm_mod_int->getFilter('pdr');}?>"><br />

[Shores]

Mmm, non capisco tanto il giro tra setFormName, il post e la sessione...

Se il nome della form vuoi che venga preso dal POST, prendilo dal post, ma allora a che ti serve poterlo settare con setFormName?

In più, fai new form_form($_POST), ma non definendo da nessuna parte la construct di quella classe dubito che il parametro $_POST che passi vada da qualche parte...

Poi non capisco perchè chiamare Filter qualcosa che in buona sostanza è ciò che l'utente ti aveva scritto nei campi...

Non so, io non procederei proprio così, ma invece creerei una classe form che contenga sia il nome della form stessa (assegnato da me e fisso) che l'elenco dei campi presenti che i valori dei campi stessi, e su ricezione del post salverei i valori nell'oggetto, che poi salverei in sessione. Per recuperare i valori, caricherei dalla sessione l'intero oggetto (oppure lo genererei nuovo se manca), e a quel punto lo userei per creare la maschera.

[tampertools]

   private $form_name;

    public function getFormName(){
                return $this->form_name;
       }

     public function setFormName($str){
               $this->form_name=$str;
       } 

Se lo puoi sia leggere che modificarlo, tanto vale dare a $form_name visibilità public

[Fractals87]

Mmm, non capisco tanto il giro tra setFormName, il post e la sessione...

Se il nome della form vuoi che venga preso dal POST, prendilo dal post, ma allora a che ti serve poterlo settare con setFormName?

Hai ragione.
In primis il codice è sbagliato perchè al massimo dovrebbe essere

$frm_mod_int = new form_form();
$frm_mod_int->setFormName($_POST['form_name']);
$frm_mod_int->createSessionFilter($_POST);
unset($frm_mod_int); 

Il setFormName lo avevo inserito per poter eliminare questo if... non mi sembrava elegante

            if($key == "form_name"){
                $name = $value;
                continue;
            } 

Il fatto che l'ho chiamato Filter non ha ragioni specifiche... magari parametro era più corretto...

Non so, io non procederei proprio così, ma invece creerei una classe form che contenga sia il nome della form stessa (assegnato da me e fisso) che l'elenco dei campi presenti che i valori dei campi stessi, e su ricezione del post salverei i valori nell'oggetto, che poi salverei in sessione. Per recuperare i valori, caricherei dalla sessione l'intero oggetto (oppure lo genererei nuovo se manca), e a quel punto lo userei per creare la maschera.

Anche io avevo pensato di creare una classe form specifica per ogni filtro...
Però ho molti filtri nella mia applicazione e volevo creare qualcosa di riutilizzabile, dici che è proprio sbagliato procedere in questo modo?

e a quel punto lo userei per creare la maschera

Questa non l'ho capita.

Se lo puoi sia leggere che modificarlo, tanto vale dare a $form_name visibilità public

Guarda ti do ragione, ma sul foro php ci sono molti javisti (sperando si dica cosi) che suggeriscono di fare sempre classi con getter e setter per non contraddire la regola di incapsulamento... e l'ho presa come abitudine... non so se ho fatto bene o male.

GRAZIE MILLE PER LE RISPOSTE