[MySQL][PHP]Upload wielu plików + dodawanie do bazy Opcje

[Krkol]

Witam.
Od paru godzin głowię się nad stworzeniem skryptu, który prześle kilka plików na serwer oraz zapisze do bazy informacje o nich.

Formularz:

[HTML] pobierz, plaintext 
 <form method="POST" name="add" action="add.php" enctype="multipart/form-data" accept="jpg, jpeg, png, gif">
		   	<table>
		   		<tr>
		   			<td>
		   				<input type="text" name="tytul" placeholder="Tytuł">
		   			</td>
		   		</tr>
		   		<tr>
		   			<td>
		   				Zdjęcie główne
		   			</td>
		   			<td>
		   				<input type="file" name="obraz-glowny" />
		   			</td>
		   		</tr>
		   		<tr>
		   			<td>
		   				Zdjęcia do galerii
		   			</td>
		   			<td>
		   				<input type="file" name="file[]" multiple="multiple" />
		   			</td>
		   		</tr>
		   		<tr>
		   			<td>
		   				<input type="submit" name="submit" value="Wyślij">
		   			</td>
		   		</tr>
		   	</table>
		   </form>
[HTML] pobierz, plaintext 

Tu sprawa jest prosta... Zdjęcie główne ładnie się uploaduje i zapisuje do bazy. Problem jest, gdy chcę wrzucić kilka zdjęć przez "Zdjęcia do galerii".
Skrypt obsługujący wysyłkę:

[PHP] pobierz, plaintext 
function losuj () {
	$znaki = "ABCDEFGHIJKLMNPQRSTUVWXYZ123456789";
	$i3 = 0;
        while ($i3 <= 25) {
        $numer3 = rand() % 33;
        $tmp3 = substr($znaki, $numer3, 1);
        $haslo3 = $haslo3 . $tmp3;
        $i3++;
    }
    	return $haslo3;
}
 
foreach ($_FILES['file']['name'] as $filename) {
 
	$nazwa2 = explode( '/' , $filename );
	$ile2 = count( $nazwa2 );
	$rozszerzenie2 = $nazwa2[$ile2-1];
 
	losuj ();
 
	$filename = $haslo3 . '.' . $rozszerzenie2;
	move_uploaded_file($filename, $_SERVER['DOCUMENT_ROOT'] . 'uploads/' . $haslo2 . '/' . $filename);
 
	mysql_query("INSERT INTO `images` (`dir`, `url`)
	VALUES ('".$haslo2."', '".$filename."');");
}
[PHP] pobierz, plaintext 

Skrypt tworzy losową nazwę katalogu ($haslo2) i tam uploaduje wszystkie obrazy. Zmienna $haslo3 to kolejny losowy ciąg znaków, który ma być nazwą pliku. Problem w tym, że do bazy dodaje się wszystko z wyjątkiem `url` - czyli nazwą pliku. Dodaje się samo rozszerzenie. Podejrzewam, że funkcja losuj() nie zwraca mi $haslo3, tylko nie wiem dlaczego :/

Ten post edytował Krkol 2.05.2018, 16:59:48

[Neutral]

[HTML] pobierz, plaintext 
<form method="post" name="add" action="" enctype="multipart/form-data">
	<table>
		<tr>
			<td>You write name picture(s):</td>
			<td><input type="text" name="own_name" style="background:green"/></td>
		</tr>
    	<tr>
    		<td>Picture 1 - Galerry, Picture 2 - Main</td>
    		<td><input type="file" name="pictures[]" multiple="multiple" /></td>
    	</tr>
    	<tr>
    		<td><input type="submit" name="submit" value="send" /></td>
    	</tr>
    </table>
</form>
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php 
 
function random_name_file() {
 
	if(!empty($_FILES['pictures']['name'][0])){
		$extension = (explode('.',$_FILES['pictures']['name'][0]));
 
		$arr = ['a','b','c','d','e','f','g'];
		$randomed_characters = 'x'.$arr[rand(0,6)].'yuo'.$arr[rand(0,6)];
 
		if(getcwd()=='C:\xampp\htdocs\forumphp' && !file_exists($randomed_characters.'.'.$extension[1])){
			return $randomed_characters;	
		}else if(getcwd()=='C:\xampp\htdocs\forumphp' && !file_exists($_POST['own_name'].'.'.$extension[1]) && !empty($_POST['own_name']) && ctype_alpha($_POST['own_name'])){
			return $_POST['own_name'];
		}else if(getcwd()=='C:\xampp\htdocs\forumphp' && !file_exists($extension[0].'.'.$extension[1])){
			return $extension[0];
		}else{ 
			return 1;
		}
	}else { 
		return 1;
	}
}
 
$arr_empty = [random_name_file()];
 
if(!empty($_FILES['pictures']['name'][0])){ 		 
	$extension = (explode('.',$_FILES['pictures']['name'][0]));
	if($arr_empty[0]==1){
		echo 'This name is busy or consist from disallowed characters, please choose other';
	}else{ 
		if(move_uploaded_file($_FILES['pictures']['tmp_name'][0],$arr_empty[0].'.'.$extension[1])===true){
			echo 'Successfully uploaded!';
		}else{ 
			echo 'Failed uploaded!';
		}
	}
}else{ 
 	echo 'Not send file to server, because you don\'t clicked to "send" and you don\'t choose file to upload';
}	
?>
[PHP] pobierz, plaintext 

Trzeba to lekko poprawić, aby było zgodne z zasadą DRY (Don't Repeat Yourseflt (Nie powtarzaj się)). Chodzi mi o tę linię:

[PHP] pobierz, plaintext 
if(!empty($_FILES['pictures']['name'][0])){ // ... 
[PHP] pobierz, plaintext 

w funkcji i poza nią. Chyba, żeby przenieść tę linijkę przed funkcję i żeby to był jeden wielki if. Skrypt napisałem pod plik pojedynczy, nie multiple, gdybyś chciał wiele plików upload'ować, to musiałbyś przerobić ten skrypt.

Ten post edytował Neutral 8.05.2018, 13:19:13