[AJAX][MySQL][PHP]Zmiana wartości pola w bazie Opcje

[djgarsi]

Witam. Próbuję właśnie zrobić regulację wartości pola (+/-) w bazie danych.
Wygląda to mniej więcej tak: + 10 -, po kilknięciu na + chcę aby zwiększyła się wartość pola w bazie np o 5 i w miejscu 10 pojawiło się 15.

Obecnie stworzyłem skrypt ajax'a ale nie działa.

Kod

$(window).load(function(){
    function zwieksz(id){
        $.ajax({
           type: "POST",
           url: "zm_ob.php?id=" + id + "",
           data: "akcja=dodaj",
           success: function(wynik) {
            $("#ile").html(wynik)
           }
          });
          return false;
    }
    function zmniejsz(id){
        $.ajax({
           type: "POST",
           url: "zm_ob.php?id=" + id + "",
           data: "akcja=odejmij",
           success: function(wynik) {
            $("#ile").html(wynik)
           }
          });
          return false;
    }
});

plik zm_ob.php zawiera:

[PHP] pobierz, plaintext 
//polaczenie z baza jest
$id = $_GET['id.'];
$Query= 'SELECT id,wartosc FROM tabela WHERE id = "'.$id.'"';
$result = mysql_query($Query) or die(mysql_error());
while($rt = mysql_fetch_array($result)) {
	$how = $rt['wartosc'];
}
$decrease = 'UPDATE tabela SET wartosc=wartosc-5 WHERE id = "'.$id.'"';
$increase = 'UPDATE tabela SET wartosc=wartosc+5 WHERE id = "'.$id.'"';
if($_POST['akcja']=='dodaj') {
	mysql_query($increase);
	$how += 5;
	echo $how;
}
elseif($_POST['akcja']=='odejmij') {
	mysql_query($decrease);
	$how -= 5;
	echo $how;
}
[PHP] pobierz, plaintext 

I teraz pytanie czy da się coś z tym zrobić?

Ten post edytował djgarsi 30.03.2014, 18:54:00

[Turson]

data: "akcja=odejmij",
sprawdz jeszcze raz jak to powinno wygladac

[trueblue]

[PHP] pobierz, plaintext 
$id = $_GET['id'];
[PHP] pobierz, plaintext 

?
A oprócz tego obydwa skrypty zmieniłbym tak:

[HTML] pobierz, plaintext 
function zmien(id,typ){
        $.ajax({
           type: "POST",
           url: "zm_ob.php?id=" + id + "",
           data: {'akcja':typ}, //Turson ma rację
           success: function(wynik) {
            $("#ile").html(wynik)
           }
          }); 
          return false;
    }
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
if($_POST['akcja']=="dodaj"||$_POST['akcja']=="odejmij"){
 $zmien = 'UPDATE tabela SET wartosc=wartosc+'.($_POST['akcja']=="odejmij"?"-":"").'5 WHERE id = "'.$id.'"';
 mysql_query($zmien);
 $how=$how+($_POST['akcja']=="odejmij"?-5:5);
 echo $how;
}
[PHP] pobierz, plaintext 

Ten post edytował trueblue 30.03.2014, 19:37:30

[djgarsi]

rozumiem że link wywołujący funkcję powinien wyglądać np tak:

[HTML] pobierz, plaintext 
<a href="java script:zmien(1,dodaj); return false;">Dodaj</a> <span id="ile">tu wartosc</span> <a href="java script:zmien(1,odejmij); return false;">Odejmij</a>
[HTML] pobierz, plaintext 

Dobrze myślę?

[trueblue]

Prawie, bo typ przesyłasz jako string, więc:

[HTML] pobierz, plaintext 
<a href="java script:zmien(1,'dodaj'); return false;">Dodaj</a> <span id="ile">tu wartosc</span> <a href="java script:zmien(1,'odejmij'); return false;">Odejmij</a>
[HTML] pobierz, plaintext 

Ale nie musisz tego przesyłać tak, równie dobrze możesz podać 0 (dla dodaj), 1 (dla odejmij) i odpowiednio zmodyfikować skrypt PHP.

[djgarsi]

Kod

SyntaxError: return not in function
zmien(1,'dodaj'); return false;

Konsola wywala coś takiego.

[trueblue]

