Forum PHP.pl > [MySQL] Błąd odejmowania punktów

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [MySQL] Błąd odejmowania punktów

Stron: 1, 2

Dominator

6.08.2011, 11:18:35

Witajcie, mam taki problem, mam kolumny array (kolumna array to są użytkownicy) i hs_nagrody (tam gdzie są nagrody), no i każdy użytkownik ma powiedzmy 500 punktów, aby odjęło punkty muszę zrobić 2 zapytania, jedno do użytkownika o stan ile ma punktów, a drugie zapytanie to ile kosztuje nagroda dla danego użytkownika, i mam problem, wywala mi błąd, ciągle naprawiam i dupa.

[PHP] pobierz, plaintext 
$zapytanie = mysql_query("SELECT punkty FROM array where login='".$login."'", $polaczenie) or die(mysql_error());
$zapytanie = mysql_query("SELECT koszt FROM hs_nagrody where koszt='".$koszt."'", $polaczenie) or die(mysql_error());
 
$pytanie = mysql_fetch_assoc($zapytanie);
$punkty = $pytanie['punkty'];
$koszt = $pytanie['koszt'];
 
$newitem = $punkty - $koszt;
 
$aktualizacja = mysql_query("UPDATE array SET punkty='".$newitem."' WHERE login='".$login."'", $polaczenie) or die(mysql_error());
[PHP] pobierz, plaintext

Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in ...

drozdii07

6.08.2011, 11:53:23

Nie może być czegoś takiego: " "" ". Te " w środku można zastąpić ' ' albo escapować za pomocą \

Dominator

6.08.2011, 12:06:44

O którą konkretnie linijkę Ci chodzi ?

drozdii07

6.08.2011, 12:10:04

W każdym zapytaniu tak masz

Napisz poprostu:

[SQL] pobierz, plaintext 
SELECT ... WHERE login = '$login'
[SQL] pobierz, plaintext

i po sprawie.

Dominator

6.08.2011, 12:28:49

Ale poza tym wszystko dobrze zrobiłem ?

Po zastosowaniu twoich opcji:

Parse error: syntax error, unexpected T_STRING in

Fifi209

6.08.2011, 12:32:57

Nie.
Najpierw wykonujesz zapytanie, potem je nadpisujesz.

$zapytanie = ...
$zapytanie = ...

drozdii07

6.08.2011, 12:33:11

A dokladnie to jak wygląda twoje zapytanie po zmianach ?

Dominator

6.08.2011, 12:34:41

Cytat(Fifi209 @ 6.08.2011, 13:32:57 )

Nie.
Najpierw wykonujesz zapytanie, potem je nadpisujesz.

$zapytanie = ...
$zapytanie = ...

Przecież tak zrobiłem -,- Co nie tak ?

drozdii07

6.08.2011, 12:35:48

Zrób tak:

$zapytanie = pierwsze zapytanie
$zapytanie_drugie = drugie zapytanie

Dominator

6.08.2011, 12:35:51

Cytat(drozdii07 @ 6.08.2011, 13:33:11 )

A dokladnie to jak wygląda twoje zapytanie po zmianach ?

