[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.

[eminiasty]

php obiektowe miało być proste

chodziło o to, że łatwiejsze od strukturalnego

[PHP] pobierz, plaintext 
 __construct? Wiesz co to extends? 
[PHP] pobierz, plaintext 

dzieki extends dziedziczymy klase nadrzędna w tym wypadku jest to CI_Controller, który jest klasa bazową frameworka i (ponoć) nie powinienem go ruszać
jeśli chodzi o konstruktor jest to specjalna metoda ktora ma taka sama nazwe jak klasa, jest nazywany w php obiekt "magiczna metoda" i wywolywany jako __construct(), np. można przekazywać przez nia różne elementy

Wiec, wracajac:

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

chyba nie moge edytować domyslnej klasy, a jednoczesnie klasa moze dziedziczyc tylko z jednej klsay na raz, co staje sie problemowe

[Evinek]

chodziło o to, że łatwiejsze od strukturalnego

Nie, nie jest łatwiejsze. Można zrobić aplikacje, która będzie łatwiejsza w utrzymaniu, ale samo w sobie programowanie obiektowe jest trudniejsze.

Masz CI_Controller, potem MY_Controller, a potem już konkretny kontroler, który już robi swoje.

[PHP] pobierz, plaintext 
<?php
 
class A {
	public $a = 1;	
}
 
class B extends A {
	public $b = 2;	
}
 
class C extends B {
	public $c = 3;	
}
 
$c = new C;
 
echo $c->a; // 1
echo $c->b; // 2
echo $c->c; // 3
[PHP] pobierz, plaintext 

Ten post edytował Evinek 8.07.2017, 21:21:35

[markuz]

__construct() ma taką cechę (jedną z kilku), że jego zawartość jest wykonywana za każdym razem gdy tworzysz instancje danej klasy. Czyli to jest dobre miejsce na kod który podałeś wyżej.

Nie możesz edytować CI_Controller, ale możesz stworzyć nową klasę MY_Controller która dziedziczy po CI_Controller, a następnie każdy inny kontroler np. News dziedziczy po MY_Controller.

[Tomplus]

Pamiętaj, że tworzenie klas nie musi, a często nie powinno zawierać wszystkich funkcji jakie ma mieć twój skrypt.
Np. funkcje autoryzacji powinny być w innym obiekcie. Wtedy powtórzeń nie będzie.

A co do samego OOP - nikt nie powiedział że człowiek od razu nauczy się programowania obiektowego po przeczytaniu jednej książki, artykułu czy kursu PHP-OOP. Człowiek uczy się poprzez praktykę. Dzisiaj robisz spieszysz kod ale jutro będziesz się śmiał z tego że byłeś taki głupi że nie potrafiłeś tak prostych rzeczy

Nie poddawaj się!