[PHP][MySQL]Blad w zapytaniu sql Opcje

[eminiasty]

Witam, mam pewnien problem dotyczacy zwracania wynikow w paginacji, kod paginacji jest tu:(raczej samego przetwarzania)

[PHP] pobierz, plaintext 
   $result= mysqli_query($link,$query) or die(mysqli_error());
        $row = mysqli_fetch_array($result);
        extract($row);
 
		$onpage = 10; 
        $navnum = 7;
        $allpages = ceil($all_posts/$onpage); 
 
		if(!isset($_GET['page']) or $_GET['page'] > $allpages or !is_numeric($_GET['page']) or $_GET['page'] <= 0){
                $page = 1;
        }else{
                $page = $_GET['page'];
        }
        $limit = ($page - 1) * $onpage;
[PHP] pobierz, plaintext 

Zapytanie wyglada nastepujaco:
Problem polega na tym, iż dane sa zwracana na podstawie kategori a nie newsow. Czyli jak powinno byc
$onpage = 10; newsow na stronie to wyswietla ich np 2 jesli sa 2 po 5 kategori w jednym. Jak przerobic to zapytanie by wyswietlal po newsach, a nie po kategoriach?
Probowalem GROUP BY news_id lecz to nie w tym rzecz bo potem sa kategorie nie wyswietlane ..;/

[PHP] 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 
 
		LIMIT $limit, $onpage";
[PHP] pobierz, plaintext 

Zauwazylem ze glowny blad jest w

[PHP] pobierz, plaintext 
		JOIN ".$prefix."news_categories 
			 ON ".$prefix."news.news_id = ".$prefix."news_categories.news_id
[PHP] pobierz, plaintext 

bo przyrownuje jednego newsa (jedno id) do kilku kategori czyli tego samego id , tylko nie wiem jak to zmienic zeby bylo poprawnie..;/


Ten post edytował eminiasty 4.02.2016, 13:12:18

[com]

".$prefix."news.*,".$prefix."users.*,".$prefix."news_categories.*,".$prefix."categories.* już Ci mówiłem zastąp to *, bo to niczego nie daje a tylko zaciemnia wydłuża niepotrzebnie kod, czemu masz potrójnego joina?

Ten post edytował com 4.02.2016, 13:14:58

[eminiasty]

[PHP] pobierz, plaintext 
już Ci mówiłem zastąp to *
[PHP] pobierz, plaintext 

tzn? przeciez jest *?

1 join - z tabeli users wyciaga loginy(zeby byl wyswietlany potem)

[PHP] pobierz, plaintext 
JOIN ".$prefix."users 
		 ON ".$prefix."news.users_id = ".$prefix."users.users_id
[PHP] pobierz, plaintext 

2 join - potrzebny jest w sumie zeby zrobic 3 joina(chyba) , bo potrzebuje miec powiazanie z categoreis

[PHP] pobierz, plaintext 
JOIN ".$prefix."news_categories 
	 ON ".$prefix."news.news_id = ".$prefix."news_categories.news_id
[PHP] pobierz, plaintext 

3 join - nazy kategori sa trzymane w tabeli categories wiec musze je jakos pobrac? id kategori do newsa jest przypisywane w news_categories wiec stad chyba to potrzebne

[PHP] pobierz, plaintext 
 JOIN ".$prefix."categories 
			ON ".$prefix."categories.categories_id = ".$prefix."news_categories.categories_id  
[PHP] pobierz, plaintext 

[com]

tak ale cale to ".$prefix."news.*,".$prefix."users.*,".$prefix."news_categories.*,".$prefix."categories.*" wystarczyło by zapisać jako * i efekt jest ten sam, ok no to musisz jeszcze zgrupować te wyniki odpowiednio, poczytaj o group by (IMG:style_emoticons/default/wink.gif)

[eminiasty]

pisalem ze probowalem je grupowac
np. tak

[PHP] pobierz, plaintext 
GROUP BY ".$prefix."news_categories.news_id
[PHP] pobierz, plaintext 

lecz rodza sie dwa problemy
1. nie wyswietla wszystkich newsow
2. po zgrupowaniu nie wyswietla wszystkich kategori dla danego newsa

[com]

zaraz zaraz, z tego co widzę wyświetla te kategorie do których należą newsy, wiec jak nalezą do 2 np to nie wyświetli ich 5 przecież

[eminiasty]

