Credditshop probleem

Jeroen.G · 5 Januari 2013

Hallo allemaal ik heb een credits shop gemaakt voor het vendetta source (deels gehaald uit criminolz)

Maar nu voert het script niks uit, zou iemand er naar kunnen kijken.

Hier het script

PHP

<?php


  include('config.php');
 $dbres = mysql_query("SELECT *,UNIX_TIMESTAMP(`pc`) AS `pc`,UNIX_TIMESTAMP(`transport`) AS `transport`,UNIX_TIMESTAMP(`bc`) AS `bc`,UNIX_TIMESTAMP(`slaap`) AS `slaap`,UNIX_TIMESTAMP(`kc`) AS `kc`,UNIX_TIMESTAMP(`start`) AS `start`,UNIX_TIMESTAMP(`crime`) AS `crime`,UNIX_TIMESTAMP(`ac`) AS `ac` FROM `users` WHERE `login`='{$_SESSION['login']}'");
  $data	= mysql_fetch_object($dbres);
 
  if(! check_login()) {
    header('Location: login.php');
    exit;
 


 }


?>


<html>
<title>Criminalita - De Online Mafia Game</title>
<link href="style.css" rel="stylesheet" type="text/css">
<meta name="language" content="english">
</head>


<table width="100%">
  <tr> 
    <td class="subTitle"><b>Callcreditsshop Info</b></td>
  </tr>
  <tr><td class="mainTxt"><br />
<a href="belservice.php">Hier kan u callcredits kopen</a> <br />
<?php
$callcredits = number_format($data->callcredits, 0, ',' , ','); 
print <<<ENDHTML
<b>Je hebt momenteel $callcredits callcredits</b><br />
ENDHTML;
?> 
<br /></td></tr>
  <tr> 
    <td class="mainTxt">
<?php


if(isset($_POST['submit']) && isset($_POST['product'])) {
  $fout = array();
  $p = $_POST['product'];
  unset($_POST['product']);
  $cost = array("undefined", 25, 25, 50, 25, 200, 5, 20, 13);
  if($data['callcredits'] < $cost[$p]) {
    $fout[] = " - Je hebt niet genoeg callcredits hiervoor";
  }
  if(empty($fout[0])) {
    switch($p) {
    
    case 1;
      $query = "UPDATE users SET nrofcrime = nrofcrime + '100' WHERE login = '" . $data['login'] . "'";
    break;
    
    case 2;
      $query = "UPDATE users SET nrofcar = nrofcar + '75' WHERE login = '" . $data['login'] . "'";
    break;
    
    case 3;
      $query = "UPDATE users SET cash = cash + '1000000' WHERE login = '" . $data['login'] . "'";
    break;
    
    case 4;
      $query = "UPDATE users SET kogels = kogels + '100' WHERE login = '" . $data['login'] . "'";
    break;
    
    case 5;
      $query = "UPDATE users SET prestige = prestige + '1' WHERE login = '" . $data['login'] . "'";
    break;
    
    case 6;
      $query = "UPDATE users SET storten = storten + '5', stortenreset = stortenreset + '5' WHERE login = '" . $data['login'] . "'";
    break;
    
    case 7;
      $query = "UPDATE users SET rankvordering = rankvordering + '25' WHERE login = '" . $data['login'] . "'";
    break;
    
    }
    
    mysql_query($query);
    mysql_query("UPDATE users SET callcredits = callcredits - '" . $cost[$p] . "' WHERE login = '" . $data['login'] . "'");
    echo '<font color="green"><b>Aankoop geslaagd!</b></font><br />';
  } else {
    foreach($fout as $tekst) {
      @$foutje .= "<br />\r\n" . $tekst;
    }
    echo "<font color=\"red\" style=\"font-weight: bold\">De volgende dingen gingen verkeerd:</font>" . $foutje . "<br />";
  }
}


echo'
<center>
<form method="post">
<table width="500" cellpadding="0" cellspacing="0">
  <tr>
    <td class="subtitle">&nbsp;</td>
    <td class="subtitle">Artikel</td>
    <td class="subtitle">Prijs</td>
  </tr>
  
  <tr>
    <td class="list"><input type="radio" name="product" value="1" /></td>
    <td class="list">100 extra voltooide misdaden (hoger slagings percentage)</td>
    <td class="list">25 callcredits</td>
  </tr>
  
  <tr>
    <td class="list"><input type="radio" name="product" value="2" /></td>
    <td class="list">75 extra voltooide auto diefstallen (hoger slagings percentage)</td>
    <td class="list">25 callcredits</td>
  </tr>
  
  <tr>
    <td class="list"><input type="radio" name="product" value="3" /></td>
    <td class="list">&euro; 1.000.000,-</td>
    <td class="list">50 callcredits</td>
  </tr>
  
  <tr>
    <td class="list"><input type="radio" name="product" value="4" /></td>
    <td class="list">100 kogels</td>
    <td class="list">25 callcredits</td>
  </tr>
  
  <tr>
    <td class="list"><input type="radio" name="product" value="5" /></td>
    <td class="list">1 Prestige omhoog</td>
    <td class="list">200 callcredits</td>
  </tr>
  


</table>
<input type="submit" name="submit" class="submit" value="Kopen!" />
</form>
</center>';
?>
</td>
</tr>
<table width="441" align="center">
</td></tr>
</table>

Toon Meer

Al vast bedank

