Forum PHP.pl

Witam,
czy ten skrypt zalicza się do skryptu zgodnego z wzorcami MVC?
Funkcje hexpass oraz safe email są w innym pliku pokazuje je tylko dla tego tematu

[PHP] pobierz, plaintext 
<?php
function safeEmail($str)
{
	$pattern = '/^[a-zA-Z0-9.-_]+@[a-z0-9]+.([a-z]+){2,4}$/';
	if (!preg_match($pattern, $str))
	{
		return false;
	}	
	else
	{
		return true;
	}
}
function hexPass($str)
{
	$str = md5('Damned'.$str.'Land');
	$str = sha1('PHP5.///-0-'.md5($str).'-0-.5PHP');
	$str = md5($str);
	return $str;
}
class LoginModel
{
	static public function getData($email, $pass)
	{
	$result=@mysql_fetch_assoc(mysql_query("SELECT id FROM players WHERE email='".safeEmail($email)."' AND pass='".hexPass($pass)."'"));
		return (!isset($result['id'])) ? false : $result;
	}
}
 
 
Class LoginView
{
	static public function getVista($tpl, $actionFile)
	{
		$tpl->assign('actionFile', $actionFile);
		$tpl->parse('login.tpl');
	}
}
 
class LoginController
{
	static public function Loging ($email, $pass)
	{
		$test=LoginModel::getData($email, $pass);
		if ($test == false)
		{
			return false;
		}	else
		{
			return $test['id'];
		}
	}
}
?>
[PHP] pobierz, plaintext 

Ten post edytował daniel1302 18.03.2008, 19:59:44

Może się mylę (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) , ale wydaje mi się, że kod, który dałeś w "widoku" powinien być w "kontrolerze". Widok to sam szablon - ja to przynajmniej tak rozumiem...

@qqrq

Nie myl widoku z systemem szablonów, widok to pewien ściśle określony interfejs dzięki któremu kontroler przekazuje dane do niego. Co widok z nimi zrobi kontrolera nic nie obchodzi. Najczęściej będzie to wykorzystanie jakiegoś systemu szablonów, np. Smarty i wygenerowanie html i wyświetlenie, ale równie dobrze może być to zapis danych do pliku xml, czy cokolwiek. (to może nie jest dokładnie to co niektórzy uważają za MVC ale tak w aplikacjach web'owych się zazwyczaj przyjmuje.)

Co do kodu to raczej autorowi doradzam naukę postaw, a potem zabawę z tworzeniem czegoś większego.
A czemu? chociażby safeEmail() zwraca boolean a jest wykorzystywane w kwerendzie jako string. (jedna z wielu rzeczy które rażą)

Właśnie się zorietowałem
$email=(safeEmail($email)==true) ? $email : 'Zły mail';

Ale chodzi mi o budowe klasy bardziej

- czy to jest ładny samochód?
- ale on nie ma kół
- oj to drobny szczegół, ale ładny?

Jak piszesz to pisz coś porządnie.