Witam. Potrzebuje funkcji do wyświetlania news'ów na stronie. Napisałem Coś takiego

[PHP] pobierz, plaintext 
<?php
function wyswietl_art ($rodzaj='News')
{
require_once('funkcje_bazy.php');
			$wynik= wykonaj_w_bazie("select id, naglowek, tekst , utworzony from artykuly where strona = '$rodzaj' order by utworzony desc "); 
				if (!$wynik)
				{
					exit("Wystąpił błąd bazy danych.");
				}	
				else
				{							//DOPRACUJ TO!!!
					while($wiersz = mysql_fetch_array($wynik))
					{
						echo '<div class="news">';
						echo '<a href="strona.php?show=szablon&name='.$wiersz['id'].'"><p class="news_tytul">'.htmlspecialchars(stripslashes($wiersz['naglowek'])).'</p></a>';
						echo '<p class="data">'.strftime("%d.%m.%Y", $wiersz['utworzony']).'</p>';	
						echo '<p class="news_tresc">'.htmlspecialchars(substr(stripslashes($wiersz['tekst']), 0, 300)).'...';
						echo '<a href="strona.php?show=szablon&name='.$wiersz['id'].'">['.więcej.']</a></p>';		
						echo '</div>';	
					}	
					return;	
				}
}
?>
[PHP] pobierz, plaintext 

Zastanawiam się czy to poprawne rozwiązanie. Chodzi o to że w funkcji dane są już umieszczane w divie itp. Czy nie powinno być tak że funkcja zwraca same dane, a reszta (pęta & div) są juz po za nią?

A drugie pytanie dotyczy jakiegoś edytora (coś w stylu Tinny mc). Czy znacie jakieś strony lub poadniki jak to zainstalować i dołączyć tak by działało na mojej stronie?

a moze funkcja niech zwraca tablice z danymi. nie div, tabele,i inne znaczki HTML. oddziel to a potem np w smarty juz wyswetlisz.

Wszystko zależy od tego, jak rozbudowany serwis tworzysz. Budowanie według modelu warstwowego (czyli m.in oddzielenie danych od ich prezentacji) ma sens wyłącznie przy dużych aplikacjach, przy malutkich stronkach jest to przerost formy nad treścią. Model ów wprowadzono po to, by łatwiej można było połapać się w kodzie i łatwiej dokonywać wszelakich zmian. Musisz sam ocenić, czy takie rozwiązanie jak te napisane przez Ciebie będzie w przyszłości łatwe do modyfikacji czy też nie...