Hoe moet ik de selectie (MySQL) juist aanpakken?

Door John De Zon op 14-07-2014 11:49 gewijzigd op 14-07-2014 11:49

5.744 views

Hallo iedereen,
Ik ben bezig met mijn browsergame en ik loop compleet vast bij het opzetten van de eerste keer dat iemand het speelt.
Het is de bedoeling dat als de user business_life 0 in de database heeft staan. Hij nog nooit gespeeld heeft dan moet de onderstaande html pagina gegeven worden maar ik gebruik MySQL! Ik heb al een query toegevoegd maar dan loop ik vast! Hopelijk kan iemand mijn hiermee helpen.


<?php
include 'config/mysql.php';
$userName = $_SESSION['userName'];
$query = mysql_query("SELECT * FROM `logins` WHERE `userName`='$userName' AND `business_life`='0'");
if(isset($_SESSION['userName'])) {
?>
<!DOCTYPE html>
<html>
    <head>
        <title>TeamCrafted: Business Life</title>
        <meta charset="UTF-8">
        <meta name="viewport" content="width=device-width, initial-scale=1.0">
        <link rel="stylesheet" type="text/css" href="http://www.teamgame.tk/css/game.css">
    </head>
    <body>
        <?php include_once("header.php"); ?>
    <div class="content">
            <div>
                <h2>Welkom <?php echo $_SESSION['userName'];?>,</h2>
              <p>op BUSINESS LIFE!</p>
              <p>We hebben vernomen dat dit de eerste keer is dat je dit spel speelt.</p>
              <p>Je kan starten als je wilt door op de startknop te klikken.</p>
              <form href="install_business_life.php">
              <button type="submit" class="button_kort" >Start</button>
              </form>
            </div>
    </div>
    </body>
</html>
<?php
} else {
    header('Location: index.php');
}
?>

- Ariën -

14-07-2014 11:58

met mysqli_num_rows() controleren of er een record gevonden is.
En pas ook goede foutafhandeling toe op $query (hoewel $result een beter dekkende naam is).

J John De Zon

14-07-2014 11:59

Ik begrijp niet hoe ik dit in mijn code moet toepassen.

- Ariën -

14-07-2014 12:03

Kijk eerst met een if-je of je $query gelukt is, en zo niet toon een foutmelding.


<?php
if($query) {
// gelukt, doe de rest...
	if(mysqli_num_rows($query)==1) {
		// een enkel item is gevonden, laat hier de HTML zien.
	} else {
		// niks gevonden.
	}

} else {
// toon een error mysqli_error()
}
?>

P Pipo Clown

14-07-2014 12:05 gewijzigd op 14-07-2014 12:07

Ik zou ook even het gebruik van de quotes omdraaien:


<?php
$query = mysql_query('SELECT * FROM logins WHERE userName="'.$userName.'" AND business_life = 0 ');
?>

Aangezien ik niet kan zien hoe je verbinding maakt is het gebruik van mysqli wat lastig mee te nemen voor mij.

- Ariën -

14-07-2014 12:09

En verder is de stap van MySQL naar MySQLi niet zo heel groot. Standaard gebruik ik MySQLi.

J John De Zon

14-07-2014 13:05

Dit is mijn resultaat dat uiteindelijk niet werkt.

<?php
include 'config/mysql.php';
$query = mysql_query('SELECT * FROM logins WHERE userName="'.$userName.'" AND business_life = 0 ');
if($query) {
echo 'Nieuwe account';
    if(mysql_num_rows($query)==1) {
        echo 'Bestaande account';
    } else {
        echo 'Niets gevonden!';
    }

} else {
echo 'MySQL error, contacteer [email protected]!';
}
if(isset($_SESSION['userName'])) {
?>
<!DOCTYPE html>
<html>
    <head>
        <title>TeamCrafted: Business Life</title>
        <meta charset="UTF-8">
        <meta name="viewport" content="width=device-width, initial-scale=1.0">
        <link rel="stylesheet" type="text/css" href="http://www.teamgame.tk/css/game.css">
    </head>
    <body>
        <?php include_once("header.php"); ?>
    <div class="content">
            <div>
                <h2>Welkom <?php echo $_SESSION['userName'];?>,</h2>
              <p>op BUSINESS LIFE!</p>
              <p>We hebben vernomen dat dit de eerste keer is dat je dit spel speelt.</p>
              <p>Je kan starten als je wilt door op de startknop te klikken.</p>
              <form href="install_business_life.php">
              <button type="submit" class="button_kort" >Start</button>
              </form>
            </div>
    </div>
    </body>
</html>
<?php
} else {
    header('Location: index.php');
}
?>

O Obelix Idefix

14-07-2014 13:05 gewijzigd op 03-04-2017 18:45

**knip** op 14/07/2014 11:49:10


<?php
include 'config/mysql.php';
$userName = $_SESSION['userName'];
$query = mysql_query("SELECT * FROM `logins` WHERE `userName`='$userName' AND `business_life`='0'");
if(isset($_SESSION['userName'])) {
?>

Dus je gaat eerst een nieuwe variabele maken (overigens onnodig).
Vervolgens een query uitvoeren met mysql ipv mysqli/pdo en zonder foutafhandeling.
En daarna (!!) ga je pas kijken of de sessie wel bestaat???
Lijkt mij een verkeerde volgorde.

- Ariën -

14-07-2014 13:24

Blijkbaar was Obelix en Idefix en je even voor, en je foutafhandeling is nu wel goed.
Maar op de plek van 'Bestaande account' moet je uiteraard wel de info tonen die er dan ook getoond meot worden.

Gebruik wel mysql_fetch_assoc() om die gegevens uit de query om te zetten in een array. Eventueel met een while() als je meerdere items verwacht.

J John De Zon

14-07-2014 16:38

Kan je dit ook in een code zetten, leer ik meer uit dan iets onbekend.

- Ariën -

14-07-2014 16:40


<?php
$sql = "SELECT ... FROM... WHERE iets = '....'";
$result= mysqli_query($con,$sql);
if($result) {
// query gelukt, doe de rest...
    if(mysqli_num_rows($query)==1) {
    // ALS HET OM EEN ENKEL RECORD GAAT:
    // $data = mysqli_fetch_assoc($result);
    // echo $data['veldnaam'];
    // ....OF BIJ MEERDERE...........
    // while($data=mysqli_fetch_assoc($result)) {
    // echo $data['veldnaam'];
    //    }
    } else {
        // niks gevonden.
    }

} else {
    // toon een error mysqli_error()
}
?>

Reageren

Inloggen om te reageren