Una cuestion general:
Si estas usando una clase para la conexion, te conviene que esta tenga encapsulados todos los datos y que ademas, la conexión sea estática y se haga de forma atumatica para no tener que estar llamandola a cada rato.
Ej:
Código PHP:
Ver original<?php
class conexion
{
const HOST = "localhost";
const USER = "root";
const PASS = "123456";
const DB = "almacen";
private static $mysqli = null;
private static $instances = 0;
public function __construct()
{
self::$instances++; // Registras la nueva instancia.
{
self::$mysqli = new mysqli(self::HOST, self::USER, self::PASS, self::DB);
if($this->mysqli->errno)
throw new Exception("Error al conectarse a la base de datos.");
}
}
public function __destruct()
{
self::$instances--; // Eliminas esta instancia.
if(!self::$instances) // Si es la ultima instancia.
{
self::$mysqli->close();
self::$mysqli = null;
}
}
Por otro lado tu problema pasa porque no estas encerrando las variables entre comillas, podrias hacer eso o bien bindear los paraetros.