te faltan cosas basicas como saber el ultimo Id, y tiene un fallo logico.
Código PHP:
public function consulta($consulta)
{
$resultado = mysql_query($consulta,$this->conex);
if(!$resultado)
{
$resultado = 'Error de Consulta: '.' '.mysql_error();
exit;
}
return $resultado;
}
si hay un error en la consulta, se ejecuta el if, guardas el error en la variable $resultado, posteriormente haces exit(); ahi el error logico, todo se detiene, por consiguiente nunca retornas el error si llegase a existir, se me hace que no la has sometido a pruebas.
esto es una sugerencia nada mas:
Código PHP:
function __construct()
{
$this->user='usuario';
$this->cla='clave';
$this->db='base_datos';
$this->server='localhost';
$this->conex = (mysql_connect($this->server,$this->user,$this->cla)) or die(mysql_error());
mysql_select_db($this->db,$this->conex) or die(mysql_error());
}
no veo porque asignar una variable para cada valor(usuario, clave, base de datos, servidor) si estan protegidas, el usuario no las puede modificar desde afuera, y solo las ocupas en un metodo, lo que me parece algo "inutilisable", y en mi opinion eso debe ser modificable, sea por X razon el usuario pueda necesitar cambiar de servidor de base de datos, deberian ser publicas :P