Forum PHP.pl

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

Proponowałbym Tobie "zaorać" ten skrypt i napisać go od nowa, najlepiej już na PHP 7.

[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

@neutral tak z czystej ciekawosci bo nie ogarniam...
leonpro778 juz dawno podal rozwiazanie problemu a ty po paru dniach podajesz skrypt ktory:
- jest jakims tam twoim skryptem, ktory robi zupelnie co innego niz oczekuje autor
- zawiera lokalne sciezki do twoich katalogow
- mowisz autorowi by zaoral skrypt i przepisal go na php7 po czym sam podajesz skrypt, ktory z php7 nie ma nic wspolnego
- i generalnie ma sie nijak do problemu autora i trzeba by go przerabiac by dzialalo jak autor chce.
Po co? Mozesz mnie oswiecic jaki byl cel tego posta?

ps:
to jest tylko wierzcholek gory lodowej w rzeczach ktore tam masz do poprawy

No i te "lekko poprawić". Raczej zaorać jak już sam wspomniałeś.

jest tyle tam rzeczy do których można by się przyczepić aż mi się nie chce. Ja rozumiem chęć niesienia pomocy ale bądźmy ludź sukcesu a nie bylejakości

Wiem, że istnieje takie coś jak:

? ... :
? ?

Oczywiście bez spacji w drugim przypadku, napisałem tak, bo się emotikona pojawiła.
http://php.net/manual/en/migration70.php

Zacząłem pisać, więc chciałem już dokończyć, a że wiele rzeczy jest do poprawy, szczególnie te if'y, to wiem. Może komuś się przyda. Ścieżki do katalogów przecież można sobie pozmieniać na swoje. Gdybym chciał to pisać w OOP chociażby to dłużej by mi to zeszło, zresztą już chciałem zakończyć ten projekt. Może do tego wrócę i poprawię, nie wiem.

Ten post edytował Neutral 8.05.2018, 14:50:35

Tu nie chodzi o ?: ale o caloksztalt.
Generalnie bym sie nie wtracal gdybys sam najpierw userowi nie pisal by to zaoral i przepisal na php7 a potem ni z gruszki nie z pietruszki wstawiasz takie "cos" co wcale nie jest lepsze od kodu, ktory zaleciles komus innemu zaorac :/

ps:

zacznijmy od tego ze te sciezki wpisane z palca sa tam totalnie ZLE

Nie wiem za bardzo, co masz na myśli. U mnie jakoś działa. Czy mógłbyś rozwinąć myśl o tym getcwd()?

A czy ja mowie ze nie dziala.... Ja mowie, ze to jest zle zrobione. Przenies teraz chociazby ten skrypt do innego katalogu i zobacz czy bez zmian kodu bedzie dzialac. A to tylko jeden z powodow dlaczego te sciezki tam sa ZLE.

Dzięki @nospor, teraz zrozumiałem o co chodzi. W tym kodzie getcwd(), to taki jakby constant (stała), a to, czy zostaną pliki umieszczone w innym katalogu, czy nie to już inna sprawa. Można nawet przypisać do zmiennej tę ścieżkę 'C:\xampp\htdocs\forumphp' i w jednym miejscu tylko zmienić. Jeśli chodzi oto, że byłoby lepiej bez getcwd(), to może i racja, nie pomyślałem chyba, jak kto woli zresztą. Moim zdaniem, aż takie złe to nie jest, chodź może się mylę. Gdybym wykonał jakiś większy projekt z tym, to może bym zobaczył jakie to powoduje kłopoty, ale nie wydaje mi się.

Ten post edytował Neutral 8.05.2018, 15:12:05

W tym zdaniu jest wiele sprzecznosci

Ok, mylisz sie. To jest ZLE - przyjmij to jako aksjomat

Co do poprawnego rozwiazania, to gdzies tam, w pliku konfiguracyjnym definiuje sie sciezke, do ktorej wrzuca sie pliki z UPLOAD i poprostu te pliki tam sie wrzuca a nie sprawdza, czy przypadkiem working directory jest wlasnie w tej sciezce. Totalnie podszedles od zlej strony do tego.