[Kohana]problem z zapytaniem Opcje

[grzes999]

Mam mały problem z zapytaniem. Załóżmy że mam dwie tabele, w jednej mam: id,obiekt a w drugiej inne_id i id_obiektu. Czyli tabele są połączone poprzez id obiektu. I teraz jeden obiekt może być kilka razy w drugiej.I teraz jak piszę:

[PHP] pobierz, plaintext 
 DB::select()->from('ogloszenie_wyp')->join('ogloszenie')->on('id_ogl', '=', 'id_ogloszenia')->execute(); 
[PHP] pobierz, plaintext 

to pobiera mi się tylko jedna wartość z drugiej tabeli a potrzebuję wszystkie. Proszę o pomoc.

Ten post edytował grzes999 27.03.2012, 23:46:15

[thek]

A składnię join() i from() w Kohanie tyś widział?
Wersja 2.x http://docs.kohanaphp.com/libraries/database/builder#join
Wersja 3.x http://kohanaframework.org/3.0/guide/datab...y/builder#joins
Teraz popatrz, porównaj ze swoim. Ja już nawet literówek nie wypominam bo dwie widzę od razu...

[grzes999]

Widziałem i ten przykład pisałem na szybko i pominąłem i ja nie mam problemu z tym że mi zapytanie nie dzial tylko z tym że pobiera tylko jeden rekord z drugiej tabeli.

[r4xz]

1. popraw temat
2. wklej w takim razie twoje zapytanie to może poznamy przyczynę takiego, a nie innego zachowania

[thek]

Grześ... W takim razie z czym masz problem? Z ubraniem zapytania w query buildera Kohany (przy okazji która wersja?) czy z samym skleceniem zapytania? Jeśli to pierwsze to jeszcze od biedy temat tu może zostać, ale jeśli to drugie to trochę niezbyt prawidłowy dział wybrałeś

[grzes999]

Wersja Kohany 3.2 i zapytanie działa tylko z tej drugiej tabelipobiera mi tylko jeden rekord a nie wszystkie jakie są do danego id.

Moje zapytanie

[PHP] pobierz, plaintext 
DB::select()->from('ogloszenie_wyp')->join('ogloszenie')->on('id_ogl', '=', 'id_ogloszenia')->execute();
[PHP] pobierz, plaintext 

Ewewntualnie może jakoś to rozbić na dwa zapytania tylko jak je póżniej połączyć

[thek]

Takie coś powinno wygenerować zapytanie:
SELECT tu_kolumny FROM ogloszenie_wyp JOIN ogloszenie ON ig_ogl = id_ogloszenia
Pytanie się rodzi czy o takie Ci chodzi? Czy wrzucone prosto do bazy daje Ci rzeczywiście to co chcesz? Bo może to nie jest problem z Kohaną, ale nieprawidłowo utworzonym zapytaniem? Zrób debug zapytania jakie generuje query builder (jest przechowywane w obiekcie) i porównaj z tym, jakie chcesz uzyskać. Możesz się zdziwić czasem

[skowron-line]

@grzes999 najpierw napisz sobie zapytanie z palca, takie które zwraca żądany wynik w PMA, a później do QB wpisuj.

[PHP] pobierz, plaintext 
echo DB::select()->from('ogloszenie_wyp')->join('ogloszenie')->on('id_ogl', '=', 'id_ogloszenia');
[PHP] pobierz, plaintext 

i wklej do PMA

[thek]

Nie echo Query builder zwraca obiekt, więc nic to nie da. Poza tym kohana ma wbudowaną funkcję wyświetlającą, odpowiednik var_dump -> Kohana::debug($zmienne) http://kohanaframework.org/3.0/guide/kohana/debugging

[skowron-line]

Nie echo Query builder zwraca obiekt, więc nic to nie da.

Zadziała też, wiem bo sam tak robię
z manuala

[PHP] pobierz, plaintext 
echo Debug::vars((string) $query);
// Should display:
// SELECT `username`, `password` FROM `users` WHERE `username` = 'john'
[PHP] pobierz, plaintext 

[grzes999]

No i sprawdziłem i zapytanie jest takie jak chce i jeżeli wpisuje je ręcznie do bazy to otrzymuję to co chce. Może źle to wypisuje.

[PHP] pobierz, plaintext 
foreach($oferty as $oferta) : 
    echo $oferta['gdzie']; echo $oferta['od'];  echo $oferta['do']; 
    echo $oferta['opis']; 
    echo $oferta['cena'];
    switch ($oferta['has_wyposazenie'])
    {
        case('1'):
        echo "lazienka";
        break;
        case('2'):
        echo "telewizor";
        break;
        case('3'):
        echo "radio";
        break;
        case('4'):
        echo "ognisko";
        break;
        case('5'):
        echo "gril";
        break;
        default:
        echo "Brak dodatowych opcji";
        break;
    }
 
endforeach; 
[PHP] pobierz, plaintext 

Ten post edytował grzes999 28.03.2012, 10:02:30

[skowron-line]

A przypadkiem break w switch Ci nie kończy pętli foreach odrazu

@down
odpowiedz w formie pytania

Ten post edytował skowron-line 28.03.2012, 11:43:20

[phpion]

@up:
No co Ty

[grzes999]

A przypadkiem break w switch Ci nie kończy pętli foreach odrazu

@down
odpowiedz w formie pytania

