[SF][SF2][Symfony2]Nie pobiera danych z bazy danych Opcje

[Crash89]

Mam taki problem, otóż wszystko działa prawie prawidłowo. Chcę pobrać z bazy informację czy użytkownik juz zagłosował na dany product i na jeden produkt są zwracane dane ale na inne nie.
Mimo tego że gdy w phpmyadmin ręcznie wpisując dane wszystko się zgadza.

To jest kod który to obsługuje:

[PHP] pobierz, plaintext 
 public function checkIfUserVote($user, $product)
    {
        if($user === 'anon.')
        {
            return true;
        }
 
        $rate = $this->em->getRepository('AppBundle:Rate')->getRateProductUser($user->getId(),$product->getId());
 
 
 
        if(!empty($rate))
        {
            return false;
        }else{
            return true;
        }
    }
[PHP] pobierz, plaintext 

i w repozytorium:

[PHP] pobierz, plaintext 
 
 public function getRateProductUser($productId, $userId)
    {
        return $this->createQueryBuilder('r')
                ->where('r.product = :product AND r.user = :user')
                ->setParameter('product',$productId)
                ->setParameter('user',$userId)
                ->getQuery()
                ->getResult();
    }
[PHP] pobierz, plaintext 

Ten post edytował Crash89 27.11.2016, 11:40:41

[lukaskolista]

po getQuery() zamiast getResult() daj getSql() i zobacz, jakie zapytanie jest wykonywane

[Crash89]

po getQuery() zamiast getResult() daj getSql() i zobacz, jakie zapytanie jest wykonywane

Takie jest wykonywane, czyli dobre:

[SQL] pobierz, plaintext 
string 'SELECT r0_.id AS id_0, r0_.rate AS rate_1, r0_.deletedAt AS deletedAt_2, r0_.created AS created_3, r0_.updated AS updated_4, r0_.contentChanged AS contentChanged_5, r0_.product_id AS product_id_6, r0_.user_id AS user_id_7 FROM rate r0_ WHERE r0_.product_id = ? AND r0_.user_id = ?'
[SQL] pobierz, plaintext 

Dla produktu który znajduje i nie, tworzy takie same zapytania.
Użytkownika narazie używam jednego, więc tutaj to nie będzie problem.

[lukaskolista]

Skoro zapytanie jest dobrze, to znaczy, że problem tkwi gdzieś indziej.

[Crash89]

Skoro zapytanie jest dobrze, to znaczy, że problem tkwi gdzieś indziej.

Już wiem gdzie tkwił problem, odwrotnie wprowadziłem zmienne do zapytania...

[lukaskolista]

Do głowy przychodzą mi 2 sposoby zapobiegania takim pomyłkom:
1. przekazuj całe obiekty zamiast ich id
2. niech id będą "value objects" - nie wiem jak to po polsku się nazywa. Np. UserId i ProductId

Czemu dobrze zastosować jeden z nich? Niby błąd programisty i kod jest ok, ale gdyby taki błąd był w dużej i skomplikowanej aplikacji (która nie powinna powstać, bo powinna być rozbita na mniejsze usługi, ale załóżmy, że istnieje), to jego znalezienie mogło by być problemem.

Dobrze jest po prostu zapobiegać takim sytuacjom.

Ten post edytował lukaskolista 27.11.2016, 13:23:36

[Pilsener]

po getQuery() zamiast getResult() daj getSql() i zobacz, jakie zapytanie jest wykonywane

- nie ma takiej potrzeby, każde zapytanie jest wyświetlone w profilerze.

[PHP] pobierz, plaintext 
        if($user === 'anon.')
        {
            return true;
        }
[PHP] pobierz, plaintext 

1. User to jest nazwa użytkownika, login, obiekt "user", status jakiś? Nazwa zmiennej powinna odzwierciedlać jej zawartość.
2. Metoda nazywa się "checkIfUserVote" a sprawdza czy user to string "anon.", za chwilę założysz temat "dlaczego użytkownik ma zaznaczone, że głosował jak nie oddał głosu"

[PHP] pobierz, plaintext 
$this->em->getRepository('AppBundle:Rate')
[PHP] pobierz, plaintext 

- nie powinno się tworzyć repozytoriów w taki sposób, lecz deklarować je jako serwisy oraz dostarczać jako zależność. Inaczej znów tworzymy sobie problem.

I po co używasz Doctrine skoro i tak nie używasz obiektów a do najprostszych zapytań piszesz DQL?
Przecież wystarczy:

[PHP] pobierz, plaintext 
$product = $user->getProduct();
//lub, gdy user ma wiele produktów
$product = $user->getProducts()->filter();
[PHP] pobierz, plaintext 

Ewentualnie ->matching lub ->findBy, do DQLa jeszcze dużo brakuje. A jeśli nie chcesz używać ORMa (bo nie ma takiego obowiązku przecież) to nie prościej używać tylko PDO?

Trzeba po prostu lepiej organizować kod bo inaczej zawsze będą problemy a w dodatku ciężko będzie cokolwiek poprawić.

[Crash89]

- nie ma takiej potrzeby, każde zapytanie jest wyświetlone w profilerze.

[PHP] pobierz, plaintext 
        if($user === 'anon.')
        {
            return true;
        }
[PHP] pobierz, plaintext 

1. User to jest nazwa użytkownika, login, obiekt "user", status jakiś? Nazwa zmiennej powinna odzwierciedlać jej zawartość.
2. Metoda nazywa się "checkIfUserVote" a sprawdza czy user to string "anon.", za chwilę założysz temat "dlaczego użytkownik ma zaznaczone, że głosował jak nie oddał głosu"

[PHP] pobierz, plaintext 
$this->em->getRepository('AppBundle:Rate')
[PHP] pobierz, plaintext 

- nie powinno się tworzyć repozytoriów w taki sposób, lecz deklarować je jako serwisy oraz dostarczać jako zależność. Inaczej znów tworzymy sobie problem.

I po co używasz Doctrine skoro i tak nie używasz obiektów a do najprostszych zapytań piszesz DQL?
Przecież wystarczy:

[PHP] pobierz, plaintext 
$product = $user->getProduct();
//lub, gdy user ma wiele produktów
$product = $user->getProducts()->filter();
[PHP] pobierz, plaintext 

Ewentualnie ->matching lub ->findBy, do DQLa jeszcze dużo brakuje. A jeśli nie chcesz używać ORMa (bo nie ma takiego obowiązku przecież) to nie prościej używać tylko PDO?

Trzeba po prostu lepiej organizować kod bo inaczej zawsze będą problemy a w dodatku ciężko będzie cokolwiek poprawić.

Metoda IfUserVote dokładnie sprawdza czy użytkownik ma prawo głosu na dany produkt.
Sprawdzam czy user jest stringiem bo taką daną zwraca metoda z kontrolera, która pobiera zalogowanego użytkownika. Jeśli nie jest zalogowany to zwraca 'anon.'.
Zwracam prawdę bo i tak jeśli nie zalogowany użytkownik zagłosuje to wyświetli mu się strona do logowania, i problem załatwiony.

Z tymi zapytaniami do doctrine to chciałem stworzyć zapytanie, które pobierze dany głos (rate) oddany przez zalogowanego użytkownika i dotyczący danego produktu.
teraz się dowiedziałem że można to zrobić tak i dql nie jest potrzebny:

[PHP] pobierz, plaintext 
$product = $repository->findOneBy(array('name' => 'foo', 'price' => 19.99));
[PHP] pobierz, plaintext 

Myślałem że to jest ograniczone do jednego parametru.