Shop-Script Problem

[AnqeL]

Hey Com,

Ich habe ein Problem mit meinem Shop-Script.

Warning:  mysql_num_rows() expects parameter 1 to be resource, boolean given in /usr/local/www/apache22/data/scripts/itemshop/shopbuy.php on line 31
Kaufen fehlgeschlagen. 

Hier das Script:

Spoiler:

<?php
session_start();
    if($_SESSION['id'] == "") {
        die('Logge dich bitte ein!');
    }
    else {
    
    include('../../inc/dbsettings.php');
    $db    = "account";
    mysql_connect($dbhost, $dbuser, $dbpw) OR
    die("ERROR: Connection failed. ".mysql_error());        
    mysql_select_db($db) OR
    die("ERROR: DB allready open. ".mysql_error());
        
        $getuser = "SELECT * from account where login = '" . $_SESSION['id'] . "'";
            $checkuser = mysql_query($getuser);
                $num = mysql_num_rows($checkuser);
            
        if($num > 0) {
        
            $array = mysql_fetch_array($checkuser);
    
            $itemid = $_GET['id'];
            $user = $_SESSION['id'];
            $coins = $_SESSION['coins'];
                        
                    
                    
            $check = "SELECT * from itemshop where value = '" . $itemid . "'";
                $query = mysql_query($check);
                    $num = mysql_num_rows($query);
            if($num > 0) {
                $array2 = mysql_fetch_array($query);
            $rescash = $array['coins'] - $array2['coins'];
                        
        if($num > 0 && $rescash >= 0) {
                mysql_select_db('player');
            $i = 0;    
        $giveitem = "INSERT INTO item SET owner_id = '" . $array['id'] . "', window = 'MALL', pos = '" . $_SESSION['pos'] . "', count = '1', vnum = '" . $itemid . "', socket0 = '0', socket1 = '0', socket2 = '0'";
            $result = mysql_query($giveitem);
            
                if($result) {
                
                
                if($_SESSION['pos'] == '42') {
                 
                 $_SESSION['pos'] = '0'; }
                 else { $_SESSION['pos'] = $_SESSION['pos'] + 1; }
                 
                    mysql_select_db('account');
                    
                        $cashresult = $array['coins'] - $array2['coins'];
                    
                    $coins = "UPDATE account set coins = '" . $cashresult . "' where login = '" . $_SESSION['id'] . "'";
                        $coinsresult = mysql_query($coins);
                        
                    if($coinsresult) {
                    $_SESSION['coins'] = $cashresult;
                    
                    echo "<meta http-equiv="refresh" content="0;url=index.php?page=buysuccess">";
                    } else {
                    echo "Kaufen fehlgeschlagen.";
                    }
                
                }
                else { echo "Kaufen fehlgeschlagen."; }
                
            } else { echo "Du hast zu wenig Coins."; }
            
        } else { echo "Kaufen fehlgeschlagen."; }
        
    } else { echo "Bitte logge dich ein."; }

    }

?>

Hoffe ihr könnt mir helfen.

[Paradox]

Bei PHP + Mysql immer mysql_error() benutzen. Dann wird dir der Error meist genau beschrieben.

Und ich würde mal versuchen mit http://php.net/manual/de/function.mysql-fetch-assoc.php zu Arbeiten

[AnqeL]

Hab ich noch nie ^^
ich versteh aber nicht was an $num = mysql_num_rows($query); falsch sein soll?

[Comu]

Den Fehler kriegst du bei mysql_num_rows, wenn dein Query nichts zurückliefert (dieser also nicht funktioniert...). Boolean is given, heißt true oder false wurde gegeben anstatt 1 oder 0, da es jedoch nur "true" bei mysql_num_rows in Verbindung mit 1 gibt, und der Error dann nicht ausgegeben würde, ist dein Query fehlerhaft.

[AnqeL]

Den Fehler kriegst du bei mysql_num_rows, wenn dein Query nichts zurückliefert (dieser also nicht funktioniert...). Boolean is given, heißt true oder false wurde gegeben anstatt 1 oder 0, da es jedoch nur "true" bei mysql_num_rows in Verbindung mit 1 gibt, und der Error dann nicht ausgegeben würde, ist dein Query fehlerhaft.

Okay, Danke ^^

Oh man ich kapier es gerade nicht mehr viel zu lange in der scheiß db heute gewesen x.x
möchte einfach nur den dummen fehler weg haben gerade ^^

[Comu]

By the way:

               mysql_select_db('player'); 

davon sehe ich viele verschiedene in deinem Script.. Ich glaub kaum, dass du für jedes eine eigene Datenbank angelegt hast, oder? Kann doch viel eher eine Tabelle sein. Dann musst du den Query so ausführen: SELECT * FROM `player` WHERE ...

[AnqeL]

By the way:

               mysql_select_db('player'); 

davon sehe ich viele verschiedene in deinem Script.. Ich glaub kaum, dass du für jedes eine eigene Datenbank angelegt hast, oder? Kann doch viel eher eine Tabelle sein. Dann musst du den Query so ausführen: SELECT * FROM `player` WHERE ...

Joa es ist keine Tabelle die Datenbank heist player die tabelle account

[Comu]

Weiter unten machste aber mit:

mysql_select_db('account');

weiter.. Und oben ebenfalls mit mysql_select_db($db). Das wären dann 3 verschiedene Datenbanken, mit jeweils einer Tabelle?

[AnqeL]

Weiter unten machste aber mit:

mysql_select_db('account');

weiter.. Und oben ebenfalls mit mysql_select_db($db). Das wären dann 3 verschiedene Datenbanken, mit jeweils einer Tabelle?

Oh man Comu du bringst am frühen morgen meinen Kopf zum Rauchen x'D

Okay wie geht das den mit diesem mysql_fetch_assoc ?
oder wie sollte es dann aussehen xDD?!

[Paradox]

Habe dir dazu ein Link gegeben. Viel anders ist es nicht, macht nur nicht soviele Error's. Kleiner Tip noch was Comu schon Angesprochen hat.. ich würde EINDEUTIGE DB namen und ebenso Tablenamen nehmen. Üblicherweise wäre alles was mit Player zu tun hat dann in der Datenbank von Account zu finden. Kommentiere deine Schritte damit du hinterher selber noch weist was du gemacht hast. Viele Anfänger denken Kommentieren ist nicht Wichtig weil sie Denken, ich weis doch was ich da mache , aber wenn du gerade 2 Stunden mit einer Datenbank gearbeitet hast kommt dir son netter Komment // Bewirkt dies und das ganz gelegen sonst findest du dich in deinem eigenen Code nicht mehr zurecht.