zwracanie (wartości?) przez funkcję Opcje

[bliitz]

Witam
Powiedzmy że jest przykładowa funkcja w postaci:

[PHP] pobierz, plaintext 
<?php
function aktualizujTabele($nazwaKlubu, $punkty, $punktyZdobyte,
						   $punktyStracone,$kolejka, $idKlubu) {
	global $lacz;
	$nazwaKlubu = safeOutput($nazwaKlubu);
	$punkty = safeOutput($punkty);
	$punktyZdobyte = safeOutput($punktyZdobyte);
	$punktyStracone = safeOutput($punktyStracone);
	$kolejka = safeOutput($kolejka);
	$idKlubu = intval($idKlubu);
 
	if (!preg_match('/^-?[0-9]+$/', $punkty)) {
		return 0;
	} elseif (!preg_match('/^[0-9]+$/', $punktyZdobyte))	{
		return 0;
	} elseif (!preg_match('/^[0-9]+$/', $punktyStracone)) {
		return 0;
	} elseif (!preg_match('/^[0-9]+$/', $kolejka)) {
		return 0;
	} elseif (!$nazwaKlubu) {
		return 0;
	} else {
		$aktualizuj = "UPDATE `tabela` SET `nazwa_klubu` = '$nazwaKlubu', "
					. "`kolejka` = '$kolejka', `punkty` = '$punkty', "
					. "`punkty_zdobyte` = '$punktyZdobyte', "
					. "`punkty_stracone` = '$punktyStracone' "
					. "WHERE `id_klubu` = '$idKlubu' ";
		if (mysql_query ($aktualizuj, $lacz)) {
		  return 1;
		} else {
		  return 0;
		}
	}
}
?>
[PHP] pobierz, plaintext 

Jako, że funkcje nie powinny nic wypisywać czy właściwe jest zastosowanie po zapytaniu mysql_query fragmetu or die z komunikatem, czy tak jak jest zwracając wartość?
W przypadku pokazanym wyżej nie mogę uzależnić komunikatu błędu od zwracanej wartości, czy to jest błąd złych danych wprowadzonych przez użytkownika czy błąd bazy danych.
Proszę o wskazówkę bo zwracanie wartości w postaci return0, 1, 2, 3 itp. nie jest chyba właściwą praktyką?

Ten post edytował bliitz 17.08.2008, 13:52:33

[Cysiaczek]

Możesz dodawać nowe zwracane wartości 1,2,3,4,5 ... n i je obsługiwać. Dobrą praktyką jest jednak zwracanie po prostu true|false, a w razie poważniejszych błędów - rzucanie wyjątkami.

[PHP] pobierz, plaintext 
<?php
function doSomething()
{
	if(1!==2)
	{
		throw new Exception('Error');
	}
}
 
try
{
	doSomething();
}
catch(Exception $e)
{
	print $e->getMessage();
}
?>
[PHP] pobierz, plaintext 

Pozdrawiam.

[bliitz]

[PHP] pobierz, plaintext 
<?php
function output($string)
{
  print '<p class="ostrzezenia">'.$string.'</p>';
}
 
function aktualizujTabele($nazwaKlubu, $punkty, $punktyZdobyte,
						   $punktyStracone,$kolejka, $idKlubu) {
	global $lacz;
	$nazwaKlubu = safeOutput($nazwaKlubu);
	$punkty = safeOutput($punkty);
	$punktyZdobyte = safeOutput($punktyZdobyte);
	$punktyStracone = safeOutput($punktyStracone);
	$kolejka = safeOutput($kolejka);
	$idKlubu = intval($idKlubu);
 
	if (!preg_match('/^-?[0-9]+$/', $punkty)) {
		throw new Exception ('Zła wartość w polu punkty.');
	} elseif (!preg_match('/^[0-9]+$/', $punktyZdobyte))	{
		throw new Exception ('Zła wartość w polu punkty +.');
	} elseif (!preg_match('/^[0-9]+$/', $punktyStracone)) {
		throw new Exception ('Zła wartość w polu punkty -.');
	} elseif (!preg_match('/^[0-9]+$/', $kolejka)) {
		throw new Exception ('Zła wartość w polu kolejka.');
	} elseif (!$nazwaKlubu) {
		throw new Exception ('Zła wartość w polu klub.');
	} else {
		$aktualizuj = "UPDATE `tabela` SET `nazwa_klubu` = '$nazwaKlubu', "
					. "`kolejka` = '$kolejka', `punkty` = '$punkty', "
					. "`punkty_zdobyte` = '$punktyZdobyte', "
					. "`punkty_stracone` = '$punktyStracone' "
					. "WHERE `id_klubu` = '$idKlubu' ";
		if (mysql_query ($aktualizuj, $lacz)) {
			return output('Wprowadzono zmiany.');
		} else {
			throw new Exception ('Błąd bazy danych.');
		}
	}
}
?>
[PHP] pobierz, plaintext 

czy coś takiego jest poprawne?
co prawda pierwsza funkcja wypisuje tekst, ale jest to jedyna czynność jaką wykonuje.