Mærkeligste fejl i mysql_query ...

Du lægger en streng til et tal... Det er en type konflikt og vil derfor give et uventet resultat.

Hvad mener du med "flytter ud af if'en"?


Eneste fejl jeg lige kan se er at du har skrevet '1' i stedet for bare 1. Er ikke helt sikker på, om mysql gør noget underligt når du specifict siger noget er en streng men bruger det som et tal.

Ellers vil jeg antage at du faktisk har en query stående et andet det der gør det samme og at det er derfor du får 2.

prøv wins = wins++
ved ikke om det virker, men prøv

Undskyld for bump, men er enig med Valdemar. Prøv wins++
Er heller ikke sikker på at det virker, men prøv alligevel!

 if (isset($_POST['win'])) {
 $antalWins = "1";
 $userSession = $_SESSION['userdata']['user_name'];
 
 $updatestats = "UPDATE stats SET wins = wins + $antalWins  WHERE
 user = '$userSession' ";
 
 mysql_query($updatestats) or die(mysql_error());
 
 }

Prøv den kode

... Hvis jeg flytter $updatestats ud af if'en tæller den rigtigt med 1 hver gang man opdaterer siden...

Den kode du paster her ser rigtig ud, måske problemet er et andet sted end du leder?

Vil du poster lidt mere af den omgivende kode?

--- Så lige at det er en gammel tråd..

Man kan altid omkranse værdier (strenge såvel som integers) med singlequotes. Det er unødvendigt i ovenstående tilfælde, men det er normalt en god idé. Især når det er data fra brugeren.

wins++ vil ikke virke, for det er ikke en lovlig SQL construct.

Prøv at logge alle dine database kald, jeg vil næsten skyde på, at du får den udført to gange.

Man kan altid omkranse værdier (strenge såvel som integers) med singlequotes. Det er unødvendigt i ovenstående tilfælde, men det er normalt en god idé. Især når det er data fra brugeren.

At bruge quotes, single som double, typecaster altid dine værdier til strenge - MySQL er normalt logisk nok til at kunne fortolke en streng udelukkende bestående af numeriske tegn som værende en numerisk værdi. Men undlader man at bruge citationstegn omkring sine numeriske værdier vil MySQL vanligvis også forstå værdiens type som numerisk.

...

Til tidligere nævnte problem ville jeg nok udskrive en tekst til browseren for at lure om koden bliver afviklet to gange.

Problemet kan også opstå af at du aflæser databasen før du opdatere den - hvorved at wins+1 først vil være vist ved næste POST request. Løsning: byt rundt og opdater databasen før du aflæser den.

Hent evt. værdi'en ud i array, og læg et enkel til, og så update databasen. nok ikke den mest professionelle måde, men har erfaring med det virker

--

MySQL (og alle andre hvis de overholder kravene) kan se, at en streng er nummerisk hvis den bruges i en matematisk operation:

mysql> select 'Hello, ' + 'world';
+---------------------+
| 'Hello, ' + 'world' |
+---------------------+
|                   0 |
+---------------------+
1 row in set, 2 warnings (0.00 sec)

mysql> select '3' + '7';
+-----------+
| '3' + '7' |
+-----------+
|        10 |
+-----------+
1 row in set (0.00 sec)

mysql>