Czy używania takiej klasy ma jakiś sens, i czy jest ona dobrze napisana?

[PHP] pobierz, plaintext 
	class mysql{
 
		public $result;
 
		public $results;
 
		function __construct($dbhost, $dbuser, $dbpass, $dbname){
			$connectc = mysql_connect($dbhost, $dbuser, $dbpass) or die (mysql_error());
			$connects = mysql_select_db($dbname) or die (mysql_error());
		}
 
		function __destruct(){
			mysql_close();
		}
 
		function query($query){
			$this->result = mysql_query($query);
			if($this->result){
				return $this->result;
			} else {
				die(mysql_error());
			}
		}
 
		function fetch_array(){
			return mysql_fetch_array($this->result);
		}
 
		function fetch_array_all(){
			$results = array();
			while($return = mysql_fetch_array($this->result)){
				$results[] = $return;
			}
			return $results;
		}
 
		function num_rows(){
			return mysql_num_rows($this->result);
		}
 
	}
[PHP] pobierz, plaintext 

Najmnijeszego sensu nie ma, nic pożytecznego się w tej klasie nie dzieje, spowalniasz tylko program.

Użyj ActiveRecords, jeśli chcesz mogę Ci podesłać wersję standalone (wydłubaną z CodeIgnitera).
ActiveRecords dostarcza API do tworzenia zapytań, możesz podmieniać system bazy danych bez najmniejszej modyfikacji kodu.. Warunkiem jest stronienie od skomplikowanych zapytań (jak np. podzapytania).
Do tego dochodzią takie ciekawostki jak cache, forge i wiele innych.