Cron Bug

jgdonleach

Hallo,

bij mijn spel doet het uurloon van de clan het niet goed.

Ik denk zelf dat dit aan de cron hour doet. Kan iemand hier na kijken.
Dit is mijn script:

PHP

<?php /* ------------------------- */


  if($cron_pass != "secretcronpassword")
    exit;




    $data2				= mysql_query("SELECT * FROM `[users]` WHERE `login`='{$_SESSION['login']}'");
    $data				= mysql_fetch_object($data2);


$geld                		= $data2->hoeren;
$geld2                      = $geld*5;
$geld3               		= $data2->hoerenwerkend;
$geld4                      = $geld*15;
$geld5                      = $geld4+$geld2;
$inkomen                    = $geld5; 
$geld7                      = $data->bank/100*1;
$rente                      = $geld7; 


mysql_query("UPDATE `[users]` SET `bank`=`bank`+'$inkomen' WHERE `hoeren`>'0'");
  mysql_query("UPDATE `[users]` SET `bank`=`bank`+5 WHERE `type`=3 AND `activated`=1");
mysql_query("UPDATE `[users]` SET `gijzel1`=10 WHERE `activated`=1");
mysql_query("UPDATE `[users]` SET `schiet`=10 WHERE `activated`=1");
mysql_query("UPDATE `[users]` SET `belcredit1`=5 WHERE `activated`=1");
mysql_query("UPDATE `[users]` SET `dealen1`=10 WHERE `activated`=1");
mysql_query("UPDATE `[users]` SET `type`=1 WHERE `activated`=1");
mysql_query("UPDATE `[users]` SET `overval1`=1 WHERE `activated`=1");
mysql_query("UPDATE `[users]` SET `overval2`=1 WHERE `activated`=1");
mysql_query("UPDATE `[users]` SET `singlejob`=1 WHERE `activated`=1");
mysql_query("UPDATE `[users]` SET `gaandealen`=1 WHERE `activated`=1");
mysql_query("UPDATE `[clans]` SET `cash`=`cash`+10,`bank`=`bank`+10");
mysql_query("UPDATE `[users]` SET `bankberoof`=0 WHERE `activated`=1");
  mysql_query("DELETE FROM `[logs]` WHERE `area`='click'");
  mysql_query("UPDATE `[users]` SET `ocsdoen`='2'");
mysql_query("UPDATE `[users]` SET `kraak`='0'");




  mysql_query("UPDATE `[users]` SET `landvlieg`='5'");
  mysql_query("UPDATE `[users]` SET `bankmax`='5000000'");
  mysql_query("UPDATE `[users]` SET `energie`='10' WHERE `energie`<'0'"); 
  mysql_query("UPDATE `[users]` SET `bankleft`='20'");
  mysql_query("UPDATE `[users]` SET `jointtijd`='0'");
  mysql_query("UPDATE `[users]` SET `beroofkansen`='10'");
  mysql_query("UPDATE `[users]` SET `maffiamode`='0'");
     mysql_query("UPDATE `[users]` set `klikmissie`='0'"); 
     mysql_query("UPDATE `[users]` set `klikmissie1`='0'"); 