[PHP] pobierz, plaintext 
$zapytanie = mysql_query("SELECT punkty FROM array where login='$login', $polaczenie) or die(mysql_error());
$zapytanie = mysql_query("SELECT koszt FROM hs_nagrody where koszt='$koszt', $polaczenie) or die(mysql_error());
 
$pytanie = mysql_fetch_assoc($zapytanie);
$punkty = $pytanie['punkty'];
$koszt = $pytanie['koszt'];
 
$newitem = $punkty - $koszt;
 
$aktualizacja = mysql_query("UPDATE array SET punkty='$newitem' WHERE login='$login', $polaczenie) or die(mysql_error());
[PHP] pobierz, plaintext

Fifi209

6.08.2011, 12:36:44

No właśnie. A ja Cię upominam, że tak zrobiłeś. Skoro jedno nadpisujesz drugim to jak potem chcesz z tego nadpisanego coś wyciągnąć?

Dominator

6.08.2011, 12:37:54

Kurna, mam ciągle

Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in

drozdii07

6.08.2011, 12:38:12

W pierwszej linijce brakuje " na końcu zapytania

I w drugiej też

Dominator

6.08.2011, 12:40:54

Cytat(Fifi209 @ 6.08.2011, 13:36:44 )

No właśnie. A ja Cię upominam, że tak zrobiłeś. Skoro jedno nadpisujesz drugim to jak potem chcesz z tego nadpisanego coś wyciągnąć?

$zapytanie=
$zapytaniedwa=

potem

$pytanie = mysql_fetch_assoc($zapytanie, $zapytaniedwa);

?

Cytat(drozdii07 @ 6.08.2011, 13:38:12 )

W pierwszej linijce brakuje " na końcu zapytania

I w drugiej też

Wiem, już wcześniej to naprawiłem

drozdii07

6.08.2011, 12:42:51

Jeśli wcześniej to naprawiłeś to dlaczego dajesz kod z błędami ?

Fifi209

6.08.2011, 12:43:35

No z tym zap1 i zap2 ok, tylko mysql_fetch_assoc przyjmie max jeden argument

Dominator

6.08.2011, 12:44:39

Cytat(Fifi209 @ 6.08.2011, 13:43:35 )

No z tym zap1 i zap2 ok, tylko mysql_fetch_assoc przyjmie max jeden argument

To co zrobić aby przyjęło 2 argumenty ?

Fifi209

6.08.2011, 12:47:30

Użyć dwa razy? Po razie na każde zapytanie?

Dominator

6.08.2011, 12:49:23

Cytat(Fifi209 @ 6.08.2011, 13:47:30 )

Użyć dwa razy? Po razie na każde zapytanie?

Mógłbyś trochę konkretniej ? Czy o to Ci chodzi ?

$dwazapytania = $zapytanie + zapytaniedwa; ?

drozdii07

6.08.2011, 12:51:30

[PHP] pobierz, plaintext 
$pytanie = mysql_fetch_assoc($zapytanie);
...
 
$pytaniedwa = mysql_fetch_assoc($zapytaniedwa);
[PHP] pobierz, plaintext

Dominator

6.08.2011, 13:02:47

Mój kod wygląda następująco:

[PHP] pobierz, plaintext 
$zapytanie1 = mysql_query("SELECT punkty FROM array where login='$login'", $polaczenie) or die(mysql_error());
 
$pytanie = mysql_fetch_assoc($zapytanie1);
 
$zapytanie2 = mysql_query("SELECT koszt FROM hs_nagrody where koszt='$koszt'", $polaczenie) or die(mysql_error());
 
$pytaniedwa = mysql_fetch_assoc($zapytanie2);
 
$punkty = $pytanie['punkty'];
$koszt = $pytaniedwa['koszt'];
 
$newitem = $punkty - $koszt;
 
$aktualizacja = mysql_query("UPDATE array SET punkty='$newitem' WHERE login='$login'", $polaczenie) or die(mysql_error());
[PHP] pobierz, plaintext

ciągle mi wywala błąd Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in

ten błąd jest gdzie $zapytanie1

Wsadziłem w złe miejsce i wywalał błąd

Ale punktów nie odejmuje ...

drozdii07

6.08.2011, 13:05:19

Zrób:

[PHP] pobierz, plaintext 
echo $punkty;
echo $koszt;
[PHP] pobierz, plaintext

I napisz co się pojawi

Dominator

6.08.2011, 13:11:11

Nic się nie pojawia

mortus

6.08.2011, 13:31:18

Nie nawiązałeś połączenia z bazą danych. Jeśli połączenie nawiązujesz w innym pliku, to może po prostu go nie wstawiłeś (include).
W przypadku dalszych problemów podaj kod odpowiedzialny za połączenie z mysql. Coś w stylu:

[PHP] pobierz, plaintext 
$polaczenie = mysql_connect(...)
[PHP] pobierz, plaintext

Pamiętaj, żeby usunąć nazwę użytkownika i hasło.

EDIT
@down No przecież o tym mówi komunikat błędu.

drozdii07

6.08.2011, 13:32:41

Mi tamten kod działa, więc to musi być tylko i wyłącznie połączenie

Dominator

6.08.2011, 13:38:40

Include jest cały czas, błędu żadnego nie wywala ale punktów nie odejmuje - czemu ?

drozdii07

6.08.2011, 13:40:25

No bo nie pobiera danych z bazy ! Pokaz jak się łączysz z bazą albo cały plik razem z tym include a nie tylko urywek z zapytaniem..

Dominator

6.08.2011, 13:46:37

Cytat(drozdii07 @ 6.08.2011, 14:40:25 )

No bo nie pobiera danych z bazy ! Pokaz jak się łączysz z bazą albo cały plik razem z tym include a nie tylko urywek z zapytaniem..

Uporałem się z tym, punkty zabiera ale zabiera wszystkie punkty zamiast np: dwustu oraz zabiera każdemu użytkownikowi a nie temu co akurat odejmuje sobie

drozdii07

6.08.2011, 13:49:43

Czyli, że zostaje 0 pkt. czy jak ? Wyświetl sobie $newitem i sprawdź czy dobra wartość wychodzi..

EDIT: Masz phpmyadmin ? Jeśli masz to weź konsole SQL i wykonaj to zapytanie UPDATE z jakimiś przykładowymi danymi, np. pkt. 30 a login jakiś jaki masz w bazie

Dominator

6.08.2011, 13:54:27

Cytat(drozdii07 @ 6.08.2011, 14:49:43 )

Wyświetla mi ile punktów mają wszyscy użytkownicy, ja ich mam 58, a ty dajmy że masz 100 to pokazuje mi "Posiadasz 58100" punktów :/

drozdii07

6.08.2011, 14:00:09

Pokaz jak masz skonstruowane tabele

Dominator

6.08.2011, 14:01:55

Cytat(drozdii07 @ 6.08.2011, 15:00:09 )

Pokaz jak masz skonstruowane tabele

Z użytkownikami czy z nagrodami ?

drozdii07

6.08.2011, 14:03:43

array i hs_nagrody i czy na pewno masz login w zmiennej $login ?

Dominator

6.08.2011, 14:07:59

Cytat(drozdii07 @ 6.08.2011, 15:03:43 )

array i hs_nagrody i czy na pewno masz login w zmiennej $login ?

hs_nagrody

CREATE TABLE IF NOT EXISTS `hs_nagrody` (
`id` int(11) NOT NULL AUTO_INCREMENT,
`skrot` varchar(30) NOT NULL,
`nazwa` varchar(50) NOT NULL,
`opis` varchar(100) NOT NULL,
`pelnyopis` varchar(2000) NOT NULL,
`link` varchar(200) NOT NULL,
`ilosc` int(5) NOT NULL,
`koszt` varchar(30) NOT NULL,
`trescboxa` varchar(200) NOT NULL,
PRIMARY KEY (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=latin2 AUTO_INCREMENT=27 ;

array

CREATE TABLE `array` (
`id` INT NOT NULL AUTO_INCREMENT PRIMARY KEY ,
`login` VARCHAR( 32 ) NOT NULL ,
`password` VARCHAR( 40 ) NOT NULL ,
`email` VARCHAR( 50 ) NOT NULL ,
`token` TEXT NOT NULL ,
`ban`TINYINT NOT NULL DEFAULT '0',
`punkty`TINYINT NOT NULL DEFAULT '0',
`potwierdzenie` TINYINT NOT NULL DEFAULT '0'
) ENGINE = MYISAM ;

drozdii07

6.08.2011, 14:10:41

To w koncu ta druga tabela to array czy users ? Bo w tym CREATE.. jest users a ty pobierasz z array

Dominator

6.08.2011, 14:11:10

Cytat(drozdii07 @ 6.08.2011, 15:10:41 )

To w koncu ta druga tabela to array czy users ? Bo w tym CREATE.. jest users a ty pobierasz z array

ta druga z uzytkownikami - array

drozdii07

6.08.2011, 14:15:28

Punkty mogą przyjąć wartość tylko do 127. Zmień typ na INT i zobacz czy działa

Dominator

6.08.2011, 14:16:24

Ok, mogę dać powyżej 127 ale nadal pokazuje mi to co pokazywało - 58100 pkt

drozdii07

6.08.2011, 14:17:58

Daj mi cały kod jaki teraz używasz żeby to pobrać

Dominator

6.08.2011, 14:24:09

Cytat(drozdii07 @ 6.08.2011, 15:17:58 )

Daj mi cały kod jaki teraz używasz żeby to pobrać

[PHP] pobierz, plaintext 
 
... przeniesienie do kategorii jesli ilosc nagrody jest równa zerze
 
$zapytanie1 = mysql_query("SELECT punkty FROM array where login='$login' ORDER BY id", $polaczenie) or die(mysql_error());
 
    $pytanie = mysql_fetch_assoc($zapytanie1);
 
    $zapytanie2 = mysql_query("SELECT koszt FROM hs_nagrody where koszt='$koszt' ORDER BY id", $polaczenie) or die(mysql_error());
 
    $pytaniedwa = mysql_fetch_assoc($zapytanie2);
 
    $punkty = $pytanie['punkty'];
    $koszt = $pytaniedwa['koszt'];
 
    $newitem = $punkty - $koszt;
 
    $aktualizacja = mysql_query("UPDATE array SET punkty='$newitem' WHERE login='$login' ORDER BY id", $polaczenie) or die(mysql_error());
 
odejmowanie jednej sztuki nagrody oraz gratulacje - zamowiles nagrode tratata
[PHP] pobierz, plaintext

mortus

6.08.2011, 14:29:22

Zauważ, że to Twoje drugie zapytanie jest w ogóle niepotrzebne. Wybierasz z bazy danych koszt, który jest równy wartości zmiennej $koszt, bez sensu.
Sprawdź zawartość tych trzech zmiennych przed i po odejmowaniu (kod poniżej):

[PHP] pobierz, plaintext 
var_dump($punkty, $koszt);
$newitem = $punkty - $koszt;
var_dump($punkty, $koszt, $newitem);
[PHP] pobierz, plaintext

@down W takim razie, który koszt ma zostać pobrany, skoro może ich być ile dusza zapragnie? Źle zorganizowany jest cały skrypt.

Do drugiego zapytania powinieneś przekazać id nagrody:

[SQL] pobierz, plaintext 
SELECT `koszt` FROM `hs_nagrody` WHERE `id` = $id_nagrody
[SQL] pobierz, plaintext

To id nagrody powinno natomiast być przekazywane wcześniej przez $_GET lub $_POST.

drozdii07

6.08.2011, 14:31:12

[PHP] pobierz, plaintext 
$zapytanie2 = mysql_query("SELECT koszt FROM hs_nagrody ORDER BY id", $polaczenie)
[PHP] pobierz, plaintext

Chciałeś pobrać z bazy koszt gdzie koszt równy jest zmiennej, której nie ma przed zapytaniem

Teraz działa, przynajmniej mi.

@up Szybszy ;p Szukał rozwiązania dlaczego mu nie pobiera dobrze i dlaczego nie odejmuje. Jak on sobie później zrobi pobieranie kosztów to mnie szczerze nie obchodzi. Nie to jest tematem tego wątku więc nie chciałem się już w to zagłębiać

Dominator

6.08.2011, 14:38:05

Ogarnąłem to że pokazuje ile kto ma punktów, i jeśli mam 58 to pokaże że mam 58

Ahh.. odjąć nie chce - 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, wiecie co to za błąd ?, sorki że was tak męczę

drozdii07

6.08.2011, 14:41:09

No jakim cudem ? Pokaż co zmieniłeś, bo to zapytanie działa na 100%

[SQL] pobierz, plaintext 
$aktualizacja = mysql_query("UPDATE array SET punkty='$newitem' WHERE login='$login'") OR die(mysql_error());
[SQL] pobierz, plaintext

Dominator

6.08.2011, 14:45:42

Na wyświetlanie kto ile ma punktów to miałem

[PHP] pobierz, plaintext 
<?php
include('db.php');
 
 
$zapytanie = "SELECT punkty FROM array";
$pobierz = mysql_query($zapytanie);
 
while($dane = mysql_fetch_array($pobierz))
{
    echo $dane['punkty'];
}
 
?>
[PHP] pobierz, plaintext

ale zmodyfikowałem go tak

[PHP] pobierz, plaintext 
<?php
include('db.php');
 
$login = $_SESSION['login'];
 
$zapytanie = "SELECT punkty FROM array WHERE login='$login'";
$pobierz = mysql_query($zapytanie);
 
while($dane = mysql_fetch_array($pobierz))
{
    echo $dane['punkty'];
}
 
?>
[PHP] pobierz, plaintext

---

Cytat(drozdii07 @ 6.08.2011, 15:41:09 )

No jakim cudem ? Pokaż co zmieniłeś, bo to zapytanie działa na 100%

[SQL] pobierz, plaintext 
$aktualizacja = mysql_query("UPDATE array SET punkty='$newitem' WHERE login='$login'") OR die(mysql_error());
[SQL] pobierz, plaintext

takie coś mam

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

drozdii07

6.08.2011, 14:50:47

Nie rozumiem, mi wszystko działa

Może jakiś przypadkowy " ci sie zawieruszył gdzies ?

Dominator

6.08.2011, 14:55:10

Nic mi nie pouciekało, wszystko sprawdziłem, jestem pewien na 100% że wszystko jest dobrze

, gdyby nie ten błąd ... kurde no

drozdii07

6.08.2011, 15:00:04

No to ja znowu proszę o cały kod

Bo z tego co już jest w temacie to nic nie widzę, żadnych błędów czy cos

Dominator

6.08.2011, 15:16:35

[PHP] pobierz, plaintext 
$zapytanie1 = mysql_query("SELECT punkty FROM array where login=$login", $polaczenie) or die(mysql_error());
 
    $pytanie = mysql_fetch_assoc($zapytanie1);
 
    $zapytanie2 = mysql_query("SELECT koszt FROM hs_nagrody where koszt=$koszt", $polaczenie) or die(mysql_error());
 
    $pytaniedwa = mysql_fetch_assoc($zapytanie2);
 
    $login = $_SESSION['login'];
 
    $punkty = $pytanie['punkty'];
    $koszt = $pytaniedwa['koszt'];
 
    $newitem = $punkty - $koszt;
 
    $aktualizacja = mysql_query("UPDATE array WHERE login=$login SET punkty=$newitem") OR die(mysql_error());;
[PHP] pobierz, plaintext

edit: wiem juz do czego się dowala - do " oraz '
Wychwyciłem błąd, dowala się do tego

[PHP] pobierz, plaintext 
$zapytanie1 = mysql_query("SELECT punkty FROM array where login=$login", $polaczenie) or die(mysql_error());
[PHP] pobierz, plaintext

drozdii07

6.08.2011, 15:18:28

To jednak nie ta linijka

W ostatniej linijce masz o jeden średnik za dużo

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.