輸出錯誤

Question

我的代碼如下：

class Database 
{ 
    private $db_host; 
    private $db_user; 
    private $db_pass; 
    private $db_name; 
    private $con; 

    public function __construct() { 
     $this->db_host = "localhost"; 
     $this->db_user = "admin"; 
     $this->db_pass = 'password'; 
     $this->db_name = 'test';  
     $this->con = ''; 
    } 

    public function connect() { 
     $db_name = "test";  
     $this->con = mysql_connect($this->db_host, $this->db_user, $this->db_pass); 
    } 

    public function select(){ 
     $q = "SELECT name, city FROM customers;"; 
     mysql_select_db($this->db_name, $this->con); 
     $result = mysql_query($q); 
     return mysql_fetch_assoc($result); 
    } 
} 


$db = new Database(); 
$db->connect(); 
$tempArray = Array(); 
$rs = $db->select('customers', 'name, suburb'); 
foreach ($rs as $row) 
{ 
    echo $rs['name'] . "<br>"; 
} 

而我的表中的數據是

name | city 
-------------- 
Anne | Sydney 
Jane | London 

實際的輸出是：

Anne 
Anne 

所需的輸出是：

Anne 
Jane 

有人能告訴我我做錯了什麼。這似乎是我錯過了一些基本的東西。我讀過50篇文章，似乎沒有任何解釋我做錯了什麼。

注意：這是我的代碼的縮小版本。我打算用這個來創建一個更通用的對象，從我的數據庫中提取信息。

感謝，

佈雷特

Answer 1

你需要調用mysql_fetch_assoc爲每個行。它只返回一行數據，而不是全套數據。例如，你可以將它移出到循環：

class Database 
{ 
    /* ... */ 

    public function select(){ 
     $q = "SELECT name, city FROM customers;"; 
     mysql_select_db($this->db_name, $this->con); 
     return mysql_query($q); 
     /* Remove your line here, returning the query result, not the first row */ 
    } 
} 

$db = new Database(); 
$db->connect(); 
$tempArray = Array(); 
$result = $db->select('customers', 'name, suburb'); 
/* Note that I'm now using mysql_fetch_assoc to get each row from the result */ 
while ($row = mysql_fetch_assoc($result)); 
    echo $row['name'] . "<br>"; 
} 

您可以使用while循環那裏，因爲最後一排已經被檢索後，mysql_fetch_assoc將返回FALSE和退出循環。

Answer 2

在代碼中的這一部分：

return mysql_fetch_assoc($result); 

你只是返回第一行。我建議你創建一個數組。

public function select(){ 
    $q = "SELECT name, city FROM customers;"; 
    mysql_select_db($this->db_name, $this->con); 
    $result = mysql_query($q); 
    $toReturn = array(); 
    while($row = mysql_fetch_assoc($result)) 
     $toReturn[] = $row; 
    return $toReturn; 
} 

Answer 3

Should'nt它是：

foreach ($rs as $row) 
{ 
    echo $row['name'] . "<br>"; 
} 

和：

$resArr = array(); 
while($res = mysql_fetch_assoc($result)) { 
    $resArr[] = $res; 
} 
return $resArr; 

Answer 4

你的班級的一個理智的版本。它沒有當然的要害部位，在現實生活中使用，只是你的素描：

class Database 
{ 
    private $con; 

    public function __construct() { 
     $this->con = mysql_connect("localhost", "admin", 'password'); 
     mysql_select_db("test",$this->con); 
    } 
    public function query($sql,$this->con){ 
     return mysql_query($sql); 
    } 
    public function get_all($sql,$this->con){ 
     $ret = array(); 
     $res = mysql_query($sql); 
     while ($row = mysql_fetch_array($row)) { 
      $ret[] = $row; 
     } 
     return $ret; 
    } 
} 

$db = new Database(); 
$rs = $db->get_all("SELECT name, city FROM customers"); 
foreach ($rs as $row) 
{ 
    echo $rs['name'] . "<br>"; 
} 

Answer 5

似乎有點奇怪的是，沒有人在代碼拿起有點魚叉這裏。

你定義select()像這樣：

public function select(){ 
    $q = "SELECT name, city FROM customers;"; 
    mysql_select_db($this->db_name, $this->con); 
    $result = mysql_query($q); 
    return mysql_fetch_assoc($result); 
} 

但是，你這樣稱呼它：

$rs = $db->select('customers', 'name, suburb'); 

我假設你的意圖是能夠指定表和字段選擇來自數據庫。如果是，你select功能應該看起來更像是這樣的：

public function select($table, $fields){ 
    $q = "SELECT $fields FROM $table;"; 
    mysql_select_db($this->db_name, $this->con); 

    return mysql_query($q); 
} 

從那裏，你會關注@ BENLEE的例子在他的回答，因爲你需要遍歷一個結果。每個字段成爲關聯數組中的一個鍵。

我不會推薦在生產代碼中實際執行字符串插入，但我認爲它更接近你的意圖。

HTH。