system newsów - kategorie Opcje

[xxdrago]

Witam, ja się chciałem zapytać czy to jest dobry pomysł z kategoriami newsów najpierw dam tabele:

Nazwy kategorii, id , typ

[SQL] pobierz, plaintext 
CREATE TABLE `kategorie` (
  `kategoria_id` int(11) NOT NULL AUTO_INCREMENT,
  `kategoria_nazwa` text collate utf8_polish_ci NOT NULL, 
  `typ` int(11) NOT NULL DEFAULT '0',
  KEY `kategoria_id` (`kategoria_id`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=3 ;
[SQL] pobierz, plaintext 

W tej tabeli mam newsy:

[SQL] pobierz, plaintext 
CREATE TABLE `newsy_strony` (
  `id` smallint(6) NOT NULL AUTO_INCREMENT,
  `nazwa` text collate utf8_polish_ci NOT NULL,
  `data` datetime NOT NULL DEFAULT '0000-00-00 00:00:00',
  `autor` varchar(255) collate utf8_polish_ci NOT NULL DEFAULT '',
  `tresc` text collate utf8_polish_ci NOT NULL,
  `typ` text collate utf8_polish_ci NOT NULL,
  `kategoria_id` int(11) NOT NULL,
  `link` text collate utf8_polish_ci NOT NULL,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=127 ;
[SQL] pobierz, plaintext 

`kategoria_id` int(11) NOT NULL, - id kategorii.

Myślałem sobie zrobić tak żeby była możliwość, kilku kategorii dla jednego newsa, więc zastosowałbym funkcje explode.

A przykładowy rekord z kategoria_id wyglądałby tak:

Kod

5,7,3

Potem funkcją

[PHP] pobierz, plaintext 
 explode(','$rekord['kategoria_id'])
[PHP] pobierz, plaintext 

bym to ładnie podzielił do tabeli, następnie wyszukałbym w bazie nazwy kategorii, po id. Czy to jest doby pomysł?

[Adi32]

Łamiesz tym zasadę atomowości.
Powinieneś tutaj zastosować relację wiele do wielu (czyli zrobić tabelę pośrednią "id_news,id_kategoria").

[Crozin]

OT: @Adi32: http://en.wikipedia.org/wiki/Atomicity_(database_systems)

[xxdrago]

Zrobiłem tak jak mówicie, zrobiłem sobie tabele:

Co tutaj można by było z optymalizować?

[PHP] pobierz, plaintext 
if (is_null($news)) {
    $dane = 'SELECT * FROM  `newsy_strony` WHERE typ =1 ORDER BY `data` DESC limit ' . $start . ',' . ($end - $start + 1);
    $sql = $db->query($dane);
 
 
    while ($newss = $sql->fetch_assoc()) {
        $bb = new BbCode();     // Dodanie BB-CODE
        $bb->parse($newss['tresc']); // Prasowanie
        $length = 500; // Sktracanie tekstu...
        $cutText = $bb->cutText($length);
 
        // Ilosc komentarzy
        $sql_ilosc = 'SELECT COUNT( * ) FROM komentarze WHERE id =' . $newss['id'];
        $sql_iloscc = $db->query($sql_ilosc);
        $newse = $sql_iloscc->fetch_assoc();
 
        // Kategoria
        $kategoria_sql = 'SELECT *  FROM  `kategorie_newsy` where news_id=' . $newss['id'];
        $kategoria = $db->query($kategoria_sql);
        $kategoriaa = $kategoria->fetch_assoc();
 
		if (empty($kategoriaa['kategoria_id']))
		{
		$id_k =1;
		}
		else
		{
		$id_k = $kategoriaa['kategoria_id'];
		}
 
		// Zap2
		$kategoria2_sql = 'SELECT *  FROM  `kategorie` where kategoria_id=' . $id_k;
        $kategoria2 = $db->query($kategoria2_sql);
        $kategoriaa2 = $kategoria2->fetch_assoc();
 
 
 
// To poniżej dodajemy co cache
 
        $news[] = array(
            'nazwa' => znaki('znaki', $newss['nazwa']),
            'id' => $newss['id'],
            'link' => $newss['link'],
            'tresc' => $cutText,
            'autor' => $newss['autor'],
            'data' => $newss['data'],
            'komentarze' => $newse['COUNT( * )'],
			'kategoria' => $kategoriaa2['kategoria_nazwa'],
        );
    }
    $cache->Put('news' . $actualPage, $news, '@queries');
}
[PHP] pobierz, plaintext 

[wujek2009]

Nie wyświetlaj od razu newsów tylko zapisz je wszystkie do tablicy najpierw.
Czyli przerób pętle z #6 linijki na mniej więcej taki zarys;

[PHP] pobierz, plaintext 
$newsy = $id_newsow = array();
 
while ($newss = $sql->fetch_assoc()) {
  $newsy[] = $newss;
  $id_newsow[] = $newss['news_id']; // przekaż id newsa
}
[PHP] pobierz, plaintext 

Po co? Bo wyświetlając 15 newsów na stronę powielasz 15x zapytanie z linijki 18 a można to zrobić mniej więcej tak:

[PHP] pobierz, plaintext 
 
$newsy = $id_newsow = array();
 
while ($newss = $sql->fetch_assoc()) {
  $newsy[] = $newss;
  $id_newsow[] = $newss['news_id']; // przekaż id newsa
}
 
if ( sizeof($newsy) )
{
   // sa newsy
 
 
// dla $id_newsow możesz zastosować też array_unique aby usunęło z tablicy te same ID do pobrnaia
// bo po co ma pobierać 3x kategorie o ID = 19?
$pobierzKategorie = "select * from bla bla where ID_KATEGORI IN(". implode(',', $id_newsow) .")";
// i teraz
$kategorie_row = array();
 
foreach ( $pobierzKategorie as $k => $v )
	$kategorie_row[$v['id_kategori']] = $v;
 
// wyświetlamy teraz newsy:
 
foreach ( $newsy as $k => $v )
{
   $tytul_newsa = $v['title']; // taki przykład;
   $nazwa_kategori = isset($kategorie_row[$v['id_kategori']]) ? $kategorie_row[$v['id_kategori']]['nazwa'] : NULL;
 
   echo 'tytuł: '. $tytul_newsa .' jest w kategori: '. $nazwa_kategori;
}
 
}
else
die('nie ma newsów');
[PHP] pobierz, plaintext 

co można jeszcze zoptymalizować? zapytanie z #13 linjki.
otwórz w tabeli newsów kolumnę "liczba_komentarzy" i gdy ktoś doda nowy komentarz to rób zwykłe UPDATE +1 (aby liczba się zwiększała); oczywiście przy usuwaniu komentarzy to -1 :-)

i wtedy te zapytanie z #13 kompletnie to wyrzucenia.

// pisałem z palca, nie analizowałem kolumn z Twojej bazy.

Ten post edytował wujek2009 7.04.2012, 16:23:23

[xxdrago]

Wiesz co, patrz całość wygląda tak:

[PHP] pobierz, plaintext 
<?php
 
# --- Dodajemy naglowki --- #
ob_start();             // Buforowanie
error_reporting(E_ALL); // Raportowanie bledow
session_start();        // Sesja
# --- Odpalamy odpowienie pliki --- #
require('includes/BbCode.php');      // BB-CODE
require('includes/function.php');    // Funkcje
require('includes/template.php');    // Klasa szablonow
require('config.php');               // Plik konfiguracyjny
require('includes/Pager.class.php'); // Pager
require('includes/Cache.class.php');
require('includes/CacheFileDriver.class.php');
# --- Klasy --- #
$bb = new BbCode();                 // BB-CODE
$tpl = new Template($sciezka_styl); // Szablon
$db = lacz();                       // Baza danych MYSQL
$cache = new Cache();
$ile_komentarzy = new ile_komentarzy();
 
/*
 
SELECT newsy_strony.id,newsy_strony.nazwa,newsy_strony.data,newsy_strony.autor,newsy_st
rony.tresc,newsy_strony.typ,newsy_strony.link, COUNT( komentarze.id ) ilosc
FROM newsy_strony
LEFT JOIN komentarze ON komentarze.id = newsy_strony.id
GROUP BY newsy_strony.id
 
*/
$cache->AddDriver('fileDriver', new CacheFileDriver('Cache'));
 
$cache->AddGroup('@queries', array('lifetime' => 300, 'driver' => 'fileDriver', 'hashid' => true));
# --- Konfiguracja Templeate ---#
$nazwa_plikuu = 'index';
$rozszezeniee = '.html'; // Rozszezenie HTML/TPL
$nazwa_Podstrony = 'Strona Główna';
$nastronie = 5;
# --- Skrypt --- #
$tpl->load_file($nazwa_plikuu . $rozszezeniee, $nazwa_plikuu); // Ladujemy plik
//Obiekt nalezy do grupy queries
// --- Cache PAGER
 
$pager = $cache->Get('pager', '@queries');
if (is_null($pager)) {
    $sql_pager = $db->query('SELECT count( * ) FROM newsy_strony where typ=1');
    $row = $sql_pager->fetch_array();
    $pager = array();
    $pager['ilosc_rekordow'] = $row['0'];
 
    $cache->Put('pager', $pager, '@queries');
}
// $pager['ilosc_rekordow']; - Zwraaca ilosc rekordow
// --- Cache pager 
 
if (komentarze_on_off() == true) {
    $komentarze = 1;
} else {
    $komentarze = 0;
}
$pagers = new Pager('id');
$pagers->SetTotalRecords($pager['ilosc_rekordow']);
$pagers->SetRecordsPerPage($nastronie);
$pagers->SetPagesPerNav(3);
$pagers->Make(true);
$pag = $pagers->Render();
$actualPage = $pagers->GetActualPage(); //tu aktualna strona 
$start = $pagers->GetIndexRecordStart();
$end = $pagers->GetIndexRecordEnd();
 
 
$news = $cache->Get('news' . $actualPage, '@queries');
 
if (is_null($news)) {
    $dane = 'SELECT * FROM  `newsy_strony` WHERE typ =1 ORDER BY `data` DESC limit ' . $start . ',' . ($end - $start + 1);
    $sql = $db->query($dane);
 
 
    while ($newss = $sql->fetch_assoc()) {
        $bb = new BbCode();     // Dodanie BB-CODE
        $bb->parse($newss['tresc']); // Prasowanie
        $length = 500; // Sktracanie tekstu...
        $cutText = $bb->cutText($length);
 
        // Ilosc komentarzy
        $sql_ilosc = 'SELECT COUNT( * ) FROM komentarze WHERE id =' . $newss['id'];
        $sql_iloscc = $db->query($sql_ilosc);
        $newse = $sql_iloscc->fetch_assoc();
 
        // Kategoria
        $kategoria_sql = 'SELECT *  FROM  `kategorie_newsy` where news_id=' . $newss['id'];
        $kategoria = $db->query($kategoria_sql);
        $kategoriaa = $kategoria->fetch_assoc();
 
		if (empty($kategoriaa['kategoria_id']))
		{
		$id_k =1;
		}
		else
		{
		$id_k = $kategoriaa['kategoria_id'];
		}
 
		// Zap2
		$kategoria2_sql = 'SELECT *  FROM  `kategorie` where kategoria_id=' . $id_k;
        $kategoria2 = $db->query($kategoria2_sql);
        $kategoriaa2 = $kategoria2->fetch_assoc();
 
 
 
 
        $news[] = array(
            'nazwa' => znaki('znaki', $newss['nazwa']),
            'id' => $newss['id'],
            'link' => $newss['link'],
            'tresc' => $cutText,
            'autor' => $newss['autor'],
            'data' => $newss['data'],
            'komentarze' => $newse['COUNT( * )'],
			'kategoria' => $kategoriaa2['kategoria_nazwa'],
        );
    }
    $cache->Put('news' . $actualPage, $news, '@queries');
}
foreach ($news as $baza) {
    $tpl->set_var('nazwa', $baza['nazwa']);
    $tpl->set_var('link', $baza['link']);
    $tpl->set_var('id', $baza['id']);
    $tpl->set_var('tresc', $baza['tresc']);
    $tpl->set_var('autor', $baza['autor']);
    $tpl->set_var('data', $baza['data']);
    $tpl->set_var('cat', $baza['kategoria']);
    $tpl->set_var('coments', $ile_komentarzy->ile($baza['komentarze'], 0)); // Ilosc komentarzy
    $tpl->parse("NEWS", true);
}
$tpl->set_var('pager', $pag);
 
include 'hs_header.php'; // Zalaczamy header
$tpl->pparse($nazwa_plikuu, false); // Prasujemy plik
include 'hs_stopka.php'; // Zalaczamy stopke
[PHP] pobierz, plaintext 

Zapytanie z tymi komentarzami przerobiłbym tak:

[SQL] pobierz, plaintext 
SELECT newsy_strony.id,newsy_strony.nazwa,newsy_strony.DATA,newsy_strony.autor,newsy_strony.tresc,newsy_strony.typ,newsy_strony.link, COUNT( komentarze.id ) ilosc
FROM newsy_strony
LEFT JOIN komentarze ON komentarze.id = newsy_strony.id
GROUP BY newsy_strony.id
[SQL] pobierz, plaintext 

(kategorie podobnie)
Według mnie to jest bezsensu z tym liczba_komentarzy. To co dałeś wyżej to też jest trochę bez sensu....

Ten post edytował xxdrago 7.04.2012, 16:35:43