Forum PHP.pl > [AJAX][MySQL][PHP]Zmiana wartości pola w bazie

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [AJAX][MySQL][PHP]Zmiana wartości pola w bazie

djgarsi

30.03.2014, 18:52:48

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ć?

Turson

30.03.2014, 19:01:41

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

trueblue

30.03.2014, 19:02:20

[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

djgarsi

30.03.2014, 19:28:48

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

30.03.2014, 19:34:23

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

30.03.2014, 19:49:24

Kod

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

Konsola wywala coś takiego.

trueblue

30.03.2014, 19:52:15

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

djgarsi

30.03.2014, 19:57:38

Kod

ReferenceError: zmien is not defined

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

trueblue

30.03.2014, 21:31:43

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

djgarsi

30.03.2014, 21:56:39

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

30.03.2014, 22:47:20

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

djgarsi

31.03.2014, 12:19:10

Cytat(IProSoft @ 30.03.2014, 23:47:20 )

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

Nic to nie daje.

trueblue

31.03.2014, 12:51:30

Może ta literówka:

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

djgarsi

31.03.2014, 13:12:09

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

trueblue

31.03.2014, 13:27:28

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

djgarsi

31.03.2014, 13:47:22

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

trueblue

31.03.2014, 13:58:23

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

31.03.2014, 14:08:32

Tu jest link do działania.

nospor

31.03.2014, 14:12:57

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

31.03.2014, 14:21:53

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

nospor

31.03.2014, 14:23:15

Mi nie dziala

Mnie przenosi na strone: java script:.....

Moim zdaniem dawanie js w HREF jest brzydkim rozwiązaniem. Uwazam, ze lepszym rozwiązaniem jest to co podalem. Chocby z takiego powodu, ze jak ktos nie ma wlaczonego js, to go bez sensu nie bedzie przekierowywac

trueblue

31.03.2014, 14:25:13

Bo poprawnym protokołem jest java script: a nie java script:, pisał o tym djgrasi (parser BB tak przeformatowuje).

Najlepszym będzie dodanie zdarzeń z wykorzystaniem jQuery, skoro takowe jest wykorzystywane gdzie indziej.

nospor

31.03.2014, 14:27:23

Cytat

Bo poprawnym protokołem jest java script: a nie java script:, pisał o tym djgrasi (parser BB tak przeformatowuje).

Przeciez wiem, nie musisz mi tego tlumaczyc. Poza tym widze jak ma na stronie.....
Jak pisalem: mi przekierowuje na java script:..... (spacje dodaje parser forum, wiec sie nie czepiaj ze wyswietla sie spacja...)

Cytat

Najlepszym będzie dodanie zdarzeń z wykorzystaniem jQuery, skoro takowe jest wykorzystywane gdzie indziej.

Lub to co ja napisalem

trueblue

31.03.2014, 14:29:17

Nie czepiam się, nie wiem w których miejscach dodał parser, a w których występuje faktyczna literówka.

djgarsi

31.03.2014, 14:29:23

Już działa super

Dziękuję

Ajax trueblue, html nospor

nospor

31.03.2014, 14:29:51

Cytat

Nie czepiam się, nie wiem w których miejscach dodał parser, a w których występuje faktyczna literówka.

No to juz wiesz. Parser rozdziela kazde slowo javascript z dwukropkiem
Poza tym pisalem, ze mi przenosi z jego strony, a na stronie mial poprawnie napisane jak mu kazales, wiec nie bylo mowy bym czepial sie literowki

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.