[PHP]Aktualizowanie kilku checkboxów poprzez foreach Opcje

[casperii]

Witam panowie, mam kod , który ma za zadanie po array_key_exsist polecieć foreachem wszystkie zaznaczone checkboxy i jeżeli w bazie mam 1 to robi 0 , jeżeli mam 0 to robi 1.

[PHP] pobierz, plaintext 
$odbieram = $_POST['plik'];
if(isset($odbieram)){
	foreach ($odbieram as $cos=>$value){
		$sprawdz = mysql_query("SELECT * FROM `files` WHERE `plik` = '".$value."'") or die(mysql_error());
		$sprawdzam=mysql_fetch_array($sprawdz,MYSQL_ASSOC);					
		$wyciagam = $sprawdzam['onof'];
 
		if($wyciagam=='1'){
			$zapisuje = '0';
		}else{
			$zapisuje = '1';
		}
 
 
		$aktualizuje = ("UPDATE `files` SET `onof` = '".$zapisuje."' WHERE `plik` = '".$value."'") or die (mysql_error());
		$wykonuje_aktualizuje = mysql_query($aktualizuje);
		if($result_upd){
			print 'skonfigurowałem:'.$value.'<br>';
		}								
	}
}else{
	print('Komunikat jeżeli nie wybrano nic');
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<form...
...
zapytanie do bazy w celu sprawdzenia czy checkbox mam zaznaczony
print('<input type="checkbox" name="plik[]" value="'.$row['plik'].'"');if($row['onof']){print('checked="checked"');}print('>'.$row['plik'].'<br>');
..
</form>
[PHP] pobierz, plaintext 

Niestety powyższy kod jakoś dziwnie działa. zaptaszkuje zapisze to 1 zapisuje drugi nie zapisuje. Albo jak mam np 2 checkboxy zaznaczone i 1 odznacze to mi odznacza wszystkie checkboxy.

Ten post edytował casperii 17.09.2015, 09:05:58

[nospor]

Czemu do sprawdzania pobierasz dane z tabelka a updatujesz w files?

[casperii]

Źle napisałem :-) oczywiście tabelka files

[nospor]

Pokaz jak wyglada tabelka files oraz co zwraca
print_r($_POST);
w sytuacji, gdy rzekomo źle działa.

[Tomplus]

W linii 15: dajesz do stringa funkcję die() która powinna być dla mysql_query()
W ogóle zbędna rzecz, bo lepiej sprawdzić przed wrzuceniem do UPDATE/SELECT czy nasze wartości są prawidłowe.

czyli if (is_integer($value)) mysql_query ($update);


Potem: $sprawdz = mysql_query("SELECT * FROM `files` WHERE `plik` = '".$value."'") or die(mysql_error());

Jeżeli sprawdzasz czy wartość ma 1 lub 0, to po co wykonujesz dalej polecenie UPDATE?


zrób if ($sprawdz = mysql_query("SELECT * FROM `files` WHERE `plik` = '".$value."'")) { /* UPDATE */}
else echo " Wartość nie zmieniła się. ";


array_key_exist to sprawdzenie czy klucz w tablicy dla wartości o którą pytam istnieje, ale jeżeli już to powinnieneś pytać się o:

if (array_key_exist('plik',$_POST)) ...
lub if (array_key_exist($cos,$odbieram)) ale to bez sensu.

[KsaR]

Skróciłem ci ten kod z 1 Postu do identycznego z nudów ]:-).
Tylko troche zaciemniony;

[PHP] pobierz, plaintext 
if (isset($_POST['plik'])) foreach ($_POST['plik'] as $v) (false!==mysql_query("UPDATE `files` SET `onof`=IF(`onof`='0','1', '0') WHERE `plik` = '".$v."'")) ? print('skonfigurowałem:'.$v.'<br>'):exit(mysql_error());
else echo 'Komunikat jeżeli nie wybrano nic';
[PHP] pobierz, plaintext 

if($result_upd){
print 'skonfigurowałem:'.$value.'<br>';
}
^ a to sie nigdy nie wykona u ciebie bo zmienna nie istnieje :-P
--
Moze malo czytelny, ale zawsze mozna dodac { } . Wedlug mnie taki kod szybciej ogarniesz bo jest 12x krótszy, i zarazem dosc logiczny.

