Problema con istanziazione classe

[fermat]

ho scoperto che così funziona:

if (!$objConn->verificaSessione()) {
    echo '<script type="text/javascript">document.location.href="index.php";</script>';
} 

per quanto riguarda Singleton VS NON Singleton nn saprei dirvi.
in java lo uso per la classe che gestisce i db perchè a rigor di logica mi pare inutile istanziare ogni volta una classe usata praticamente ovunque.

[_debo]

Come ho detto il singleton ha i suoi campi di applicazione ma va usato con parsimonia.

Per quanto riguarda il codice io li darei fuoco a quella cosa li

Prova cosi:

if (!$objConn->verificaSessione()) { 
    header('Location: index.php'); 
} 

[fermat]

Originariamente inviato da _debo
Come ho detto il singleton ha i suoi campi di applicazione ma va usato con parsimonia.

Per quanto riguarda il codice io li darei fuoco a quella cosa li

Prova cosi:

if (!$objConn->verificaSessione()) { 
    header('Location: index.php'); 
} 

concordo con te, ma anche così nn funziona.
e nn mi da nessun errore.

[_debo]

Non ci credo nemmeno se lo vedo

[_debo]

Aspetta un momento, non è che generi dell'output html prima di usare header?

[fermat]

si ora che mi ci fai pensare.
perchè in ogni pagina ci sta la testata HTML che è comune per tutti.

[Virus_101]

Originariamente inviato da Al_katraz984
Ciao Viruz permettimi di dissentire....

Un pattern singleton PHP su una classe database è abbastanza comune..

l'errore che c'era sul primo post è:

class Login {

    private $user;
    private $pwd;
    // non posso istanziare una classe sulla definizione di una proprietà !!!!!
    private $conn = Connessione::getInstance(); // ERRORE 

da risolvere cosi:

class Login {

    private $user;
    private $pwd;
    private $conn;

    function __construct($user, $pwd) {
        $this->user = $user;
        $this->pwd = $pwd;
        $this->conn = Connessione::getInstance();
    }
 } 

come disse Satifal... che poi sarebbe $this->db e non $this->conn ma comunque come dici tu:


se ho una e una sola unica istanza del database è meglio no? a quello serve il singleton..

NOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOOO

Si sintatticcamente e' corretto e funzionante a runtime(solo grazie alla natura del linguaggio).
MA la classe connessione::getInstance a mio avviso e' una cosa OSCENA !

la classe connessione se e' statica e' statica punto e fine connessione::connect();

Altrimenti fai una classe $connessione = new connessione() ;

Mescolare ste 2 cose e' 1 cosa oscena!
Personalmente e' una cosa che odio le classi statiche vanno usate correttamente ossia come set di funzioni. Pensa alle classi matematiche di java. Non e' che fai
Math m = Math.getInstance() ;
m.abs( numero ) ;
NO NON LO FAI!!!!

Il fatto che php ti consenta di farlo non e' una scusa per fare acrocchi del tipo connession::getInstance();
Queste classi vanno usate opportunamente per il fatocrying o per definire funzioni in un contesto completamente OOP . Tutto il resto sono appunto accrocchio obrobiosi.

Poi fate come volete e alla fine se funziona funziona ..... nulla da dire ma PERSONALMENTE odio questi approcci ibiridi.

[_debo]

Virus_101 non è una cosa oscena si chiama singleton pattern ed ha i suoi campi di utilizzo.
Bisogna solo usarla nei contesti adeguati e sapere perché la si sta usando.

[_debo]

Originariamente inviato da fermat
si ora che mi ci fai pensare.
perchè in ogni pagina ci sta la testata HTML che è comune per tutti.

Ecco spiegato il problema allora, non puoi inviare gli header due volte. Questo però mi fa comunque strano perché dovrebbe darti errore.

[fermat]

eh si lo so.
infatti pare strano anche a me.