Cron Bug

jgdonleach · 3 December 2009

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}'");
  }


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

Toon Meer

Stefan.J · 3 December 2009

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 · 3 December 2009

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 · 3 December 2009

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

Toon Meer

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 · 3 December 2009

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


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

Toon Meer

Zo dus?

Stefan.J · 3 December 2009

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

jgdonleach · 3 December 2009

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

Toon Meer

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

Stefan.J · 3 December 2009

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");


?>

Toon Meer

Het is één query.

jgdonleach · 6 December 2009

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

Melvin · 6 December 2009

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}'");
  }


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

Toon Meer

jgdonleach · 6 December 2009

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

Melvin · 6 December 2009

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

jgdonleach · 6 December 2009

Hij doet het TNQ Melvin

Graaag :slotje:

Stefan.J · 6 December 2009

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 · 6 December 2009

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

Darsstar · 6 December 2009

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;

Toon Meer

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 · 6 December 2009

|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 · 6 December 2009

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

Darsstar · 6 December 2009

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

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

[FREE] WeFact Hosting module

Help testers nodig voor android app Urgent

Versio vervanger

Afspraken systeem met planbeperking

Partner Gezocht om meerdere NFT Collecties op Open Sea te Plaatsen

Plesk installer

cpanel / whm themes

PHP Point of Sale

213 Nieuwe domeinnamen September 2025

180 Nieuwe domeinnamen Augustus 2025

206 Nieuwe domeinnamen Juli 2025

Cron Bug

Participate now!

Fijne feestdagen

Kritieke update voor Really Simple Security-plug-in

ING Nederland streeft naar ondersteuning van Google Pay tegen eind februari

Functioneel ontwerp

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

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

Delen