mysql_query("UPDATE `[users]` SET `hoerpimped`='0'");
mysql_query("UPDATE `[users]` SET `hoeren`='0'");
mysql_query("UPDATE `[users]` SET `hoerenwerkend`='0'");
mysql_query("UPDATE `[users]` SET `ramen`='0'");
mysql_query("UPDATE `[users]` SET `bezet`='0'");


  mysql_query("DELETE FROM `[linkex]` WHERE `siteid`='1'");


  mysql_query("OPTIMIZE TABLE `[clans]`");
  mysql_query("OPTIMIZE TABLE `[logs]`");
  mysql_query("OPTIMIZE TABLE `[temp]`");
  mysql_query("OPTIMIZE TABLE `[users]`");


  mysql_query("UPDATE `[users]` SET `IPs`=''"); 
  mysql_query("UPDATE `[clans]` SET `IPs`=''"); 


  $dbres				= mysql_query("SELECT * FROM `[clans]` WHERE `money_lvl1` > 0");
  while($clan = mysql_fetch_object($dbres)) {
    if($clan->type == 1)
      mysql_query("UPDATE `[users]` SET `cash`=`cash`+". (50*$clan->money_lvl1) .",`bank`=`bank`+". (150*$clan->money_lvl1) ." WHERE `clan`='{$clan->name}'");
    else if($clan->type == 2)
      mysql_query("UPDATE `[users]` SET `cash`=`cash`+". (100*$clan->money_lvl1) .",`bank`=`bank`+". (100*$clan->money_lvl1) ." WHERE `clan`='{$clan->name}'");
    else if($clan->type == 3)
      mysql_query("UPDATE `[users]` SET `bank`=`bank`+". (rand(150,300)*$clan->money_lvl1) ." WHERE `clan`='{$clan->name}'");
  }


/* ------------------------- */ ?>

Meer tonen

Stefan.J

Het volgende hoort niet in je cron thuis:

PHP

$data2                = mysql_query("SELECT * FROM `[users]` WHERE `login`='{$_SESSION['login']}'");
    $data                = mysql_fetch_object($data2);

Meestal (Altijd), worden crons niet per user uitgevoerd, en bestaat de SESSION array al helemaal niet (of is irrelevant).

Daarbij kunnen alle updates van de users tabel worden samengevoegd. Dat scheelt een hele hoop in snelheid. Als het mogelijk is ook de WHERE statement weghalen: Dit versloomd je query enorm.

Daarnaast zal ik even nadenken over een mogelijkheid om de uurloon van de clans in ??n query te proppen.

Darsstar

PHP

<?php // kleur
      mysql_query("UPDATE `[users]`, `[clans]` SET `[users]`.`cash`=`[users]`.`cash`+(50*`[clans]`.`money_lvl1`),`[users]`.`bank`=`[users]`.`bank`+(150*`[clans]`.`money_lvl1`) WHERE `[users]`.`clan` = `[clans]`.`name` AND `[clans]`.`type`=1");
      mysql_query("UPDATE `[users]`, `[clans]` SET `[users]`.`cash`=`[users]`.`cash`+(100*`[clans]`.`money_lvl1`),`[users]`.`bank`=`[users]`.`bank`+(100*`[clans]`.`money_lvl1`) WHERE `[users]`.`clan` = `[clans]`.`name` AND `[clans]`.`type`=2");
      mysql_query("UPDATE `[users]`, `[clans]` SET `[users]`.`bank`=`[users]`.`bank`+(FLOOR(150+RAND()*(301-150))*`[clans]`.`money_lvl1`) WHERE `[users]`.`clan` = `[clans]`.`name` AND `[clans]`.`type`=3");

Dat zou de betere versie voor uurloon van clans moeten zijn...

Stefan.J

Heey, ik zei één query!:p

PHP

UPDATE `[users]` u LEFT JOIN `[clan]` c ON u.clan = c.name 
SET 
 cash = CASE 
  WHEN c.type = '1' THEN u.cash + 50 * c.money_lvl1 
  WHEN c.type = '2' THEN u.cash + 100 * c.money_lvl1  
  WHEN c.type = '3' THEN u.cash
 END,
 bank = CASE
  WHEN c.type = '1' THEN u.bank + 150 * c.money_lvl1 
  WHEN c.type = '2' THEN u.bank + 100 * c.money_lvl1  
  WHEN c.type = '3' THEN u.bank + ROUND(RAND()*150+150 * c.money_lvl1)
 END

Meer tonen

Eén afwijking ten opzichte van de originele (ook bij die van Darsstar), alle users van type 3, krijgen nu een afzonderlijke random, dus allemaal andere hoeveelheden van hun clan.

Ook een pluim voor jou query Darsstar, mooi gemaakt.;)

jgdonleach

PHP

