[PHP]Sprawdzanie rozmiaru pliku przed przesłaniem, uploader php + renamer Opcje

[camilost]

Witam, próbowałem sam sobie zrobić, ale coś mi nie idzie.
Chciałbym, aby przed przesłaniem pliku, sprawdzało jego rozmiar pomiędzy plikiem gotowym już na serwerze

Mój kod:

[PHP] pobierz, plaintext 
function rename_if_exists($dir, $filename) {
	$ext = strrchr($filename, '.');
	$prefix = substr($filename, 0, -strlen($ext));
	$rozmiar = filesize($filename);
 
	$i = 0;
	while(file_exists($dir . $filename)) { // If file exists, add a number to it.
		$rozmiar = $plik_rozmiar;
		$filename = $prefix . ++$i . $ext;
	}
 
	return $filename;
}
 
 
if (!empty($_FILES))
{
	$plik_nazwa = rename_if_exists('pliki/', $plik_nazwa);
	$plik_tmp = $_FILES['plik']['tmp_name'];
	$plik_nazwa = $_FILES['plik']['name'];
	$plik_rozmiar = $_FILES['plik']['size'];
 
	if(is_uploaded_file($plik_tmp)) {
		if($rozmiar != $plik_rozmiar) { // probowalem tez z <>
			move_uploaded_file($plik_tmp, "pliki/$plik_nazwa");
			echo "Plik: <strong>$plik_nazwa</strong> o rozmiarze <strong>$plik_rozmiar bajtów</strong> został przesłany na serwer!";
		}
	}
}
[PHP] pobierz, plaintext 

Ciągle przesyła pliki o tych samych rozmiarach.

Ten post edytował camilost 12.04.2012, 17:32:04

[nospor]

$rozmiar = $plik_rozmiar;

A magiczną zmienną $plik_rozmiar to niby skad bierzesz?

[camilost]

Dałem

[PHP] pobierz, plaintext 
$plik_rozmiar = $_FILES['plik']['size'];
[PHP] pobierz, plaintext 

przed funkcją i dalej nici, kombinowałem
dalej nici, nie potrafię dobrze zbudować tego ;/
Ktoś pomoże? (IMG:style_emoticons/default/sad.gif)

[nospor]

No ale co przed funkcją... poczytaj w manualu o zasięgu zmiennych. Masz tam to dokładnie wyjaśnione.

[redeemer]

W ogóle ten skrypt wygląda jak jakiś chaotyczny zlepek (IMG:style_emoticons/default/wacko.gif) , ale jeżeli dobrze zrozumiałem idee to:
Linia 4 i 8 do zmiany na

[PHP] pobierz, plaintext 
$rozmiar = filesize($dir . $filename);
[PHP] pobierz, plaintext 

W funkcji zwróć sobie i rozmiar pliku i "wolną" nazwę:

[PHP] pobierz, plaintext 
return array('filesize'=>$rozmiar,'filename'=>$filename);
[PHP] pobierz, plaintext 

Linia 18 zmiana na:

[PHP] pobierz, plaintext 
list($rozmiar, $plik_nazwa) = rename_if_exists('pliki/', $_FILES['plik']['name']);
[PHP] pobierz, plaintext 

Linia 20 do wywalenia.

Ten post edytował redeemer 12.04.2012, 14:07:44

[camilost]

