[PHP]konstruktor, Error: Call to a member function get() on a non-object Opcje

[twojastara]

uczę się. Dlaczego konstruktor w wersji drugiej wywala komunikat Error: Call to a member function get() on a non-object
1.

[PHP] pobierz, plaintext 
namespace AppBundle\Utils;
 
 
class Cart {
 
    public $request;
    public $session;
    public $cart;
 
    public function __construct(Request $request)
    {
        $this->request = $request;
        $this->session = $this->request->getSession();
        $this->cart = $this->session->get('cart');
    }
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$cart_obiekt = new Cart($request);
[PHP] pobierz, plaintext 

2.

[PHP] pobierz, plaintext 
namespace AppBundle\Utils;
use Symfony\Component\HttpFoundation\Request;
 
class Cart {
 
    public $request;
    public $session;
    public $cart;
 
    public function __construct()
    {
        $this->request = new Request();
        $this->session = $this->request->getSession();
        $this->cart = $this->session->get('cart');
    }
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$cart_obiekt = new Cart();
[PHP] pobierz, plaintext 

Ten post edytował twojastara 16.03.2015, 21:14:18

[Mariner]

Ponieważ $this->request->getSession() nie zwraca obiektu. Sprawdź co zwraca. Jest to związane ze sposobem w jaki przypisywany jest obiekt $request. W pierwszej wersji z zewnątrz, a więc możemy się domyślać że jest zainicjowany i posiada już przypisany obiekt session. W drugim przypadku obiekt Request tworzony jest w kontrolerze a więc nie posiada danych żądania ani session.

Ten post edytował Mariner 16.03.2015, 20:11:30

[twojastara]

Ponieważ $this->request->getSession() nie zwraca obiektu.

a dlaczego ma znaczenie co zwraca?

Dla ścisłości
Error: Call to a member function get() on a non-object
wywala dla lini nr 14.

[PHP] pobierz, plaintext 
$this->cart = $this->session->get('cart');
[PHP] pobierz, plaintext 

a w ogóle jak sensownie przetłumaczyć ten komunikat na polski?

Ten post edytował twojastara 16.03.2015, 23:01:32

[Aqu]

Odwołujesz się tutaj do metody get, więc $this->session musi być obiektem, a $this->request->getSession(); widocznie Ci tego nie zwraca.

[Damonsson]

Mniej więcej tłumaczenie tak czytaj: Odwołujesz się do metody get(), w czymś ($this->session) co nie jest obiektem(, więc nie może zawierać żadnych metod).

W przypadku 1. przekazujesz zawartość obiektu $request do konstruktora. W przypadku 2. tworzysz nowy obiekt $request, który nie ma pojęcia o istnieniu sesji.

Ten post edytował Damonsson 16.03.2015, 23:20:30

[twojastara]

acha,

no tak, Mariner w pierwszej odpowiedzi pisał coś podobnego, (ale zamiast 'w konstruktorze' napisał 'w kontrolerze').

Rozumiem już komunikat i dlaczego się pojawia, ale nie bardzo rozumiem dlaczego 'nie ma pojecia o istnieniu sesji', no chyba na tym polega sesja, że jest widoczna wszędzie.

faktycznie $cart_obiekt = new Cart(); print_r($cart_obiekt->session) nic nie zwraca przy drugiej wersji.

"W przypadku 2. tworzysz nowy obiekt $request, który nie ma pojęcia o istnieniu sesji. "

nie pojmuję dlaczego ten obiekt nie widzi sesji.

Ten post edytował twojastara 16.03.2015, 23:41:56

[Damonsson]

Bo tworzysz nowy obiekt new Request() w którym normalnie jest zawsze trzymana sesja. W którym nic nie ma, sesja jest równa null. W Symfony nie masz dostępnej superglobalnej zmiennej $_SESSION dostępnej z każdego miejsca, tylko odwołujesz się bodajże zawsze do Request().

[Mariner]

a dlaczego ma znaczenie co zwraca?

Dla ścisłości

Kod

Error: Call to a member function get() on a non-object

wywala dla lini nr 14.

Ponieważ w następnej linii, właśnie nr. 14, używasz składowej session tak jak obiektu, wywołując na niej metodę get('cart'). Jak zapewne wiesz, nie można wywołać metody na zmiennej która nie przechowuje w sobie obiektu a taka próba kończy się błędem.

Powinieneś wstrzyknąć obiekt klasy Request już gotowy z zewnątrz, dokładnie tak jak w pierwszym przypadku a nie tworzyć go od nowa w konstruktorze ponieważ wtedy będzie pusty i zwracał będzie puste wartości. W drugim przypadku, sposób pozyskiwania obiektu klasy Request sprawia że jest on pusty. Nie możesz oczekiwać że już samo utworzenie obiektu Request sprawi że będzie miał on w sobie dane które może Ci zwrócić.

W systemie Symfony, obiekt Request możesz pozyskać z kontenera $request = $container->get('request'). Jeżeli masz problem z dostępem do obiektu Request w klasie Card, spraw aby obiekty klasy Cart tworzone były jako usługi (services). Przykładowy plik services.xml mógłby wyglądać tak:

CODE

<?xml version="1.0" ?>

<container xmlns="http://symfony.com/schema/dic/services"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://symfony.com/schema/dic/services http://symfony.com/schema/dic/services/ser...0.xsd">

<services>

<service id="app_utils.card.service" class="\AppBundle\Util\Card">
<argument type="service" id="request"/>
</service>

</services>
</container>

Dzięki temu, w kontrolerach, obiekty klasy Cart możemy pozyskiwać tak $card = $this->get('app_utils.card.service') o ile klasę Card zadeklarujesz jak w pierwszym przykładzie.
A za powoływanie obiektów klasy Card i wstrzykiwanie im obiektu Request martwił będzie się kontener zależności Symfony.

acha,

no tak, Mariner w pierwszej odpowiedzi pisał coś podobnego, (ale zamiast 'w konstruktorze' napisał 'w kontrolerze').

Fakt. Mój błąd, chodziło oczywiście o konstruktor a nie kontroler.

[Damonsson]

Na chłopski rozum:

Masz pudełko w którym trzymasz batonika. Zamiast wrzucić przez okno to pudełko z batonikiem. Ty kupujesz nowe puste pudełko, wrzucasz je przez okno i dziwisz się, że nie ma w nim batonika.

Ten post edytował Damonsson 16.03.2015, 23:52:37

[twojastara]

dzięki Wam obu.

Tylko, że ja wcale nie uważam by to pudełko w pierwszej wersji miało w środku jakiegoś batona.

W pierwszej wersji jakos specjalnie nie staram się by inicjować obiekt Cart z parametrem ... z batonem.

(Za 2 minuty edytuję ten post ze szczegółami)

bo w pierwszej wersji inicjalizacja obiektu Cart wygląda tak:

w kontrolerze w akcji

[PHP] pobierz, plaintext 
 
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\Request;
use AppBundle\Utils\Cart;
 
public function test1Action(Request $request)
    {
        $cart_obiekt = new Cart($request);
 
 
        return array();          
    }
[PHP] pobierz, plaintext 

I tę akcję inicjuję jako pierwszą, tj. obiekt request jest tutaj pierwszy raz powoływany.


Wychodzi na to, że nie rozumiem podstaw zwiazanych z inicjowaniem zmiennych/obiektów. Sądziłem, że taki zapis

[PHP] pobierz, plaintext 
 function test1Action(Request $request)
[PHP] pobierz, plaintext 

równy jest takiemu

[PHP] pobierz, plaintext 
 function test1Action(){$request= new Request();}
[PHP] pobierz, plaintext 

Ten post edytował twojastara 17.03.2015, 00:10:21

[Damonsson]

Symfony robi to za Ciebie. Sprawdź:

[PHP] pobierz, plaintext 
namespace AppBundle\Utils;
 
 
class Cart {
 
