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.

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

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.

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

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ś

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ć

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

@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

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

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 

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 

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

@down
odpowiedz w formie pytania

@up:
No co Ty

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.

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

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

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.

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.

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.

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



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 

Zobacz jakie zapytanie Ci wypluwa na ekran

[SQL] pobierz, plaintext 
SELECT * FROM `pokoje` JOIN `has_wyposazenie` ON (`id_pokoju` = `id_pokl`) WHERE `id_wyposazenia` IN () GROUP BY `id_pokoju`
[SQL] pobierz, plaintext 

nie ma czegoś takiego jak IN (), między nawiasami coś musi się znajdować. Twoja zmienna jest pusta.

Bo $tablica powinna zawierać pobrane z formularza id_wyposażenia. Nie może być pusta. Właśnie dlatego napisałem, że JOINujesz i dajesz to WHERE tylko gdy jakiekolwiek wyposażenie zostało wybrane. Query Builder istnieje, by sobie palnąć IF i wykryć czy mamy to zrobić "po drodze". Ja na Twoim miejscu przekazywałbym dane $_POST jako tablicę do metody szukającej. Metoda by ją sobie przefiltrowała i na podstawie danych uzupełniała odpowiednie warunki czy Joinowała odpowiednie tabele. Przykładowo bym w formularzu umieścił checkboxy jako tablicę (name="wyposazenie[]"), gdzie jako ich value podrzucałbym id_wyposażenia. Odczyt takiej tablicy, walnięcie części wspólnej z prawidłową tablicą id_wyposażenia (w ramach walidacji) i posłanie do warunku where() to wszystko co wystarczy zrobić.

Tylko problem jest taki że ja nie mam formularza i nie wiem skąd ten pomysł ;p
Ja to pobieram z bazy i wyświetlam na stronie. Ogólnie większość treści jest pobierana z bazy
Tylko z tym problemem nie mogę sobie poradzić.

Jeśli to tylko pobór z bazy, to where niepotrzebne. Robisz group by i w select dajesz GROUP_CONCAT(wyposazenie_id), co Ci wszystkie id_wyposażenie dla pokoju walnie po przecinku.

Dzięki wielkie wszystko działa

I jeszcze mam małe pytanie jeżeli chciał bym zrobić sobie funkcję, która będzie w switchu w zależności od id wstawiała obrazek.
To piszę sobie nową metodę w modelu coś takiego

[PHP] pobierz, plaintext 
public function obrazki($id)
			{
				foreach ($id as $wyposazenie)
						{
								switch ($wyposazenie)
								{
									case('1'):
									echo "<p>lazienka</p>";
									break;
									case('2'):
									echo "<p>telewizor</p>";
									break;
									case('3'):
									echo "<p>radio</p>";
									break;
									case('4'):
									echo "<p>ognisko</p>";
									break;
									case('5'):
									echo "<p>gril</p>";
									break;
									default:
									echo "<p>Brak dodatowych opcji</p>";
									break;
								}
						}
			}
[PHP] pobierz, plaintext 

W kontrolerze piszę takie coś

[PHP] pobierz, plaintext 
$oferta = Model::factory('gielda');
$oferty = $oferta->oferta();
$this->template->content->oferty = $oferty;
$wyp=$oferta->obrazki($param);
$this->template->content->wyp = $wyp;
 
[PHP] pobierz, plaintext 

I w kontrolerze przy jej wywołaniu muszę wstawić parametr i teraz moje pytanie. Co wstawić
W widoku to wiadomo wstawiam id_wyposazenia i w pętli wyświetlam a co z kotrolerem??
W powyższym przypadku wyświetli wiadomy błąd.

[PHP] pobierz, plaintext 
Undefined variable: param
[PHP] pobierz, plaintext 

Jak dla mnie to źle podszedłeś do tego... Skoro funkcja wstawia obrazki, to ja to widze tak:
1. W kontrolerze do widoku wrzucasz jedynie tablicę z id_wyposażenia, albo string zawierający wszystkie id_wyposażenia oddzielone przecinkami lub innym delimiterem (czyli tak, jak dostajesz to z bazy)
2. Funkcja obrazki to Helper widoku, która odbiera ową tablicę lub string i w zależności od tego co to jest - reaguje.
a. jeśli to tablica to sprawdź ilość elementów i zwaliduj je pod kątem zgodności z bankiem id_wyposażenia. Jeśli po walidacji jest przynajmniej jeden element, puść na switch w pętli po elementach zwalidowanych, przy czym default ma być pusty.
b. jeśli to string to explode do tablicy i dalej już jak w punkcie a.

Ok tylko że w zapytaniu oprócz id_wyposazenia pobieram też inne dane dotyczące pokoju.
I trochę głupie pytanie mógł byś podesłać jakiegoś linka jak zrobić takiego helpera.
Gdyż w Kohanie nie jestem tak bardzo obeznany.
Znalazłem takie coś tylko nie jestem pewien czy o to ci chodzi.

Kohana->helper

