problem ze sfromuowaniem zapytania Opcje

[hmmm]

mam problem ze sformuowaniem zapytania ...

mam dwie tabele:
1. kategorie
- id
- kategoria (np. 'niemiecki', 'programy')
2. download
- id
- kategoria (numery odpowiadajace dla kategorie.id)
- plik (nazwa pliku)
- nazwa (opis pliku)
- wielkosc

chcialbym teraz za pomoca najchetniej jednego zapytania, jezeli sie nie da, to dwoch (przypuszczam, ze petla w petli) wyswietlic to w takiej formie:

Kod

nazwa_kategorii1:
nazwa_pliku  |  opis_pliku  |  wielkosc_pliku
nazwa_kategorii2:
nazwa_pliku  |  opis_pliku  |  wielkosc_pliku

i tak dalej

czyli podzielic najpierw na kategorie i pod nazwa kazdej kategorii wyswietlic pliki nalezace do tej kategorii.
probowalem, ale nie wychodzi.
wszystko bylo dobrze, dopoki mialem jedna tabele - kategorie i zamiast wartosci liczbowych w kolumnie kategorie mialem nazwy kategorii.

[patigo]

napisz to co sam wymyśliłeś, mam pewną propozycję ale chciałbym zobaczyć twój tok myślenia

[hmmm]

sorki, ze dopiero, ale probowalem jeszcze cos stworzyc i ostetecznie, zupelnie sie pogubilem.
mam tylko moje pierwsze rozwiazanie, reszta przepadla razem w probami, bo nie zapisuje tych, ktore i tak sa zle ...

na poczatku robilem to tak:
mialem wowczas jedna tabele - download i w niej byla kolumna kategoria, do ktorej wpisywalem wartosci liczbowe, np. 1,2,3,4,...

a potem mialem taki skrypcik php:

