[PHP]Dublowanie kodu w OPP Opcje

[eminiasty]

Bardzo proszę o pomoc bo juz nie wiem w czym rzecz. Czym więcej dopisuje funkcji do mojej strony muszę kopiować wcześniej napisane linijki kodu. Przecież php obiektowe miało być proste. Mialo sie tu raczej nic nie powtarzać. Nie wiem co robię źle i jak można uprościć napisany przeze mnie kod. O co chodzi, tak na prawde?
Napisałem cały moduł newsów (bardzo uproszczony na razie), aczkolwiek są już funkcje takie jak index, view, add, edit, del (czyli podstawa). Problemem jest to, że muszę pomiędzy kontrolerami i wszystkimi/większością funkcji (tam gdzie wywołuje widoki) kopiować ciągle ten sam kod! W moim przypadku:

[PHP] pobierz, plaintext 
 if(isset($this->session->userdata['logged_in'])) {
           $session_data =$this->session->userdata('logged_in');
           $data['session_id'] = $session_data['id'];
           $data['session_login'] = $session_data['login'];
           $data['admin'] = $this->model_login->checkAdmin();
       }  
      $data['setPageName'] = $this->model_site->setPageName();
      $data['returnSubpage'] = $this->model_panel->returnSubpage();
      @$data['breadcrumbs'] = $this->model_panel->breadcrumbs(); 
[PHP] pobierz, plaintext 

Odpowiada on za sesje, sprawdzenie czy jestes adminem, oraz inne funkcje z modeli.

PYTANIE DO WAS: Czy jest jakas możliwość by tego nie kopiwac a by nie rzucalo bledu o braku dostepi do zmiennej np $admin (juz w widoku)?
Próbowałem to wywoływać w konstruktorze jednak bez efektu ;/

Dorzucam całość jednego kontrolera:
http://wklejto.pl/288309

Dopiero się uczę obiektowości ...

Ten post edytował eminiasty 8.07.2017, 19:05:27

[markuz]

php obiektowe miało być proste

Ktoś Cię oszukał.

Mialo sie tu raczej nic nie powtarzać

Jak napiszesz bez powtórzeń to się powtarzać nie będzie.

Czym według Ciebie jest __construct? Wiesz co to extends? (php obiektowe nie może być proste bez znajomości podstaw programowania obiektowego).
Poczytaj o tym, wtedy:

[PHP] pobierz, plaintext 
if(isset($this->session->userdata['logged_in'])) {
           $session_data =$this->session->userdata('logged_in');
           $data['session_id'] = $session_data['id'];
           $data['session_login'] = $session_data['login'];
           $data['admin'] = $this->model_login->checkAdmin();
        }
[PHP] pobierz, plaintext 

zostanie tylko w 1 miejscu.

Mała podpowiedź:

Jeżeli w każdym kontolerze potrzebujesz danych o zalogowanym użytkowniku to te informacje mogą być pozyskiwane w nadrzędnej klasie np. MY_Controller - po której dziedziczyć będą wszystkie inne kontrolery Twojej aplikacji. Potem będzie jeszcze z problem przekazania tych danych do widoku, ale możesz utworzyć właściwość klasy MY_Controller o nazwie np. viewData do której będziesz przekazywał wszystkie zmienne które mają znaleźć się w widoku.