0

私はこれを持っています（データベースクラスでの最初の試みから派生した他の誰かから）：

    require_once( "declarations.php" ); 

    class Database{            
        private static $mysqli;   
        private static $dbName = '';
        private static $username = '';
        private static $password = '';   
        private static $host = 'localhost';
        private static $prefix = '';   

        public function __construct(){  
            if( self::$host & self::$username & self::$password & self::$dbName )
            {
                self::$mysqli = new mysqli( self::$host, self::$username, self::$password, self::$dbName );

                 if (self::$mysqli->connect_error) {
                die('Connect Error (' . self::$mysqli->connect_errno . ') '
                    . self::$mysqli->connect_error);
                }
            }
            else
            {
                echo "You forgot to fill in your database connection details";
            }
        }

        public function Query( $query ){
            $query = self::$mysqli->real_escape_string( $query );

            if ($query = self::$mysqli->prepare($query)) {
                $query->execute();
                $query->store_result();            


                $stmt = $query->result;        
                //$query->mysql_num_rows = $stmt->num_rows();                 
                $query->close();
                return $stmt;                   
            }
        }  

        public function Close()
        {
            self::$mysqli->close(); 
        }
    }

これは私がそれを呼んでいる方法です：

    include_once( "system/database.php" );

    $query = "SELECT * FROM app";     
    $dbr = new Database();  

    //Change this here since your method is query and not $mysqli
    while( $row = $dbr->Query( $query )->fetch_object() ){
       echo '<td>'. $row['id'] . '</td>' ;
       echo '<td>'. $row['title'] . '</td>' ;   
    }  

    Database::Close();

fetch_object()while ループで、非オブジェクトのメンバー関数への呼び出しというエラーが発生します。

何か案は？

4

2 に答える 2

-1

さて、あなたの最初の試みは完全に使用できないコードで生じました。
2つの重大な障害と1つの重大な障害があります。

  1. すでにお話ししたように、やること$query = self::$mysqli->real_escape_string( $query );は役に立たず、一度に有害です。この行を削除する必要があります。完全にそして永遠に。
  2. 変数をバインドせずにクエリを準備することはまったく役に立ちません。
  3. mysqlエラーをチェックする必要があります。

したがって、少なくともquery（）関数は

public function query($query)
{
    $res = self::$mysqli->query($query);
    if (!$res)
    {
        throw new Exception(self::$mysqli->error);
    }
    return $res;
}

ただし、この関数は、クエリ内のデータを置き換えるプレースホルダーを実装していないため、安全ではありません。

于 2013-02-11T16:09:19.667 に答える
-1

fetch_object は、クエリが次のようなメソッドで実行された後に返される結果セットで動作します: mysql_query または代わりに fetch_assoc を使用します

$query->execute();
$result = $query->get_result();
while ($myrow = $result->fetch_assoc()) {
    //Your logic
}
于 2013-02-11T14:44:58.490 に答える