[PHP] pobierz, plaintext 
<?php
		$kategorie = array(1 => 'matematyka', 'niemiecki', 'do pobrania', 'skrypty');
		// wartosci tablicy $kategorie
		// 1 => matematyka
		// 2 => niemiecki
		// 3 => do pobrania
		// 4 => skrypty
 
		// pobierz wszystkie grupy plikow
		$polecenie2 = mysql_query(&#092;"SELECT kategoria FROM download GROUP BY kategoria\");
		while ($wynik2 = mysql_fetch_array($polecenie2, MYSQL_NUM)) {
				$download_grupy[] = $wynik2[0];
		}
 
		// wyswietl wszystkie grupy plikow
		foreach ($download_grupy as $download_grupa) {
 
?>
		<table class=\"download\" cellspacing=\"0\" cellpadding=\"0\">
				<tr>
						<td class=\"td2\" colspan=\"2\">
						<?=$kategorie[$download_grupa] . &#092;"n\";?>
						</td>
				</tr>
<?php
 
				// pobierz wszystkie pliki dla danej grupy plikow
				// wyswietl wszystkie pliki dla danej grupy plikow
				$polecenie3 = mysql_query(&#092;"SELECT plik, nazwa, wielkosc FROM download WHERE kategoria = '\" . $download_grupa . \"'\");
				$download_numer = 1;
				while ($wynik3 = mysql_fetch_array($polecenie3, MYSQL_NUM)) {
?>
				<tr>
						<td class=\"td1\">
						<?=$download_numer++;?>.
						</td>
						<td class=\"td2\">
						<a href=\"download/<?=$wynik3[0];?>\"><?=$wynik3[1];?> <span class=\"szary\">[<?=$wynik3[2];?>KB]</span></a>
						</td>
				</tr>
<?php
				}
				mysql_free_result($polecenie3);
 
				echo &#092;"t\" . '</table>' . \"n\";
?>
[PHP] pobierz, plaintext 

prymitywnie - wiem.

chce to zrobic lepiej, ale nie moge sobie poradzic ...

[bigZbig]

Proponuje abys zrobil takie zapytanie

[PHP] pobierz, plaintext 
<?php
$query=&#092;"SELECT kat.kategoria as 'nazwa_kategorii', plik as 'nazwa_pliku', nazwa as 'opis_pliku', wielkosc as 'wielkosc_pliku'
FROM kategorie as 'kat', download as 'down' 
WHERE down.kategoria = kat.id;&#092;";
?>
[PHP] pobierz, plaintext 

Tak na marginesie, kto ci wymyslil takie nazwy tabel i pol w tabelach?

Nie prosciej byloby:

kategorie
1. id_kategorie //najlepiej aby nazwa klucza podstawowego zawierala nazwe tabeli
2. nazwa //albo nazwa_kategorii

pliki
1. id_pliki
2. id_kategorie //od razu widac ze to jest klucz obcy
3. nazwa
4. opis
5. wielkosc

[hmmm]

to znaczy, to co podalem w pierwszym poscie - nazwy tabel i kolumn, to czytac nalezy bez tych nawiasow.
to, ze dobrze jest nazywac klucze obce i glowne np. id_kategorie, to wiem, no ale tak jakos sobie utrudniam zawsze

no ale zmienilem juz nazwy kolumn:
download:
- id
- kategoria
- nazwa
- opis
- wielkosc
kategorie:
- id
- kategoria

twoje zapytanie dziala i nawet je rozumiem, ale nie wiem, jak mam je wykorzystac, aby wyswietlic je juz na stronie w takiej formie, jakbym chcial.
poza tym, zwraca mi wszystkie rekordy, nie wiem, czy o to chodzi ...
w formie uproszczonej chodzi mi o cos takiego:

Kod

nazwa kategorii1
wyswietlenie wszystkich plikow z tej kategorii

nazwa kategorii2
wyswietlenie wszystkich plikow z tej kategorii

itd.

nie wiem, jak to zrobic w php.
w tabelke wsadzic to zaden problem, ale nie wiem, jak wyswietlic


E D I T :
wymyslilem cos, tylko zastanawiam sie, czy mozna to jakos skrocic.
ja niestety nie mam zadnych pomyslow ...

[PHP] pobierz, plaintext 
<?php
include('katalog/plik.inc');
$db = mysql_connect($host, $user, $pass);
mysql_select_db($name, $db);
 
// zliczam ilosc rekordow w tabeli kategorie
// i tworze tablice z id wszystkich kategorii
$polecenie3 = mysql_query(&#092;"SELECT count(id), id FROM kategorie GROUP BY id ORDER BY kategoria\");
while ($wynik3 = mysql_fetch_array($polecenie3, MYSQL_NUM)) {
		$download_ilosc = $wynik3[0];
		$kategorie[] = $wynik3[1];
}
 
if ($download_ilosc > 0) {
 
		echo &#092;"t\" . '<table id=\"download\" cellspacing=\"0\" cellpadding=\"0\">' . \"n\";
 
		// funckja tworzaca linki do plikow
		function plik ($zmienna1, $zmienna2, $zmienna3) {
				return '<a href=\"download/' . $zmienna1 . '\">' . $zmienna2 . ' <span class=\"szary\">[' . $zmienna3 . 'KB]</span></a>';
		}
 
		// wyswietlenie wszystkich rodzajow kategorii
		foreach ($kategorie as $pokaz) {
				// pobieram odpowiednia nazwe kategorii dla danego id
				$polecenie4 = mysql_query(&#092;"SELECT kategoria FROM kategorie WHERE id = $pokaz\");
				while ($wynik4 = mysql_fetch_array($polecenie4, MYSQL_NUM)) {
						$kategoria = $wynik4[0];
				}
?>
				<tr>
						<td class=\"td2\" colspan=\"2\">
						<?=$kategoria . &#092;"n\";?>
						</td>
				</tr>
<?php
				// pobieram wartosci dla kazdego pliku
				// i wyswietlam je
				$polecenie5 = mysql_query(&#092;"SELECT nazwa, opis, wielkosc FROM download WHERE download.kategoria = $poka
z\");
				$numer = 1;
				while ($wynik5 = mysql_fetch_array($polecenie5, MYSQL_NUM)) {
?>
				<tr>
						<td class=\"td1\">
						<?=$numer++;?>.
						</td>
						<td class=\"td2\">
						<?=plik($wynik5[0], $wynik5[1], $wynik5[2]);?>
						</td>
				</tr>
<?php
				}
		}
 
		echo &#092;"t\" . '</table>' . \"n\";
 
} else {
?>
		<div id=\"download\">
 
		jak dotad brak wpisow w ksiedze gosci.
 
		</div>
<?php
}
?>
[PHP] pobierz, plaintext 

Ten post edytował hmmm 5.04.2005, 19:13:04

[bigZbig]

Wiekszosc zadan mozna wykonac na wiele roznych sposobow. Dobrze jest jezeli masz zoptymalizowany kod, ale jeszcze lepiej jesli to jest Twoj kod - nawet gdy nie jest w pelni profesjonalny. Piszac cos samemu uczysz sie o wiele szybciej niz wtedy gdy kozystasz z gotowych, cudzych rozwiazan.

Co do zademonstrowanego przez Ciebie fragmentu kodu mam jednak kilka uwag.

[PHP] pobierz, plaintext 
<?php
 
$polecenie3 = mysql_query(&#092;"SELECT count(id), id FROM kategorie GROUP BY id ORDER BY kategoria\");
while ($wynik3 = mysql_fetch_array($polecenie3, MYSQL_NUM)) {
		$download_ilosc = $wynik3[0];
		$kategorie[] = $wynik3[1];
}
 
if ($download_ilosc > 0) {
 
?>
[PHP] pobierz, plaintext 

Wykonywanie wbudowanej funkcji count() na kluczu podstawowym z uzyta klauzula GROUP BY na tym samym parametrze mija sie z celem poniewaz, jak w dym mozesz przyjac ze wyjdzie 1 (w koncu wartosci klucza podstawowego sa unikalne), no chyba, ze tabela jest pusta. Co za tym idzie warunek "if ($download_ilosc > 0)" jest bez sensu bo jesli tylko zapytanie zwroci wynik to ten warunek jest zawsze prawdziwy. Proponuje to zastapic kodem

[PHP] pobierz, plaintext 
<?php
 
$polecenie3 = mysql_query(&#092;"SELECT id FROM kategorie ORDER BY kategoria\");
if ($polecenie3==true) {
	while ($wynik3 = mysql_fetch_array($polecenie3, MYSQL_NUM)) {
		 $kategorie[] = $wynik3[0]; 
	}
?>
[PHP] pobierz, plaintext 

Jesli zamiast MYSQL_NUM uzylbys MYSQL_ASSOC dzialal by zapis:
$kategorie[] = $wynik3['id'];

Jesli uzylbys MYSQL_BOTH, ktory jest wartoscia domyslna
oba zapisy moglbys stosowac zamiennie, jak wygodniej.

Musisz koniecznie ale to naprawde koniecznie popracowac nad wlasciwym nadawaniem nazw funkcji i zmiennym bo w przeciwnym razie szybko sie pogubisz w swoim wlasnym projekcie.

[PHP] pobierz, plaintext 
<?php
function plik ($zmienna1, $zmienna2, $zmienna3)
?>
[PHP] pobierz, plaintext 

Nazwa funkcji powinna zawierac jakis czasownik bo przeciez funkcja cos robi, a ta konkretna funkcja tworzy link. Nazwij ja wiec np. tworzLink(). Poza tym co przechowuja zmienne: $zmienna1, $zmienna2, $zmienna3? Nazwa zmiennej powinna odzwierciedlac jej zawartosc.

[PHP] pobierz, plaintext 
<?php
foreach ($kategorie as $pokaz) {
				// pobieram odpowiednia nazwe kategorii dla danego id
				$polecenie4 = mysql_query(&#092;"SELECT kategoria FROM kategorie WHERE id = $pokaz\");
?>
[PHP] pobierz, plaintext 

Rozumiem, ze tablice w ktorej przechowujesz identyfikatory kategorii nazwales $kategorie ale dlaczego uzyles nazwy $pokaz dla zmiennej, ktora przechowuje pojedynczy identyfikator kategorii? Poza tym Twoj komentarz jest nieprawdziwy bo ty nie pobierasz nazw kategorii tylko identyfikatory kategorii, a tak na marginesie gdybys mial dobrze dobrane nazwy zmiennych to akurat w tym wypadku komentarz bylby zbedny.

Ten post edytował bigZbig 6.04.2005, 08:02:53

[hmmm]

dziekuje za wszystkie uwagi
nauka nie pojdzie w pole i na pewno skorzystam z niej.

mam tylko jedno zastrzezenie do podanych przez ciebie rozwiazan, mianowicie do pierwszego. napisales warunek

[PHP] pobierz, plaintext 
<?php
if ($polecenie3==true)
?>
[PHP] pobierz, plaintext 

ale mi nie chodzi o to, czy polecenie wykona, czy nie, tylko o ilosc elementow w tablicy i zaleznosci od tego, ma wyswietlic albo kategorie i pliki, albo wiadomosc, ze nie ma zadnych plikow.
nie wiem, czy dobrze kombinuje, ale jezeli ilosc id bedzie rowna zero, czyli nie bedzie kategorii, a polecenie zostanie wykonane to i tak $polecenie3 bedzie rownalo sie true.
napisalem cos takiego:

[PHP] pobierz, plaintext 
<?php
$polecenie3 = mysql_query(&#092;"SELECT id FROM kategorie ORDER BY kategoria\");
while ($wynik3 = mysql_fetch_array($polecenie3, MYSQL_NUM)) {
	$kategorie[] = $wynik3[0];
}
$ilosc_kategorii = count($kategorie);
if ($ilosc_kategorii > 0) {
	// wyswietl wszystkie kategorie i pliki do nich nalezace
} else {
	echo 'jak dotad brak plikow w bazie.';
}
?>
[PHP] pobierz, plaintext 

z dalszej lektory rozumiem, ze wcale nie musze pisac MYSQL_NUM, tylko nazwe zmiennej i potem w petli while bede sie odwolywal albo do nazw, albo do numerow ...

a to nowy, poprawiony skrypt:

[PHP] pobierz, plaintext 
<?php
include('katalog/plik.inc');
$db = mysql_connect($host, $user, $pass);
mysql_select_db($name, $db);
 
// zlicz ilosc rekordow w tabeli kategorie
// utworz tablice zawierajaca id wszystkich kategorii
$polecenie3 = mysql_query(&#092;"SELECT id FROM kategorie ORDER BY kategoria\");
while ($wynik3 = mysql_fetch_array($polecenie3, MYSQL_NUM)) {
		$kategorie[] = $wynik3[0];
}
mysql_free_result($polecenie3);
 
// zlicz ilosc wszystkich kategorii
$ilosc_kategorii = count($kategorie);
if ($ilosc_kategorii > 0) {
 
		echo &#092;"t\" . '<table id=\"download\" cellspacing=\"0\" cellpadding=\"0\">' . \"n\";
 
		// funckja tworzaca linki do plikow
		function tworzlink ($nazwa_pliku, $opis_pliku, $wielkosc_pliku) {
				return '<a href=\"download/' . $nazwa_pliku . '\">' . $opis_pliku . ' <span class=\"szary\">[' . $wielkosc_pliku . 'KB]</span></a>';
		}
 
		// wyswietl wszystkie rodzaje kategorii
		foreach ($kategorie as $id) {
				// pobierz odpowiednia nazwe kategorii dla danego id
				$polecenie4 = mysql_query(&#092;"SELECT kategoria FROM kategorie WHERE id = $id\");
				while ($wynik4 = mysql_fetch_array($polecenie4, MYSQL_NUM)) {
						$kategoria = $wynik4[0];
				}
				mysql_free_result($polecenie4);
?>
				<tr>
						<td class=\"td2\" colspan=\"2\">
						<?=$kategoria . &#092;"n\";?>
						</td>
				</tr>
<?php
				// pobierz wartosci dla kazdego pliku i wyswietl je
				$polecenie5 = mysql_query(&#092;"SELECT nazwa, opis, wielkosc FROM download WHERE download.kategoria = $id\");
				$numer = 1;
				while ($wynik5 = mysql_fetch_array($polecenie5, MYSQL_NUM)) {
?>
				<tr>
						<td class=\"td1\">
						<?=$numer++;?>.
						</td>
						<td class=\"td2\">
						<?=tworzlink($wynik5[0], $wynik5[1], $wynik5[2]);?>
						</td>
				</tr>
<?php
				}
				mysql_free_result($polecenie5);
		}
 
		echo &#092;"t\" . '</table>' . \"n\";
 
} else {
?>
		<div id=\"download\">
 
		jak dotad brak plikow w bazie.
 
		</div>
<?php
}
 
mysql_close();
?>
[PHP] pobierz, plaintext 

Ten post edytował hmmm 6.04.2005, 19:50:53