[MySQL][PHP]Upload większej ilości plików, Więcej niż jedna fotka Opcje

[Barcelona]

Witam, niedawno rozwiązałem jeden problem i od razu natknąłem się na kolejny.
Z tego co mi wiadomo serwer na wykonanie jednego skryptu ma 30 sekund i dlatego stwarza to pewne problemy.
Planuje do mojego formularza dodać możliwość uploadu zdjęć w ilości większej niż 1.
Zatem jeżeli dam możliwość uploadu max 5 zdjęć i jeżeli user będzie wgrywał jednocześnie 5 zdjęć to wtedy serwer nie wyrobi się w czasie (wszystko zależy od łącza oczywiście, ale zakładam czarny scenariusz).
A zatem moje pytanko brzmi jak ten problem rozwiązać ?

Oto kod formularza:

[PHP] pobierz, plaintext 
<?
 
 
function f_rand($ile) {
for ($i=1; $i<=$ile; $i++)
{$l_rand .= dechex(mt_rand());}
return $l_rand;
}
 
 
function resize($foto, $szerokosc, $folder, $napis) {
 
$fotos = "./tmp/".$foto;
$mini = $folder."/".$foto;
if (!file_exists($mini)) {
$img = imagecreatefromjpeg($fotos);
$width  = imagesx($img); // szerokosc jpga
$height = imagesy($img); // wysokosc jpga
 
if ($width > $szerokosc || $height > $szerokosc){
//poziomy / szerokosc
if ($width>=$height) {$width_mini = $szerokosc;
		      $x = $width/$width_mini; 
		      $height_mini = $height/$x;
			  $litery_wielkosc = $width/60;}
 
//pionowy / wysokosc
if ($width<$height) {$height_mini = $szerokosc;
		     $x = $height/$height_mini; 
		     $width_mini = $width/$x;
			 $litery_wielkosc = $height/60;}
 
$img_mini = imagecreatetruecolor($width_mini, $height_mini);
}
else 
{
$img_mini = $img;
}
$txt = $napis;
$fontpath = realpath('.');
$kolor_tla = imagecolorallocate ( $img, 255, 255, 255 );
 
imagecopyresampled($img_mini, $img, 0, 0, 0, 0, $width_mini , $height_mini, $width, $height);
imagejpeg($img_mini, $mini, 100);
@imagedestroy($img);
@imagedestroy($img_mini);
}
}
 
 
 