@nospor, właśnie też kombinowałem, ale albo coś innego źle zrobiłem i nic mi nie zwracało, albo gdzieś indziej błąd popełniałem.
Złe kombinowanie (IMG:style_emoticons/default/sad.gif)
@redeemer Wciąż źle (IMG:style_emoticons/default/sad.gif) Nie wiem co dalej (IMG:style_emoticons/default/sad.gif) próbowałem też różnie przestawiaać/zamieniać. Wyniki były 3
1. Wyskakuje 'Array' zamiast nazwy piku oraz rozmiar
2. Nic nie wyskakuje w obu
3. Wyskakuje nazwa podstawowa (taka jaką wysyłam),a błąd z rozmiarem
Być może źle wkleiłem kod? ; / I były nietrafne moje próby? Można prosić o aktualizację powyższych co do poniższego? Próbowałem już wiele razy, lecz bez skutku (IMG:style_emoticons/default/sad.gif) (.
Próbuję coś stworzyć, ale nawet manual np. o list i wiele przykładów z google, za wiele mi nie dały do myślenia (IMG:style_emoticons/default/sad.gif)

Powinno: zmieniać nazwę pliku, jeśli już istnieje na $i++, gdy rozmiar pliku, który jest pod tą samą nazwą początkową: $plik_nazwa = $_FILES['plik']['name']; jest inny od aktualnego przesyłać plik.
W innym wypadków, usuwać plik tymczasowy, a orginału o tym samy rozmiarze i nazwie nie nadpisywać.

Oryginalny kod :

[PHP] pobierz, plaintext 
<?php
function rename_if_exists($dir, $filename) {
	$ext = strrchr($filename, '.');
	$prefix = substr($filename, 0, -strlen($ext));
 
	$i = 0;
	while(file_exists($dir . $filename)) { // If file exists, add a number to it.
		$filename = $prefix . ++$i . $ext;
	}
	return $filename;
}
 
 
if (!empty($_FILES))
{
	$plik_nazwa = $_FILES['plik']['name'];
	$plik_nazwa = rename_if_exists('pliki/', $plik_nazwa);
	$plik_tmp = $_FILES['plik']['tmp_name'];
	$plik_rozmiar = $_FILES['plik']['size'];
 
	if(is_uploaded_file($plik_tmp)) {
		move_uploaded_file($plik_tmp, "pliki/$plik_nazwa");
		echo "Plik: <strong>$plik_nazwa</strong> o rozmiarze <strong>$plik_rozmiar bajtów</strong> został przesłany na serwer!";
	}
}
?>
[PHP] pobierz, plaintext 

Dobranoc wszystkim (IMG:style_emoticons/default/smile.gif) )

Ten post edytował camilost 13.04.2012, 07:41:29

[IProSoft]

Nie do końca rozumiem Twoje zdanie w którym piszesz co robić jeśli plik istnieje i ma ten sam rozmiar, wg schematu nic nie robić więc tak jest:

[PHP] pobierz, plaintext 
function rename_if_exists($dir, $filename) {
	$ext = strrchr($filename, '.');
	$prefix = substr($filename, 0, -strlen($ext));
	$rozmiar = 0;
 
	$i = 0;
	while(file_exists($dir . $filename)) { // If file exists, add a number to it.
		$rozmiar = filesize($dir . $filename); // rozmiar zwracamy dopiero gdy plik istnieje
		$filename = $prefix . ++$i . $ext;
	}
 
	return array(
		'filesize' => $rozmiar,
		'filename' => $filename
	);
}
 
 
if (!empty($_FILES))
{	
 
	$array = rename_if_exists('pliki/', $_FILES['plik']['name']);
	$plik_tmp = $_FILES['plik']['tmp_name'];
	$plik_nazwa = $_FILES['plik']['name'];
	$plik_rozmiar = $_FILES['plik']['size'];
 
	if(is_uploaded_file($plik_tmp)) {
		if($array['filesize'] != $plik_rozmiar){
			move_uploaded_file($plik_tmp, "pliki/$plik_nazwa");
			echo "Plik: <strong>$plik_nazwa</strong> o rozmiarze <strong>$plik_rozmiar bajtów</strong> został przesłany na serwer!";
		}
	}
}
[PHP] pobierz, plaintext 

[camilost]

Niestety, może Twój kod zadziałał co do tego co chciałem, ale nazwy nie zmieniał, gdy plik innej wielkości ).
Odkrył się nowy problem, który sam rozwiązałem (mam nadzieję, że w miarę dobrze) - proszę zobaczyć(porównać) linie:

Musiałem zmienić na takie, i wszystko działa należycie!

[PHP] pobierz, plaintext 
    $array = rename_if_exists('pliki/', $_FILES['plik']['name']);
    $plik_nazwa = $array['filename'];
    $plik_tmp = $_FILES['plik']['tmp_name'];
    $plik_rozmiar = $_FILES['plik']['size'];
[PHP] pobierz, plaintext 

Proszę o zamknięcie tematu.

Ten post edytował camilost 14.04.2012, 11:18:42