Evitare cicli infiniti con l'incapsulamento

[iJoker]

Se ad esempio devo istanziare Titanic e Leonardo Di Caprio, e gli attributi $protagonist e $best_film, sono rispettivamente delle istanze di Actor e Film, in fase di creazione degli oggetti mi trovo in un loop infinito.

C'è un pattern per evitare queste situazioni?

class Film
{
  private $title;
  private $year;
  private $protagonist;
}

class Actor
{
  private $name;
  private $surname;
  private $best_film;
} 

[satifal]

Di cosa stiamo parlando? Quale loop

[iJoker]

per motivi di sintesi avevo tralasciato le funzioni di creazione delle istanze, adesso le ho messe.

class Film
{
  private $title;
  private $year;
  private $protagonist; // ISTANZA DELLA CLASSE ACTOR

  public static function createById($id)
  {
    $film = new Film();
    $result = mysql_query("SELECT * FROM film WHERE id=".$id;);
    $row = mysql_fetch_row($result)
    $film->title = $row['title'];
    $film->year = $row['year'];
    $this->protagonist = Actor::createById($row['id_protagonist']);
    return $film;
  }
}

class Actor
{
  private $name;
  private $surname;
  private $best_film; // ISTANZA DELLA CLASSE FILM

  public static function createById($id)
  {
    $actor = new Actor();
    $result = mysql_query("SELECT * FROM actor WHERE id=".$id;);
    $row = mysql_fetch_row($result)
    $actor->name = $row['name'];
    $actor->surname = $row['surname'];
    $actor->best_film = Film::createById($row['id_best_film']);
    return $actor;
  }
} 

[Santino83_02]

Se fai in modo che Actor e Film si ricordino le istanze che creano, allora puoi interrompere la ricorsione, altrimenti non puoi o altrimenti togli il load dell'utente da Film

[iJoker]

Se io invece nel film salvo solamente l'id dell'attore può essere una buona soluzione?
In questo modo però non ci sarebbero oggetti incapsulati in altri oggetti.

[Santino83_02]

Se io invece nel film salvo solamente l'id dell'attore può essere una buona soluzione?
In questo modo però non ci sarebbero oggetti incapsulati in altri oggetti.

Potresti usare un Laxy Load, ovvero carichi l'utente in Film solo quando in film richiami il metodo getActor, viceversa in Actor. Cmq te lo dico pure a te: se stai facendo questa cosa per studio, va bene (anche se ad occhio dovresti studiare un bel pò di teoria), se stai facendo un qualcosa di lavoro evita e usa una framework già pronto per gestire la persistenza (Doctrine? chi ha detto Doctrine? io dico Doctrine ). Sopratutto quei metodi statici, per me, sono una pugnalata al cuore con tanto di ravanata di coltello, usa classi repository esterne per istanziare oggetti Film e Actor, che risulteranno semplici oggetti con solo getter e setter

[fermat]

ciao!
mi intrometto un attimo perchè vedo che spesso citi Doctrine.
nello specifico parli di Database Abstraction Layer o Object Relational Mapper?

[ciro78]

la seconda. sia doctrine che propel sono orm

[Santino83_02]

ciao!
mi intrometto un attimo perchè vedo che spesso citi Doctrine.
nello specifico parli di Database Abstraction Layer o Object Relational Mapper?

Doctrine (propel non lo conosco ma penso che sia lo stesso) fornisce api per entrambi i livelli, quindi dipende che ti serve. In alcuni progetti uso direttamente l'orm, in qualcosa di piu semplice mi è bastato solo il dbal. nel caso specifico jocker sta facendo un orm

[fermat]

Doctrine (propel non lo conosco ma penso che sia lo stesso) fornisce api per entrambi i livelli, quindi dipende che ti serve. In alcuni progetti uso direttamente l'orm, in qualcosa di piu semplice mi è bastato solo il dbal. nel caso specifico jocker sta facendo un orm

ok grazie per il chiarimento!