Ten post edytował KsaR 17.09.2015, 10:10:39

[casperii]

@Tomplus mam kilka checkboxów wyciąganych z bazy.

To wyciągam zapytaniem z bazy i ptaszkuje jeżeli ma w bazie status onof = 1

[PHP] pobierz, plaintext 
print('<input type="checkbox" name="plik[]" value="'.$row['plik'].'"');if($row['onof']=='1'){print('checked="checked"');}print('>'.$row['plik'].'<br>');
[PHP] pobierz, plaintext 

To wszystko jest oplecione formem. Po kliknięciu buttona wykonuje się kolejny kod:
Ok niech będzie tu kod użytkownika @KsaR

[PHP] pobierz, plaintext 
if (isset($_POST['plik'])) foreach ($_POST['plik'] as $v) (false!==mysql_query("UPDATE `files` SET `onof`=IF(`onof`='0','1', '0') WHERE `plik` = '".$v."'")) ? print('skonfigurowałem:'.$v.'<br>'):exit(mysql_error());
else echo 'Komunikat jeżeli nie wybrano nic';
[PHP] pobierz, plaintext 

i po wykonaniu tego kodu są jaja. Bo updatuje mi się nawet ten plik , którego nie zaznaczyłem
Czyli reasumując mam 3 pliki plikA , plikB , plikC, plikA i plikB mają w bazie onof=1 czyli są zaptaszkowane, teraz ptaszkuje sobie plikC i daje zapisz to owszem mi się zaptaszkuje ten plikC ale plikA i plikB odptaszkowuje mi się Chociaż powinny być dalej zaptaszkowane bo ja ich nie odptaszkowywałem ręcznie.

[KsaR]

casperii, moj kod robi to co twoj..
Obkomentuje go:

[PHP] pobierz, plaintext 
if (isset($_POST['plik'])) # istnieje w tablicy $_POST klucz plik.
 foreach ($_POST['plik'] as $v) # więc przeglądamy wartosci wyslanych post-ów.
 (false!==mysql_query("UPDATE `files` SET `onof`=IF(`onof`='0','1', '0') WHERE `plik` = '".$v."'")) #czy "skonfigurowalo", oraz zapytanie do bazy, w ktorym jesli `onof` mialo 0 da mu 1 inaczej pozostawi 0.
 ? print('skonfigurowałem:'.$v.'<br>') # jesli wyzszy warunek dal prawde
 :exit(mysql_error()); # jesli nie dal prawdy.
else echo 'Komunikat jeżeli nie wybrano nic'; # $_POST['plik']; nie istnieje, (klucz).
[PHP] pobierz, plaintext 

Aha, ten if ale w zapytaniu zastepuje te liniki:

[PHP] pobierz, plaintext 
	$sprawdz = mysql_query("SELECT * FROM `files` WHERE `plik` = '".$value."'") or die(mysql_error());
		$sprawdzam=mysql_fetch_array($sprawdz,MYSQL_ASSOC);					
		$wyciagam = $sprawdzam['onof'];
 
		if($wyciagam=='1'){
			$zapisuje = '0';
		}else{
			$zapisuje = '1';
		}
[PHP] pobierz, plaintext 

Czyli troche inaczej jednak:
powinno byc zamiast:
IF(`onof`='0','1', '0')
W zapytaniu, tak:
IF(`onof`='1','0', '1')
Wtedy moj kod zrobi dokladnie to co twoj, z tym że dodatkowo zadziala mu ten komunikat ze skonfigurowano, jak i komunikat bledu

Ten post edytował KsaR 17.09.2015, 18:02:49

[casperii]

@KsaR i dOOpa ten twój kod coś nie działa.
Dalej jest tak, że odptaszkowuje mi pliki które je ręcznie nie odptaszkowałem w htmlu.

[Tomplus]

Ja może szukam błędu często po chłopsku, ale

zrób sobie echo/print na różnych stopniach dzialania skryptu. Sprawdź zawartość zmiennych i tablic
np. echo "<pre>".print_r($_POST,1)."</pre>"; aby wyświetlić preformatowaną listę informacji z tablicy. Sprawdź co leci do bazy, jaką wartość masz przed, a jaką po.

[casperii]

tabela files:
plik varchar (64)
onof int(1) - domyślnie 0

