Witam, Czy jeżeli stosuję bibliotekę PDO do komunikacji z bazą danych i robie to za pomocą bloku try {}catch{} to czy należy jeszcze przed np. wyświetleniem dodanych danych sprawdzić np ifem czy dane się dodały? Czy jest to sprawdzanie bez sensu? i ten if jest zbędny?

[PHP] pobierz, plaintext 
try
{
    $dodaj = $sth->prepare('INSERT INTO t_dane
                        SET 
                        imie_zm=:imie_zm,
                        nazwisko_zm=:nazwisko_zm,
                        id_miejsce=:id_miejsce,
                        data_p_zm=NOW()');
 
    $dodaj->bindValue(':imie_zm',$imie_zm,PDO::PARAM_STR);
    $dodaj->bindValue(':nazwisko_zm',$nazwisko_zm,PDO::PARAM_STR);
    $dodaj->bindValue(':id_miejsce',$id_miejsce,PDO::PARAM_INT);
    $dodaj->execute();
    if ($dodaj->rowCount() == 1)
    {
        $dodaj->closeCursor();
        header("Location: index.php?id=wyswietl_miejsce.php&id_miejsce=$id_miejsce");
    } else
    {
        echo 'Nieoczekiwany błąd';
        exit();
    }
}
catch (PDOException $e)
{
    echo 'Error!: Wystąpił błąd podczas wstawiania rekordów'.$e->getMessage();
    die();
}
 
[PHP] pobierz, plaintext 

Z góry dziękuję za odpowiedź!

A co ma try, catch do sprawdzania danych? Zawsze sprawdzaj działanie kodu i tego czy wynik jest oczekiwany.

To tak jak kod jest podany powyżej to jest dobrze czy coś tam trzeba zmienić?Np ifa dać za catch?

Sprawdź pierw czym są wyjątki http://pl.wikibooks.org/wiki/PHP/Wyjątki oraz jak ich używać (po co Ci echo?)
Kod jest ok, poza tym że nigdzie nie widzę, abyś walidował dane które przesyłasz.

Jeżeli skonfigurowałeś PDO by w przypadku błędu rzucało wyjątkiem, nie ma potrzeby sprawdzania czy PDOStatement::execute() wykonało się poprawnie. W przypadku błędu blok catch wychwyci go.

konfiguracja pdo

[PHP] pobierz, plaintext 
$sth->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
[PHP] pobierz, plaintext 

dane sprawdzam w innym pliku np w taki sposób dla nazwiska:

[PHP] pobierz, plaintext 
if (isset($_POST['nazwisko_zm']))
{   
    $dlugosc_nazwisko = strlen($_POST['nazwisko_zm']);
 
    if ($dlugosc_nazwisko >= 2 && $dlugosc_nazwisko <= 60)
    {
        //ok
    } else
    {
        echo "Pole nazwisko ma minimalną ilość znaków - 2, maksymalną - 60<br />";
        exit();
    }
 
    if (preg_match('/[^A-Za-z \'\-]/',$_POST['nazwisko_zm']))
    {
        echo"Pole nazwisko nie może zawierać znaków innych niż alfabetyczne i apostrofy";
        exit();
    } else
    {
        $nazwisko_zm=trim(ucwords($_POST['nazwisko_zm']));
    }
     if (!$nazwisko_zm)
    {
        echo "Brak nazwiska zmarłego<br />";
        exit;
    }
}
[PHP] pobierz, plaintext 

czy to jest dobrze?
to czyli cały kod może wyglądać tak?

[PHP] pobierz, plaintext 
try
{
$dodaj = $sth->prepare('INSERT INTO t_dane
SET
imie_zm=:imie_zm,
nazwisko_zm=:nazwisko_zm,
id_miejsce=:id_miejsce,
data_p_zm=NOW()');
 
$dodaj->bindValue(':imie_zm',$imie_zm,PDO::PARAM_STR);
$dodaj->bindValue(':nazwisko_zm',$nazwisko_zm,PDO::PARAM_STR);
$dodaj->bindValue(':id_miejsce',$id_miejsce,PDO::PARAM_INT);
$dodaj->execute();
$dodaj->closeCursor();
header("Location: index.php?id=wyswietl_miejsce.php&id_miejsce=$id_miejsce");
}
catch (PDOException $e)
{
echo 'Error!: Wystąpił błąd podczas wstawiania rekordów'.$e->getMessage();
die();
}
[PHP] pobierz, plaintext 

spojrzy ktoś? Nie wiem czy dobre jest moje rozumowanie.

Bez sensu.

- długość znaków sprawdzaj przy pomocy mb_strlen, ponieważ zwraca poprawną jej ilość. Sprawdź ile zwróci dla "abc" a "abć".
- jeśli dane się nie zgadzają to zatrzymujesz wykonywanie skryptu przez exit, po co? Zrób gdzieś na początku zmienną

[PHP] pobierz, plaintext 
$error = false
[PHP] pobierz, plaintext 

i gdy wystąpi błąd, zamieniasz ją na true. Później sprawdzasz jej status i wyświetlasz komunikat.
- Twoje wyrażenie regularne nie przepuści nazwisk innych niż zakres A-Za-z, a co z ółśćźż i innymi, zamiast tego użyj \w