Kodzę sobie wieczorem i wpadłem na niegłupi chyba pomysł, aby nazwa kategorii wyświetlała się nad listingiem filmów.

Dodałem więc stosowny blok, ale mam 2 x zapytanie do bazy i taki oto bałagan... :

[PHP] pobierz, plaintext 
<?php 
	$kat_id = $_GET['kat_id']; 
 
	$result = $db->query("SELECT kat_nazwa FROM kategorie
		where kategorie.kat_id = $kat_id");
 
	foreach($result as $row) { echo '<div class="section_header">
		<div class="row">
			<div class="col-lg-12">
			<h3>'.$row['kat_nazwa'].'</h3><hr /></br>
			</div>
		</div>
	</div>	';}
 
	$result = $db->query("SELECT Okladka, Nazwa, short_opis, terapeuta_nazwa FROM filmy
		inner join terapeuci
on terapeuci.terapeuta_id = filmy.terapeuta_id
							inner join filmy_kategorie
on filmy.film_id = filmy_kategorie.film_id
where filmy_kategorie.kat_id = $kat_id
 
		");
 
foreach($result as $row)
	{ echo '<div id="filmlista" class="col-lg-6"><center>
 
 
 
					<div class="col-lg-4">
					<img src="'.$row['Okladka'].'" class="picColor"/> 
					</div></center>';
 
echo '<center>		<div class="col-lg-8"><h5><strong>'.$row['Nazwa'].'</strong></h5></br><strong>'.$row['terapeuta_nazwa'].'</strong><br/><br/>'.$row['short_opis'].'<br/><br/>
					<button type="button" class="btn btn-default btn-block">Zobacz opis i fragment filmu</button>
					</div>
 
				</div></center>';
 
 
	}
?>
[PHP] pobierz, plaintext 

Czy można to zrobić estetyczniej? Np. w jednym zapytaniu, albo bez for - each dla jednej nazwy, tj. kat_nazwa ?

Dziękuję.

Estetycznej nie by było usunąć to spaghetti i zrobić porządnie, jak matka natura nakazuje: z wykorzystaniem jakiegoś systemu szablonów i oddzielić logikę od warstwy prezentacji na początek.

pedro84: Tak, masz rację, ale ja MVC ( bo chyba o tym piszesz ) nie znam ni cholery, tym bardziej żadnego frameworka. Przymierzam się do Laravela, ale to jeszcze chwila.

Jak to spaghetti uprościć w tradycyjny sposób?

Nie trzeba od razu MVC bo nie o to chodzi, ale nie wsadzać kodu html w echo

Jak nie w echo to w co wsadzać?

( nie wiedziałem, że stanę przed takimi dylematami, w co wsadzać...

[PHP] pobierz, plaintext 
<?php foreach($sth as $v): ?>
<tr><td><?= $v ?> </td></tr>
<?php endforeach; ?>
[PHP] pobierz, plaintext 

mały przykład

Dziękuję, już bardziej łapię.

A zapytań sql nie da się jakoś uprościć, zrobić w jednym np. ?

Do drugiego apytania dorzuć jeszcze tabelę z kategoriami. Będiesz mial jedno zapytanie
I dla nagłówka pobierasz zpierwszego rekordu kategorię, a ptem to już tradycyjnie...