Forum PHP.pl

Witam,
Dla wielu z was jest to pewnie banalne..
Mam funkcje przykładowo dodawania zdj

[PHP] pobierz, plaintext 
function upload_images_file($images_file) {
	$id = $_SESSION['user_id'];
	$BASE_DIR = $_SERVER['DOCUMENT_ROOT'];
	$location_images_file = "$BASE_DIR/admin/images/user$id";
	$name_file = $images_file['name'];
	$type_file = $images_file['type'];
	$tmp_name_file = $images_file['tmp_name'];
	$size_file = $images_file['size'];
	$max_size_file = 2000000;
 
	$info = pathinfo($name_file);
	$enlargement = $info['extension'];
 
	if(($type_file == "image/jpeg") || ($type_file == "image/jpg") || ($type_file == "image/png")){
		if($size_file < $max_size_file){
			if (!file_exists($location_images_file))  
			{
				mkdir("$location_images_file",  0777);
				if(is_uploaded_file($tmp_name_file)) { 
					move_uploaded_file($tmp_name_file, "$location_images_file/avatar.$enlargement"); 
					return true;
				}else{
					return false;
				}
			}else{
				if(is_uploaded_file($tmp_name_file)) { 
					move_uploaded_file($tmp_name_file, "$location_images_file/avatar.$enlargement"); 
					return true;
				}else{
					return false;
				}
			}
		}else{
			return false;
		}
	}else{
		return false;
	}
 
};
[PHP] pobierz, plaintext 

mam tam wiele if'ów i chciałbym się was zapytać skąd mam wiedzieć w którym ifie mi funkcja zwaraca true a w którym false. Jest mi to potrzebne bo np. chciałbym wyświetlać na tego podstawie komunikat czy np obraz ma zły format czy rozmiar.

Ten post edytował matti9410 22.04.2017, 16:09:37

Zamiast false możesz zwracać coś innego, chociażby treść lub kod błędu.
Możesz też rzucać wyjątkiem (try catch).

Generalnie w dobrze napisanej aplikacji walidacja danych wejściowych przeprowadzana jest wcześniej jako osobny fragment. Potem już trzeba zostają jakieś naprawdę nietypowe sytuacje, których nie przewidzieliśmy (wtedy zazwyczaj wyświetlamy już coś w stylu "Nieoczekiwany błąd").

Ten post edytował markonix 22.04.2017, 16:30:47

Czyli na false/true tego nie wykonam
Zmieniłem kod na takie coś

[PHP] pobierz, plaintext 
function upload_images_file($images_file){
	$id = $_SESSION['user_id'];
	$BASE_DIR = $_SERVER['DOCUMENT_ROOT'];
	$location_images_file = "$BASE_DIR/admin/images/user$id";
	$name_file = $images_file['name'];
	$type_file = $images_file['type'];
	$tmp_name_file = $images_file['tmp_name'];
	$size_file = $images_file['size'];
	$max_size_file = 2000000;
 
	$info = pathinfo($name_file);
	$enlargement = $info['extension'];
 
	if(($type_file == "image/jpeg") || ($type_file == "image/jpg") || ($type_file == "image/png")){
		if($size_file < $max_size_file){
			if(!file_exists($location_images_file)){
				mkdir("$location_images_file",  0777);
				if(is_uploaded_file($tmp_name_file)){ 
					move_uploaded_file($tmp_name_file, "$location_images_file/avatar.$enlargement"); 
					return true;
				}else{
					return "error3";
				}
			}else{
				if(is_uploaded_file($tmp_name_file)){ 
					move_uploaded_file($tmp_name_file, "$location_images_file/avatar.$enlargement"); 
					return true;
				}else{
					return "error3";
				}
			}
		}else{
			return "error2";
		}
	}else{
		return "error1";
	}
};
[PHP] pobierz, plaintext 

oraz część wyświetlająca błędy

[PHP] pobierz, plaintext 
	if(isset($_POST["submit_avatar_user"])){
		if($_FILES["avatar_user"]["size"] == 0){
			$alert = '<div class="alert alert-warning" role="alert">
				<button type="button" class="close" data-dismiss="alert" aria-label="Close"><span aria-hidden="true">×</span></button>
				<h2>Upsss...</h2> 
				<p>Nie wybrano pliku do wysłania.</p>
			</div>';
		}else{
			if(upload_images_file($_FILES["avatar_user"]) == "error1"){
				$alert = '<div class="alert alert-warning" role="alert">
					<button type="button" class="close" data-dismiss="alert" aria-label="Close"><span aria-hidden="true">×</span></button>
					<h2>Upsss...</h2> 
					<p>Na serwer można wysłać tylko i wyłącznie pliki jpeg, jpg, png.</p>
				</div>';
			}else{
				if(upload_images_file($_FILES["avatar_user"]) == "error2"){
					$alert = '<div class="alert alert-warning" role="alert">
						<button type="button" class="close" data-dismiss="alert" aria-label="Close"><span aria-hidden="true">×</span></button>
						<h2>Upsss...</h2> 
						<p>Maksymalny rozmiar obrazu jaki można przesłać na serwer to 2MB.</p>
					</div>';
				}else{
					if(upload_images_file($_FILES["avatar_user"]) == "error3"){
						$alert = '<div class="alert alert-warning" role="alert">
							<button type="button" class="close" data-dismiss="alert" aria-label="Close"><span aria-hidden="true">×</span></button>
							<h2>Upsss...</h2> 
							<p>Niestety nie udało się zaaktualizować avatar.</p>
						</div>';
					}else{
						$alert = '<div class="alert alert-succes" role="alert">
							<button type="button" class="close" data-dismiss="alert" aria-label="Close"><span aria-hidden="true">×</span></button>
							<h2>Zaaktualizowano !</h2> 
							<p>Twój avatar został zaaktualizowany.</p>
						</div>';
					}
				}
			}
		}
	}
[PHP] pobierz, plaintext 

I nie potrafię sb poradzić ponieważ kod jak by zawiesza się na pierwszym if...

Edit

Dodam że wgrywam plik jpg.

Ten post edytował matti9410 22.04.2017, 16:44:52

Jak już nie masz zamiaru robić tego porządnie to chociaż tak..

if
elseif
elseif
elseif
else
-tu ostateczny kod

Pyton_000 mógłbyś podrzucić pomysł jak tego uniknąć ?

markonix możesz pokazać przykład porządnego rozwiązania ?

Ten post edytował matti9410 22.04.2017, 17:55:20

https://www.google.pl/search?q=php+validati...alidation+class ale najlepiej jakiś framework, każdy ma walidacje już wbudowaną.

Ten post edytował markonix 22.04.2017, 18:07:28