probleem bij query in if statement.

ruttydm · 20 Januari 2014

Ik zot momenteel te sukkelen met mijn energy systeem op mijn spel, hij reageert namelijk niet op mijn if, heb ik ergens een fout in de syntax?

PHP

<?php
include 'connect.php';
require_once("functions.php");
$player=$_SESSION['player'];
$userstats="SELECT * from km_users where playername='$player'";
$userstats2=mysql_query($userstats) or die("Could not get user stats");
$userstats3=mysql_fetch_array($userstats2);
$player=$_SESSION['player'];	




$addenergy=$userstats3[energycron] * 10;


	if($userstats3[energycron]>0){
		if($userstats3[energycron]>10){
			mysql_query("Update km_users set energycron='10' where ID='$userstats3[ID]'") or die("cron failed");
		}
		if($userstats3[numturns]>100){
			mysql_query("Update km_users set numturns='100' where ID='$userstats3[ID]'") or die("energy failed");
			mysql_query("Update km_users set energycron='0' where ID='$userstats3[ID]'") or die("cron failed");
		}
		else{
		mysql_query("Update km_users set numturns=numturns+'$addenergy' where ID='$userstats3[ID]'") or die("energy failed");
		mysql_query("Update km_users set energycron='0' where ID='$userstats3[ID]'") or die("cron failed");
		}
	}


?>

Toon Meer

J.Rijdes · 20 Januari 2014

Even quote's gebruiken in het oproepen. Dus als volgt:

PHP

$userstats3['energycron'] en $userstats3['numturns']

ruttydm · 20 Januari 2014

Code

Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in /home/rutgerx99/domains/rutgerx99.ninetynine.axc.nl/public_html/up_html.php on line 15

dit heb ik nu

J.Rijdes · 20 Januari 2014

Geen probleem, kan iedereen overkomen. Opgelost nu dus?

ruttydm · 20 Januari 2014

heb bericht aangepast, deze error meerdere malen gehad maar weet niet door wat het komt

J.Rijdes · 20 Januari 2014

Ook in je query's moet je quote's gebruiken.

Je hebt dit:

PHP

where ID='$userstats3[ID]'

Dat moet worden:

PHP

where ID = '".$userstats3['ID']."'

Bij elke query die je uitvoert moet je goed controleren op je single en double quote's.

ruttydm · 20 Januari 2014

Het reageert steeds niet op het if statement en de query werkt niet?

J.Rijdes · 20 Januari 2014

Welke code heb je nu sinds je aanpassingen?

ruttydm · 20 Januari 2014

PHP

<?php
include 'connect.php';
require_once("functions.php");
$player=$_SESSION['player'];
$userstats="SELECT * from km_users where playername='$player'";
$userstats2=mysql_query($userstats) or die("Could not get user stats");
$userstats3=mysql_fetch_array($userstats2);
$player=$_SESSION['player'];	




$addenergy=$userstats3['energycron'] * 10;


	if($userstats3['energycron']>0){
		if($userstats3['energycron']>10){
			mysql_query("Update km_users set energycron='10' where ID = '".$userstats3['ID']."'") or die("cron failed");
		}
		if($userstats3['numturns']>100){
			mysql_query("Update km_users set numturns='100' where ID = '".$userstats3['ID']."'") or die("energy failed");
			mysql_query("Update km_users set energycron='0' where ID = '".$userstats3['ID']."'") or die("cron failed");
		}
		else{
		mysql_query("Update km_users set numturns=numturns+'$addenergy' where ID = '".$userstats3['ID']."'") or die("energy failed");
		mysql_query("Update km_users set energycron='0' where ID = '".$userstats3['ID']."'") or die("cron failed");
		}
	}


?>

Toon Meer

Reza · 20 Januari 2014

Ik raad je aan om bij gevallen waar je niet zeker bent wat de fout is en wanneer je geen foutmelding krijgt als eerste te kijken naar de query's. Meestal ligt hier dan ook de fout.

Gebruik de functie var_dump eens, door bijvoorbeeld dit te doen:

PHP

<?php
var_dump($player, $userstats, $userstats2, $userstats3);
?>

Je kan eventueel nog meer variabelen erin verwerken, meestal als er iets niet bij mij werkt ga ik d.m.v. var_dump even alles na, kijken of alles wel werkt zoals het hoort en door zo'n overzicht kom je al snel te weten waar het precies fout gaat.

ruttydm · 20 Januari 2014

ik heb de var dump al geprobeerd maar kan hier niets uit opmaken

Reza · 20 Januari 2014

Citaat van ruttydm

ik heb de var dump al geprobeerd maar kan hier niets uit opmaken

Zou je het resultaat willen posten? Het scheelt een hoop werk om het resultaat van var_dump te bekijken dan je hele code na te kijken naar wat er precies mis is.

ruttydm · 21 Januari 2014

Het zegt altijd string "89" , dit is toch normaal?

SC-Webmedia · 21 Januari 2014

