[PHP]Zapis właściwości checkbox do bazy danych Opcje

[Kerth]

Witam,
mam pewien problem. Z bazy danych pobieram zdjęcia, które tworzą pewnego rodzaju galerię(a raczej nazwy zdjęć), które później są odpowiednio wyświetlane. Obok każdego zdjęcia mam pole typu checkbox. Do każdego artykułu można będzie dodać dajmy na to 12 zdjęć. I tu pojawia się pytanie. W jaki sposób najlepiej zapisać wartość tych zdjęć do bazy danych? Z ustawieniem parametry value nie ma problemu, bo pobieram to z bazy danych i potem tylko wstawiam. Zastanawia mnie kwestia, czy do tabeli z artykułami(`article`) wstawić kolumny typu `image1`, image2`, image3` itd, czy może skorzystać z pól typu ENUM/SET?

Tak to wygląda, zdjęcia po kopiowane więc nazwy są takie same - bez znaczenia:
{{Image:edit.png}}

I teraz jak zaznaczę 2 zdjęcia to do bazy zapiszą się 2, jak 5 to 5, etc. Ma ktoś jakiś pomysł jak to rozwiązać? Czekam na kreatywne pomysły.
Pozdrawiam

Zdecydowałem się na tą drugą opcję. W ten sposób tworzę checkbox dla każdego zdjęcia:

[PHP] pobierz, plaintext 
<input type="checkbox" value="'.$getImage['name'].'" name="'.$getImage['name'].'" style="vertical-align: middle;"/>
[PHP] pobierz, plaintext 

Przypisuje mu dla atrybutu value indywidualną wartość, która ma zostać zapisana w bazie danych. Problem jest z atrybutem name. Jak go dynamicznie utworzyć?

---Edit:
W ten sposób przypisuję zmienną $_POST do metod setImage1 oraz setImage2

[PHP] pobierz, plaintext 
if($rowAllImage){
	foreach($rowAllImage as $getImage){
		$news->setImage1($_POST[''.$getImage['name'].'']);
		$news->setImage2($_POST[''.$getImage['name'].'']); 
	}
}
[PHP] pobierz, plaintext 

Co skutkuje wyświetleniem notatek typu: Undefined variables...

Pytane brzmi, jak zapisać do bazy TYLKO TE POLA co są zaznaczone?

[nospor]

Name ma miec stala nazwe tablicowa, np: images[]

<input type="checkbox" value="'.$getImage['name'].'" name="images[]" style="vertical-align: middle;"/>

A postem poleca tylko te checkboxy, ktore zostaly zaznaczone:

print_r($_POST['images']);

I wszystko jasne

[Kerth]

Ok, niby "przesyła" zawartość zaznaczonych opcji tablicą POST, lecz pojawia się ostrzeżenie:

[PHP] pobierz, plaintext 
Array ( [0] => photo3.jpg [1] => photo4.jpg ) 
Warning: trim() expects parameter 1 to be string, array given in C:\xampp\htdocs\img\news\model\news.class.php on line 19
 
Warning: trim() expects parameter 1 to be string, array given in C:\xampp\htdocs\img\news\model\news.class.php on line 20
[PHP] pobierz, plaintext 

To są te 2 linijki:
public function doFilter(){
$this->subject = trim($this->subject);
$this->content = trim($this->content);
$this->status = trim($this->status);
$this->headImage = trim($this->headImage);
$this->image1 = trim($this->image1);
$this->image2 = trim($this->image2);
return $this;
}

Metody image1, image2:

[PHP] pobierz, plaintext 
public function setImage1($image1){
	$this->image1 = $image1;
	return $this;
}
public function setImage2($image2){
	$this->image2 = $image2;
	return $this;
}
[PHP] pobierz, plaintext 

Dlaczego tak się dzieje?

[nospor]

No ale pokaz kod po zmianach jak teraz ustawiasz to setImage

[Kerth]

No w ten sposób:

[PHP] pobierz, plaintext 
$news->setImage1($_POST['images']);
$news->setImage2($_POST['images']); 
print_r($_POST['images']);
[PHP] pobierz, plaintext 