    public $request;
    public $session;
    public $cart;
 
    public function __construct(Request $request)
    {
        echo '<pre>';
        var_dump($request); 
        die;
        $this->request = $request;
        $this->session = $this->request->getSession();
        $this->cart = $this->session->get('cart');
    }
}
[PHP] pobierz, plaintext 

poiwnno być w środku session



Edit: Nie korzystam z Symfony, więc szczegółowo Ci nie powiem, ale prześledź sobie jakimś XDEBUG co się dzieje po kolei w kodzie i zobaczysz, że gdzieś tam na samym początku jest tworzony nowy obiekt Request() i później jest tylko PRZEKAZYWANY ciągle ten sam obiekt, wszędzie.

[PHP] pobierz, plaintext 
function test1Action(Request $request)
[PHP] pobierz, plaintext 

Tu przekazujesz zmienną $request, która musi być instancją obiektu(czy jakkolwiek to fachowo nazwać) Request().

[PHP] pobierz, plaintext 
function test1Action(){$request= new Request();}
[PHP] pobierz, plaintext 

Tutaj tworzysz nowy obiekt Request().

Ten post edytował Damonsson 17.03.2015, 00:15:03

[twojastara]

tak, to już przekonałes mnie wcześniej.

Muszę się zadowolić tłumaczeniem, że Symfony jakoś tam robi za mnie.


że powołowanie do życia $request = new Request(); ma inną zawartość w kontrolerze i innej/zwykłej klasie.
----

edit

Takie wytłumaczenie mi wystraczy. Dzięki za poświęcony czas!

Ten post edytował twojastara 17.03.2015, 00:19:58

[Mariner]

Obiekty Request w kontrolerach Symfony2 są tak często używane że zdecydowano się uprościć programistom sprawę z ich pozyskiwaniem w kontrolerach. Dlatego jeżeli zadeklarujesz obiekt request w parametrach metody to otrzymasz gotowy obiekt wypełniony danymi. Nie trzeba chyba wyjaśniać że normalnie tak to nie działa.

To nie jest tak że w Symfony konstrukcja new Request() nie jest tożsama jak w php. Obiekty są tworzone tak samo, z tym że obiekt Request w Symfony2, zanim go otrzymasz, w metodzie kontrolera wypełniany jest przez framework danymi z żądania. Tu nie ma żadnej "magii".

[twojastara]

[PHP] pobierz, plaintext 
function test1Action(Request $request)
[PHP] pobierz, plaintext 

Tu przekazujesz zmienną $request, która musi być instancją obiektu(czy jakkolwiek to fachowo nazwać) Request().

Recz w tym, że to pierwsza i jedyna funkcja w tej przykładowej aplikacji, że wcześniej zmienna $request nie jest nigdzie manipulowana, więc to tu obiekt $request jest powoływany do ż.

Tu nie ma żadnej "magii".

Dzięki. Wszystko już wystarczająco jasne.

[Mariner]

Recz w tym, że to pierwsza i jedyna funkcja w tej przykładowej aplikacji, że wcześniej zmienna $request nie jest nigdzie manipulowana, więc to tu obiekt $request jest powoływany do ż.

Framework Symfony2 używa Twojego kontrolera w taki mniej więcej sposób:

CODE

$request = new Request();

$request->bind($_POST, $_GET);

$controller = new YourController();

$response = $controller->test1Action($request);

Tylko tak teraz patrzę w Twój kod ale nie widzę tam kontrolera tylko funkcję. Czy u Ciebie kontroler jest obiektem klasy Controller?

Ten post edytował Mariner 17.03.2015, 00:36:17

[twojastara]

Tylko tak teraz patrzę w Twój kod ale nie widzę tam kontrolera tylko funkcję. Czy u Ciebie kontroler jest obiektem klasy Controller?

tak, ta funkcja/akcja jest wewnątrz:

[PHP] pobierz, plaintext 
class CartController extends Controller
{
[PHP] pobierz, plaintext 

Ten post edytował twojastara 17.03.2015, 00:45:01

[Mariner]

Wybacz to pytanie ale lepiej się upewnić. Jeżeli coś jeszcze mogę doradzić to tak jak polecał kolega wcześniej, żeby zrozumieć lepiej działanie symfony, dobrze jest choć raz prześledzić pracę całego programu. Jeżeli nie masz do zainstaluj xdebug oraz ide którym będziesz mógł debugować skrypty i na bieżąco śledzić czy nawet zatrzymywać pracę programu w dowolnym momencie. Być może na początku wydawać by się to mogło trudną sprawą ale wiedz że wcześniej czy później i tak tego nie unikniesz. Im szybciej się na to zdecydujesz tym lepiej dla Ciebie. Ze swojej strony mogę polecić eclipse z php dev doinstaluj xdebuga, czego nie będziesz wiedział zapytaj na forum. Naprawdę warto. Polecam i pozdrawiam.

Ten post edytował Mariner 17.03.2015, 00:57:30

[twojastara]

tak, już przy wcześniejszym poście dot. xdebug zaznaczyłem sobie, by jutro szczegółowo sie temu przyglądnąć.

Korzystam z netbeans, więc będę szukał coś pod niego. (Jeszcze raz dzięki, kupe wartościowej dla mnie treści w tym temacie)

[com]

Poczytaj o depedency injection a zrozumiesz czemu to tak działa (IMG:style_emoticons/default/smile.gif)