Probeer dit eens:

PHP

<?php
include 'connect.php';
require_once("functions.php");


$player 	= $_SESSION['player'];


$userstats 	= "SELECT
					*
				FROM
					km_users
				WHERE
					playername = '".$player."'";
$userstats2 = mysql_query($userstats) or die ("Could not get user stats");
$userstats3 = mysql_fetch_array($userstats2);


$addenergy 	= $userstats3['energycron'] * 10;


if($userstats3['energycron'] > 0)
{
    if($userstats3['energycron'] > 10)
    {
        mysql_query("UPDATE
        				km_users
        			SET
        				energycron='10'
        			WHERE
        				ID = '".$userstats3['ID']."'") or die ("cron failed");
    }
    if($userstats3['numturns'] > 100)
    {
        mysql_query("UPDATE
        				km_users
        			SET
        				numturns='100',
        				energycron='0'
        			WHERE
        				ID = '".$userstats3['ID']."'") or die("energy failed");
    }
    else
    {
	    mysql_query("UPDATE
	    				km_users
	    			SET
	    				numturns=numturns+'".$addenergy."',
	    				energycron='0'
	    			WHERE
	    				ID = '".$userstats3['ID']."'") or die("energy failed");
    }
}
?>

Toon Meer

Je gebruikte 2x $player = $_SESSION['player'];.
Query's waren niet netjes of je gebruikte meerdere query's, waarbij 1 voldoende was.

Reza · 21 Januari 2014

Citaat van ruttydm

Het zegt altijd string "89" , dit is toch normaal?

Hij laat waarschijnlijk niet alleen string 89 zien, string 89 betekent dat het een string met 89 karakters is.
Ik heb even een print gemaakt van een resultaat van var_dump: http://prntscr.com/2l9gsp

ruttydm · 22 Januari 2014

Het is nu opgelost ik heb de eerste if verwijderd en mijn structuur nagekeken.

link: http://rutgerx99.ninetynine.axc.nl/

cakemasher · 23 Januari 2014

Ik zou netjes en overzichterlijk programmeren als ik jou was. Met dingen als $userstats, $userstats1, $userstats2 raak je het overzicht snel kwijt. Daarnaast maak je geen gebruik van mysql_error wat in mijn mening erg slecht is als je bezig bent met debuggen. Zodra een MySQL error plaatst vind, zorg er dan voor dat je aan de hand van de tekst wat de 'or die ()' functie (of een andere functie) geeft, je kan lokaliseren welke query het over gaat en wat er fout mee is.

PHP

<?php


	include ('connect.php');
	require_once ('functions.php');
	
	$player		= mysql_real_escape_string ($player);
	
	$userstats	= "SELECT * from km_users where playername='" . $player . "'";
	$query		= mysql_query ($userstats) or die ("Could not get user stats!<br /><br />" . mysql_error ());
	
		if (mysql_num_rows ($query) != 0)
		{
			$array		= mysql_fetch_assoc ($query);
			
			$addEnergy	= $array['energycron'] * 10;
			
				if ($array['energycron'] > 0)
				{
					if ($array['energycron'] > 10)
						mysql_query ("Update km_users set energycron='10' where `ID`='" . $array['ID'] . "'") or die ("Cron failed! [0]<br /><br />" . mysql_error ());
					
					if ($array['numturns'] > 100)
					{
						mysql_query ("Update km_users set numturns='100' where `ID`='" . $array['ID'] . "'") or die("Energy failed! [0]<br /><br />" . mysql_error ());
						mysql_query ("Update km_users set energycron='0' where `ID`='" . $array['ID'] . "'") or die("Cron failed! [1]<br /><br />" . mysql_error ());
					}
					else
					{
						mysql_query ("Update km_users set numturns=numturns+" . $addEnergy . " where `ID`='" . $array['ID'] . "'") or die("Energy failed! [1]<br /><br />" . mysql_error ());
						mysql_query ("Update km_users set energycron='0' where `ID`='" . $array['ID'] . "'") or die("Cron failed! [2]<br /><br />" . mysql_error ());
					}
				}
		}
		else
			die ("User '" . $player . "' does not exists!");
	
?>

Toon Meer

ruttydm · 23 Januari 2014

Ok, wat bijgeleerd.

een eigen cloud sever

DNS update Directadmin

Graag advies voor mijn website

Ik wil op afstand studeren, kennen jullie Tech?

PC probleem

Let op Vimexx klanten

wie kan mij helpen met Security aanscherpen voor nmijn website

Ik ben op zoek naar een goede hosting

Dating software

250 Nieuwe domeinnamen April 2024

Xenforo buckaroo plugin

probleem bij query in if statement.

Participate now!

ING Nederland streeft naar ondersteuning van Google Pay tegen eind februari

Het Belang van Standaardisatie

De Revolutionaire Rol van IT in Alzheimeronderzoek

Functioneel ontwerp

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

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

Delen