[PHP][SQL]Łączenie zapytań Opcje

[sledzik]

Czy da się połączyć takie dwa zapytania w jedno?
Wszystko się opiera o tabele news ale nie wychodzi mi to.. ;/

[SQL] pobierz, plaintext 
 $query1 ="
		SELECT sos_categories.* 
		FROM sos_news 
		JOIN sos_categories 
		ON sos_categories.categories_id = sos_categories.categories_id 
		ORDER BY news_date_add DESC LIMIT 0, 5 
		";
[SQL] pobierz, plaintext 

[SQL] pobierz, plaintext 
$query2 ="
		SELECT ".$prefix."news.*,".$prefix."users.*,".$prefix."news_categories.*
		FROM ".$prefix."news
		JOIN ".$prefix."users 
		ON ".$prefix."news.users_id = ".$prefix."users.users_id
		JOIN ".$prefix."news_categories 
		ON ".$prefix."news.news_id = ".$prefix."news_categories.news_id
		ORDER BY news_date_add 
		DESC LIMIT $limit, $onpage
		";
[SQL] pobierz, plaintext 

Prubuje tak ale to nie dziala:

[SQL] pobierz, plaintext 
$query3 ="
		SELECT ".$prefix."news.*,".$prefix."users.*,".$prefix."news_categories.*,".$prefix."categories.*
		FROM ".$prefix."news
		JOIN ".$prefix."users 
		ON ".$prefix."news.users_id = ".$prefix."users.users_id
		JOIN ".$prefix."news_categories 
		ON ".$prefix."news.news_id = ".$prefix."news_categories.news_id
		JOIN ".$prefix."sos_categories 
		ON ".$prefix."sos_categories.categories_id = ".$prefix."sos_categories.categories_id 
		ORDER BY news_date_add 
		DESC LIMIT $limit, $onpage
		";
[SQL] pobierz, plaintext 

Ten post edytował sledzik 1.02.2016, 20:48:09

[trueblue]

[SQL] pobierz, plaintext 
ON sos_categories.categories_id = sos_categories.categories_id 
[SQL] pobierz, plaintext 

Czym się różnią tabele categories oraz news_categories?

[sledzik]

categories czyli(np)
spozywcze
cukiernicze
ala ma kota...
itp

news
id
content
itd newsa..

categories news(laczy te dwie)
news_id categories_id

//wroc moj pomysl to..

[SQL] pobierz, plaintext 
ON ".$prefix."sos_categories.categories_id = ".$prefix."sos_news_categories.categories_id 
[SQL] pobierz, plaintext 

lecz to bledne myslenie


Ten post edytował sledzik 1.02.2016, 21:24:00

[trueblue]

Wskazałem Ci błąd.

[sledzik]

no ok ale szkopul w tym ze w tabeli news nie ma zadnego nawiazania do categories wiec nie moge chyba jako tak uzyc JOIN?

[trueblue]

No, więc łączysz je przez news_categories.

[sledzik]

kurde nie rozumiem , dalbys mi strone gdzie sa jakies przyklady lub napisal jakies przykladowe zapytanie jesli nie to

[com]

no żeby coś załączyć musisz mieć element wspólny(klucz), na tym polegają relacyjne bazy danych (IMG:style_emoticons/default/smile.gif)

[sledzik]

no tak ..

dlatego tabela news_categories laczy -> news i categories poprzez news_id oraz categories_id
tylko ze w zapytaniu chce jeszcze pobrac users_id z tabeeli users (czyli jakis id) uzytkownika ktory pisze news...

czy robiac JOIN pomiedzy dwoma pierwszymi tabelami nie musze juz brac pod uwage tabeli uzytkownikow bo w news jest pole users_id i tam jest deklarowane kto pisze posta i bede mogl to wywolac laczac te dwie pierwsze tylko tabele?

[trueblue]

Łączysz (i wybierasz we FROM) tylko te tabele, z których chcesz pobrać dane.

Oczywiście jeśli tabela przejściowa łączy jakieś tabele, z której to najczęściej nie wybierasz danych, to też ją umieszczasz w klauzuli FROM.

[sledzik]

Teoretycznie chyba mi sie udalo, blad byl gupi,a raczej polegal na slepocie jak zazywczaj.

