Forum PHP.pl

Witam tworze obecnie cms na własne potrzeby i mam problem mam taki kod

[PHP] pobierz, plaintext 
    while($r = mysql_fetch_assoc($wynik)) {
	echo('
<div class="n">
<div class="nt">');
$tytul=$r['news_tytul'];
$tytul=htmlspecialchars($tytul);
 
        echo ('<h1>');
		echo ($tytul);
		echo ('</h1>');
		echo "".$r['news_skrot']."";
		echo ('</div><div class="nb">Napisał:');
		echo "".$r['news_autor']."";
		echo(' | dn.');
		echo "".$r['news_data']."";
		echo('<div><a href="newsy.php?id=');
		echo "".$r['news_id']."";
		echo('">[czytaj więcej/komentuj]</a></div></div></div>');
    }
[PHP] pobierz, plaintext 

tojest pętla wyświetlająca newsy
tylko ze jak ktos wpisze wnewsie html np </div> to strona layout siś sypie na łeb na szyje;;] chcialbym temu zapobiec;]

w powyzszym kodzie tytul przelecialem chtmlspecjalcharts lecz nadal html sie wykonuje
jak zrobic do tego bbcody?
zeby bylo np tekst



edit udało mi sie juz zrobic zeby mi sie nieinterpretwal html tylko ze znaczniki html się wyświetlają tym razem;] chcialbym zrobic do tego bbcode:P

Napisz bibliotekę do obsługi BBCode lub zainstaluj jakąś gotową. PHP nie ma wbudowanej takiej funkcjonalności domyślnie (jest w PECL-u moduł, ale trzeba go samodzielnie skompilować i na 99% Twój serwer nie będzie go mieć). Zazwyczaj BBCode parsuje się przy wstawianiu treści do bazy danych, a nie przy wyświetlaniu, ze względów wydajnościowych. Gdy chcemy edytować, albo treść źródłowa jest zapisywana obok (skompresowana), albo stosuje się parser odwrotny, który HTML konwertuje z powrotem na BBCode.

Nawiasem mówiąc popracuj jeszcze nad jakością kodu.

[PHP] pobierz, plaintext 
$tytul=$r['news_tytul'];
$tytul=htmlspecialchars($tytul);
 
        echo ('<h1>');
		echo ($tytul);
[PHP] pobierz, plaintext 

Pamiętaj, że można zrobić następującą rzecz:

[PHP] pobierz, plaintext 
echo '<h1>'.htmlspecialchars($r['news_tytul']).'</h1>';
[PHP] pobierz, plaintext 

Zmienne tymczasowe powinno się wprowadzać tylko wtedy, gdy są one nam naprawdę z jakichś względów potrzebne. Ty tymczasem wykonujesz kupę przypisań zupełnie bez sensu - nie ma to żadnego znaczenia praktycznego. Klepiesz jedynie za dużo kodu i dokładasz niepotrzebną robotę interpreterowi.

Tak na marginesie: echo nie jest funkcją (odsyłam do dokumentacji), więc nawiasy są niepotrzebne.

ja jestem początkującym koderem to sa moje pierwsze działające skrypty. Za krytykę dziekuje i pozdrawiam rozwiązałem ten problemz html tylko to bbcode mnie meczy;/