Forum » Server-Side scripting » Random functie werkt niet goed.
sukel
Berichten: 409
avatar
Online Stuur privebericht
Beste,

Ik heb op mijn site een nieuwe functie gemaakt, een target systeem. Nu zit er alleen een hele nare bug in. Het is de bedoeling dat hij 1 iemand in de database pakt, en daarbij een beloning zet. Dit doet hij goed, alleen inplaats van dat hij 1 beloning pakt, pakt hij de beloning 104x.

Dit is de melding:

Code | Selecteer Alles
minimaliseren
1
Hij werkt....


Hij pakt dus 1 iemand, en die houd hij ook. Maar de beloning maakt hij 104x aan?

Dit is de script:


Code | Selecteer Alles
minimaliseren
1
Hij werkt....


Kan iemand mij helpen?

Alvast bedankt!
Laatst gewijzigd door sukel op 2011-05-22 16:14:21
21-05-2011 20:15
Dit topic is 252 keer bekeken door 49 verschillende leden
Reacties op: "Random functie werkt niet goed."
1
habib10
Hmm
Berichten: 931
avatar
Offline Stuur privébericht
maak voor elke beloning een apparte functie
21-05-2011 20:20
sukel
Berichten: 409
avatar
Online Stuur privébericht
Quote: habib10

maak voor elke beloning een apparte functie



Dat zijn ze toch al?
21-05-2011 20:26
BrokenTrack
Informatica Student
Berichten: 1406
avatar
Online Stuur privébericht
Code | Selecteer Alles
minimaliseren
1
2
$select = mysql_query("SELECT * FROM `speldata` WHERE leven>'0'"); 
                while($listm5 = mysql_fetch_object($select)) {

Zorgt voor een loop he?
21-05-2011 21:05
sukel
Berichten: 409
avatar
Online Stuur privébericht
Quote: BrokenTrack

Code | Selecteer Alles
minimaliseren
1
2
$select = mysql_query("SELECT * FROM `speldata` WHERE leven>'0'"); 
                while($listm5 = mysql_fetch_object($select)) {

Zorgt voor een loop he?



Wat is er dan fout aan? Ik snap het niet precies.
21-05-2011 21:07
DonM
Berichten: 52
avatar
Offline Stuur privébericht
While betekend dat hij onderelkaar alle alles pakt van de opgevraagde optie,
21-05-2011 23:06
L.Groot
Rapifia Europe
Berichten: 3716
avatar
Offline Stuur privébericht
In dit geval gaat hij alle rijen in de tabel speldata langs. Daar zijn 104 rijen, dus word de code 104x uitgevoerd. Gewoon de loop verwijderen en klaar.
22-05-2011 09:37
NielsB
Webdeveloper & Programmeu
Berichten: 1001
avatar
Online Stuur privébericht
Haal die regel weg :
Code | Selecteer Alles
minimaliseren
1
2
$select = mysql_query("SELECT * FROM `speldata` WHERE leven>'0'"); 
                while($listm5 = mysql_fetch_object($select)) {


volgens mij gebruik je hem toch nergens voor! Anders maak je een if wanneer level > 0 moet zijn, maar volgens mij is dat niet eens nodig als ik even zo snel kijk.
22-05-2011 11:13
sukel
Berichten: 409
avatar
Online Stuur privébericht
Quote: L.Groot

In dit geval gaat hij alle rijen in de tabel speldata langs. Daar zijn 104 rijen, dus word de code 104x uitgevoerd. Gewoon de loop verwijderen en klaar.



Bedankt L.Groot en BrokenTrack! Ik snap hem nu, het scriptje werkt nu ook!

Nieuwe reactie samengevoegd met originele reactie op 22.05.11 11:59:16:
Quote: NielsB

Haal die regel weg :

Code | Selecteer Alles
minimaliseren
1
2
$select = mysql_query("SELECT * FROM `speldata` WHERE leven>'0'"); 
                while($listm5 = mysql_fetch_object($select)) {


Het scriptje kies een willekeurige lid uit de db, als ik deze regel weghaal kan hij dus ook een verbannen lid of een dode lid pakken. Dit stukje is dus zeker nodig.

22-05-2011 11:13
Reageer op: "Random functie werkt niet goed."
1
Je kan niet reageren omdat je niet bent ingelogd. Inloggen of Aanmelden