[SQL] pobierz, plaintext 
$query1 ="
		SELECT ".$prefix."news.*,".$prefix."users.*,".$prefix."news_categories.*,".$prefix."categories.*
		FROM ".$prefix."news
		JOIN ".$prefix."users 
			ON ".$prefix."news.users_id = ".$prefix."users.users_id
		JOIN ".$prefix."news_categories 
			ON ".$prefix."news.news_id = ".$prefix."news_categories.news_id
		JOIN ".$prefix."categories 
			ON ".$prefix."categories.categories_id = ".$prefix."news_categories.categories_id 
		ORDER BY news_date_add 
		DESC LIMIT $limit, $onpage
		";
[SQL] pobierz, plaintext 

Mam jeszce jeden problem.
Posiadam taki kod wyswietlajacy:
I teraz jak jakis news ma wiecej niz 2 kategorie to pokazywany jest w nowym wierszu jako odrebny z nowa kategoria.
Jak to poprawic?

[PHP] pobierz, plaintext 
if(mysqli_num_rows($result) > 0) {		
 
	echo '<div class="table-responsive"><table class="table table-hover">';
	echo"<tr><thead>";
 
	echo'<th>KATEGORIA <a href="m.php?s=news&a=v&c=s';
	if(isset($_GET['page'])) echo '&page='.$_GET['page'];
		echo'" ><img src="images/s.png" alt="sort" /></th>';
		echo"</thead></tr>";
 
		while($r = mysqli_fetch_assoc($result)) {
			echo"<td><span class='rama label' style='color:#".$r['categories_color_text']."; background-color:".$r['categories_color_bg'].";'>".$r['categories_name']."</span></td>";
			echo "</tr>";
		}
echo "</table></div>";
}
[PHP] pobierz, plaintext 

[nospor]

Musis wpierw pogrupowac dane a potem wyswietlac jak ci pasuje
http://nospor.pl/grupowanie-wynikow.html

[sledzik]

mógłby mi ktoś pomoc z tym grupowaniem? bo pomimo przykladow nie umiem tego zrobic;/
wydawało mi się, że na podstawie przykładów coś wymyśle , ale siedzę już tyle czasu i nie bardzo
ew moge zaloczyc swoje smieciowe kody



Ten post edytował sledzik 2.02.2016, 23:38:58

[nospor]

if (!isset($categories[$nid]))
$kat[$nid] = array('name' => $r['news_title'],'categories' => array());
Sprawdzasz $categories, wpisujesz do $kat....

Tak da sie to zrobic, tylko trzeba pisac uwaznie.

[sledzik]

No ok, literowka po ciaglych modyfikacjach.. lecz to nie to
Jest tak:
tytul newsta (liczba kategori: 1)
Kategoria o naz Ważne i id 1

tytul newsta (liczba kategori: 2)
Kategoria o naz Ważne i id 1
Kategoria o naz Ciacho i id 2

tytul newsta (liczba kategori: 3)
Kategoria o naz Ważne i id 1
Kategoria o naz Ciacho i id 2
Kategoria o naz Grzybki i id 3

a powinno byc tak:
tytul newsta (liczba kategori: 3)
Kategoria o naz Ważne i id 1
Kategoria o naz Ciacho i id 2
Kategoria o naz Grzybki i id 3

jesli to dobrze rozumiem

i teraz tak?
wedlug mnie jest tak gdyz

ta linai wykonuje skrypt x razy(tyle ile jest kategorii)

