Mam kontener do którego ładuję 2 moduły, z tych modułów chce mieć dostęp do konenera.
Macie na to jakiś lepszy sposób niż ten który stosuje:

[PHP] pobierz, plaintext 
class container{
 private $lang;
 public $user;//moduł
 public $board;//moduł
 function __construct(){
  $this->user = new user($this);//w tym miejscu przekazuje obiekt do modułu, da się to zautomatyzować?
  $this->board = new board($this);
 }
 private function getLanguage(){
  return $lang;
 }
}
class user{
 private $parent;
 function __construct($parent){
  $this->parent = $parent;
 }
 function welcome(){
  echo $this->parent->getLanguage();
 }
}
class board{
 private $parent;
 function __construct($parent){
  $this->parent = $parent;
 }
 function welcomeOnBoard(){
  echo $this->parent->getLanguage();
 }
}
[PHP] pobierz, plaintext 

Jeszcze uprzedzę: dziedziczenie kontenera dla każdego moduły z będzie nieintuicyjne.

1. Utwórz sobie interfejs na podstawie, którego będziesz mógł sprawdzić czy dany obiekt (User, Board) powinien mieć wstrzyknięty obiekt kontenera:

[PHP] pobierz, plaintext 
interface ContainerAwareInterface {
    public function setContainer(Container $container); // oczywiście lepiej jest stosować type hinting po interfejsie, nie klasie
    public function getContainer();
}
[PHP] pobierz, plaintext 

2. Zaimplementuj ten interfejs dla obu klas.
3. Teraz wstrzykiwanie zależności możesz sobie zautomatyzować:

[PHP] pobierz, plaintext 
foreach ($objects as $object) {
    if ($object instanceof ContainerAwareInterface) {
        $object->setContainer($this);
    }
}
[PHP] pobierz, plaintext 

Pozostaje jednak problem utworzenia kolekcji obiektów dla tej pętli z osobnych argumentów. Możesz:
1. Zamiast wstrzykiwać każdą zależność z osobna (co jest bardzo dobrą praktyką) wstrzyknąć jakąś tablicę tych zależności (co jest już nie tak dobrą praktyką).
2. Utworzyć sobie jakąś pomocniczą metodę robiącą to:

[PHP] pobierz, plaintext 
class Container {
    ... __construct($a, $b, $c) {
      $this->injectMyself($a, $b, $c);
 
      $this-> .. = $a;
      ...
   }
 
   private function injectMyself() {
       foreach (func_get_args() as $object) {
          ....
       }
   }
}
[PHP] pobierz, plaintext 

Ale to rozwiązanie też nie jest zbyt ładne. IMO najlepiej jednak ręcznie to zrobić - nie ma przy tym aż tak dużo pracy, a kod pozostaje bez "hacków".

Wielkie dzięki. Po konsultacjach z znajomym otrzymałem taką samą odpowiedź jak Twoja, że tak jak teraz jest dobrze.

Dodałem tylko żeby zmienną przy konstruktorach modułów przekazywał przez referencje:

[PHP] pobierz, plaintext 
class module{
 private parent;
 public function __construct(container &$parent){
  $this->$parent;
 }
 ...
}
[PHP] pobierz, plaintext 

Przekazywanie obiektów przez referencje jest bez sensu, powinno nawet wyrzucić jakieś błędy o ile dobrze pamiętam. Obiekty są z automatu przekazywane przez coś w rodzaju wskaźników, więcej oczywiście w Google: PHP object pass by reference

Nie wiem czy jest sens pisać tak krótki post w którym muszę napisać: masz racje.

http://stackoverflow.com/questions/9331519...9332219#9332219

Ten & nie ma tam sensu.