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


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

Toon Meer

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

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

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

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

Toon Meer

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


?>

Toon Meer

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


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

Toon Meer

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;

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

|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

Beta-testers gezocht voor Crypto-oefenplatform

Na 15 jaar terug van weggeweest: iCriminals.nl is terug (BETA)!

Developer Gezocht

Op zoek naar de legends

[FREE] WeFact Hosting module

Help testers nodig voor android app Urgent

Versio vervanger

Afspraken systeem met planbeperking

350 Nieuwe Domeinnamen Januari 2026

321 Nieuwe Domeinnamen December 2025

Meerdere mafia game template te koop

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

Gebruikers die dit topic bekijken