Problem z MySQLi OOP, nie chce wykonać zapytania w metodzie by zwrócić wynik Opcje

[ksenonlogin]

Witam

Niżej przedstawiam klasę której głównym celem ma być realizacja zapytań do bazy przez mysqli i zwracanie wyniku końcowego przez wywołaną metodę.

Wszystko było by dobrze gdybym doszedł do etapu wykonywania zapytań, jednak zatrzymuję się na metodzie Query(), to właśnie tutaj ma być realizowane zapytanie gdzie metoda ma zwracać np.tablicę rekordów czy co tam mi przyjdzie do głowy.
Problem polega na tym że skrypt nie chce mi wykonać zapytanie tylko zatrzymuje się na nim i tyle.

Może robię coś źle, a raczej na pewno skoro nie działa

class

[PHP] pobierz, plaintext 
#-------------------+-----------------------------------------------------------------------+
# TYPE              |   CLASS                                                               |
#-------------------+-----------------------------------------------------------------------+
# CLASS             |   class C_DBMysqliOOP extends C_DBConnectionParameters                |
#-------------------+-----------------------------------------------------------------------+
# ATRYBUTES         |   private $host     = 'default';                                      |
#                   |   private $user     = 'default';                                      |
#                   |   private $password = 'default';                                      |
#                   |   private $database = 'default';                                      |
#-------------------+-----------------------------------------------------------------------+
# METHODS           |   public function __construct()                                       |
#                   |   public function M_Query($query)                                     |
#                   |   public function __destruct()                                        |
#-------------------+-----------------------------------------------------------------------+
 
 
class C_DBMysqliOOP extends C_DBConnectionParameters {
 
	private $host     = 'default';
	private $user     = 'default';
	private $password = 'default';
	private $database = 'default';
 
	public function __construct() {
 
		parent::M_DBDecoderParameters();
		$this -> host     = parent::M_Host();
		$this -> user     = parent::M_User();
		$this -> password = parent::M_Password();
		$this -> database = parent::M_Database();
 
		$mysqli = @new mysqli($this -> host, $this -> user, $this -> password, $this -> database);
 
		if ($mysqli -> connect_errno || $mysqli -> connect_error) {
			throw new Exception('Error Connect DB - ' . $mysqli -> connect_error.' NR: '.$mysqli -> connect_errno);
 
		}
	}
 
 
	public function M_Query($query) {//problematyczna metoda w której nie wiem jak zrealizować zapytanie oraz zwrócić winik...
 
		echo $query;
		//wyświetla zapytanie porawnie jednak go nierealizuje
 
		$result = $mysqli -> query($query);
		$row = $result -> fetch_array(MYSQLI_ASSOC);
		printf("%s (%s)\n", $row["user_id"], $row["user_login"]);
 
	}
 
	public function __destruct() {
		echo '</br></br>--------------------------------------------</br></br>';
	}
 
}
[PHP] pobierz, plaintext 

Wywołanie

[PHP] pobierz, plaintext 
	$ms = new C_DBMysqliOOP();
	$ms->M_Query("SELECT * FROM user WHERE user_id=2");
[PHP] pobierz, plaintext 

Ten post edytował ksenonlogin 2.09.2012, 22:51:27

[Greg0]

Metoda M_Query() nic nie zwraca, nie ma żadnego returna printf nie załatwia spawy

[ksenonlogin]

Metoda M_Query() nic nie zwraca, nie ma żadnego returna printf nie załatwia spawy

W tym przypadku to niema znaczenia, w sytuacji gdyby return coś zwracał to bym musiał to wydrukować poza klasą (wychodzi na to samo), ale mniejsza o to; wszystko już działa.
Problem był w tym że połączenie z bazą danych przypisywałem do zmiennej $mysqli (linijka 32 starego skryptu) co powodowało konflikt przy podpinaniu tej zmiennej do zapytania (linijka 46 starego skryptu).

Obecnie wprowadziłem kilka zmian i działa - niżej poprawny skrypt dla tych którzy mają podobny problem
Co prawda wymaga jeszcze kilka korekt, ale to kosmetyczne poprawki.


class

[PHP] pobierz, plaintext 
#-------------------+-----------------------------------------------------------------------+
# TYPE              |   CLASS                                                               |
#-------------------+-----------------------------------------------------------------------+
# CLASS             |   class C_DBMysqliOOP extends C_DBConnectionParameters                |
#-------------------+-----------------------------------------------------------------------+
# ATRYBUTES         |   private $host     = 'default';                                      |
#                   |   private $user     = 'default';                                      |
#                   |   private $password = 'default';                                      |
#                   |   private $database = 'default';                                      |
#-------------------+-----------------------------------------------------------------------+
# METHODS           |   public function __construct()                                       |
#                   |   public function M_Query($query)                                     |
#                   |   public function __destruct()                                        |
#-------------------+-----------------------------------------------------------------------+
 
 
class C_DBMysqliOOP extends C_DBConnectionParameters {
 
	private $host       = 'default';
	private $user       = 'default';
	private $password   = 'default';
	private $database   = 'default';
 
    private $db_connect = 'default';
 
 
 
	public function __construct() {
 
		                      parent::M_DBDecoderParameters();
		$this -> host       = parent::M_Host();
		$this -> user       = parent::M_User();
		$this -> password   = parent::M_Password();
		$this -> database   = parent::M_Database();
 
		$this -> db_connect = @new mysqli($this -> host, $this -> user, $this -> password, $this -> database);
 
		if ($this -> db_connect -> connect_errno || $this -> db_connect -> connect_error) {
			throw new Exception('Error Connect DB - ' . $this -> db_connect -> connect_error . ' NR: ' . $this -> db_connect -> connect_errno);
			}
			}
 
			public function M_Query($query) {
 
				if ($result = $this -> db_connect -> query($query)) {
					while ($row = $result -> fetch_assoc()) {
						$return[] = $row;
					}
				}
				return $return;
			}
 
			public function __destruct() {
			echo
 '</br></br>--------------------------------------------</br></br>';
	}
 
}
[PHP] pobierz, plaintext 

Wywołanie

[PHP] pobierz, plaintext 
$ms = new C_DBMysqliOOP();
$return = $ms -> M_Query("SELECT * FROM user");
 
foreach ($return as $row) {
	echo $row['user_id'] . ' ' . $row['user_login'] . '</br>';
}
[PHP] pobierz, plaintext 

Ten post edytował ksenonlogin 3.09.2012, 01:49:15

[Greg0]

Tak czy siak IMO klasa nie jest napisana poprawnie. Klasa nie powinna się zajmować prezentacją danych

[ksenonlogin]

Tak czy siak IMO klasa nie jest napisana poprawnie. Klasa nie powinna się zajmować prezentacją danych

A dokładniej?
chodzi o drukowanie danych czy ich zwracanie?
w których linijkach robię błędy?

Będę wdzięczny za wskazówki

[cudny]

Greg0wi chyba chodziło o to że jest to część biznesowa aplikacji (coś w rodzaju modelu) i nie powinieneś tam używać echo czy print ale z tego co widzę robisz to tylko dla sprawdzenia czy wszystko jest w porządku, więc problemu nie widzę

[ksenonlogin]

Greg0wi chyba chodziło o to że jest to część biznesowa aplikacji (coś w rodzaju modelu) i nie powinieneś tam używać echo czy print ale z tego co widzę robisz to tylko dla sprawdzenia czy wszystko jest w porządku, więc problemu nie widzę

Dokładnie tak.