@KsaR tak wygląda obecnie kod:

[PHP] pobierz, plaintext 
<?php
if(array_key_exists('zapiszmnie', $_POST)){
	$cosik = $_POST['plik'];
	if(isset($cosik)){
		foreach ($cosik as $v){
			(false!==mysql_query("UPDATE `files` SET `onof`=IF(`onof`='0','1', '0') WHERE `plik` = '".$v."'")) 
			? print('skonfigurowałem:'.$v.'<br>')
			 :exit(mysql_error());
		}
	}else{
		echo 'Nie wybrano pliku.';
	}
}
?>
 
<form name="zapiszmnie" method="post" action="">
<?php
$sql = mysql_query("SELECT * FROM `files`") or die(mysql_error());
if(mysql_num_rows($sql)>0){
	for($i=0;$i<mysql_num_rows($sql);$i++){
		$row = mysql_fetch_array($sql, MYSQL_ASSOC);
		print('<input type="checkbox" name="plik[]" value="'$row['plik'].'"');;if($row['onof']=='1'){print('checked="checked"');}print('>'.$row['plik'].'<br>');
	}
}
?>
<br><br>
<input type="submit" name="zapiszmnie" value="Zapisz">
</form>
[PHP] pobierz, plaintext 

Nie potrafię znaleźć błędu. Dlaczego bierze pod uwagę te checkboxy które nie odznaczyłem i sam nadaje im update z 1 na 0. Czyli nadpisuje jak mu się podoba.

[nospor]

Dlaczego bierze pod uwagę te checkboxy które nie odznaczyłem

Poniewaz formularzem wysylane są tylko zaznaczone checkboxy. To są podstawy HTML a sam bys to zobaczyl, gdybyś raczył posluchac porad, ktore ci dawalismy, a ktorze brzmialy:
zajrzyl w koncu co idzie postem. Ale zignorowales to dwukrotnie i meczysz sie od tygodnia z totalną pierdułą.

[casperii]

@nospor nie zrozumiałeś mnie.
Wiem, że tylko zaznaczone checkboxy idą POSTem. Chodzi mi o to, że jeżeli już jakiś checkbox jest już zaznaczony po wczytaniu strony zaznaczę inne checkboxy, to te wcześniejsze co były zaznaczone odptaszkowują się (czyli nadpisują w bazie onof=0).

[nospor]

Bo tak dziala skrypt, ktory napisales.
Skoro napisales, ze w bazie zmieniasz zaznaczone checkboxy na przeciwne wartosci jakie mają aktualnie w bazie, to skrypt robi co mu każesz.

[casperii]

@nospor racja, skrypt robi to co się jemu dyktuje, źle może na początku wyraziłem się o to co chcę osiągnąć.

obecnie skrypt wygląda tak:

[PHP] pobierz, plaintext 
<?php
if(array_key_exists('zapiszmnie', $_POST)){
	$cosik = $_POST['plik'];
	if(isset($cosik)){
		foreach ($cosik as $v){
			$wykonajmnie= mysql_query("UPDATE `files` SET `onof` = '1' WHERE `plik` = '".$v."'");
		}
	}else{
		echo 'Nie wybrano pliku.';
	}
}
?>
 
<form name="zapiszmnie" method="post" action="">
<?php
$sql = mysql_query("SELECT * FROM `files`") or die(mysql_error());
if(mysql_num_rows($sql)>0){
	for($i=0;$i<mysql_num_rows($sql);$i++){
		$row = mysql_fetch_array($sql, MYSQL_ASSOC);
		print('<input type="checkbox" name="plik[]" value="'$row['plik'].'"');;if($row['onof']=='1'){print('checked="checked"');}print('>'.$row['plik'].'<br>');
	}
}
?>
<br><br>
<input type="submit" name="zapiszmnie" value="Zapisz">
</form>
[PHP] pobierz, plaintext 

Oczywiście powyższy kod jest dla mnie jak najbardziej zrozumiały i zapisuje mi 1 dla tych checkboxów które wybrałem. No ale nasuwa się pytanie w jaki sposób teraz odptaszkowywać je? Mam tworzyć kolejny foreach z empty i w nim update onof=0?
Proszę o porady.

Ten post edytował casperii 3.10.2015, 12:48:49