Classi e parametri

[Katija]

Salve a tutti, stavo cercando di capire meglio le tecniche di programmazione OOP e dando un'occhiata a molti esempi online mi é venuto un dubbio che non so quanto sia pertinente.
Prendete queste due classi.
LA PRIMA

<?php

class Persona {
    
    private $nome = null;
    private $cognome = null;
    
    function __construct($nome, $cognome) {
        $this->nome = $nome;
        $this->cognome = $cognome;
    }    
    
    public function scriviNomeCompleto() {
        
        return $this->nome.' '.$this->cognome;
    }
    
}

$p = new Persona("Mario", "Bianchi");

echo $p->scriviNomeCompleto();
?>

LA SECONDA

class Persona {
    
    public $nome = null;
    public $cognome = null;
    
    public function scriviNomeCompleto() {
        
        return $this->nome.' '.$this->cognome;
    }
    
}

$p = new Persona;
$p->nome = "Mario";
$p->cognome = "Bianchi";

echo $p->scriviNomeCompleto();

?> 

Entrambe restituiscono lo stesso risultato, il mio dubbio é: che differenza sostanziale c'é? E' solo una differenza di stile di programmazione oppure una é preferibile all'altra?
Dal punto di vista dell'utilizzo delle risorse e della sicurezza, quale delle due é preferibile?

Grazie.

[mtx_maurizio]

Quelli che hai postato sono esempi di utilizzo di classi che ti mostrano come puoi ottenere lo stesso risultato in modi diversi. Nel primo caso utilizzando il costruttore, nel secondo valorizzano le proprietà.

Non esiste un modo migliore. Dipende dai casi e su classi del genere è difficile spiegarlo, ma quando ti troverai di fronte a progetti di una certa dimensione, è chiaro che scegliere una strada piuttosto che un'altra può fare differenza.

[luca200]

In generale, viene considerata buona pratica tenere private le variabili di classe, come nel primo esempio, e fornire dei metodi di accesso (i cosiddetti "getter" e "setter").
Il motivo di ciò è che in questo modo si mantiene il controllo su ciò che viene impostato nei campi.
Immagina di avere un campo numerico che, per motivi di logica della classe, deve avere un valore compreso fra 1 e 10. Se il campo è pubblico, chiunque usi quella classe può fare

$oggetto->campo = 11

stravolgendo la logica di base.
Se invece il campo è privato e hai un metodo setter di questo tipo

function setCampo($val) {
  $this->campo = $val;
  if ($this->campo < 1) $this->campo = 1;
  if ($this->campo > 10) $this->campo = 10;
} 

non avrai mai un valore incongruente in quel campo.
Questo è un esempio banale, potresti anche gestire un sistema di notifica di eventuali valori non validi passati, ad esempio lanciando eccezioni, o prevedere che un valore non valido lascia immutato il valore precedente, invece di impostare il valore limite come ho fatto io. Insomma puoi fare quello che ti pare


Il codice del primo esempio che hai postato si ispira a questa filosofia, anche se per impostare i valori utilizza il costruttore e non metodi setter. In questo caso la cosa può essere giustificata, ad esempio, considerando che il nome di una persona è immutabile. Ma in un'ottica più generica, quel codice ha il difetto di non consentire la modifica dei valori dopo la creazione dell'oggetto.

[Katija]

Perfetto, gentilissimi.
Sinceramente mi piace molto di piú l'idea di evitare dove possibile l'invio di parametri tipo stringhe o integer, mi sembra piú flessibile, il dubbio era che ci fossero delle 'controindicazioni'.
Per fare un esempi concreto, mettiamo che voglia inserire i dati dell'oggetto in un db, qualcosa di simile

$p = new Persona;
$p->nome = 'nome';
$p->cognome = 'cognome';
$p->insertToDb(); 

Mi sembra piú flessibile che mandare parametri tipo $p->insertTodb($nome,$cognome).
Ovviamente é giusto il concetto della validazione dei dati, per cui invece di proprietá pubbliche si potrebbero creare metodi pubblici che settano proprietá private.
Altro dubbio, usare i metodo magici __set e __get come pubblici con proprietá private, non é la stessa cosa che avere proprietá pubbliche, con l'unica differenza che i controlli sui dati li fai solo dentro il metodo __set?

[luca200]

Originariamente inviato da Katija
Altro dubbio, usare i metodo magici __set e __get come pubblici con proprietá private, non é la stessa cosa che avere proprietá pubbliche, con l'unica differenza che i controlli sui dati li fai solo dentro il metodo __set?

Non è una differenza da poco... nell'altro caso dove li fai?

[Katija]

Originariamente inviato da luca200
Non è una differenza da poco... nell'altro caso dove li fai?

Forse mi sono spiegata male.
Facciamo un esempio pratico.

Ho due variabili private var1, var1 e due metodo pubblici setVar1 e setVar2.
Ho la libertá per ciascuna variabile di fari gli opportuni controlli.
Con __set invece dovrei anche intercettare che proprietá sto controllando.
Qualcosa tipo switch ($variabile) case "var1": etc. etc.
Mi sbaglio? Mi sfugge qualcosa?

[luca200]

No, non sbagli.

Però la tua frase precedente mi rimane poco chiara