Nie zwracaj nic jeśli jesteś w "href", czyli samo wywołanie funkcji, bez return false;

[djgarsi]

Kod

ReferenceError: zmien is not defined

Teraz to. Plik z funkcją js jest dodany do head.

Ten post edytował djgarsi 30.03.2014, 19:58:23

[trueblue]

Wrzuć trochę więcej kodu.
Wytnij to czego nie chcesz pokazywać.


Ten post edytował trueblue 30.03.2014, 21:32:01

[djgarsi]

HTML gdzie dane będą prezentowane:

[HTML] pobierz, plaintext 
<a href="java script:zmien(<?php echo $id_ob; ?>,'dodaj');" ><i class="glyphicon glyphicon-plus"></i></a> <span id="ile"><?php echo $obciazenie; ?></span> <a href="java script:zmien(<?php echo $id_ob; ?>,'odejmij');"><i class="glyphicon glyphicon-minus"></i></a></p>
[HTML] pobierz, plaintext 

JS:

[HTML] pobierz, plaintext 
$(window).load(function(){
function zmien(id,typ){
        $.ajax({
           type: "POST",
           url: "zm_ob.php?id=" + id + "",
           data: {'akcja':typ}, //Turson ma rację
           success: function(wynik) {
            $("#ile").html(wynik)
           }
          }); 
          return false;
    }
});
[HTML] pobierz, plaintext 

PHP:

[PHP] pobierz, plaintext 
include('db_connect.php');
$id = $_GET['id'];
if($_POST['akcja']=="dodaj") {
	$zmien = 'UPDATE treningi SET obciazenie=obciazenie+5 WHERE id = "'.$id.'"';
	mysql_query($zmien);
	$how=$how+5;
	echo $how;
}
if($_POST['akcja']=="odejmij") {
	$zmien = 'UPDATE treningi SET obciazenie=obciazenie-5 WHERE id = "'.$id.'"';
	mysql_query($zmien);
	$how=$how-5;
	echo $how;
}
[PHP] pobierz, plaintext 

Co może być nie tak?

[IProSoft]

Wyrzuć funkcję zmien() poza $(window).load i wtedy sprawdź.

[djgarsi]

Wyrzuć funkcję zmien() poza $(window).load i wtedy sprawdź.

Nic to nie daje.

[trueblue]

Może ta literówka:

[HTML] pobierz, plaintext 
href="java script:
[HTML] pobierz, plaintext 

[djgarsi]

No właśnie to parser forum tę spację dodaje. W kodzie jest ok.

[trueblue]

A po radzie IProSoft, aby wyrzucić poza "load", gdzie wrzuciłeś funkcję? Nie czasem do "ready"?

[djgarsi]

Też na to wpadłem. Próbowałem w load, ready i bez obu. to samo za każdym razem.

[trueblue]

Zostaw tak jak jest niżej, bez żadnych zmian, działa (oczywiście status żądania będzie 404)?

[HTML] pobierz, plaintext 
<!DOCTYPE>
<html>
<head>
<script src="http://ajax.googleapis.com/ajax/libs/jquery/1.10.2/jquery.min.js"></script>
<script type="text/javascript">
function zmien(id,typ){
	$.ajax({
	   type: "POST",
	   url: "blabla.php?id=" + id + "",
	   data: {'akcja':typ},
	   success: function(wynik) {
		$("#ile").html(wynik)
	   }
	  }); 
	  return false;
}
 
</script>
</head>
<body>
<a href="java script:zmien(1,'dodaj');" >dodaj<i class="glyphicon glyphicon-plus"></i></a>
<span id="ile">1</span>
<a href="java script:zmien(1,'odejmij');">odejmij<i class="glyphicon glyphicon-minus"></i></a>
</body>
</html>
[HTML] pobierz, plaintext 

[djgarsi]

Tu jest link do działania.

[nospor]

Nie:
<a href="java script:zmien(1,'dodaj');" >dodaj<i class="glyphicon glyphicon-plus"></i></a>

a:
<a href="#" onclick="zmien(1,'dodaj');return false;" >dodaj<i class="glyphicon glyphicon-plus"></i></a>

Analogicznie reszta

[trueblue]

nospor, tyle, że w przypadku podpięcia pod href również powinno działać (i działa).