Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> Parser szablonów
akurczyk
post
Post #1





Grupa: Zarejestrowani
Postów: 201
Pomógł: 5
Dołączył: 15.07.2008
Skąd: Kłodzko

Ostrzeżenie: (10%)
X----

Napisałem coś takiego:
[PHP] pobierz, plaintext 
  1. <?php
  2. $szablon = file_get_contents("szablon.html");
  3. preg_match("#{newsy}(.*?){/newsy}#si", $szablon, $znalezione);
  4. $tablica[0]["tytul"] = "Pierwszy news";
  5. $tablica[0]["tresc"] = "Treść pierwszego newsa";
  6. $tablica[1]["tytul"] = "Drugi news";
  7. $tablica[1]["tresc"] = "Treść drugiego newsa";
  8. $tablica[2]["tytul"] = "Trzeci news";
  9. $tablica[2]["tresc"] = "Treść trzeciego newsa";
  10. foreach($tablica as $wartosc) {
  11.     $temp = str_replace("{tytul}", $wartosc["tytul"], $znalezione[1]);
  12.     $newsy_array[] = str_replace("{tresc}", $wartosc["tresc"], $temp);
  13. }
  14. $newsy = join("", $newsy_array);
  15. $szablon = preg_replace("#{newsy}(.*?){/newsy}#si", $newsy, $szablon);
  16. echo $szablon;
  17. ?>
[PHP] pobierz, plaintext


Szablon:
[HTML] pobierz, plaintext 
  1. <html>
  2. <head>
  3. <title>Newsy</title>
  4. </head>
  5. <body>
  6. {newsy}
  7. <h2>{tytul}</h2>
  8. <p>{tresc}</p>
  9. {/newsy}
  10. </body>
  11. </html>
[HTML] pobierz, plaintext


Wydaje mi się że jest to nieoptymalene, i nieprofesjonalnie napisane. Proszę o jakieś sugestje. Co powinienem w tym zmienić??
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi
Riklaunim
post
Post #2





Grupa: Zarejestrowani
Postów: 640
Pomógł: 44
Dołączył: 8.02.2004

Ostrzeżenie: (0%)
-----

Pomyśl trochę. Jeżeli pobiera wszystkie fora to znaczy że warunek WHERE jest błędny. Zakładając że masz relację jeden do wielu w tabeli for powinna być kolumna z numerem ID danej kategorii - i wstawiając w warunku where id obecnie przemiatanej kategorii musi zadziałać (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Dla przykładu fragment struktury mojego forum:
Kod
CREATE TABLE IF NOT EXISTS `rk_category1` (
  `id` int(11) NOT NULL auto_increment,
  `name` varchar(255) NOT NULL,
  `order` smallint(5) unsigned NOT NULL,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8 AUTO_INCREMENT=1;

CREATE TABLE IF NOT EXISTS `rk_forum1` (
  `id` int(11) NOT NULL auto_increment,
  `category_id` int(11) NOT NULL,
  `name` varchar(255) NOT NULL,
  `description` varchar(255) NOT NULL,
  `topics` int(10) unsigned NOT NULL,
  `posts` int(10) unsigned NOT NULL,
  `lastposter` varchar(255) default NULL,
  `lasttopic` varchar(255) default NULL,
  `modification_date` datetime NOT NULL,
  `order` smallint(5) unsigned NOT NULL,
  `use_prefixes` tinyint(1) NOT NULL,
  `allow_anonymous` tinyint(1) NOT NULL,
  `use_moderators` tinyint(1) NOT NULL,
  PRIMARY KEY  (`id`),
  KEY `rk_forum1_category_id` (`category_id`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8 AUTO_INCREMENT=1;

Tabela rk_forum1 ma kolumnę "category_id" przechowującą wartość "id" kategorii (rk_category1), do której jest przypisane (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
Go to the top of the page
+Quote Post

Posty w temacie
- akurczyk   Parser szablonów   21.12.2008, 18:19:21
- - Riklaunim   Napisz to w Javie to będzie bardzo profesjonalne ...   21.12.2008, 18:41:25
- - l0ud   Istota działania jest zła. Systemy szabl...   21.12.2008, 18:53:39
|- - Riklaunim   Cytat(l0ud @ 21.12.2008, 20:53:39 ) I...   21.12.2008, 19:07:16
- - akurczyk   A co z optymalnością mojego skryptu??   21.12.2008, 19:18:13
- - Riklaunim   Duży ruch ci raczej nie grozi, więc i trudno mówić...   21.12.2008, 19:28:38
- - l0ud   [PHP] pobierz, plaintext <?phpIMHO obecność jęz...   21.12.2008, 19:33:34
- - akurczyk   Jeszcze jedno: Załużmy że w szablonie mam: Kod...   21.12.2008, 19:44:27
|- - Riklaunim   Cytat(l0ud @ 21.12.2008, 21:33:34 ) W...   21.12.2008, 20:11:46
- - Crozin   [PHP] pobierz, plaintext <?php}?>[PHP] ...   21.12.2008, 20:26:17
- - akurczyk   @Crozin: Fajne rozwiązanie, spróbuję. Wrućmy do mo...   21.12.2008, 21:25:38
|- - Riklaunim   Cytat(akurczyk @ 21.12.2008, 23:25:38...   21.12.2008, 22:28:43
- - Mize   Jeślibyś używał czystego PHP w szablonach to byłob...   21.12.2008, 21:37:18
- - Mize   To co wyżej napisałem to był przykład, i chyba nie...   21.12.2008, 22:38:44
- - l0ud   Riklaunim, nikt nie miał na myśli całkowitego unik...   21.12.2008, 22:43:15
- - Riklaunim   CytatOk, piszesz że szablony są kompilowane do czy...   21.12.2008, 23:00:03
- - akurczyk   Zroiłem tak jak mi radziliście. Fajnie t...   21.12.2008, 23:03:59
- - Crozin   CytatBo jak np. pisałem firmowego CRMa to osoba od...   21.12.2008, 23:06:11
|- - Riklaunim   Cytat(Crozin @ 22.12.2008, 01:06:11 )...   21.12.2008, 23:35:38
- - akurczyk   POMOCY!!   21.12.2008, 23:18:29
- - erix   CytatMam problem, więcej w drugim zapytaniu (dzies...   21.12.2008, 23:31:25
- - akurczyk   Niema żadnego błądu, tylko niewiem co napisać zoba...   21.12.2008, 23:59:00
- - Riklaunim   Coś jakby musisz umieścić zapytania o fora z danej...   22.12.2008, 00:17:54
- - akurczyk   http://www.codemaster2222.yoyo.pl/skrypt.php - nie...   22.12.2008, 10:47:23
- - Riklaunim   Pomyśl trochę. Jeżeli pobiera wszystkie fora to zn...   22.12.2008, 12:35:00
- - akurczyk   Jak to ma działeć to ja wie, przecież jak robiłem ...   22.12.2008, 12:52:56
- - Riklaunim   Po wierszu z $zapytanie2 możesz dać $tab...   22.12.2008, 13:52:17

« Następny starszy · PHP · Następny nowszy »
 

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Przełącz na: Standardowy · Przełącz na: Linearny+ · Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Aktualny czas: 28.12.2025 - 05:26
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn