classe php sbilenca

[jappoz]

salve a tutti, ho provato a creare una classe che abbia i metodi necessari a ricevere i dati da un form e salvarli in un file, dopo aver verificato l' input dell' utente e di avere i permessi sui file.
ecco cosa mi è uscito

<?php
 class Scrivifile {
  public $file="iscritti.txt";
  public $nome;
  public $cognome;
  public $mail;
 
public function __construct() {   
 
 $this->nome=$_POST['nome'];
 $this->cognome=$_POST['cognome'];
 $this->mail=$_POST['mail'];
}
 
public function Controllodati ($nome,$cognome,$mail) {
 if(!isset($this->nome) ||!$this->nome || !isset($this->cognome) || !$this->cognome )
 return FALSE;
 else return TRUE;
 }
public function Controllomail($mail) {
 trim($this->mail);
 
    if(!$email)    
        return FALSE;   
 
    $num_at = count(explode( '@', $this->mail )) - 1;   
    if($num_at != 1)    
        return FAlSE;   
 
 
    if(strpos($this->mail,';') || strpos($this->mail,',') || strpos($this->mail,' '))    
        return FALSE;   
 
    if(!preg_match( '/^[\w\.\-]+@\w+[\w\.\-]*?\.\w{1,4}$/', $this->mail))   
        return FALSE;  
 
return TRUE;
 
}
public function Controllofile ($file) {
if (is_writable($this->file) && feof($this->file))
       return TRUE;
       else return FALSE;
}
public function Scrittura ($file) {
if($this->Controllodati($nome,$cognome,$mail) && $this->Controllomail($mail) && Controllofile($file)) {
 if(fopen($this->file,a)) {
 $dascrivere="$this->nome | $this->cognome |  $this->mail/r/n";
 fwrite($this->file,$dascrivere);
 fclose($this->file);
 }
    else echo "errore nell' apertura del file, ci scusiamo per l' inconveniente";
}
else if(!$this-Controllodati($nome, $cognome, $mail) || !$this->Controllomail($mail))
echo "I dati da te immessi sono errati, la preghiamo di ricontrollarli" ;
}
}
?>

facendo il debug non è sorto nessun errore, ma ho molte perplessità sul codice.
E' concettualmente giusto come sono strutturate le funzioni?
Come potrei indentare e rendere più efficente e funzionale questa stessa classe' Cosa è consigliabile che modifichi?
Insomma, secondo voi è scritta bene o si poteva scrivere notevolmente meglio?
Grazie delle risposte , ciao a tutti

 

[borgo italia]

per qullo che conosco php mi sembra giusta, uniche osservazioni una sul cadice (ma forse si tratta di una mia abitudine) ed una no

codice: io chiudo sempre gli if(){}elseif{}

non codice:

"I dati da te immessi sono errati, la preghiamo di ricontrollarli"

correggerie in

i dati da te..., ti prego di...
oppure
i dati da lei ..., la preghiamo...

 

[jappoz]

grazie della risposta , quello vedrò di modificarlo ma
ho un altro problema proprio con questa classe
facendo partire questa mini applicazione formata da
un form html (pagina1.htm)

<form action="script1.php" method="POST">
<input type="text" name="nome" /> Immetti il nome <br>
<input type="text" name="cognome" /> Immetti il cognome <br>
<input type="text" name="mail" /> Metti qui la tua mail <br>
<input type="submit" value="invia" />
</form>

una classe .inc (vedere quella postata sopra)
uno script php linkato al method del form (script1.php)

<?php
include ("classe1.inc");
$scrivifile = new Scrivifile;
$scrivifile->Controllodati($nome,$cognome,$mail);
$scrivifile->Controllomail($mail);
$scrivifile->Controllofile($file);
$scrivifile->Scrittura($file);
?>

Quando faccio una prova, immettendo dati sbagliati mi da il messaggio di errore, ma anche mettendo quelli giusti
Mi sto scervellando da ore e ho capito che c'è qualcosa che non va nella classe, ma non so cosa! Ho provato di tutto, potete aiutarmi perfavore?

 