Dokladnie, a powinno chyba je wyswietlac.
W sesnie ze jeden newst ma dwie kategorie to je wyswietla przy nim. nie osobno.
To wyglada tak:

bez grupowania
(IMG:http://iv.pl/images/55276996056448780391_thumb.jpg)

z grupowaniem
(IMG:http://iv.pl/images/12197208131381307762_thumb.jpg)

Chce zeby bylo jak na pierwszym tylko nie zliczalo po ilosci kategori na stronie tylko ilosci postow.

Ten post edytował eminiasty 4.02.2016, 13:47:57

[com]

on niczego nie zlicza, bo dalej masz 10 rekordów tylko zgrupowanych wiec na stronie robi Ci się ich np 5. Policz sobie ile masz katergorii, bo tyle jest rekordów, czyli 10 (IMG:style_emoticons/default/smile.gif)

[eminiasty]

Ja wiem jak wyglada problem, nie umeim go tylko rozwiazac bo grupowanie nie dziala chyba..

To co mowisz jest prawda.
Tylko nie chce zeby na pierwszej stronie bylo 5 a na drugiej juz 6 nesow, a na 3 juz 7 np.
Prosze o pomoc w rozwiazaniu tej sytuacji.

[com]

no to musisz ograniczyć wyświetlanie w inny sposób, powiedzmy przyjac pesymistyczny przypadek czyli 10xkategoria, a podział na strony zrobić w samym php (IMG:style_emoticons/default/smile.gif)

[eminiasty]

rzuca mi sie tylko slowo jak?
W ktorym miejscu takie ograniczegnia powinny byc umieszczone? to powinno byc zrobione w postaci ifa czy inaczej?

no to musisz ograniczyć wyświetlanie w inny sposób, powiedzmy przyjac pesymistyczny przypadek czyli 10xkategoria

czy chodzi ci o to aby ilosc wyswietlanych posotw na stronie podniesc np do 100? co w przypadku postow z kiludziesieciowam kategoriami zmniejszy liczebnosc? przeciez to nie rozwiazuje problemu, pozatym kazda strona bedzie zawiaerac inna liczbe postow

to jest moje wyswietlanie:

[PHP] pobierz, plaintext 
....
   //wyswietlanie danych
echo '<table class="table">
    <thead>
        <tr>';
        echo'<th>LOGIN </th>';
            echo'<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>
		';	
		echo'<td>
		<a href="mod.php?s=news&a=czytaj&id='.$id.'" data-original-title="Czytaj" data-toggle="tooltip" type="button" class="btn btn-xs btn-success"><i class="fa fa-fw fa fa-comment"></i></a>
		</td>';
		echo'<td>
		<a href="mod.php?s=news&a=edytuj&id='.$id.'" data-original-title="Edytuj" data-toggle="tooltip" type="button" class="btn btn-xs btn-primary"><i class="fa fa-fw fa fa-pencil-square-o"></i></a>
		</td>'; 
		echo'<td>
		<a href="mod.php?s=news&a=usun&id='.$id.'" id="js-news-usun" data-original-title="Usuń" data-toggle="tooltip" type="button" class="btn btn-xs btn-danger js-are-you-sure"><i class="fa fa-fw fa fa-trash"></i></a>
		</td>'; 			
 
}
echo '</tr>
    </tbody>
</table>'; 
 
}
[PHP] pobierz, plaintext 

Ten post edytował eminiasty 4.02.2016, 15:14:56

[com]

chodzi o to żeby pobrać x rekordów z bazy, a ograniczenie i podział na strony nie robić przez limit a poprzez ograniczenie w tablicy przy wyświetlaniu. Ale ze względu na to że rekordów może być sporo dlatego sugerowałem limit ilość kategorii razy ilość elementów, których niech jest 5 zamiast 10 np.

[eminiasty]

ok ok
chodzi ci o wywalnie LIMIT $limit, $onpage
tylko ze o to opiera sie paginacja ktora wyswietla newsy na kolejnych stronach.

[com]

tak, i te paginacje zrób w php nie poprzez bazę, limit daj ale np 5 rekordów x 5 kategorii (IMG:style_emoticons/default/wink.gif) dodasz warunek modulo i tam ograniczysz na strony (IMG:style_emoticons/default/smile.gif)

[eminiasty]

fajnie niby rozumiem.. ale nie wiem jak zrobic paginacje bez uzycia LIMIT i jak ten LIMIT zaimplementowac w kod php, najsmieszniejsze jest to ze kiedy nie uzywalem JOINow to wszystko bylo ok.