W takim razie w jaki sposób wykonać tylko te akcje, kótre chcę a nie wszystkie??
I jeszcze jedno pytanie bo teraz wyświetla mi kilka razy to samo.
To znaczy jeżeli do jednego obiektu jest kilka rekordów w drugiej tabeli to wyświetla tle razy ile jest rekordów w tej drugiej tabeli.
Można to jakoś ograniczyć. Wiem, że to jest bardziej pytanie z SQL ale nie będę już zakładał drugiego tematu.

[skowron-line]

Napisz co chcesz wyciągnąć z bazy, będzie prościej.

[grzes999]

W bazie mam w jednej tabeli mam hotele a w drugiej tabeli wyposażenie dodatkowe takie ognisko,gril,wifi,parking,itd i chcę wyciągnąć wszystkie informacje o pokoju. Z podstawowymi nie ma problemu tylko z tymi dodatkowymi się troszkę meczę.

hotel

id | gdzie | opis

wyposazenie

id_wyposazenia | co

has_wyposazenie

id_has | id_pokoju | id_wyposazenia

Mniej więcej tak wygląda moja baza i niestety musi tak zostać bo jest częścią większej całości i nie opłaca się już przerabiać.

[thek]

To w takim razie pchaj to jako tablicę, czyli where('pole', 'IN', TABLICA_Z_ID_WYPOSAŻENIA)
U Ciebie nie jest bowiem JOINowanie problemem, ale kryterium wyszukiwawcze
Twoje zapytanie więc to...
1. Jeśli jest podane w kryteriach wyposażenie dorób join na tabelę z wyposażeniem
2. Jeśli już zjoinowałes, dorzuć kryterium wyszukiwawcze, gdzie id_wyposażenia wybranego pchniesz jako tablicę do where('id_wyposazenia', 'in', $tablica_wybranych_id)
3. Dodatkowo pogrupuj to po id_pokoju lub id obiektu, zależnie co chcesz pokazać.
Tu następuje rozwidlenie bo mozna do sprawy podejść na 2 sposoby:
4.1. Nie muszą wystapić wszystkie kryteria wyszukiwawcze, ale musi być trafione choć jedno.
a. tylko order malejacy po zliczonej ilości różnych kryteriów (kilka pokoi w jednym obiekcie to kilka trafionych a ma być w takim wypadku zliczone jako 1 a nie kilka)
4.2. Muszą wystąpić wszystkie kryteria:
a. tu jest zabawa Musisz policzyć ile masz kryteriów wyszukiwawczych dla każdego obiektu (znów uwaga jak w alternatywnym podejściu. Kilka trafień w to samo kryterium = 1, anie przypuśćmy 5) i wybrać te, gdzie liczba ta jest równa ilości kryteriów w zapytaniu. Having się kłania

Jak rozwiązać kwestię wyposażenia? Prosto... Albo z cache, albo na początku skryptu pobierz/utwórz tablicę gdzie masz inna tablicę typu klucz_id => wyposażenie. Teraz z formularza pobierasz id wybranych elementów wyposażenia. Dla pewności sprawdź, czy Ci ktoś nie spreparował formularza i zrób częśc wspólną tego co przyszło z formularza oraz wzorcowych. array_intersect przy czym dla wygody można wszystkie klucze wzorcowe machnąć jako array_keys.

Jak sam widzisz, nie Kohana Ci sprawia problem, ale logika skryptu.

[grzes999]

thek dzięki za wyjaśnienie ale samo się trochę teraz zamotałem w tym co napisałeś ale na spokojnie powinienem to ogarnąć.

I może lepiej jest zrobić dwa zapytania?? To znaczy w modelu zrobił bym dwie funkcje. Jedna pobierała by wszystko i zwracała id_obiektu a druga by pobierała wszystkie atrybuty. Tylko nie wiem czy tak można :/

I tam jest małe niedopatrzenie w schemacie mojej bazy.

pokoju

id_pokoju | gdzie | opis

wyposazenie

id_wyposazenia | co

has_wyposazenie

id_has | id_pokoju | id_wyposazenia

Bo chyba rozważałeś, że id w tej pierwszej tabeli to co innego niż w tej ostatniej a to jest to samo.

Ten post edytował grzes999 28.03.2012, 15:27:15

[thek]

Metody modelu pisze się tak, by był przydatne Co do tabeli to w ostatniej id_has nawet nie jest potrzebne. Wystarczy nadać klucz unique na połączone id_pokoju i id_wyposażenia. Co do reszty to join rób tylko gdy w searchu masz wybrane wyposażenie. Inaczej join jest zbędny.

[grzes999]

Próbuję pisać zapytanie tak jak mi poradziłeś thek ale mam problem z tablicą bo gdy ją deklaruję jako pustą to mi wywala błąd a gdy nie deklaruję to wiadomo błąd że nie jest zadeklarowana

Kod

Database_Exception [ 1064 ]: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ') GROUP BY `id_pokoju'`' at line 1 [ SELECT * FROM `pokoje` JOIN `has_wyposazenie` ON (`id_pokoju` = `id_pokl`) WHERE `id_wyposazenia` IN () GROUP BY `id_pokoju` ]

Moje zapytanie

[PHP] pobierz, plaintext 
$result = DB::select()->from('pokoje)->join('has_wyposazenie')->on('id_pokoju', '=', 'id_pokl')
				->where('id_wyposazenia', 'IN', $tablica)->group_by('id_pokoju')->execute();
[PHP] pobierz, plaintext 

Ten post edytował grzes999 29.03.2012, 13:23:30