if ($_SESSION["zalogowany"]=="tak") {
include('menu.php');
 
$tytul = mysql_real_escape_string (trim($_POST['tytul']));
$data = mysql_real_escape_string (trim($_POST['data']));
$tresc = $_POST['tresc'];
$stan = mysql_real_escape_string (trim($_POST['stan']));
$rodzaj = mysql_real_escape_string (trim($_POST['rodzaj']));
$marka = mysql_real_escape_string (trim($_POST['marka']));
$model = mysql_real_escape_string (trim($_POST['model']));
$rocznik = mysql_real_escape_string (trim($_POST['rocznik']));
$miejsca = mysql_real_escape_string (trim($_POST['miejsca']));
$klimatyzacja = mysql_real_escape_string (trim($_POST['klimatyzacja']));
$rejestracja = mysql_real_escape_string (trim($_POST['rejestracja']));
$cena = mysql_real_escape_string (trim($_POST['cena']));
$ul_plik = mysql_real_escape_string (trim($_POST['foto']));
 
 
if (strlen($tytul)>2) {
 
  ////////////////////////////////////////
 
 //ustawienie wielkosci zmniejszanych obrazow
$w_1 = 175;
$w_2 = 400;
 
if ($_FILES){ 
foreach( $_FILES as $ul_plik )
{
	if( is_uploaded_file( $ul_plik['tmp_name'] ) )
        {
$rand = f_rand(3);
 
		$new_name = $rand.".jpg";
            $tmp_name = $ul_plik['tmp_name'];
            $name = $ul_plik['name'];
 
            copy($tmp_name, './tmp/'.$new_name);
 
 
	    resize($new_name,$w_1,"./img/mini/", "");
	    resize($new_name,$w_2,"./img/", "");	
 
 
		unlink('./tmp/'.$new_name);
 
	mysql_query("
        INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
        VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','$rand','$cena')
      ");
 
 
        }
		else {
  mysql_query("
        INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
        VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','nazwa','$cena')
      ");
	  }
  }
 
  }
 
 
  echo "
  <br><div class=\"potwierdzenie\"><center>Ogłoszenie zostało dodane</center>
    </div>";
 
 
	}
$data = date("Y-m-d H:i:s");
echo <<<KONIEC
<br>
<form action="" method="post" enctype="multipart/form-data" class="uniForm">
 
<fieldset><legend>Dane Samochodu</legend>
<div class="ctrlHolder">
  <label for="foto">Dodaj zdjęcie do ogłoszenia:</label><br>
  <input type="file" name="foto"><br>
  <label for="marka">Podaj marke samochodu:</label>
  <input type="text" name="marka" style="width:100%;" class="textInput small" />
  <label for="model">Podaj model samochodu:</label>
  <input type="text" name="model" style="width:100%;" class="textInput small" />
  <label for="rocznik">Podaj rocznik samochodu:</label>
  <input type="text" name="rocznik" style="width:100%;" class="textInput small" />
  <label for="miejsca">Podaj ilość miejsc siedzących:</label>
  <input type="text" name="miejsca" style="width:100%;" class="textInput small" />
  <label for="klimatyzacja">Klimatyzacja:</label><br>
  <select id="" name="klimatyzacja"><optgroup label='Wybierz jedną opcję'><option value='TAK'>TAK</option><option value='NIE'>NIE</option></optgroup></select><br>
  <label for="rejestracja">Podaj rejestracje samochodu*:</label>
  <input type="text" name="rejestracja" style="width:100%;" class="textInput small" />
  </div></fieldset>
  <fieldset><legend>Ogłoszenie</legend>
<div class="ctrlHolder">
  <label for="tytul">Podaj tytuł ogłoszenia:</label>
  <input type="text" name="tytul" style="width:100%;" class="textInput small" />
  <label for="model">Podaj date ogłoszenia:</label>
  <input type="text" name="data" style="width:100%;" class="textInput small" value="$data" />
  <label for="rocznik">Napisz treść ogłoszenia:</label><br>
  <textarea name="tresc" id="editor"></textarea><br>
KONIEC;
  ?>
<script type="text/javascript">
			//<![CDATA[
 
				CKEDITOR.replace( 'editor',
					{
						fullPage : true
					});
 
			//]]>
			</script>
<?
echo <<<KONIEC
<label for="cena">Podaj cene (zł):</label>
  <input type="text" name="cena" style="width:100%;" class="textInput small" />
</div></fieldset>
<fieldset><legend>Informacje dodatkowe</legend>
<div class="ctrlHolder">
	  <p class="label">
            Wybierz rodzaj ogłoszenia, oraz czy ma on zostać wyświetlony na stronie
          </p><table width="100%">
          
 
		  <tr><td align="middle"><ul class="alternate">
            <li><label for="rodzaj" align="top">Rodzaj <select id="" name="rodzaj"><optgroup label='Wybierz jedną opcję'><option value='wynajem_autobusow'>Wynajem autobusów</option><option value='wynajem_busow'>Wynajem busów</option><option value='sprzedaz_busow'>Sprzedaż busów</option><option value='transport_ludzi'>Transport ludzi</option></optgroup></select></label></li></td><td align="middle">
            <li><label for="stan" align="middle">Stan <select id="" name="stan"><optgroup label='Aktualne'><option value='1'>Ogłoszenie widoczne</optgroup><optgroup label='Wybierz jedną opcję'><option value='1'>Ogłoszenie widoczne</option><option value='0'>Ogłoszenie niewidoczne</option></optgroup></select></label></li>
          </ul></td></tr></table>
        </div></fieldset>
  <div class="buttonHolder">
<input type="submit" value=" wyślij" class="primaryAction" />
</div>
KONIEC;
} 
else {
echo "<div class=\"potwierdzenie\"><center>Proszę się zalogować!</center>
    </div>";
 
}
 
?>
[PHP] pobierz, plaintext 

[evolucja]

Przepraszam bardzo, ale co ma czas wgrywania plików do czasu wykonywania skryptu? No, chyba, że masz na myśli problem w którym serwer dostanie 20 zdjęć i ich obróbka zajmie mu dłużej niż 30 sekund.

Skrypt zostaje uruchomiony dopiero wtedy, jak całość 'od użytkownika' zostanie wysłana. W twoim mniemaniu skrypt zaczyna działać(przerabiać zdjęcia) jeszcze zanim całkowicie je odebrał Inaczej nie potrafię tego wytłumaczyc.

Ten post edytował evolucja 21.07.2010, 00:45:37

[Barcelona]

Jak w formularzu wskaże lokalizacje do 5 fotek i wcisnę przycisk wyślij to wtedy zaczyna wykonywanie skryptu? @evolucja mówisz że nie bedzie problemu czasowego ?

[tehaha]

jeżeli obawiasz, że obróbka zdjęć może potrwać za długo i skrypt przestanie działać, to ustaw na samym początku:

[PHP] pobierz, plaintext 
ini_set('max_execution_time', 300);
[PHP] pobierz, plaintext 

[evolucja]

Jak w formularzu wskaże lokalizacje do 5 fotek i wcisnę przycisk wyślij to wtedy zaczyna wykonywanie skryptu? @evolucja mówisz że nie bedzie problemu czasowego ?

Nie. Przecież napisałem wyraźnie. Podałem nawet rozwinięcie twojego myślenia i sam nie wyłapałeś błędu?
Skrypt po stronie serwera może zostać uruchomiony dopiero wtedy, jak całość danych zostanie przesłana. Bo niby w jaki inny sposób miały by działać hostingi plików?

jeżeli obawiasz, że obróbka zdjęć może potrwać za długo i skrypt przestanie działać, to ustaw na samym początku:

[PHP] pobierz, plaintext 
ini_set('max_execution_time', 300);
[PHP] pobierz, plaintext 

Na localu może sobie ustawiać... Nie znam jeszcze shared hostingu który by miał możliwość zmiany

[Barcelona]

@evolucja rozumiem o co chodzi.
A możecie mi wskazać jak zrobić żeby skrypt robił miniaturki kolejnych zdjęć. Bo teraz mam tylko jednego inputa z type=file.

----------------------------------------------

Witam, sorki że nadpisuje ale nie mogę sobie poradzić z tym problemem.
Zrobiłem kolejnego inputa na wrzucanie pliku.
$ul_plik --> odnosi się do pierwszego inputa
$foto_dwa --> to drugi input.

Zmierzam do tego żeby zrobić upload większej ilości uploadu plików graficznych i tak żeby skalowało mi te pliki do miniatury. Wszystko działa, ale tworzy mi podwójną ilość miniatur oraz dodaje do bazy podwójna ilość ogłoszeń.
Dodaje ogłoszenie i daje 2 fotki. Na ftp tworzy się 8 plików: 4 miniatury i 4 oryginalne. Do bazy dodają się 2 ogłoszenia.
Nie wiem co jest nie tak.
Jak był pojedynczy upload to wszystko działało dobrze.
Proszę o pomoc.

[PHP] pobierz, plaintext 
if ($_FILES){ 
foreach( $_FILES as $foto_dwa )
{
	if( is_uploaded_file( $foto_dwa['tmp_name'] ) )
        {
$rand = f_rand(3);
 
		$new_name = $rand.".jpg";
            $tmp_name = $foto_dwa['tmp_name'];
            $name = $foto_dwa['name'];
 
            copy($tmp_name, './tmp/'.$new_name);
 
	    resize($new_name,$w_1,"./img/mini/", "");
	    resize($new_name,$w_2,"./img/", "");	
 
 
		unlink('./tmp/'.$new_name);
 
        }
  }
 
  }
 
 
if ($_FILES){ 
foreach( $_FILES as $ul_plik )
{
	if( is_uploaded_file( $ul_plik['tmp_name'] ) )
        {
$rand = f_rand(3);
 
 
		$new_name = $rand.".jpg";
            $tmp_name = $ul_plik['tmp_name'];
            $name = $ul_plik['name'];
 
            copy($tmp_name, './tmp/'.$new_name);
 
 
	    resize($new_name,$w_1,"./img/mini/", "");
	    resize($new_name,$w_2,"./img/", "");	
 
 
		unlink('./tmp/'.$new_name);
 
 
	mysql_query("
        INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
        VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','$rand','$cena')
      ");
 
 
        }
		else {
  mysql_query("
        INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
        VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','nazwa','$cena')
      ");
	  }
  }
 
  }
[PHP] pobierz, plaintext 

Ten post edytował Barcelona 22.07.2010, 11:22:34

[Wilu88]

[PHP] pobierz, plaintext 
else {
 
mysql_query("
 
INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
 
VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','nazwa','$cena')
 
");
 
}
 
}
[PHP] pobierz, plaintext 

Na pewno w tym kodzie masz błąd w treści wysyłanej do kolumny obrazek, we wcześniejszym zapytaniu wrzucam tam $rand a w drugim jakaś nazwa

Ten post edytował Wilu88 22.07.2010, 11:50:48

[evolucja]

A po co ci dwa foreachy działające dokładnie tak samo? A, nie dokładnie. Ten pierwszy nie wysyła zapytania Po prostu usuń pierwszą część a zostaw tylko:

[PHP] pobierz, plaintext 
if ($_FILES){ 
foreach( $_FILES as $ul_plik )
{
	if( is_uploaded_file( $ul_plik['tmp_name'] ) )
        {
$rand = f_rand(3);
 
 
		$new_name = $rand.".jpg";
            $tmp_name = $ul_plik['tmp_name'];
            $name = $ul_plik['name'];
 
            copy($tmp_name, './tmp/'.$new_name);
 
 
	    resize($new_name,$w_1,"./img/mini/", "");
	    resize($new_name,$w_2,"./img/", "");	
 
 
		unlink('./tmp/'.$new_name);
 
 
	mysql_query("
        INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
        VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','$rand','$cena')
      ");
 
 
        }
		else {
  mysql_query("
        INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
        VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','nazwa','$cena')
      ");
	  }
  }
 
  }
[PHP] pobierz, plaintext 

Dodawanie kolejnych zdjęć to już tylko stworzenie odpowiednich inputów i ich nazwanie - bez zmian w kodzie.

[Wilu88]

A nie prościej zrobić tak:

Ujednolicić nazwy inputów z plikami np: $ul_plik1, $ul_plik2 itd. I jeśli z góry zakładasz zż będzie np 5 plików zrobić pętle z 5 przebiegami i za każdym przebiegiem robić INSERTA?

Ten post edytował Wilu88 22.07.2010, 12:02:06

[Barcelona]

@evolucja, ale czy jak usunę pierwszą część to czy będą się tworzyły miniaturki do kolejnych wgrywanych zdjęć. Zależy mi jeszcze żeby nazwy tych plików dodawać do bazy mysql.

Na pewno w tym kodzie masz błąd w treści wysyłanej do kolumny obrazek, we wcześniejszym zapytaniu wrzucam tam $rand a w drugim jakaś nazwa

zapytanie jest dobre. Ta 'nazwa' w zapytaniu jest nazwą pliku. Działa to tak --> jak nie zostanie dodane zdjęcie to wtedy w polu obrazek zostaje dodany domyslny obrazek o nazwie "nazwa" a jak obrazek zostaje dodany to wtedy leci rand z nazwa pliku.

@Wilu88 mógłbyś napisać jak to zrobić w pętli bo właśnie rozważam również takie rozwiązanie ale nie wiem jak się za nie zabrać.

[evolucja]

A nie prościej zrobić tak:

Ujednolicić nazwy inputów z plikami np: $ul_plik1, $ul_plik2 itd. I jeśli z góry zakładasz zż będzie np 5 plików zrobić pętle z 5 przebiegami i za każdym przebiegiem robić INSERTA?

To rozwiązanie jest po prostu głupie. Po co ma przerabiać to na statyczne skoro już ma teraz zrobione dynamicznie?

@evolucja, ale czy jak usunę pierwszą część to czy będą się tworzyły miniaturki do kolejnych wgrywanych zdjęć. Zależy mi jeszcze żeby nazwy tych plików dodawać do bazy mysql.

A sprawdziłeś to w ogóle czy tylko tak gadasz?

[Wilu88]

[PHP] pobierz, plaintext 
<?
for( $a=1 ; $a<=5 ; $a++) {
 
 
if( is_uploaded_file($ul_plik'.$a.'['tmp_name'] ) )
 
{
 
$rand = f_rand(3);
 
 
$new_name = $rand.".jpg";
 
$tmp_name = $ul_plik'.$a.'['tmp_name'];
 
$name = $ul_plik'.$a.'['name'];
 
 
 
copy($tmp_name, './tmp/'.$new_name);
 
 
 
resize($new_name,$w_1,"./img/mini/", "");
 
resize($new_name,$w_2,"./img/", "");
 
unlink('./tmp/'.$new_name);
 
 
mysql_query("
 
INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
 
VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','$rand','$cena')
 
");
 
}
?>
[PHP] pobierz, plaintext 

tyle że nie jestem pewien tej linijki

Kod

if( is_uploaded_file($ul_plik'.$a.'['tmp_name'] ) )

i łączenia dwóch zmiennych pokombinujesz i powinno być ok

Ten post edytował Wilu88 22.07.2010, 12:36:26

[evolucja]

#Wilu88
No i po co to? Jaki jest sens przerabiania czegoś co działa i jest dynamiczne, na coś takiego?

[Wilu88]

Kolega poprosił o kod to przedstawiłem, a czy jest sens to niech on zdecyduje.

[Barcelona]

@evolucja własnie sprawdziałem to co napisałeś, czyli usunąłem tą pierwszą część kodu. I teraz na ftp zostaje wgrana prawidłowa ilość plików, jak wrzucam jeden to wrzuca się jedno a jak dwa to wrzucają się dwa. Tylko problem jest taki że tworzą się podwójne ogłoszenia. Napisze inaczej:
Do formularza dodaje dwa pliki i jak dodam to wszystko do bazy to tworzą się dwa ogłoszenia. Pierwsze z tym zdjęciem pierwszym oraz drugie ogłoszenie z tym zdjęciem drugim. Tak jakby pętla wykonywała zapytanie do bazy dla każdego obrazka z osobna.

@Wilu88 jednak pozostane przy tym co teraz jestem bo nie ma sensu przerabiać coś co już działa, ale ma małe niedociągnięcia.

Wkleje cały kod dla pewności

[PHP] pobierz, plaintext 
<?
 
 
function f_rand($ile) {
for ($i=1; $i<=$ile; $i++)
{$l_rand .= dechex(mt_rand());}
return $l_rand;
}
 
 
function resize($foto, $szerokosc, $folder, $napis) {
 
$fotos = "./tmp/".$foto;
$mini = $folder."/".$foto;
if (!file_exists($mini)) {
$img = imagecreatefromjpeg($fotos);
$width  = imagesx($img); // szerokosc jpga
$height = imagesy($img); // wysokosc jpga
 
if ($width > $szerokosc || $height > $szerokosc){
//poziomy / szerokosc
if ($width>=$height) {$width_mini = $szerokosc;
		      $x = $width/$width_mini; 
		      $height_mini = $height/$x;
			  $litery_wielkosc = $width/60;}
 
//pionowy / wysokosc
if ($width<$height) {$height_mini = $szerokosc;
		     $x = $height/$height_mini; 
		     $width_mini = $width/$x;
			 $litery_wielkosc = $height/60;}
 
$img_mini = imagecreatetruecolor($width_mini, $height_mini);
}
else 
{
$img_mini = $img;
}
$txt = $napis;
$fontpath = realpath('.');
$kolor_tla = imagecolorallocate ( $img, 255, 255, 255 );
 
imagecopyresampled($img_mini, $img, 0, 0, 0, 0, $width_mini , $height_mini, $width, $height);
imagejpeg($img_mini, $mini, 100);
@imagedestroy($img);
@imagedestroy($img_mini);
}
}
 
 
 
if ($_SESSION["zalogowany"]=="tak") {
include('menu.php');
echo "Ogłoszenia s? najważniejsz? funkcj? tego serwisu. Masz tutaj możliwo?ć dodawania ogłoszeń do bazy danych.<br> Podczas pisania ogłoszenia należy wypełnić wszystkie pola!<br><br>
<h2>UWAGA!! Jeżeli zdjęcie nie zostanie wybrane wtedy ogłoszenie zostanie dodane ze standardowym zdjęciem.</h2>";
$tytul = mysql_real_escape_string (trim($_POST['tytul']));
$data = mysql_real_escape_string (trim($_POST['data']));
$tresc = $_POST['tresc'];
$stan = mysql_real_escape_string (trim($_POST['stan']));
$rodzaj = mysql_real_escape_string (trim($_POST['rodzaj']));
$marka = mysql_real_escape_string (trim($_POST['marka']));
$model = mysql_real_escape_string (trim($_POST['model']));
$rocznik = mysql_real_escape_string (trim($_POST['rocznik']));
$miejsca = mysql_real_escape_string (trim($_POST['miejsca']));
$klimatyzacja = mysql_real_escape_string (trim($_POST['klimatyzacja']));
$rejestracja = mysql_real_escape_string (trim($_POST['rejestracja']));
$cena = mysql_real_escape_string (trim($_POST['cena']));
$ul_plik = mysql_real_escape_string (trim($_POST['foto']));
 
 
if (strlen($tytul)>2) {
 
  ////////////////////////////////////////
 
 //ustawienie wielkosci zmniejszanych obrazow
$w_1 = 175;
$w_2 = 400;
 
 
if ($_FILES){ 
foreach( $_FILES as $ul_plik )
{
	if( is_uploaded_file( $ul_plik['tmp_name'] ) )
        {
 
$rand = f_rand(3);
 
		$new_name = $rand.".jpg";
            $tmp_name = $ul_plik['tmp_name'];
            $name = $ul_plik['name'];
 
            copy($tmp_name, './tmp/'.$new_name);
 
 
 
	    resize($new_name,$w_1,"./img/mini/", "");
	    resize($new_name,$w_2,"./img/", "");	
 
 
		unlink('./tmp/'.$new_name);
 
 
 
	mysql_query("
        INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
        VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','$rand','$cena')
      ");
 
 
        }
		else {
  mysql_query("
        INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
        VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','nazwa','$cena')
      ");
	  }
  }
 
  }
 
  echo "
  <br><div class=\"potwierdzenie\"><center>Ogłoszenie zostało dodane</center>
    </div>";
 
 
	}
$data = date("Y-m-d H:i:s");
echo <<<KONIEC
<br>
<form action="" method="post" enctype="multipart/form-data" class="uniForm">
 
<fieldset><legend>Dane Samochodu</legend>
<div class="ctrlHolder">
  <label for="foto">Dodaj zdjęcie do ogłoszenia:</label><br>
  <input type="file" name="foto"> <font size="1" color="red">*To zdjęcie będzie zdjęciem głównym, kolejne nie musz? być dodane</font><br>
  <input type="file" name="foto_dwa"><br>
  <label for="marka">Podaj marke samochodu:</label>
  <input type="text" name="marka" style="width:100%;" class="textInput small" />
  <label for="model">Podaj model samochodu:</label>
  <input type="text" name="model" style="width:100%;" class="textInput small" />
  <label for="rocznik">Podaj rocznik samochodu:</label>
  <input type="text" name="rocznik" style="width:100%;" class="textInput small" />
  <label for="miejsca">Podaj ilo?ć miejsc siedz?cych:</label>
  <input type="text" name="miejsca" style="width:100%;" class="textInput small" />
  <label for="klimatyzacja">Klimatyzacja:</label><br>
  <select id="" name="klimatyzacja"><optgroup label='Wybierz jedn? opcję'><option value='TAK'>TAK</option><option value='NIE'>NIE</option></optgroup></select><br>
  <label for="rejestracja">Podaj rejestracje samochodu*:</label>
  <input type="text" name="rejestracja" style="width:100%;" class="textInput small" />
  </div></fieldset>
  <fieldset><legend>Ogłoszenie</legend>
<div class="ctrlHolder">
  <label for="tytul">Podaj tytuł ogłoszenia:</label>
  <input type="text" name="tytul" style="width:100%;" class="textInput small" />
  <label for="model">Podaj date ogłoszenia:</label>
  <input type="text" name="data" style="width:100%;" class="textInput small" value="$data" />
  <label for="rocznik">Napisz tre?ć ogłoszenia:</label><br>
  <textarea name="tresc" id="editor"></textarea><br>
KONIEC;
  ?>
<script type="text/javascript">
			//<![CDATA[
 
				CKEDITOR.replace( 'editor',
					{
						fullPage : true
					});
 
			//]]>
			</script>
<?
echo <<<KONIEC
<label for="cena">Podaj cene (zł):</label>
  <input type="text" name="cena" style="width:100%;" class="textInput small" />
</div></fieldset>
<fieldset><legend>Informacje dodatkowe</legend>
<div class="ctrlHolder">
	  <p class="label">
            Wybierz rodzaj ogłoszenia, oraz czy ma on zostać wy?wietlony na stronie
          </p><table width="100%">
          
 
		  <tr><td align="middle"><ul class="alternate">
            <li><label for="rodzaj" align="top">Rodzaj <select id="" name="rodzaj"><optgroup label='Wybierz jedn? opcję'><option value='wynajem_autobusow'>Wynajem autobusów</option><option value='wynajem_busow'>Wynajem busów</option><option value='sprzedaz_busow'>Sprzedaż busów</option><option value='transport_ludzi'>Transport ludzi</option></optgroup></select></label></li></td><td align="middle">
            <li><label for="stan" align="middle">Stan <select id="" name="stan"><optgroup label='Aktualne'><option value='1'>Ogłoszenie widoczne</optgroup><optgroup label='Wybierz jedn? opcję'><option value='1'>Ogłoszenie widoczne</option><option value='0'>Ogłoszenie niewidoczne</option></optgroup></select></label></li>
          </ul></td></tr></table>
        </div></fieldset>
  <div class="buttonHolder">
<input type="submit" value=" wy?lij" class="primaryAction" />
</div>
KONIEC;
} 
else {
echo "<div class=\"potwierdzenie\"><center>Proszę się zalogować!</center>
    </div>";
 
}
 
?>
[PHP] pobierz, plaintext 

Ten post edytował Barcelona 22.07.2010, 14:04:00

[evolucja]

Nie chce mi się czytać całego kodu, wybacz. Cały problem w tym, że my tak naprawdę nie wiemy do czego to służy i jak ma działać A ty gdybyś popatrzył na klamry w programie który je podświetla(np. notepad++) zauważył byś że zapytania wykonują się w pętli foreach.

[PHP] pobierz, plaintext 
if ($_FILES) { 
foreach( $_FILES as $ul_plik )
{
	if( is_uploaded_file( $ul_plik['tmp_name'] ) )
        {
 
$rand = f_rand(3);
 
		$new_name = $rand.".jpg";
            $tmp_name = $ul_plik['tmp_name'];
            $name = $ul_plik['name'];
 
            copy($tmp_name, './tmp/'.$new_name);
 
 
 
	    resize($new_name,$w_1,"./img/mini/", "");
	    resize($new_name,$w_2,"./img/", "");	
 
 
		unlink('./tmp/'.$new_name);
 
 
 
  }       
  }
   	mysql_query("
        INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
        VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','$rand','$cena')
      ");
 
}  else {
	mysql_query("
        INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
        VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','nazwa','$cena')
      "); } 
[PHP] pobierz, plaintext 

[Barcelona]

Tak jak napisałeś w moim notepad++ z którego korzystam mam podświetlone klamry

A więc przeniosłem zapytanie po za pętle foreach i teraz dodaje tylko jedno ogłoszenie do bazy danych

Ale jednak teraz mam problem z opcją gdy zdjęcie nie jest dodane w formularzu.
Zasada działania jest taka, jak nie jest zadna fotka wybrana wtedy do bazy zostaje dodane ogłoszenie z nazwą domyślnej fotki która jest juz na ftp. I teraz jak nie wybiore zadnej fotki to tworzą się dwa ogłoszenia z ta domyślną (to chyba jest zależne od ilości inputów w formularzu). Tak, wiem identyczna sprawa jak w przypadku wyżej, zapytanie znajduje się w pętli, ale jednak jak to zrobić po za pętlą żeby skrypt wiedział że nie została wybrana żadna fotka i dodał jedno ogłoszenie z domyślną nazwą.
Mam tam warunek if( is_uploaded_file( $ul_plik['tmp_name'] ) ) i jak jest spełniony wtedy dodaje się ogloszenie z fotą a jak nie to z domyślną nazwą tylko to wszystko znajduje się w foreach.
Aktualnie ta część kodu wygląda tak (przeniosłem zapytanie po za foreach)

[PHP] pobierz, plaintext 
if ($_FILES){ 
foreach( $_FILES as $ul_plik )
{
	if( is_uploaded_file( $ul_plik['tmp_name'] ) )
        {
 
$rand = f_rand(3);
 
		$new_name = $rand.".jpg";
            $tmp_name = $ul_plik['tmp_name'];
            $name = $ul_plik['name'];
 
            copy($tmp_name, './tmp/'.$new_name);
 
 
 
	    resize($new_name,$w_1,"./img/mini/", "");
	    resize($new_name,$w_2,"./img/", "");	
 
 
		unlink('./tmp/'.$new_name);
 
 
        }
		else {
  mysql_query("
        INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
        VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','nazwa','$cena')
      ");
	  }
  }
 
  }
mysql_query("
        INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
        VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','$rand','$cena')
      ");
[PHP] pobierz, plaintext 

[evolucja]

Przecież dałem ci już poprawioną i działającą funkcję w poprzednim moim poście, to ty dalej kombinujesz.

Ten post edytował evolucja 22.07.2010, 16:10:49

[Barcelona]

Racja, przepraszam najmocniej Myślałem że wskazujesz mi miejsce błędu.

Już wystawiłem podziekowanie, ale jeszcze raz wielkie dzieki bo było to bardzo ważne dla mnie.

Jeszcze mam takie małe pytanko.
Jak z tej pętli wydobyć nazwy wszystkich plików bo chce w tabeli w bazie danych zapisywać te nazwy i wyświetlać (jako obrazki) przy konkretnym ogłoszeniu.
Ten pierwszy obrazek $ul_plik jest zapisywany jako $rand w bazie danych, a nie wiem jak to zrobić z kolejnymi.

--------------
Wracam z editem:
@evolucja działa prawie dobrze. Skrypt nie wykrywa że nie został dodany obrazek i dodaje puste pole w bazie.
Funkcja rand odpowiada za tworzenie unikatowych nazw plików, i tutaj nie wykrywa że nie ma pliku i nie randuje i daje pustą wartość. Próbowałem nawet w phpmyadmin zrobić domyślną wartość dla pola "obrazek" ale i tak dodawał się pusty rekord.

Ten post edytował Barcelona 22.07.2010, 17:46:12

[Wilu88]

No to w tym drugim zapytaniu przypisz kolumnie obrazek jakąś wartość która będzie zawsze wysyłana gdy nie będzie obrazka.

[PHP] pobierz, plaintext 
mysql_query("
 
INSERT INTO ogloszenia (tytul, data, tresc, stan, rodzaj, marka, model, rocznik, miejsca, klimatyzacja, rejestracja,obrazek,cena)
 
VALUES ('$tytul','$data','$tresc','$stan','$rodzaj','$marka','$model','$rocznik','$miejsca','$klimatyzacja','$rejestracja','obrazek','$cena')
 
");
[PHP] pobierz, plaintext 

W tym wypadku powinien przesłać wartość 'obrazek' do kolumny obrazek.