Walidacja pola typu select...option PHP

pytanie zadane 16 stycznia 2017 w PHP przez dawid4157 Nowicjusz (220 p.)

Witam,
w jaki sposób sprawdzić czy pole typu select jest puste przy pomocy funkcji empty()?

Mam tak:

<!DOCTYPE html>
<html>
<head>
    <meta charset="utf-8"/>
    <title>Kolor pojazdu</title>
</head>
<body>
    <form action="add-car.php" method="post">
        <p>Kolor pojazdu:
        <select name="car_color">
            <option selected="selected" disabled="disabled">Kolor pojazdu...</option>
            <option value='Czerwony'>Czerwony</option>
            <option value='Zielony'>Zielony</option>
            <option value='Żółty'>Żółty</option>
            <option value='Niebieski'>Niebieski</option>
            <option value='Granatowy'>Granatowy</option>
            <option value='Fioletowy'>Fioletowy</option>
            <option value='Czarny'>Czarny</option>
            <option value='Bordowy'>Bordowy</option>
        </select></p>
        <button type="submit" name="addCar">Dodaj kolor pojazdu</button>
    </form>
</body>
</html>

<?php
    if(isset($_POST['addCar'])){
        $carColor = trim($_POST['car_color']);
        if(empty($carColor)){
            echo '<div class="error">Musisz wybrać kolor pojazdu.</div>';
        }
    }
?>

Otrzymuję informację:

Notice: Undefined index: car_color in /opt/lampp/htdocs/vehicle/garage/new-car.php on line 47

Co robię nie tak?

2 odpowiedzi

odpowiedź 17 stycznia 2017 przez efiku Szeryf (75,160 p.)

Piszmy już w PHP7

<?php
    $carColor = $_POST['car_color'] ?? "default_color";
    echo $carColor;

https://3v4l.org/R4bu4

komentarz 17 stycznia 2017 przez dawid4157 Nowicjusz (220 p.)

Ok, dzięki za podpowiedź. Faktycznie tego nie znałem. Według tego co podałeś, "skonstruowałem" sobie takie coś:

$carColor = $_POST['car_color'] ?? $errors[] = ''.ERROR_SELECT_COLOR.'';

Niby działa lecz nie jest to niczym filtrowane. Gdy chcę wrzucić tam np. funkcję trim() aby pozbyć się białych znaków otrzymuję tą samą informację co wyżej(Notice):

$carColor = trim($_POST['car_color']) ?? $errors[] = ''.ERROR_SELECT_COLOR.'';

Dlaczego?

odpowiedź 16 stycznia 2017 przez CzikaCarry Szeryf (75,340 p.)

Najlepiej użyć funkcji isSet()

if(isset($_POST['car_color'])
{
$carColor = $_POST['car_color'];
}
else
{
throw new Exception('$_POST["car_color"] can't be null!');
}

1	komentarz 17 stycznia 2017 przez efiku Szeryf (75,160 p.) Średnio to wygląda. Tu raczej bym zastosował jakąś opcję domyślną, niż rzucał wyjątek. :)