Jeśli używasz Kohany z gałęzi 2.x to owszem, możesz tak zrobić. W Kohanie 3.x nieco inaczej, gdyż tam jest inna hierarchia w związku z HMVC, ale i tam można zrobić prosty myk, by działało jak helpery w Kohanie 2.x. Kwestia dodania jednego dodatkowego pliku extendującego "po drodze", jeśli ktoś się przyzwyczaił do używania helperów jak w 2.x

Ja osobiście używam 3.2 tak że będę musiał coś w manualu poszukać.
Ale tak ogólnie to jak trzeba zapisać tą moją poprzednią funckję bez użycia helpera bo będę potrzebował wyświetlić do danego pokoju komentarze.
I wtedy tak by chyba było najprościej.
I jest jesze jeden problem tym razem chyba bardziej z Kohana.
Otóż w jednym zapytaniu pobieram wszystkie informacje o pokoju i do kazdego pokoju mogą być komentarze. Więc w drugim zapytaniu pobieram komentarze do tego pokoju. Wiem że mogę pobrać to w jednym zapytaniu ale te zapytania i tak są dość skomplikowane dlatego nie chce ich jeszcze bardziej komplikować. I teraz moje pytanie jak przekazać do drugiego zapytania id_obiektu pobrane z pierwszego. Moje dwa modele.

Do pobierania informacji o pokoju.

[PHP] pobierz, plaintext 
public function oferta()
			{
				$result = DB::select('id_ogloszenia','gdzie','od','do','dodano','koniec','cena','opis',
				array('group_concat("has_wyposazenie")' , 'wyposazenie'))->from('ogloszenie')->join('ogloszenie_wyp')
				->on('id_ogloszenia', '=', 'id_ogl')->group_by('id_ogloszenia')->execute();
				$result = $result->as_array();
				return $result;
			}
[PHP] pobierz, plaintext 

I drugi do pobierania komentarzy do pokoju i nie tylko

[PHP] pobierz, plaintext 
public function ogloszenie()
			{
 
				$result = DB::select('nazwa','oferta.cena','oferta.opis','obiekt.zdjecie','data_dodania',
				array('group_concat("wyposazenie_id")' , 'wyposazenie'))
				->from('oferta')->join('ogloszenie')->on('oferta.id_ogloszenia','=','ogloszenie.id_ogloszenia')
				->join('obiekt')->on('obiekt.id','=','id_obiektu')->join('user')->on('oferta.id_usera','=','user.id')
				->join('obiekt_has_wyposazenie')->on('obiekt_id','=','obiekt.id')
				->where('ogloszenie.id_ogloszenia','=',2)->group_by('nazwa')->order_by('oferta.cena')->execute();
				$result = $result->as_array();
				return $result;
			}
[PHP] pobierz, plaintext 

Jak widać na razie wstawiłem na stałe id_ogloszenia=2
Wiem że trzeba przekazać parametr w funkcji i wpisać go w zapytanie ale jak to później napisać w kontrolerze??

A o parametrach funkcji słyszałeś?
$model->ogłoszenie($zmienna);
Nie mów, że wszystkie funkcje do tej pory pisałeś i wywoływałeś bezparametrowo...

No oczywiście że nie tylko co ma być w kontrolerze w tej zmiennej??
Bo jak wrzucę nie zdefiniowaną to wiadomo błąd a jak wstawiłem pustą zmienną i w widoku tak wywyływałem

$wywolanie($ogloszenie['cos'])

To wywala mi błąd że call function must by string albo jakoś tak i głównie z tym mam problem.

Napiszę tak... Przeczytaj jakiś poradnik o tym jak się tworzy i wywołuje klasy w Kohanie. Bez tego możemy Ci pomagać, pisać kod, a i tak nic nie zrozumiesz. Wpierw zapoznaj się z podstawami frameworka na jakichś tutorialach i wróć gdy już coś opanujesz. Skoro już się łapiesz za FW to wpierw go choć trochę poznaj byś wiedział co z czym ugryźć, a nie, że liznąłeś php i od razu się rzucasz na coś, czego idei nie rozumiesz. Czy jeśli nauczyłeś się jeździć na rowerze, to siadasz na motocykl bo "oba mają dwa koła, kierownicę i siedzenie oraz ramę"? Przeskoczyć z czystego php można od razu na framework, ale to wymaga zdolności już jakichś z innych języków i rozmienia zasad działania.

Rozumiem mniej więcej zasadę działanie MVC. Mogę poczytać tutorial tylko cięzko znaleźć coś dobrego w sieci.
Niby czytałem manuala ale nie wile mi pomogło to przy moim problemie i dlatego proszę o przykład jak takie coś się piszę i wtedy łatwiej będzie mi wyciągnąć wnioski i zrozumieć.
Niby zrobiłem tylko nie wiem czy to jest poprawne w widoku wpisuje coś takiego i działa

[PHP] pobierz, plaintext 
$ogloszenie = Model::factory('gielda')->ogloszenie($oferta['id_ogloszenia']);
[PHP] pobierz, plaintext 

Bo tak właśnie wygląda jedno z możliwych poprawnych wywołań. Są krótsze także. Jeśli używałeś w modelu wywołania połączenia z bazą danych (DB) to chyba wiesz już jak wygląda krótsze

Dzięki wszystkim za bardzo dobrą i szybką pomoc.
Temat uważam za zamknięty