[MySQL][PHP]Dodawanie wielu elementów tablicy $_FILES do bazy danych Opcje

[babcia_smierc]

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 

[Damonsson]

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

[Sephirus]

Ogólnie możesz zastosować wiele dodawanych wartości w INSERT w SQL przecież (IMG:style_emoticons/default/smile.gif)

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 

[Damonsson]

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

[Sephirus]

W tym przypadku - sam sobie odpowiedz (IMG:style_emoticons/default/smile.gif)

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.

Ten post edytował Sephirus 20.02.2013, 08:48:14

[Damonsson]

Gdybym wiedział, nie pytałbym (IMG:style_emoticons/default/tongue.gif)

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.

Ten post edytował Damonsson 20.02.2013, 08:49:03

[Sephirus]

Gdybym wiedział, nie pytałbym

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

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 (IMG:style_emoticons/default/smile.gif)

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:

Kod

pętla() {
    Tu generuje całe zapytanie SQL z dodawaniem wielu rekordów
}

Tu wykonuje jedno zapytanie

Niż:

Kod

pętla() {
    Każdorazowo wykonuje zapytanie
}

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 (IMG:style_emoticons/default/wink.gif)

EDIT2: Bo zapomniałem - moja metoda wymusza po prostu takie myślenie (IMG:style_emoticons/default/smile.gif)

Ten post edytował Sephirus 20.02.2013, 08:56:40

[babcia_smierc]

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 ?

[Sephirus]

[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 

(IMG:style_emoticons/default/smile.gif)

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

Ten post edytował Sephirus 20.02.2013, 09:27:54

[phpion]

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

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.

[babcia_smierc]

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.

Ten post edytował babcia_smierc 20.02.2013, 09:55:17

[Sephirus]

Na pewno dobrze przepisałeś ten kod?

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

[babcia_smierc]

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 

Ten post edytował babcia_smierc 20.02.2013, 10:04:47

[Sephirus]

ajjj takie przeoczenie (IMG:style_emoticons/default/smile.gif) sam mogłeś wpaść na to (IMG:style_emoticons/default/tongue.gif) Zapomnieliśmy obaj o tym że INPUT FILE ma swoją nazwę (IMG:style_emoticons/default/smile.gif)

[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 

[babcia_smierc]

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

Ten post edytował babcia_smierc 20.02.2013, 10:18:58

[Sephirus]

A odbierasz to ID z parametru GET w ogóle? (IMG:style_emoticons/default/smile.gif)

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

[babcia_smierc]

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 

[Sephirus]

$_GET['id']

A jak się nazywa parametr w adresie? (IMG:style_emoticons/default/smile.gif)

Co masz w atrybucie ACTION w FORM? (IMG:style_emoticons/default/smile.gif)

Chłopie! ;P (IMG:style_emoticons/default/facepalmxd.gif)

[babcia_smierc]

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 

[Sephirus]

Przecież możesz to połączyć w jeden (IMG:style_emoticons/default/smile.gif)

Liczbę zdjęć możesz określi po liczbie plików (IMG:style_emoticons/default/wink.gif)

Z tym musisz już sobie sam poradzić.