[inny][Laravel] namespace i użycie Opcje

[Randallmaster]

Cześć,

Może jest to moja nie wiedza odnośnie OOP, ale zależy mi też aby zrobić to poprawnie w Laraverze.

mam taki układ controllerów:

/controllers
---PagesController
---/Pages
-------DefaultController

plik DefaultController:

[PHP] pobierz, plaintext 
<?php
 
namespace App\Http\Controllers\Pages;
 
use Illuminate\Http\Request;
 
use App\Http\Requests;
use App\Http\Controllers\Controller;
use App\Http\Controllers\PagesController;
 
/**
 * Class DefaultController
 * @package App\Http\Controllers\Pages
 */
class DefaultController extends PagesController
{
    /**
     * Create default edit view
     * @return mixed
     */
    public function createDefaultView() // 
    {
        $viewPortlet = view('pages.edit');
        $settingsPortlet = [
            'setRedricted'     => '',
            'setClass'         => '',
            'setTitle'         => '',
            'setTitleHelper'   => '',
            'setActionButtons' => '',
        ];
        return $this->createPortlet($settingsPortlet, $viewPortlet);
    }
}
[PHP] pobierz, plaintext 

1. W jaki sposób w PagesController wywołać funkcję: createDefaultView?, wiem że jak dodam że jest to funkcja statyczna to mi zadziała: DefaultController::createDefaultView(), czy w przypadku Laravela powinno się robić to inaczej? Można jeszcze w inny sposób dostać się do tej funkcji nie używając static?

2. Jak funkcję createPortlet wywołać z Controllera głównego bo tam też ona się znajduje? //w controller.php jest jako public function createPortlet()

Z góry dziękuję za pomoc

/////////////
wszystko działa jeżeli funkcje pozamieniam na statyczne, ale czy to tak może być?

Ten post edytował Randallmaster 29.07.2015, 18:22:24

[Pyton_000]

1. Nie wywołasz metody dziecka z rodzica.
Albo musisz tworzyć instancje 'DefaultController' albo przekazać go jako dependency injection

2. Skoro dziedziczy po PagesController to zapewne tamten dziedziczy po Controller, więc tak jak masz.

Nie za bardzo rozumiem i widzę co chcesz osiągnąć, bo struktura katalogowo plikowa jest kiepska

[Randallmaster]

Pyton_000 dzięki za odpowiedź.

Są jakieś standardy które mówią jak prowadzić strukturę plików? Może mógłbyś coś polecić.

Co chcę osiągnąć.

'pages/edit' będzie dzieliła się na 5 tabsów, i te 5 tabsów chciałem rozbić na 5 różnych klas po 1 do każdego z taba. no i jedna główna classa czyli PagesController. Jak powinno wyglądać katalogowanie plików w takim przypadku?

[Pyton_000]

Nie ma standardów. Układa się katalogi raczej w logiczne całości żeby miało ręce i nogi.

Co masz na myśli 5 tabsów? Panadol? Apap? Czy może te od Goździkowej

Opisz co to ma robić, jakie są założenia to coś wymyślimy.

[Randallmaster]

Panadol i Apap nie może coś mocniejszego generalnie jak wpiszesz Tabs i wyświetlisz grafikę w Google to pokaże to co robię dokładnie "pages/edit" ma tabsy z 5 poziomami uprawnień czyli na jednej stronie bedzie 5 stron umiesCzonych w podstronach. Do każdej podstrony chce wykonać osobna klasę z controllerem

[Pyton_000]

Dobra ja już nie wnikam.

- createPortlet() wydziel do jakiejś innej klasy która będzie odpowiedzialna za tworzenie czegoś takiego z parametrów. Nie ma sensu pchanie wszystkiego do głównego kontrolera.
- DefaultController nie musi w tym momencie dziedziczyć już po PagesController, wystarczy po Controller
- Teraz w PagesController musisz odpalić sobie generowanie tych zakładek. Obiekty możesz przekazać jako parametry w konstruktorze klasy lub parametry metody:

[PHP] pobierz, plaintext 
<?php
 
/**
* 
*/
class PagesController extends Controller
{
 
	protected $controller1;
 
	protected $controller2;
 
	function __construct(Tabs1Controller $controller1, Tabs2Controller $controller2)
	{
 
		$this->controller1 = $controller1;
		$this->controller2 = $controller2;
	}
 
	// LUB
 
	public function generateTabs(Tabs1Controller $controller1, Tabs2Controller $controller2)
	{
		$tab1 = $controller1->createDefaultView();
	}
}
[PHP] pobierz, plaintext 

Oczywiście nie zapomnieć o namespace

[Randallmaster]

Popraw mnie jeżeli źle zrozumiałem i zrobiłem poniższy przykład

[PHP] pobierz, plaintext 
<?php
 
namespace App\Http\Controllers;
 
use App\Http\Controllers\Controller;
use App\Http\Controllers\Pages\DefaultController;
use App\Http\Controllers\Pages\PermissionController
 
class PagesController extends Controller
{
 
	protected $default;
 
	protected $permission;
 
	function __construct(DefaultController $default, PermissionController $permission)
	{
 
		$this->default = $default;
		$this->permission = $permission;
	}
 
	// CZY
 
	public function generateTabs()
	{
		$tab1 = DefaultController::createDefaultView();
	}
}
[PHP] pobierz, plaintext 

Jeżeli zrobiłem to poprawnie, odpowiedz mi co jest wydajniejsze. Pierwszy przykład z tworzeniem instancji w konstruktorze czy funkcja statyczna + abstract class? Wydaje mi się że raczej funkcja statyczyna + (zmiana klasy na abstrakcyjną), ponieważ znajdować się tam będą funkcję podstawowe dla PagesController?

Ten post edytował Randallmaster 30.07.2015, 07:36:47

[Pyton_000]

Wiesz, ja bym całkowicie olał te "podklasy" i to co one robią zostawił w głównym kontrolerze, a logikę zamieścił w jakimś repozytorium (Repositories).

Albo zamiast "tabów" oddielne kontrolery które będą odpalane samodzielnie. I wtedy odpalasz sobie np. PermissionsController który dziedziczy po PagesController (a ten po Controller) i tam sobie dłubiesz. Rzeczy w spólne będziesz miał w PagesController a indywidualne w konkretnej klasie.

[Randallmaster]

Dzięki wielkie za pomoc, ostatnie pytanie orientacyjne , Repozytorium tworzyć w strukturze APP, czy lepiej w Illuminate za pomocą provides Facade?

i tak na szybko:
po co tworzyć : protected $permission;
jak i tak odwołuję się $this->permission->createDefaultView();
/raczej jest to w tej sytuacji nie potrzebne?

Ten post edytował Randallmaster 30.07.2015, 08:33:37

[Pyton_000]

Żeby było ładnie. Pola się deklaruje (ułatwia to później czytanie)
Repozytorium ja trzymam w app/Repositories/

[Randallmaster]

Dzięki za pomoc

[nrm]

wstrzykiwanie kontrolerów do kontrolerów? To jakaś incepcja?

[Randallmaster]

tak na 4 poziomie jest już problem bo bączek nie chce upaść