[borgo italia]

secondo me dovresti individuare cosa ti da errore, se controlladati o email o altro

 

[christian`]

grazie della risposta , quello vedrò di modificarlo ma
ho un altro problema proprio con questa classe
facendo partire questa mini applicazione formata da
un form html (pagina1.htm)

<form action="script1.php" method="POST">
<input type="text" name="nome" /> Immetti il nome <br>
<input type="text" name="cognome" /> Immetti il cognome <br>
<input type="text" name="mail" /> Metti qui la tua mail <br>
<input type="submit" value="invia" />
</form>

una classe .inc (vedere quella postata sopra)
uno script php linkato al method del form (script1.php)

<?php
include ("classe1.inc");
$scrivifile = new Scrivifile;
$scrivifile->Controllodati($nome,$cognome,$mail);
$scrivifile->Controllomail($mail);
$scrivifile->Controllofile($file);
$scrivifile->Scrittura($file);
?>

Quando faccio una prova, immettendo dati sbagliati mi da il messaggio di errore, ma anche mettendo quelli giusti
Mi sto scervellando da ore e ho capito che c'è qualcosa che non va nella classe, ma non so cosa! Ho provato di tutto, potete aiutarmi perfavore?

Ciao io sono nuovo del forum....anche io ora stò combattendo con le classi cmq che versione di php hai?
Magari è probabile che sia quella che non riesce a gestire bene
le classi :°).
Se ho detto una cazzata scusami anticipatamente, ma a me è successo quando cercavo di far partire un carrello in php e cambiando l'interprete ho risolto :°) grazie cmq ciao..:beer:

 

[borgo italia]

guardando la classe di controllo mi sembra che per nome e cognome non dovrebbero esserci errore , in fondo verifichi solo se immessi o no

prova a guardare la verifica di mail io (non è una classe) utilizzo questa funzione che 8gioco di parole) funziona

function convalida_mail($mail) //verifica correttazza formale indirizzo email non ammesso campo vuoto
{
  $mail=trim($mail);
  if (eregi("^([a-z0-9\._\-]+)(@[a-z0-9\.\-_]+)(\.{1,2}[a-z0-9]{2,6})$", $mail)) 
  { 
    return TRUE; 
  }else{ 
    return FALSE; 
  } 
}

p.s.
ho modificato l'espr reg in quanto mi sembra migliore in

("^([-a-z0-9\._]+)(@[-a-z0-9\._]+)(\.{1,2}[a-z0-9]{2,6})

comunque funziona nei due casi

 

[Trogo]

Decisamente c'è un po' di confusione tra le variabili dichiarate all'interno della classe e quelle passate come argomento.
Queste sono interne alla classe ( e questo è giusto):

 $this->nome=$_POST['nome'];
 $this->cognome=$_POST['cognome'];
 $this->mail=$_POST['mail'];

Ma poi richiami il metodo nel file script1.php:

 $scrivifile->Controllodati($nome,$cognome,$mail);

Passando come parametri della variabili non inizializzate, ma oltretutto inutili, essendo delle variabili a cui poi, giustamente, accedi internamente alla classe:

public function Controllodati ($nome,$cognome,$mail) {
 if(!isset($this->nome) ||!$this->nome || !isset($this->cognome) || !$this->cognome )
 return FALSE;
 else return TRUE;
 }

Ma il problema più grosso è qui:

public function Controllomail($mail) {
 trim($this->mail);
 
    if(!$email)                  // ritorna sempre falso, la variabili non esiste
        return FALSE;         // usare $this->mail
 
    $num_at = count(explode( '@', $this->mail )) - 1;   
    if($num_at != 1)    
        return FAlSE;   
 
 
    if(strpos($this->mail,';') || strpos($this->mail,',') || strpos($this->mail,' '))    
        return FALSE;   
 
    if(!preg_match( '/^[\w\.\-]+@\w+[\w\.\-]*?\.\w{1,4}$/', $this->mail))   
        return FALSE;  
 
return TRUE;
 
}

Spero di esserti stato d'aiuto. Se qualcosa non è chiaro chiedi pure.

 

[jappoz]

grazie mille davvero, mi hai chiarito moltissimi punti che mi erano confusi, sei stato utilissimo.
Pensando al costruttore non sarebbe meglio erp comodità fare invece che attribuire alle proprietà della classe i vari array di post attribuirgli semplicemente un altro nome per evitare di usare sempre this-> ?
es
$this->nome=$in_nome;
//ecc

e poi nello script linkato dal form assegnare i valori del $_POST alle variabili?
ora modifico e semmai posto il nuovo codice, grazie mille delle risposte!

 

[jappoz]

ho modificato così la classe

<?php
	class Scrivifile {
	 public $file="iscritti.txt";
	 public $nome;
	 public $cognome;
	 public $mail;
	                      
public function __construct($in_nome, $in_cognome, $in_mail, $in_file) {   
                                                                                                      
	$this->nome= $in_nome;
	$this->cognome=$in_cognome;
	$this->mail=$in_mail;
	$this->file=$in_file;
}


public function Controllodati ($in_nome,$in_cognome,$in_mail) {
 if(!$in_nome || !$in_cognome )
 return FALSE;
 else return TRUE;
 }
public function Controllomail($in_mail) {
 trim($in_mail);
  $num_at = count(explode( '@', $in_mail )) - 1;
    if(!$in_mail)    
        return FALSE;   
       
    else if($num_at != 1)    
        return FAlSE;   
       
    else if(strpos($in_mail,';') || strpos($in_mail,',') || strpos($in_mail,' '))    
        return FALSE;   
       
   else  if(!preg_match( '/^[\w\.\-]+@\w+[\w\.\-]*?\.\w{1,4}$/', $in_mail))   
        return FALSE;  
        else return TRUE;
        
	
}
public function Controllofile ($in_file) {
if (is_writable($in_file) && feof($in_file))
       return TRUE;
       else return FALSE;
}
public function Scrittura ($in_file) {
if($this->Controllodati($in_nome,$in_cognome,$in_mail) && $this->Controllomail($in_mail) && Controllofile($in_file)) {
	fopen($in_file,a);
	$dascrivere="$in_nome | $in_cognome |  $in_mail/r/n";
	fwrite($in_file,$dascrivere);
	fclose($in_file);
	}
    else echo "I dati da te immessi sono errati";

}


}

?>

é giusto secondo voi?
come potrei fare a richiamare i metodi nello script che passo al form?

 

[Trogo]

Commenti sulla classe

<?php
    class Scrivifile {
     public $file="iscritti.txt";
     public $nome;
     public $cognome;
     public $mail;
                          
// Bene il costruttore che slega i dati di inizializzazione dal contesto.
// Così è più facile da riutilizzare anche se i dati non arrivano da un
// POST come era prima
public function __construct($in_nome, $in_cognome, $in_mail, $in_file) {   
                                                                                                        // Ok l'inizializzazione delle proprietà della classe
    $this->nome= $in_nome;
    $this->cognome=$in_cognome;
    $this->mail=$in_mail;
    $this->file=$in_file;
}

// Perchè passare questi parametri al metodo, se nel costruttore
// sono state inizializzate come proprietà della classe?
// Eliminare i parametri
public function Controllodati ($in_nome,$in_cognome,$in_mail) {
// Usare le proprietà della classe al posto dei parametri
// $this->nome invece di $in_nome, ecc...
 if(!$in_nome || !$in_cognome )
 return FALSE;
 else return TRUE;
 }
// Vedi sopra
public function Controllomail($in_mail) {
 trim($in_mail);
  $num_at = count(explode( '@', $in_mail )) - 1;
    if(!$in_mail)    
        return FALSE;   
       
    else if($num_at != 1)    
        return FAlSE;   
       
    else if(strpos($in_mail,';') || strpos($in_mail,',') || strpos($in_mail,' '))    
        return FALSE;   
       
   else  if(!preg_match( '/^[\w\.\-]+@\w+[\w\.\-]*?\.\w{1,4}$/', $in_mail))   
        return FALSE;  
        else return TRUE;
        
    
}

// Vedi sopra sopra
public function Controllofile ($in_file) {
if (is_writable($in_file) && feof($in_file))
       return TRUE;
       else return FALSE;
}

// Vedi sopra sopra sopra.
// Riflessione: se i metodi Controllodati, Controllomail e Controllofile
// vengono giustamente usati per validare l'input utente prima della
// scrittura, e quindi vengono richiamati dal seguente metodo, perchè
// questi metodi devono essere pubblici?

public function Scrittura ($in_file) {
if($this->Controllodati($in_nome,$in_cognome,$in_mail) && $this->Controllomail($in_mail) && Controllofile($in_file)) {
    fopen($in_file,a);
    $dascrivere="$in_nome | $in_cognome |  $in_mail/r/n";
    fwrite($in_file,$dascrivere);
    fclose($in_file);
    }
    else echo "I dati da te immessi sono errati";

}


}

?>

Forse sono un po' pedante, ma credo sia fondamentale avere ben chiari alcuni concetti da subito

 

[jappoz]

No non sei affatto pedante, mi stai schiarendo le idee moltissimo! e ti ringrazio.
Quindi ricapitolando.
Nella classe metto i metodi tutti protected e non passo parametri ai metodi. Fin qui ci siamo.
Ultima cosa , scusa se abuso del tuo aiuto ma non saprei proprio come fare.
Come posso scrivere il file script.php che utilizza i metodi di questa classe per fare appunto tutte le operazioni da me volute sui dati (ovvero controllo dei dati e scrittura nel file )?

grazie mille dell' aiuto

a presto

<?php
	class Scrivifile {
	protected $file="iscritti.txt";
	protected $nome;
	protected $cognome;
	protected $mail;
	                      
protected function __construct($in_nome, $in_cognome, $in_mail, $in_file) {   
                                                                                                      
	$this->nome= $in_nome;
	$this->cognome=$in_cognome;
	$this->mail=$in_mail;
	$this->file=$in_file;
}


protected function Controllodati () {
 if(!$this->nome || !$this->cognome )
 return FALSE;
 else return TRUE;
 }
protected function Controllomail() {
 trim($this->mail);
  $num_at = count(explode( '@', $this->mail )) - 1;
    if(!$this->mail)    
        return FALSE;   
       
    else if($num_at != 1)    
        return FAlSE;   
       
    else if(strpos($this->mail,';') || strpos($this->mail,',') || strpos($this->mail,' '))    
        return FALSE;   
       
   else  if(!preg_match( '/^[\w\.\-]+@\w+[\w\.\-]*?\.\w{1,4}$/', $this->mail))   
        return FALSE;  
        else return TRUE;
        
	
}
protected function Controllofile () {
if (is_writable($this->file) && feof($this->file))
       return TRUE;
       else return FALSE;
}
protected function Scrittura () {
if($this->Controllodati() && $this->Controllomail() && Controllofile()) {
	fopen($this->file,a);
	$dascrivere="$this->nome | $this->cognome |  $this->mail/r/n";
	fwrite($this->file,$dascrivere);
	fclose($this->file);
	}
    else echo "I dati da te immessi sono errati";

}


}

?>

 

[Trogo]

<?php
    class Scrivifile {
     public $file="iscritti.txt";
     public $nome;
     public $cognome;
     public $mail;
                          
// Bene il costruttore che slega i dati di inizializzazione dal contesto.
// Cosi e piu facile da riutilizzare anche se i dati non arrivano da un
// POST come era prima
public function __construct($in_nome, $in_cognome, $in_mail, $in_file) {   
                                                                                                        // Ok l'inizializzazione delle proprieta della classe
    $this->nome = $in_nome;
    $this->cognome =$in_cognome;
    $this->mail = $in_mail;
    $this->file = $in_file;
}

// Perche passare questi parametri al metodo, se nel costruttore
// sono state inizializzate come proprieta della classe?
// Eliminare i parametri
protected function Controllodati () {
// Usare le proprieta della classe al posto dei parametri
// $this->nome invece di $in_nome, ecc...
 // Questo per abbreviare le modifiche, ma funziona solo in lettura!!!
 $in_nome = $this->nome;
 $in_cognome = $this->cognome;
  
 if(!$in_nome || !$in_cognome )
 return FALSE;
 else return TRUE;
 }
// Vedi sopra
protected function Controllomail() {
 $in_mail = $this->mail;
 trim($in_mail);
  $num_at = count(explode( '@', $in_mail )) - 1;
    if(!$in_mail)    
        return FALSE;   
       
    else if($num_at != 1)    
        return FAlSE;   
       
    else if(strpos($in_mail,';') || strpos($in_mail,',') || strpos($in_mail,' '))    
        return FALSE;   
       
   else  if(!preg_match( '/^[\w\.\-]+@\w+[\w\.\-]*?\.\w{1,4}$/', $in_mail))   
        return FALSE;  
        else return TRUE;
        
    
}

// Vedi sopra sopra
protected function Controllofile () {
$in_file = $this->file;
if (is_writable($in_file) && feof($in_file))
       return TRUE;
       else return FALSE;
}

// Vedi sopra sopra sopra.
// Riflessione: se i metodi Controllodati, Controllomail e Controllofile
// vengono giustamente usati per validare l'input utente prima della
// scrittura, e quindi vengono richiamati dal seguente metodo, perche
// questi metodi devono essere pubblici?

public function Scrittura () {
$in_file = $this->file;
if($this->Controllodati() && $this->Controllomail() && Controllofile()) {
    fopen($in_file,a);
    $dascrivere="$in_nome | $in_cognome |  $in_mail/r/n";
    fwrite($in_file,$dascrivere);
    fclose($in_file);
    }
    else echo "I dati da te immessi sono errati";

}


}

Devi lasciare public il costruttore della classe e il metodo Scrittura, che devono poter essere richiamati esternamente da script1.php

In script1.php hai bisogno solo di questo:

<?php
include ("classe1.inc");
$scrivifile = new Scrivifile($nome,$cognome,$mail,$file);
$scrivifile->Scrittura($file);
?>

Il controllo dei dati viene fatto richiamando Scrittura($file)

 

[jappoz]

grazie mille, ma nello script1.php non devo prima assegnare a quelle variabili il valore dell' array di post?
apparte questo è tutto chiaro e limpido, ti ringrazio molto per l' aiuto (so che talvolta è irritante provare a far capire le cose a ua testa dura come me)
appena provo lo script vedrò se va ( ora è molto più probabile) o se c'è qualche errore

grazie

 

[jappoz]

ok, pare che lo script sia apposto, se inserisco qualcosa di errato restituisce il mio messaggio di errore, ma il problema è che lo fa anche se immetto tutto giusto!
ho cercato l' errore ma non capisco dove possa essere, non lo trovo.
probabilmente si trova in qualche metodo della classe, ma a me sembra tutto apposto...

p.s. ho strutturato la pag script1.php così

<?php
include ("classe1.inc");
$nome=$_POST['nome'];
$cognome=$_POST['cognome'];
$mail=$_POST['mail'];
$scrivifile = new Scrivifile($nome,$cognome,$mail,$file);
$scrivifile->Scrittura($file);
?>

é forse sbagliato?

 

[Trogo]

Giusto inizializzare le variabili dall'array POST, il metodo Scrittura va richiamato senza argomenti (errore mio nel post prima) e il metodo Scrittura nella classe è errato:

if($this->Controllodati() && $this->Controllomail() && Controllofile())

diventa così:

if($this->Controllodati() && $this->Controllomail() && $this->Controllofile())

Errori nella gestione file:

   $fp = fopen($in_file,"a");
    $dascrivere="$this->nome | $this->cognome |  $this->mail\r\n";
    fwrite($fp,$dascrivere);
    fclose($fp);

In più, il controllo se il file è scrivibile... mi raccomando, il file deve esistere!!!

 

[jappoz]

ho modificato quello che mi hai detto (ho trovato giuste le modifiche), ma lo script restituisce comunque errore.
non ho ulteriormente implementato per adesso il metodo scrittura (controllando se il file esiste) per non entrare in confusione, quindi deduco che ci sia un errore in uno dei metodi...
per facilitare le anime pie che mi vogliono aiutare riposto tutto il codice modificato come detto prima
classe1.inc

<?php
	class Scrivifile {
	public $file="iscritti.txt";
	public $nome;
	public $cognome;
	public $mail;
	                      
public function __construct($in_nome, $in_cognome, $in_mail, $in_file) {   
                                                                                                      
	$this->nome= $in_nome;
	$this->cognome=$in_cognome;
	$this->mail=$in_mail;
	$this->file=$in_file;
}


protected function Controllodati () {
 $in_nome = $this->nome;
 $in_cognome = $this->cognome;
 if(!$in_nome || !$in_cognome )
 return FALSE;
 else return TRUE;
 }
protected function Controllomail() {
 $in_mail = $this->mail;
 trim($in_mail);
  $num_at = count(explode( '@', $in_mail )) - 1;
    if(!$in_mail)    
        return FALSE;   
       
    else if($num_at != 1)    
        return FAlSE;   
       
    else if(strpos($in_mail,';') || strpos($in_mail,',') || strpos($in_mail,' '))    
        return FALSE;   
       
   else  if(!preg_match( '/^[\w\.\-]+@\w+[\w\.\-]*?\.\w{1,4}$/', $in_mail))   
        return FALSE;  
        
        else return TRUE;
       
        
	
}
protected function Controllofile () {
$in_file = $this->file;
if (is_writable($in_file) && feof($in_file))
       return TRUE;
       else return FALSE;
}
public function Scrittura () {
$in_file = $this->file;
if($this->Controllodati() && $this->Controllomail() && $this->Controllofile()) {
	$fp=fopen($in_file,"a");
	$dascrivere="$this->nome | $this->cognome |  $this->mail/r/n";
	fwrite($fp,$dascrivere);
	fclose($fp);
	}
    else echo "I dati da te immessi sono errati";

}


}

?>

script1.php

<?php
include ("classe1.inc");
$nome=$_POST['nome'];
$cognome=$_POST['cognome'];
$mail=$_POST['mail'];
$scrivifile = new Scrivifile($nome,$cognome,$mail,$file);
$scrivifile->Scrittura();
?>

pagina1.htm

<html><head><title>Form </title></head>
<body>


<form action="script1.php" method="POST">
<input type="text" name="nome" /> Immetti il nome <br>
<input type="text" name="cognome" /> Immetti il cognome <br>
<input type="text" name="mail" /> Metti qui la tua mail <br>
<input type="submit" value="invia" />

</form>
</body></html>

grazie mille dell' aiuto che mi state dando, mi state aiutando moltissimo a imparare

 

[Trogo]

Nella funzione Controllofile(), fai questa verifica feof($in_file) che ritorna sicuramente false perchè è errato l'utilizzo della funzione foef passando come argomento il path ad un file. Il giusto argomento è l'handle ad un file (in pratica quello che ritorna da una fopen)

In oltre, perchè vuoi fare questo controllo? Non ti fidi della fopen in append?

 

[jappoz]

beh, ora che mi ci fai pensare è inutile se lo apro in append, quindi lo toglierò...
grazie mille dei tuoi suggerimenti
ciao e a presto!