[PHP]Prosta rzecz Opcje

[crew123]

Cześć, nie mogę sobie poradzić z jednym problemem...
Otóż chciałbym aby nie szedl insert gdy uploadowane zdjecie bedzie mialo inne rozszerzenie niż dopuszczalne.
Nie mam jednak pojecia jak to zrobic, kombinowałem już kilka razy i nic ;/

[PHP] pobierz, plaintext 
if(isset($_POST['ok']))
			{		
				$obrazek = '';
 
				if (isset($_FILES['obrazek']['name']))
                {
                    $sp1 = explode(".", ($_FILES['obrazek']['name']));
                    if ($sp1[1] == "gif" or $sp1[1] == "jpg" or $sp1[1] == "JPG" or $sp1[1] == "GIF" or $sp1[1] == "jpeg" or $sp1[1] == "JPEG" or $sp1[1] == "png" or $sp1[1] == "PNG")
                    {
                        $data_img = date('dmYHis');
                        $uploaddir = '../images/' . $data_img . '-';
                        $_FILES['obrazek']['name'] = przyjazny_string($_FILES['obrazek']['name']);
                        if (move_uploaded_file($_FILES['obrazek']['tmp_name'], $uploaddir . $_FILES['obrazek']['name']))
                        {
                             $obrazek = 'images/' . $data_img . '-' . $_FILES['obrazek']['name'];
                        }
                    }
				}
 
                $tytul = mysql_real_escape_string($_POST['tytul']);
                $tresc = mysql_real_escape_string($_POST['tresc']);
 
                $kategoria = $_POST['kategoria'];
				$wojewodztwo = $_POST['wojewodztwo'];
                $region = $_POST['region'];
				$kod = $_POST['kod_pocztowy'];
				$miasto = $_POST['miasto'];
				$adres = $_POST['adres'];
				$tel = $_POST['telefon'];
				$www = $_POST['www'];
				$email = $_POST['email'];
 
 
				function  ValidEmail($email) {
					if (!eregi("^[_.0-9a-z-]+@([0-9a-z][0-9a-z-]+.)+[a-z]{2,4}$" , $email)) {
						return false;
					}
					return true;
				}
 
				if(empty($tytul) || empty($tresc) || empty($kategoria) || empty($wojewodztwo) || empty($wojewodztwo) || empty($kod) || empty($miasto) || empty($adres) || empty($email)) {
 
					echo 'Wypełnij wszystkie pola oznaczone *';
 
				}
				elseif (!preg_match('/^[0-9]{2}-?[0-9]{3}$/Du', $kod)) {
 
					echo 'Błędny kod pocztowy';
 
				}
				elseif (!ValidEmail($email)) {  
 
					echo "Błędny adres email"; 
 
				}
				elseif (!$sp1[1] == "gif" or $sp1[1] == "jpg" or $sp1[1] == "JPG" or $sp1[1] == "GIF" or $sp1[1] == "jpeg" or $sp1[1] == "JPEG" or $sp1[1] == "png" or $sp1[1] == "PNG") {
 
					echo 'Dopuszczalne pliki tylko w formacie gif, jpg, png!';
 
				}
 
				else {
 
				$iduser = $user_data['user_id'];
 
 
                $query = "INSERT INTO `ogloszenia` (`tytul`, `tresc`, `id_cat`, `id_woj`, `id_region`, `id_user`, `kod_pocztowy`, `miasto`, `adres`, `tel`, `www`, `email`, `zdjecie_1`) VALUES ('$tytul', '$tresc', '$kategoria', '$wojewodztwo', '$region', '$iduser', '$kod', '$miasto', '$adres', '$tel', '$www', '$email', '$obrazek')";
 
				if(mysql_query($query)) {
 
				$id = mysql_insert_id();
 
				foreach ($_POST['opcje'] as $opcja)
				{
					mysql_query("INSERT INTO dodatkowo(id_oglo, id_oglo_opcje) VALUES(".$id.", ".intval($opcja).")");
				}
 
				echo 'Pomyslnie dodano ogłoszenie! <a href="index.php">wróć</a>';
 
				}
 
                return 1;
				}
        }
[PHP] pobierz, plaintext 

Ten post edytował crew123 30.06.2012, 12:44:03

[!*!]

Trochę to chaotyczne, a samo rozszerzenie sprawdzasz błędnie (pathinfo). Na szybko aby to naprawić, stwórz zmienną $error = false, jeśli rozszerzenie będzie złe, zamieniasz na true, przed zapytaniem do bazy sprawdzasz w warunku czy error nadal ma false, jeśli tak, wykonujesz, jeśli nie, sprawdzasz wyświetlasz bład.

Ten post edytował !*! 30.06.2012, 12:48:30

[Evinek]

Nie chce mi się całego kodu analizować dokładnie, ale wystarczy wziąć kod który dodaje do bazy pod klamerkę przy instrukcji:
if ($sp1[1] == "gif" or $sp1[1] == "jpg" or $sp1[1] == "JPG" or $sp1[1] == "GIF" or $sp1[1] == "jpeg" or $sp1[1] == "JPEG" or $sp1[1] == "png" or $sp1[1] == "PNG")
A jeszcze lepiej to jak plik poprawnie się utworzy

@EDIT:
A najlepiej przepisz od nowa to.

Ten post edytował Evinek 30.06.2012, 12:48:48

[crew123]

Dobra działa, dzięki
---
Mam jeszcze pytanie, co zrobic aby gdy zostawie pisty input file nie pokazywalo bledu z rozszerzeniem?

Ten post edytował crew123 30.06.2012, 13:09:25