a W klasie binduję:

[PHP] pobierz, plaintext 
$img->bindValue(":image1", $this->image1, PDO::PARAM_STR);
$img->bindValue(":image2", $this->image2, PDO::PARAM_STR);
[PHP] pobierz, plaintext 

[nospor]

No to ci pokazalem i napisalem ze $_POST['images'] to tablica wiec czemu przypisujesz tablice jako nazwe image? Jako nazwe image masz przypisac element tej tablicy a nie tablice.

[Kerth]

A w jaki sposób mam przpypisać element z tablicy images[]?
Ja to rozumiem, tak, że jeśli tablica

[PHP] pobierz, plaintext 
$_POST['images']
[PHP] pobierz, plaintext 

zwraca mi nazwy zaznaczonych checkboxów, to też chciałbym tą zawartość tablicy zapisać do bazy danych. Jak się dowiedzieć o, który konkretny element może chodzić?

Dzięki nospor. Teraz mi trybiki zaskoczyły

Rozwiązanie:

[PHP] pobierz, plaintext 
$news->setImage1($_POST['images'][0]);
$news->setImage2($_POST['images'][1]);
[PHP] pobierz, plaintext 

Dzięki za podpowiedzi.

Link, który mi pomógł(dla potomności): http://phpedia.pl/wiki/Jak_uzyska%C4%87_do..._formularzem%3F

A jeszcze jedno pytanie, jak pozbyć się tych notatek:

[PHP] pobierz, plaintext 
Notice: Undefined offset: 2 in C:\xampp\htdocs\img\Img\controller\edit\addEditImg.php on line 26
 
Notice: Undefined offset: 3 in C:\xampp\htdocs\img\Img\controller\edit\addEditImg.php on line 27
 
Notice: Undefined offset: 4 in C:\xampp\htdocs\img\Img\controller\edit\addEditImg.php on line 28
 
Notice: Undefined offset: 5 in C:\xampp\htdocs\img\Img\controller\edit\addEditImg.php on line 29
 
Notice: Undefined offset: 6 in C:\xampp\htdocs\img\Img\controller\edit\addEditImg.php on line 30
 
Notice: Undefined offset: 7 in C:\xampp\htdocs\img\Img\controller\edit\addEditImg.php on line 31
 
Notice: Undefined offset: 8 in C:\xampp\htdocs\img\Img\controller\edit\addEditImg.php on line 32
 
Notice: Undefined offset: 9 in C:\xampp\htdocs\img\Img\controller\edit\addEditImg.php on line 33
 
Notice: Undefined offset: 10 in C:\xampp\htdocs\img\Img\controller\edit\addEditImg.php on line 34
 
Notice: Undefined offset: 11 in C:\xampp\htdocs\img\Img\controller\edit\addEditImg.php on line 35
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
Array ( [0] => photo7.jpg [1] => photo8.jpg )
[PHP] pobierz, plaintext 

Dzieje się tak wtedy gdy zaznaczę np. 2 zdjęcia z możliwych 12. I każdy numer w notatce odnosi się do elementu tablicy.Jeśli zaznaczę wszystkie możliwe 12 to notatek nie ma.

Mniemam, że trzeba wykorzystać jakiś warunek, żeby przesyłało tylko te, które mają checked na true:

[PHP] pobierz, plaintext 
$news->setImage1($_POST['images'][0]);
$news->setImage2($_POST['images'][1]);
$news->setImage3($_POST['images'][2]);
$news->setImage4($_POST['images'][3]);
$news->setImage5($_POST['images'][4]);
$news->setImage6($_POST['images'][5]);
$news->setImage7($_POST['images'][6]);
$news->setImage8($_POST['images'][7]);
$news->setImage9($_POST['images'][8]);
$news->setImage10($_POST['images'][9]);
$news->setImage11($_POST['images'][10]);
$news->setImage12($_POST['images'][11]);
[PHP] pobierz, plaintext 

Tylko jak takie coś zrobić?

