[PHP]Die Opcje

[kry100kry100]

Witam. Czy da się jakoś użyć die bez psucia szablonu? Bo trochę brzydko wygląda w pół ucięty szablon Myślałem może nad jakimś systemem błędów, ale jakoś za bardzo nie wiem jak się za niego zabrać. Dałby ktoś jakieś podpowiedzi?

[Wicepsik]

Pokaż skrypt do którego potrzebny Ci jest die.

[nexis]

Najprościej treść wyświetlać na samym końcu i wtedy masz możliwość przerwania programu z powodu jakiegoś błędu bez efektu "urwanych stron". Np.:

[PHP] pobierz, plaintext 
<?php
$output  = '<h1>Nagłówek</h1>';
$output .= '<p>Treść</p>';
if (true) {
	exit('Wystąpił błąd i przerywam działanie programu!');
}
$output .= '<h2>Stopka</h2>';
echo $output;
?>
[PHP] pobierz, plaintext 

[kry100kry100]

Jest to skrypt rejestracji:

[PHP] pobierz, plaintext 
<?php
require_once("config.php");
$wynik = mysql_query("SELECT * FROM gadu")
or die('Błąd zapytania');
 
if(mysql_num_rows($wynik) > 0) {
while($r = mysql_fetch_assoc($wynik)) 
{
if($r['ip'] == $_SERVER['REMOTE_ADDR']){
die ('1 IP = 1 konto');
}
}
}
 
echo '<form action="" method="post" enctype="multipart/form-data" />
Login:<br />
<input name="login" type="text" maxlength="15" /><br />
Hasło:<br />
<input name="haslo" type="password" /><br />
Potwierdź hasło:<br />
<input name="haslo1" type="password" /><br />
E-mail:<br />
<input name="email" type="text" /><br /><br />
Przepisz kod:<br />
<img src="token.php" alt="token" maxlength="3" /><br />
<input name="token" type="text" /><br />
<input type="submit" value="Wyslij" />
</form>';
 
 
$login = $_POST['login'];
$haslo = md5($_POST['haslo']);
$haslo1 = md5($_POST['haslo1']);
$email = $_POST['email'];
$token = $_POST['token'];
$punkty = '20';
$ip = $_SERVER['REMOTE_ADDR'];
 
 
$login = htmlentities($login, ENT_QUOTES);
$haslo = htmlentities($haslo, ENT_QUOTES);
$haslo1 = htmlentities($haslo1, ENT_QUOTES);
$email = htmlentities($email, ENT_QUOTES);
$token = htmlentities($token, ENT_QUOTES);
 
$checkuser = mysql_query("SELECT login FROM gadu WHERE login='$login'");
$username_exist = mysql_num_rows($checkuser);
 
$em = mysql_query("SELECT email FROM gadu WHERE email='$email'");
$email_check = mysql_num_rows($em);
 
if(isset($_POST['login']))
{
if(empty ($login) || empty($haslo) || empty($email))
{
die ('<br />Wszystkie pola muszą zostać wypełnione');
}
if(!ereg("[a-zA-Z0-9\-\_\.]+\@[a-zA-Z0-9\-\_\.]+\.[a-zA-Z]" , $email)){
die ('<br />Nie poprawny adres e-mail');
}
if(!$haslo==$haslo1){
die ('<br />Hasła muszą być takie same');
}
if($username_exist > 0){
die ('<br />Ta nazwa użytkownika jest już zajęta');
}
if($email_check > 0){
die ('<br />Ten adres email został juz użyty');
}
if($token==$_SESSION['captcha'])
{
$_SESSION['reg'] = ''.$login.''; 
$ins = mysql_query("INSERT INTO `gadu` (login, haslo, email, punkty, ip) VALUES ('$login','$haslo','$email','$punkty','$ip')");
}
else
{
die ('<br />Błędny kod');
}  
if($ins)
{
echo '<br />Gratulujemy, zostałeś pomyślnie zarejestrowany';
}
}
?>
[PHP] pobierz, plaintext 

[Void]

IMHO przy walidacji formularza używanie die() jest bez sensu. No bo jeśli użytkownik wypełni np. 3 pola formularza źle, to nie dowie się o tym od razu, bo pojawi się tylko jeden komunikat i skrypt zostanie przerwany. Poprawi pole, znowu wyśle formularz pojawi się kolejny błąd itd. Poza tym błędne wypełnienie pola formularza nie jest od razu powodem do zakończenia całego skryptu. Lepiej jest wygenerować stronę z formularzem (można wtedy wpisać dane do pól, które wypełnione zostały poprawnie) i ładnymi komunikatami o błędach Stwórz sobie więc jakąś zmienną i dopisuj do niej ewentualne komunikaty o błędach. A potem sprawdź czy zmienna ma jakąś wartość i jeśli tak - wyświetl komunikaty o błędach.

[bmL]

Nie chcę nikomu nic wpajać ale naprawdę lubię taką obsługę błędów formularzy, zawsze można ją szybko na poczekaniu napisać i nie jest zła:

[PHP] pobierz, plaintext 
function viewErrors($errorsArr, $backURL = 'java script:history.back()')
{
  if($errorsArr)
{
    echo '<b>Wystąpiły następujące błędy:</b> <ul class="errors">';
    foreach (errorsArr as $error)
    {
        echo '<li>' . $error . '</li>';
    }
    echo '</ul>';
    echo '<a href="' . $backURL . '">Powrót</a>';
return true;
}
return false;
} 
 
if(!$haslo==$haslo1){
$errors[] = '<br />Hasła muszą być takie same';
}
if($username_exist > 0){
$errors[] = '<br />Ta nazwa użytkownika jest już zajęta';
}
if($email_check > 0){
$errors[] = '<br />Ten adres email został juz użyty';
}
 
if(!viewErrors($errors))
{
// wszystko ok
}
[PHP] pobierz, plaintext 

Ten post edytował bmL 8.09.2009, 01:37:24

[kry100kry100]

Dzięki bmL. Wkradły się 2 błędy, ale bez problemu je usunąłem i skrypt działa jak należy