<?php /* ------------------------- */ 


  if($cron_pass != "secretcronpassword") 
    exit; 




    $data2                = mysql_query("SELECT * FROM `[users]` WHERE `login`='{$_SESSION['login']}'"); 
    $data                = mysql_fetch_object($data2); 


$geld                        = $data2->hoeren; 
$geld2                      = $geld*5; 
$geld3                       = $data2->hoerenwerkend; 
$geld4                      = $geld*15; 
$geld5                      = $geld4+$geld2; 
$inkomen                    = $geld5;  
$geld7                      = $data->bank/100*1; 
$rente                      = $geld7;  


mysql_query("UPDATE `[users]` SET `bank`=`bank`+'$inkomen' WHERE `hoeren`>'0'"); 
  mysql_query("UPDATE `[users]` SET `bank`=`bank`+5 WHERE `type`=3 AND `activated`=1"); 
mysql_query("UPDATE `[users]` SET `gijzel1`=10 WHERE `activated`=1"); 
mysql_query("UPDATE `[users]` SET `schiet`=10 WHERE `activated`=1"); 
mysql_query("UPDATE `[users]` SET `belcredit1`=5 WHERE `activated`=1"); 
mysql_query("UPDATE `[users]` SET `dealen1`=10 WHERE `activated`=1"); 
mysql_query("UPDATE `[users]` SET `type`=1 WHERE `activated`=1"); 
mysql_query("UPDATE `[users]` SET `overval1`=1 WHERE `activated`=1"); 
mysql_query("UPDATE `[users]` SET `overval2`=1 WHERE `activated`=1"); 
mysql_query("UPDATE `[users]` SET `singlejob`=1 WHERE `activated`=1"); 
mysql_query("UPDATE `[users]` SET `gaandealen`=1 WHERE `activated`=1"); 
mysql_query("UPDATE `[clans]` SET `cash`=`cash`+10,`bank`=`bank`+10"); 
mysql_query("UPDATE `[users]` SET `bankberoof`=0 WHERE `activated`=1"); 
  mysql_query("DELETE FROM `[logs]` WHERE `area`='click'"); 
  mysql_query("UPDATE `[users]` SET `ocsdoen`='2'"); 
mysql_query("UPDATE `[users]` SET `kraak`='0'"); 




  mysql_query("UPDATE `[users]` SET `landvlieg`='5'"); 
  mysql_query("UPDATE `[users]` SET `bankmax`='5000000'"); 
  mysql_query("UPDATE `[users]` SET `energie`='10' WHERE `energie`<'0'");  
  mysql_query("UPDATE `[users]` SET `bankleft`='20'"); 
  mysql_query("UPDATE `[users]` SET `jointtijd`='0'"); 
  mysql_query("UPDATE `[users]` SET `beroofkansen`='10'"); 
  mysql_query("UPDATE `[users]` SET `maffiamode`='0'"); 
     mysql_query("UPDATE `[users]` set `klikmissie`='0'");  
     mysql_query("UPDATE `[users]` set `klikmissie1`='0'");  
mysql_query("UPDATE `[users]` SET `hoerpimped`='0'"); 
mysql_query("UPDATE `[users]` SET `hoeren`='0'"); 
mysql_query("UPDATE `[users]` SET `hoerenwerkend`='0'"); 
mysql_query("UPDATE `[users]` SET `ramen`='0'"); 
mysql_query("UPDATE `[users]` SET `bezet`='0'"); 


  mysql_query("DELETE FROM `[linkex]` WHERE `siteid`='1'"); 


  mysql_query("OPTIMIZE TABLE `[clans]`"); 
  mysql_query("OPTIMIZE TABLE `[logs]`"); 
  mysql_query("OPTIMIZE TABLE `[temp]`"); 
  mysql_query("OPTIMIZE TABLE `[users]`"); 


  mysql_query("UPDATE `[users]` SET `IPs`=''");  
  mysql_query("UPDATE `[clans]` SET `IPs`=''");  