Reza · 5 Januari 2013

Heb je een online voorbeeld?
Komt er wit beeld of het script wel alleen niks verder?

Jeroen.G · 5 Januari 2013

hij geeft niks weer als ik de bestelling aanklik en dan op koop, er zou minimaal de foutmelding moeten komen dat het niet gelukt is, maar dat doet hij niet je krijgt in principe dus een blanke pagina..

Tim · 5 Januari 2013

Hallo,

Wat heb je gedaan om te debuggen? En simpelweg een script kopiëren gaat natuurlijk niet werken. Er moeten wel dingen geprogrammeerd worden.

Mvg,
Tim

Jeroen.G · 5 Januari 2013

ik heb de opdrachten aangepast uiteraard en alles een keer na gelopen of het klopt. een keer door mijn debugger gehaald in dreamweaver.

Dat heb ik momenteel geprobeerd maar zodra ik hem in de criminolz source probeer werkt deze wel.
dus daar heb ik ook nog ingame.php en de includes daarvan na gelopen of ik niet iets miste maar dit doe ik niet.

Reza · 5 Januari 2013

Zet error_reporting eens aan.
En plaats achter elke query or die (mysql_error())

Tim · 5 Januari 2013

Hallo,

Probeer eens te debuggen met steeds het weghalen van code om erachter welke code conflicteert.

Mvg,
Tim

Jeroen.G · 5 Januari 2013

Tim wat bedoel je precies?
Reza geen uitkomst

gofie · 5 Januari 2013

Bouw gewoon een nieuwe en anders heb ik nog wel 1 liggen die je voor kleine bedrag mag overnemen.

Victor · 5 Januari 2013

- Ik zie geen foutafhandeling voor je queries
- Probeer Error reporting aan te zetten
- Het is overzichtelijker als je je html en php zoveel mogelijk gescheiden probeert te houden.

Reza, or die() is een foute manier van een query afhandelen. Aangezien mysql_query() de boolean false teruggeeft als er iets mis ging, kan je een if statement gebruiken.

PHP

<?php


if($query == false)
{
    echo 'Fout in de query!';
}


?>

Error reporting:

PHP

<?php


//1 is aan, 0 is uit
ini_set('display_errors',1);
error_reporting(E_ALL | E_STRICT);


?>

Patrick · 6 Januari 2013

Victor ik wil niet vervelend doen,
maar als je een or achter een myql_query zet kan je degelijk wel de foutmelding vanuit je sql verkrijgen.

PHP

mysql_query('SELECT * FROM users') or die(mysql_error());

De pure mysql_query is wel boolean false dat hij returnt maar daarnaast lijkt het me ook niet slim om daarop te vertrouwen aangezien dat niet al te stabiel reageert wat hij ook door php wordt aangegeven.

Victor · 6 Januari 2013

Patrick, dat is correct. Redenen waarom je het toch niet moet gebruiken:

- Het is niet userfriendly.
- In de error kan informatie staan die je niet in je website wil laten zien. Het is okay in een testomgeving, maar gebruik dit nooit in een eindproduct!
- Je kan de error niet loggen.
- Script word abrupt gestopt.

Bron: http://at-byte.com/technology/…g-or-die-php-bad-practice

Je kan hierbij natuurlijk ook mooie classes schrijven die de error loggen en een nette userfriendly message teruggeven.

WHMCSAddons · 6 Januari 2013

Victor
or die(mysql_error) is inderdaad niet vriendelijk maar wel handig voor jezelf om te weten wat er fout gaat, je krijgt dus een volledige foutmelding.
Userfriendly is het zeker niet dat ben ik volledig met je eens, maar als je debuggen bent is het juist makkelijker.

Victor · 6 Januari 2013

Fils, het gaat erom dat je altijd foutafhandeling in je script moet hebben. In plaats van de niet userfriendly or die methode te gebruiken kan je net zo goed nette foutafhandeling inbouwen met hetzelfde resultaat.

WHMCSAddons · 6 Januari 2013

Hoeveel eigen coders zitten hier, niet veel dus meestal is dit oude code die allemaal gebruikt wordt.
Jou manier is wel de correcte manier maar in open-source code is dit vaak niet toegepast en dus snappen ze het ook niet, daarom is or die makkelijker voor ze.

Victor · 6 Januari 2013

Precies! Dat is de reden waarom we gewoon moeten zeggen: Je kan het beter op die en die manier doen.
We zijn een community en dit is gewoon een leerproces voor ons allemaal.

Jeroen.G · 6 Januari 2013

mensen zouden jullie ontopic willen gaan hier schiet ik ook niks mee op, ik heb alles wat jullie zeggen geprobeerd alleen ik krijg niks te zien

Tim · 6 Januari 2013

Hallo,

Met mijn post bedoelde ik dat vanuit niks begint en stukje bij beetje code er van toevoegt tot het fout gaat. Dan weet je de bron. Of andersom.

Mvg,
Tim

Partner Gezocht om meerdere NFT Collecties op Open Sea te Plaatsen

Plesk installer

cpanel / whm themes

PHP Point of Sale

Probleem omtrent Tinymce

Kunnen jullie mij helpen met deze enquete

een eigen cloud sever

Probleem C++ en CL

193 Nieuwe domeinnamen April 2025

228 Nieuwe domeinnamen Maart 2025

285 Nieuwe domeinnamen Februari 2025

Credditshop probleem

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