[PHP] pobierz, plaintext 
foreach ($kat as $idCat => $category){ 
[PHP] pobierz, plaintext 

ale z tej petli nie moge wyjsc bo ta bardziej wenetrzna przestanie dzialac

Ten post edytował sledzik 2.02.2016, 23:01:24

[nospor]

skoro chcesz grupowac po newsach to masz odwrocic sytuacje niz ja podalem w arcie. Ja ta grupowalem po kategoriach.

[sledzik]

przeciez juz newsy sa nadrzedne to nie w tym rzecz? u ciebie jest tak jakby produkt podrzebny czyli moj news, czyli syturacja tak jakby juz jest odwrocona..?

[nospor]

W kodzie co podales grupujesz po category a nie po newsach. RObisz dokladnie tak jak mam w arcie. A ja ci mowie ze masz zrobic na odwrot. masz grupowac po newsach i do nich dodawac kategorie

[sledzik]

oO , pominimy tamte kody

Ten kod jest wedlug mnie odwrocony, juz inaczej nie umiem go odwrocic, a zwraca to co w poscie powyzej...?

[PHP] pobierz, plaintext 
$newst = array();
 
while($r = mysqli_fetch_assoc($result)) {
 
    //najpierw pozyskamy niezbędne dane
 
 
  $nid = $r['news_id'];
 
  //jeśli nie było jeszcze danego newsa, to ją tworzymy
  if (!isset($newst[$nid]))
    $newst[$nid] = array('name' => $r['news_title'], 'categories' => array());
 
  //dodajemy do newsa kolejne katgorie
  if (!empty($r['categories_name'])) //jeśli istnieje news
    $newst[$nid]['categories'][] = array('name' => $r['categories_name'], 'id'=>$r['categories_id']);
 
 
    	//echo'<pre>';
    //print_r($newst); //do obejrzenia jak wygląda wygenerowana tablica 
    	//echo'</pre>';
 
 
    //a teraz wygenerujemy stronę na podstawie uzyskanych danych
echo '<ul>';
foreach ($newst as $id => $newsy){ //petla, która leci po newsach
  echo '<li>'.$newsy['name'].' (liczba kategori: '.count($newsy['categories']).')<ul>';
  foreach ($newsy['categories'] as $cat){ //pętla, która leci po kategoriach w newsach
    echo '<li>Nazwa  <b>'.$cat['name'].'</b> i id <b>'.$cat['id'].'</b></li>';
  }
  echo '</ul></li>';
}
echo '</ul>'; 
[PHP] pobierz, plaintext 

[nospor]

Kod wyswietlajacy ma byc poza petla WHILE a nie w petli WHILE

[sledzik]

No oczywiscie.. jak ja kocham takie bledy, az sie plakac chce.. Dzieki!

ostatnie pytanie
skoro to ma byc za petla:

[PHP] pobierz, plaintext 
   foreach ($newst as $id => $newsy){ //petla, która leci po newsach
  foreach ($newsy['categories'] as $cat){ //pętla, która leci po kategoriach w newsach
    echo '<td>Nazwa'.$cat['name'].'</td>';
  }
[PHP] pobierz, plaintext 

to jak to wcisnac w kod wyswietlajacy podany wczesniej? chodzi o:

[PHP] pobierz, plaintext 
    if(mysqli_num_rows($result) > 0) {		
 
    	echo '<div class="table-responsive"><table class="table table-hover">';
    	echo"<tr><thead>";
 
    	echo'<th>KATEGORIA <a href="m.php?s=news&a=v&c=s';
    	if(isset($_GET['page'])) echo '&page='.$_GET['page'];
    		echo'" ><img src="images/s.png" alt="sort" /></th>';
    		echo"</thead></tr>";
 
    		while($r = mysqli_fetch_assoc($result)) {
    			echo"<td><span class='rama label' style='color:#".$r['categories_color_text']."; background-color:".$r['categories_color_bg'].";'>".$r['categories_name']."</span></td>";
    			echo "</tr>";
    		}
    echo "</table></div>";
    }
[PHP] pobierz, plaintext 

tak jak powyzej kod jest pod tabela

Ten post edytował sledzik 3.02.2016, 00:13:08

[nospor]

Najpierw przygotowujesz dane do wyswietlania, potem dopiero dane wyswietlasz. To sie nazywa oddzielenie logiki od widoku

[sledzik]

Nie da sie tego jakos prosciej zrobic zapytaniami sql?(dzielac na dwa zapytania)?
Nie wiem jak z wydajnoscia potem tego..

Ten post edytował sledzik 3.02.2016, 12:07:59

[nospor]

Czytamy jeszcze raz ze zrozumieniem:
najpierw logika, potem wyswietlanie. Tak sie robi w normalnych aplikacjach. poczytaj o MVC jak mi nie wierzysz.

[sledzik]

wierze, wierze.. tylko MVC to kod obiektowy , a mi wystarczy napisac to strukturalnie

[nospor]

ja ci nie karze tego pisac jako obiektowka. Ja ci tylko mowie, bys zaporzyczyl dobre zachowanie, ktore jest:
NAJPIERW LOGIKA, POTEM WYSWIETLANIE
Mam ci to poraz czwarty jeszcze pisac?

[sledzik]

Nie spokojnie (IMG:style_emoticons/default/smile.gif)
bardzo cenie twoje rady, lecz nie zawsze umiem je zastosowac...

wymoncilem cos takiego(ponizej kod):

Wyswietla sie fajnie, ale problem jest taki, ze gdy np aktywuje lub dezaktywuje newsa

[PHP] pobierz, plaintext 
if ($newsy['active']=='0'){
		echo'<td>
		<a href="admin.php?s=news&a=akt&id='.$nid.'" id="js-news-aktywny" data-original-title="Przyklej" data-toggle="tooltip" type="button" class="btn btn-xs btn-danger js-are-you-sure"><i class="fa fa-fw fa fa-close"></i></a>
		</td>';
[PHP] pobierz, plaintext 

to zawsze korzysta z tego samego id . Czyli zawsze ten sam news..

[PHP] pobierz, plaintext 
if(mysqli_num_rows($result) > 0) {		
$newst = array();
while($r = mysqli_fetch_assoc($result)) {
 
  $nid = $r['news_id'];
 
  //jeśli nie było jeszcze danego newsa, to ją tworzymy
  if (!isset($newst[$nid]))
    $newst[$nid] = array(
'users' =>$r['users_login'],
'name' => $r['news_title'],
'date_add' =>$r['news_date_add'],
'date_end' =>$r['news_date_end'],
'display' =>$r['news_display'],
'active' =>$r['news_active'], 
'stick' =>$r['news_stick'],
'categories' => array());
 
  //dodajemy do newsa kolejne katgorie
  if (!empty($r['categories_name'])) //jeśli istnieje news
    $newst[$nid]['categories'][] = array('bg' => $r['categories_color_bg'], 'color' => $r['categories_color_text'], 'name' => $r['categories_name'], 'id'=>$r['categories_id']);
 
 
    	/* echo'<pre>';
    print_r($newst); //do obejrzenia jak wygląda wygenerowana tablica 
    	echo'</pre>'; */
 
	}	
 
 
   //a teraz wygenerujemy stronę na podstawie uzyskanych danych
echo '<table class="table">
    <thead>
        <tr>
            <th>LOGIN</th>
            <th>TYTUL</th>
            <th>KATEGORIA</th>
            <th>DATA ADD</th>
			<th>DATA END</th>
			<th>WYŚ</th>
        </tr>
    </thead>
	    <tbody>
        ';
foreach ($newst as $id => $newsy){ //petla, która leci po newsach
  echo '<tr>
  <td>'.$newsy['users'].'</td>
  <td>'.$newsy['name'].'</td><td>';
  foreach ($newsy['categories'] as $cat){ //pętla, która leci po kategoriach w newsach
	echo"<span class='rama label' style='color:#".$cat['color']."; background-color:".$cat['bg'].";'>".$cat['name']."</span>";
  }
  echo '</td><td>'.$newsy['date_add'].'</td>
		</td><td>'.$newsy['date_end'].'</td>
		</td><td>'.$newsy['display'].'</td>
		';
 
if ($newsy['active']=='0'){
		echo'<td>
		<a href="admin.php?s=news&a=akt&id='.$nid.'" id="js-news-aktywny" data-original-title="Przyklej" data-toggle="tooltip" type="button" class="btn btn-xs btn-danger js-are-you-sure"><i class="fa fa-fw fa fa-close"></i></a>
		</td>';
		}
		if ($newsy['active']=='1'){
		echo'<td>
		<a href="admin.php?s=news&a=dezakt&id='.$nid.'" id="js-news-dezaktywny" data-original-title="Odklej" data-toggle="tooltip" type="button" class="btn btn-xs btn-success js-are-you-sure"><i class="fa fa-fw fa fa-check"></i></a>
		</td>';
		}
 
 
 
 
 
}
echo '</tr>
    </tbody>
</table>'; 
 
 
}
[PHP] pobierz, plaintext 

[nospor]

Jak to mowia: znajdz 10 roznic
$id
$nid

(IMG:style_emoticons/default/wink.gif)

A na przyszlosc patrz jak nazywasz zmienne w roznych miejscach (IMG:style_emoticons/default/tongue.gif)

[sledzik]

dalo sie to tez rozwiazac tak:
'id' =>$r['news_id'],

(IMG:style_emoticons/default/tongue.gif)

ale chyba moje rozwiazanie jest bardziej na okretke, i oczywiscie zbedne , dziekuje.


a nid wziolem z

$nid = $r['news_id'];

i gupio myslalem ze to zaskoczy..

Ten post edytował sledzik 3.02.2016, 13:51:31

[nospor]

No skoro id masz w index, to wystarczy sie wlasnie do niego dobrac. Ale trzeba pamietac jak sie nazwalo zmienna sklerotyku jeden (IMG:style_emoticons/default/tongue.gif)