UPDATE `[users]` u LEFT JOIN `[clan]` c ON u.clan = c.name 
SET 
 cash = CASE 
  WHEN c.type = '1' THEN u.cash + 50 * c.money_lvl1 
  WHEN c.type = '2' THEN u.cash + 100 * c.money_lvl1  
  WHEN c.type = '3' THEN u.cash
 END,
 bank = CASE
  WHEN c.type = '1' THEN u.bank + 150 * c.money_lvl1 
  WHEN c.type = '2' THEN u.bank + 100 * c.money_lvl1  
  WHEN c.type = '3' THEN u.bank + ROUND(RAND()*150+150 * c.money_lvl1)
 END


/* ------------------------- */ ?>

Meer tonen

Zo dus?

Stefan.J

Euhmm... Nee niet echt, SQL query´s moet je in de mysql_query functie plaatsen...

jgdonleach

PHP

UPDATE `[users]` u LEFT JOIN `[clan]` c ON u.clan = c.name 
SET 
 cash = CASE 
   mysql_query("WHEN c.type = '1' THEN u.cash + 50 * c.money_lvl11");
   mysql_query("WHEN c.type = '2' THEN u.cash + 100 * c.money_lv1l");
   mysql_query("WHEN c.type = '3' THEN u.cash");
 END,
 bank = CASE
   mysql_query("WHEN c.type = '1' THEN u.bank + 150 * c.money_lvl1");
   mysql_query("WHEN c.type = '2' THEN u.bank + 100 * c.money_lvl1");
   mysql_query("WHEN c.type = '3' THEN u.bank + ROUND(RAND()*150+150 * c.money_lvl1)
 END

Meer tonen

ZO? Als het fout is kan je het even voor mij doen?

Stefan.J

Nee, valt mij een beetje tegen, zo:

PHP

<?php


mysql_qeury("UPDATE `[users]` u LEFT JOIN `[clan]` c ON u.clan = c.name 
SET 
 cash = CASE 
  WHEN c.type = '1' THEN u.cash + 50 * c.money_lvl1 
  WHEN c.type = '2' THEN u.cash + 100 * c.money_lvl1  
  WHEN c.type = '3' THEN u.cash
 END,
 bank = CASE
  WHEN c.type = '1' THEN u.bank + 150 * c.money_lvl1 
  WHEN c.type = '2' THEN u.bank + 100 * c.money_lvl1  
  WHEN c.type = '3' THEN u.bank + ROUND(RAND()*150+150 * c.money_lvl1)
 END");


?>

Meer tonen

Het is één query.

jgdonleach

Hij doet het nu helemaal niet meer. Eerst was het zo als je online was kreeg je wat maar nu 0,0

Melvin

Haha ok probeer deze maar eens:

PHP

<?php /* ------------------------- */ 


  if($cron_pass != "secretcronpassword") 
    exit; 




    $data2                = mysql_query("SELECT * FROM `[users]` WHERE `login`='{$_SESSION['login']}'"); 
    $data                = mysql_fetch_object($data2); 


	$geld                        = $data2->hoeren; 
	$geld2                      = $geld*5; 
	$geld3                       = $data2->hoerenwerkend; 
	$geld4                      = $geld*15; 
	$geld5                      = $geld4+$geld2; 
	$inkomen                    = $geld5;  
	$geld7                      = $data->bank/100*1; 
	$rente                      = $geld7;  


 	mysql_query("UPDATE `[users]` SET `cash`=`cash`+100 WHERE `activated`=1");
 	mysql_query("UPDATE `[users]` SET `bank`=`bank`+100 WHERE `level`=100 AND `activated`=1");
  	mysql_query("UPDATE `[users]` SET `cash`=0 WHERE `cash`<0");
  	mysql_query("UPDATE `[users]` SET `bank`=0 WHERE `bank`<0"); 
	mysql_query("UPDATE `[users]` SET `gijzel1`=10 WHERE `activated`=1"); 
	mysql_query("UPDATE `[users]` SET `schiet`=10 WHERE `activated`=1"); 
	mysql_query("UPDATE `[users]` SET `belcredit1`=5 WHERE `activated`=1"); 
	mysql_query("UPDATE `[users]` SET `dealen1`=10 WHERE `activated`=1"); 
	mysql_query("UPDATE `[users]` SET `type`=1 WHERE `activated`=1"); 
	mysql_query("UPDATE `[users]` SET `overval1`=1 WHERE `activated`=1"); 
	mysql_query("UPDATE `[users]` SET `overval2`=1 WHERE `activated`=1"); 
	mysql_query("UPDATE `[users]` SET `singlejob`=1 WHERE `activated`=1"); 
	mysql_query("UPDATE `[users]` SET `gaandealen`=1 WHERE `activated`=1"); 
	mysql_query("UPDATE `[clans]` SET `cash`=`cash`+10,`bank`=`bank`+10"); 
	mysql_query("UPDATE `[users]` SET `bankberoof`=0 WHERE `activated`=1"); 
  	mysql_query("DELETE FROM `[logs]` WHERE `area`='click'"); 
  	mysql_query("UPDATE `[users]` SET `ocsdoen`='2'"); 
	mysql_query("UPDATE `[users]` SET `kraak`='0'"); 




  	mysql_query("UPDATE `[users]` SET `landvlieg`='5'"); 
  	mysql_query("UPDATE `[users]` SET `bankmax`='5000000'"); 
  	mysql_query("UPDATE `[users]` SET `energie`='10' WHERE `energie`<'0'");  
  	mysql_query("UPDATE `[users]` SET `bankleft`='20'"); 
  	mysql_query("UPDATE `[users]` SET `jointtijd`='0'"); 
  	mysql_query("UPDATE `[users]` SET `beroofkansen`='10'"); 
  	mysql_query("UPDATE `[users]` SET `maffiamode`='0'"); 
  	mysql_query("UPDATE `[users]` set `klikmissie`='0'");  
  	mysql_query("UPDATE `[users]` set `klikmissie1`='0'");  
  	mysql_query("UPDATE `[users]` SET `hoerpimped`='0'"); 
	mysql_query("UPDATE `[users]` SET `hoeren`='0'"); 
	mysql_query("UPDATE `[users]` SET `hoerenwerkend`='0'"); 
	mysql_query("UPDATE `[users]` SET `ramen`='0'"); 
	mysql_query("UPDATE `[users]` SET `bezet`='0'"); 


  	mysql_query("DELETE FROM `[linkex]` WHERE `siteid`='1'"); 


  	mysql_query("OPTIMIZE TABLE `[clans]`"); 
  	mysql_query("OPTIMIZE TABLE `[logs]`"); 
  	mysql_query("OPTIMIZE TABLE `[temp]`"); 
  	mysql_query("OPTIMIZE TABLE `[users]`"); 


  	mysql_query("UPDATE `[users]` SET `IPs`=''");  
  	mysql_query("UPDATE `[clans]` SET `IPs`=''");  


  	$dbres        = mysql_query("SELECT * FROM `[clans]` WHERE `money_lvl1` > 0");
  	while($clan = mysql_fetch_object($dbres)) {
    if($clan->type == 1)
      mysql_query("UPDATE `[users]` SET `cash`=`cash`+". (50*$clan->money_lvl1) .",`bank`=`bank`+". (150*$clan->money_lvl1) ." WHERE `clan`='{$clan->name}'");
    else if($clan->type == 2)
      mysql_query("UPDATE `[users]` SET `cash`=`cash`+". (100*$clan->money_lvl1) .",`bank`=`bank`+". (100*$clan->money_lvl1) ." WHERE `clan`='{$clan->name}'");
    else if($clan->type == 3)
      mysql_query("UPDATE `[users]` SET `bank`=`bank`+". (rand(150,300)*$clan->money_lvl1) ." WHERE `clan`='{$clan->name}'");
    else if($clan->type == 1)
      mysql_query("UPDATE `[users]` SET `cash`=`cash`+". (100*$clan->money_lvl3) .",`bank`=`bank`+". (1250*$clan->money_lvl3) ." WHERE `clan`='{$clan->name}'");
    else if($clan->type == 2)
      mysql_query("UPDATE `[users]` SET `cash`=`cash`+". (250*$clan->money_lvl5) .",`bank`=`bank`+". (1250*$clan->money_lvl5) ." WHERE `clan`='{$clan->name}'");
    else if($clan->type == 3)
      mysql_query("UPDATE `[users]` SET `bank`=`bank`+". (rand(150,1300)*$clan->money_lvl6) ." WHERE `clan`='{$clan->name}'");
  }


/* ------------------------- */ ?>

Meer tonen

jgdonleach

Volgensmij doet hij het nog niet. Ik laat volgend uur wel weten.

Melvin

je kan ook in je database je cron een uur terug zetten zie je het gelijk

jgdonleach

Hij doet het TNQ Melvin

Graaag :slotje:

Stefan.J

Hij werkt ja, zolang je spel slechts één speler heeft. En ik ga er natuurlijk vanuit dat je er wel een paar meer hebt. Dus: Dat data-object moet uit dat script. Je ziet staan: $_SESSION, als dat in een cron-file staat, moeten de bellen toch wel gaan rinkelen.

jgdonleach

We zitten nu in demo met 4 spelers en iedereen krijgt uurloon.

Darsstar

Uurloon wel ja...

PHP

$data2                = mysql_query("SELECT * FROM `[users]` WHERE `login`='{$_SESSION['login']}'");
    $data                = mysql_fetch_object($data2);


$geld                        = $data2->hoeren;
$geld2                      = $geld*5;
$geld3                       = $data2->hoerenwerkend;
$geld4                      = $geld*15;
$geld5                      = $geld4+$geld2;
$inkomen                    = $geld5; 
$geld7                      = $data->bank/100*1;
$rente                      = $geld7;

Meer tonen

Dat laat je allemaal door PHP berekenden...
En maar voor één persoon...
Met andere woorden...
Wil het toeval het dat de cron wordt uitgevoerd door een noob...
dan krijgt IEDEREEN heel wijnig inkomen...
Wordt het juist door de nummer 1 uitgevoerd, dan krijgt IEDEREEN juist heel veel inkomen...

jgdonleach

|Dat stukje bebruik ik niet meer.

Nieuwe reactie samengevoegd met originele reactie op 06.12.09 19:09:06:
Hij geeft alleen uurloon als er iemand online is hoe kan dit?

Stefan.J

Onzin wat ik zei.:} Staan geen WHERE-clauses in.

Darsstar

Oplossing 1:
Ga met cron jobs werken...
Werkt veel beter...

Oplossing 2:
Lees dit artikel (oorspronkelijk door mij gescheven... ooit... heel lang geleden XD)

ICTscripters

Dé plek voor IT

Dé plek voor IT

Externe testers gezocht — Saphaxis AI Chat Organizer 0.1.59

nieuwe game

Criminolz

[Beta] Ottomaan: gratis browserspel met een echte, levende stad. Feedback gezocht!

Port Father: beta van een browsergame in Antwerpen 1931, feedback welkom

Maffia Hotel: een browsergame waarin je van lobbygast naar spil van het hotel groeit

Crime Club volledig herschreven vanaf nul

Malavita (nieuwe maffia game, van 0)

SEO Expired Domains for Sale | .COM, .ORG, .NET & Australian Domains | From $30

293 Nieuwe Domeinnamen Oktober 2026

[Aanbod] Stadskaart voor je browsergame: van menu naar klikbare stad, vanaf €200

Cron Bug

Registratie

PHP waarschuwt voor twee ernstige bugs die makkelijk uit te buiten lijken

Functioneel ontwerp

Access Control List implementatie in PHP/MySQL - deel 1/2

Access Control List implementatie in PHP/MySQL - deel 2/2

Gebruikers die dit topic bekijken