[Fred1485]

W pętli spróbuj

[PHP] pobierz, plaintext 
 
foreach ($_POST['images'] as $image) {
 //dodaj
}
[PHP] pobierz, plaintext 

[Kerth]

Jak taka pętla powinna wyglądać poprawnie?

Próbuję w ten sposób, jednakże żadne obrazki nie są dodawane do bazy danych.

[PHP] pobierz, plaintext 
foreach($_POST['images'] as $j){
	$img->setImage1($_POST['images'][$j]);
	$img->setImage2($_POST['images'][$j]);
	$img->setImage3($_POST['images'][$j]);
	$img->setImage4($_POST['images'][$j]);
	$img->setImage5($_POST['images'][$j]);
	$img->setImage6($_POST['images'][$j]);
	$img->setImage7($_POST['images'][$j]);
	$img->setImage8($_POST['images'][$j]);
	$img->setImage9($_POST['images'][$j]);
	$img->setImage10($_POST['images'][$j]);
	$img->setImage11($_POST['images'][$j]);
	$img->setImage12($_POST['images'][$j]);
}
[PHP] pobierz, plaintext 

[Fred1485]

Nie wiem po co ci do każdego image osobny setter
Przykładowa implementacja. Oczywiście ty masz swoją klasę Images, ja użyłem tylko dla testów.

[PHP] pobierz, plaintext 
class Images
{
    private $image1;
    private $image2;
    private $image3;
    private $image4;
    private $image5;
    private $image6;
    private $image7;
    private $image8;
    private $image9;
    private $image10;
    private $image11;
    private $image12;
 
 
    public function setImage1($image1)
    {
        $this->image1 = $image1;
    }
 
    public function setImage2($image2)
    {
        $this->image2 = $image2;
    }
 
    public function setImage3($image3)
    {
        $this->image3 = $image3;
    }
 
    public function setImage4($image4)
    {
        $this->image4 = $image4;
    }
 
    public function setImage5($image5)
    {
        $this->image5 = $image5;
    }
 
    public function setImage6($image6)
    {
        $this->image6 = $image6;
    }
 
    public function setImage7($image7)
    {
        $this->image7 = $image7;
    }
 
    public function setImage8($image8)
    {
        $this->image8 = $image8;
    }
 
    public function setImage9($image9)
    {
        $this->image9 = $image9;
    }
 
    public function setImage10($image10)
    {
        $this->image10 = $image10;
    }
 
    public function setImage11($image11)
    {
        $this->image11 = $image11;
    }
 
    public function setImage12($image12)
    {
        $this->image12 = $image12;
    }
 
    public function getImages()
    {
        $response =[];
        for ($i = 1; $i <= 12; $i++) {
            if ($this->{'image'.$i} !== null) {
                $response[] = $this->{'image'.$i};
            }
        }
        return $response;
    }
}
 
$_POST['images'] = [
    0 => 'image1',
    4 => 'image5',
    5 => 'image6',
    8 => 'image9',
    11 => 'image12',
];
 
$images = new Images();
 
foreach ($_POST['images'] as $index => $image) {
    $images->{'setImage'.++$index}($image);
}
 
echo '<pre>';
 
print_r($images->getImages());
[PHP] pobierz, plaintext 

Ten post edytował Fred1485 18.07.2016, 20:01:33

[Kerth]

Kod bardzo podobny do mojego. Po podstawieniu metody od Ciebie pojawia się Fatal error:

[PHP] pobierz, plaintext 
Fatal error: Uncaught Error: Call to a member function getImages() on string in 
C:\xampp\htdocs\img\controller\edit\addEditImg.php:21 Stack trace: #0 
C:\xampp\htdocs\img\view\edit_Img.php(30): require_once() #1 {main} thrown in 
C:\xampp\htdocs\img\controller\edit\addEditImg.php on line 21
[PHP] pobierz, plaintext 

Odnosi się to właśnie do tej metody. Wiesz co zrobić?

[Fred1485]

Wklej cały kod na http://pastebin.com/ jak możesz i daj linka.