Mam taki problem:
Użytkownik wskazuje ilosc zdjec do galeri newsa do wgrania na serwer, po czym tworzona jest tablica $_FILES. W jaki sposób dodać zawartość tej tablicy (wielu rekordów) za pomocą jednego zapytania ?

tabela news_gal do ktorej wstawic chiałbym wiele rekordów wyglada tak:(id, id_newsa, obraz)

oto kod dynamicznego tworzenia ilosci zdjec do wstawienia:

[PHP] pobierz, plaintext 
if(@$_POST['submit'] AND ($_POST['ilosc_zdj'])>0) {
//form start
echo('<form enctype="multipart/form-data" action="dodaj_zdj_akt.php?id_newsa='. $id . '" method="POST">
<input type="hidden" name="MAX_FILE_SIZE" value="1000000" />');
$ilosc = (int)$_POST['ilosc_zdj'];
for($x=1;$x<=$ilosc;$x++) {
echo('<input name="plik[]" type="file" /><br />');
}
[PHP] pobierz, plaintext 

A to kod (błędny) zapytania

[PHP] pobierz, plaintext 
if(@$_POST['submit2']) {
 
 
$add_news_pic = "INSERT INTO `wtz`.`news_gal`(id, id_newsa, obraz) VALUES (NULL, '" . $id . "')";
mysql_query($add_news_pic) or die(mysql_error());
 
}
[PHP] pobierz, plaintext 

Zrób sobie foreach tej tablicy i w nim INSERT, przy okazji możesz skorzystać z transakcji.

Ogólnie możesz zastosować wiele dodawanych wartości w INSERT w SQL przecież

Przykład:

[PHP] pobierz, plaintext 
 
$sql = "INSERT INTO tabelka (id,pole1,pole2) VALUES ";
$sqlValues = array();
 
foreach($tablica AS $element) {
    $sqlValues[] = "(NULL,'{$element['pole1']}','{$element['pole2']}')";
}
 
$sql = $sql . implode(', ',$sqlValues);
 
mysql_query($sql) OR die(mysql_error());
 
[PHP] pobierz, plaintext 

Ładowanie wszystkiego do jednego INSERT'a po przecinku, jest szybsze/wydajniejsze/lepsze, niż oddzielne wykonywanie INSERT'ów bez korzystania z transakcji/z wykorzystaniem transakcji?

W tym przypadku - sam sobie odpowiedz

EDIT: Transakcja tutaj to dodatek - możesz jej równie dobrze użyć w moim zapytaniu lub nie - to już zależy od twórcy - nie to było tematem.

Gdybym wiedział, nie pytałbym

Ale zaryzykuję, że jest równe oddzielnym insertom z wykorzystaniem transakcji?

edit: w tym konkretnym przypadku, to nie ma znaczenia pewnie, ale przy większej ilości, to już ma znaczenie, chodzi mi o ten drugi przypadek.

Ogólnie tak - zgadzam się z tym, ale tylko jeśli autor zbierze wszystkie zapytania w kupę (w jakiś jeden ciąg - łącznie ze startem transakcji) i je wyśle raz do DB - inaczej wykonywanie oddzielnych zapytań (autor korzysta z mysql_***) będzie mniej wydajne

EDIT: Dla duuuuużej liczby rekordów to nawet moją metodę trzeba by było dzielić dodając po X rekordów tak by nie przekroczyć maksymalnej długości zapytania. Tu bardziej chodzi o dobry nawyk. Lepiej od razu zrobić coś na zasadzie:



Niż:



Dla dużej liczby tych plików poszła by masa zapytań, a że autor korzysta z funkcji rodziny mysql_*** to z wydajnością było by to na bakier

EDIT2: Bo zapomniałem - moja metoda wymusza po prostu takie myślenie

A co jesli $_FILES['name'] tez jest tablica ? Tzn. w formularzy <select> wybieram wiecej niz 1 zdjecie... Wtedy to wyglada tak:

[PHP] pobierz, plaintext 
 
Array
(
    [plik] => Array
        (
            [name] => Array
                (
                    [0] => obraz12.jpg
                    [1] => obraz1.jpg
                )
)
[PHP] pobierz, plaintext 

Co trzeba zmodyfikować w:

[PHP] pobierz, plaintext 
$tablica = $_FILES;
$sql = "INSERT INTO news_gal (id,id_newsa,obraz) VALUES ";
$sqlValues = array();
 
foreach($tablica AS $element) {
$sqlValues[] = "(NULL,' . $id . ','{$element['name'][0]}')";
}
 
$sql = $sql . implode(', ',$sqlValues);
 
mysql_query($sql) OR die(mysql_error());
 
[PHP] pobierz, plaintext 

Dodać jeszcze jedną pętle foreach ?

[PHP] pobierz, plaintext 
$tablica = $_FILES['name'];
$sql = "INSERT INTO news_gal (id,id_newsa,obraz) VALUES ";
$sqlValues = array();
 
foreach($tablica AS $element) {
$sqlValues[] = "(NULL,'{$id}','{$element}')";
}
 
$sql = $sql . implode(', ',$sqlValues);
 
mysql_query($sql) OR die(mysql_error());
[PHP] pobierz, plaintext 

EDIT: miałeś mały błąd.

Zdecydowanie (przynajmniej w przypadku PostgreSQL). Przy jednym projekcie miałem okazję sprawdzić to na własnej skórze. Multi-insert jest zdecydowanie szybszy. Zależy oczywiście ile danych jest do wstawienia. Różnica będzie rosła wraz z ilością danych do wstawienia. Oczywiście przy ogromnej liczbie danych warto rozważyć zapis danych do pliku i późniejsze LOAD DATA INFILE.

A błędne czasem nie jest to ze w panelu phpmyadmin w tablei news_gal w ostatniej kolumnie 'obraz' wyswietla nie nazwe obrazu tylko 'Array' ? Bo to własnie tablice mi dodaje a nie konkretna nazwe pliku.
Oto co mi wywala:

Notice: Array to string conversion in E:\xampp\xampp\htdocs\wtz_php\dodaj_zdj_akt.php on line 26

Przypominam, że chciałbym dodać więcej niż jeden plik (obraz). W zakresie 1-10.

Na pewno dobrze przepisałeś ten kod?

Skoro $tablica to $_FILES['name'] to w foreach powinno być wszystko ok...

Zrobiłem kopiuj wklej i oto co mi wywala:

Notice: Undefined index: name in E:\xampp\xampp\htdocs\wtz_php\dodaj_zdj_akt.php on line 21

Warning: Invalid argument supplied for foreach() in E:\xampp\xampp\htdocs\wtz_php\dodaj_zdj_akt.php on line 25
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

a oto linia 25:

[PHP] pobierz, plaintext 
foreach($tablica AS $element) {
    $sqlValues[] = "(NULL,'{$id}','{$element}')";
    }
[PHP] pobierz, plaintext 

ajjj takie przeoczenie sam mogłeś wpaść na to Zapomnieliśmy obaj o tym że INPUT FILE ma swoją nazwę

[PHP] pobierz, plaintext 
// W tym miejscu powinieneś też sprawdzać czy w ogóle w $_FILES coś jest itd...
$tablica = $_FILES['plik']['name'];
$sql = "INSERT INTO news_gal (id,id_newsa,obraz) VALUES ";
$sqlValues = array();
 
foreach($tablica AS $element) {
$sqlValues[] = "(NULL,'{$id}','{$element}')";
}
 
$sql = $sql . implode(', ',$sqlValues);
 
mysql_query($sql) OR die(mysql_error());
[PHP] pobierz, plaintext 

Działa !
Tylko nie dodaje mi identyfikatora newsa do bazy (id,id_newsa,obraz). Nie wiecie dlaczego ?

A odbierasz to ID z parametru GET w ogóle?

[PHP] pobierz, plaintext 
$id = mysql_real_escape_string($_GET['id_newsa']);
 
// ... dalszy kod
[PHP] pobierz, plaintext 

Tak, na początku strony:

[PHP] pobierz, plaintext 
include("header.php");
if($_SESSION['USERLEVEL'] != 10) {
	header("Location: index.php?page=main");
}
@$id = $_GET['id'];
 
...
[PHP] pobierz, plaintext 

A jak się nazywa parametr w adresie?

Co masz w atrybucie ACTION w FORM?

Chłopie! ;P

Problem w tym ze na jednej stornie mam 2 formularze (1 do okreslenia ilosci zdjec która chce sie uploadowac a 2 do wysyłania tych zdjec).

1:

[PHP] pobierz, plaintext 
echo('<form enctype="multipart/form-data" action="dodaj_zdj_akt.php?id_newsa='. $id . '" method="POST">
[PHP] pobierz, plaintext 

2:

[PHP] pobierz, plaintext 
echo '<form action=dodaj_zdj_akt.php?id_newsa=' . $id . ' method=POST>';
echo '<p class=form>Podaj ilosc zdjęć do wstawienia</p>';
echo '<select name=ilosc_zdj>';
[PHP] pobierz, plaintext 

Przecież możesz to połączyć w jeden

Liczbę zdjęć możesz określi po liczbie plików

Z tym musisz już sobie sam poradzić.