problema al recorrer array con for

[guardarmicorreo]

estoy aprendiendo la programación orientada a objetos con php. en este caso la consulta a base de datos con clases.
he creado estos dos códigos y no imprime en pantalla el valor que hay en la base de datos, no entiendo dónde está el error. si alguien puede ayudarme se lo agradezco.

index.php

aquí un usuario solo puede consultar las habitaciones de hotel que dispone

Código PHP:

<div>
        <form method="post" action="index.php">
            <select name="habitaciones">
                <option value="simple">Habitación individual</option>
                <option value="doble">Habitación doble</option>
                <option value="triple">Habitación triple</option>
                <option value="cuadruple">Habitación cuadruple</option>
            </select>
            <input type="submit" name="enviar_plazas" value="Enviar">
        </form>
    </div>
    <?php
    if (isset($_POST['enviar_plazas'])) //si existe el boton enviar_plazas 
    {
        //instancia un nuevo objeto Plazas pasándole dos valores
        $rest = new Plazas($_SESSION['usuario'], $_POST['habitaciones']);
        $plazas = $rest->consultabd();
        for ($i = 0; $i < sizeof($plazas); $i++) //creo que es aquí donde está el problema pero no //estoy seguro
        {
            echo $plazas;
        }
     }
    ?>
 

clases.php

aquí realizo la consulta a base de datos asignando la consulta a una variable que retornaré al index.php

Código PHP:

<?php
//esta es la clase conectar que recibiendo parámetros realiza las consultas
class Conectar
{
 
    public static function conexion($TextoSQL)
    {
        $IDConexion = mysql_connect("localhost", "root", "");
 
        mysql_select_db("hoteles", $IDConexion);
 
        $IDConsulta = mysql_query($TextoSQL, $IDConexion) or die(mysql_error());
 
        if ($IDConsulta == false)
        {
            echo "ERROR: <br>\n";
            echo mysql_error($IDConexion);
        }
 
        return $IDConsulta;
    }
 
}
 
//esta es la clase Plazas
class Plazas
{
    //declaro las variables que usaré como privadas
    private $usuario;
    private $resultado_consulta = array();
    private $tipo_habita;
     //constructor que recibe los parámetros del index.php
    public function __construct($user, $habi, $num)
    {
        $this->usuario = $user;
        $this->tipo_habita = $habi;
        
    }
    //aquí se realiza la consulta a la base de datos según los parámetros
    public function consultabd()
    {
        $consulta = "SELECT usuario, " . $this->tipo_habita . " FROM usuarios, habitaciones WHERE (usuario='" . $this->usuario . "' AND " . $this->tipo_habita . " ='" . $this->tipo_habita . "')";
        $result_conexion = Conectar::conexion($consulta);
        while ($fila = mysql_fetch_array($result_conexion))
        {
            $this->resultado_consulta[] = $fila[$this->tipo_habita]; //asigno a la variable //resultado_consulta el resultado de la consulta
        }
        return $this->resultado_consulta;
    }
 
}
 

gracias! :)

[Sevillacode]

Hola
creo que el problema lo tienes en la consulta sql, si la analizas es un poco rara no crees?
Supongamos que el sesion id_usuario es el 60, y el post de tipo de habitacion 'simple'. estas diciendo
"SELECT usuario, simple FROM usuarios, habitaciones WHERE (usuario='60' AND simple ='simple');

Creo en primer lugar que deberias usar un apodo a la tabla, y debes decirla a que tabla pertenece cada campo

select u.usuario,h.". $this->tipo_habita ." from usuarios u, habitaciones h WHERE ....

Por otro lado, no entiendo el cruce de tablas, que es lo que quieres obtener? que relacion tienen los usuarios con las habitaciones? La relacion deberia ser por id.

Sea como sea te aconsejo filtrar el contenido del post enviado como norma antes de usarlo para prevenir agujeros de seguridad, ya que es muy facil que alguien use el firebug y le cambie el valor al option enviado, y te puede armar una buena.

un saludo!

[guardarmicorreo]

Iniciado por Sevillacode

Hola
creo que el problema lo tienes en la consulta sql, si la analizas es un poco rara no crees?
Supongamos que el sesion id_usuario es el 60, y el post de tipo de habitacion 'simple'. estas diciendo
"SELECT usuario, simple FROM usuarios, habitaciones WHERE (usuario='60' AND simple ='simple');

Esto lo entendí. tienes razón, con solo esto es suficiente

Código PHP:

$consulta = "SELECT usuario, " . $this->tipo_habita . " FROM usuarios, habitaciones WHERE (usuario='" . $this->usuario . "')"; //he borrado el $tipo_habita
 

Creo en primer lugar que deberias usar un apodo a la tabla, y debes decirla a que tabla pertenece cada campo

select u.usuario,h.". $this->tipo_habita ." from usuarios u, habitaciones h WHERE ....

Esto no lo entendí :S

Por otro lado, no entiendo el cruce de tablas, que es lo que quieres obtener? que relacion tienen los usuarios con las habitaciones? La relacion deberia ser por id.

la relación es que cada usuario tiene un número de habitaciones en cada tipo de habitación.

por ejemplo

usuario= q

simple=200|doble=300|triple=400|cuadruple=500

usuario es un campo de la tabla usuarios.
simple, doble, triple, cuadruple son campos de la tabla habitaciones.

cuando hago una consulta un usuario determinado solo puede consultar el número disponible de sus habitaciones de cada habitación.

Sea como sea te aconsejo filtrar el contenido del post enviado como norma antes de usarlo para prevenir agujeros de seguridad, ya que es muy facil que alguien use el firebug y le cambie el valor al option enviado, y te puede armar una buena.

un saludo!

no tengo problema porque no es para algo real, está en local y es solo para aprendizaje :D

gracias por tu ayuda, la verdad es que me has ayudado con algo que se me pasó por alto, pero el problema persiste.

igual que antes me da este error:

Notice: Array to string conversion in C:\xampp\htdocs\xampp\web\trabajo\funciones.php on line 74
Array

[andresdzphp]

Te falta imprimir el índice en el for:

Código PHP:

for($i = 0, $size = count($plazas); $i < $size; $i++) {
    echo $plazas[$i];
}
 

mejor usas un foreach:

Código PHP:

foreach ($plazas as $plaza) {
   echo $plaza;
}
 

[guardarmicorreo]

Iniciado por andresdzphp

Te falta imprimir el índice en el for:

Código PHP:

for($i = 0, $size = count($plazas); $i < $size; $i++) {
    echo $plazas[$i];
}
 

mejor usas un foreach:

Código PHP:

foreach ($plazas as $plaza) {
   echo $plaza;
}
 

gracias!

me devuelve el mismo error con for y con foreach.

si necesitais absolutamente todo el código solo dilo y te lo pongo. no comprendo por qué de este error :S

[andresdzphp]

Antes del bucle coloca:

Código PHP:

echo '<pre>';
print_r($plazas);
echo '</pre>';
 

y copia aquí el contenido. Así sabremos que estructura tiene el array y como debes recorrerlo.

Saludos.

[guardarmicorreo]

Iniciado por andresdzphp

Antes del bucle coloca:

Código PHP:

echo '<pre>';
print_r($plazas);
echo '</pre>';
 

y copia aquí el contenido. Así sabremos que estructura tiene el array y como debes recorrerlo.

Saludos.

me devuelve esto

Código PHP:

Array
(
    [0] => 100
)
 

[Sevillacode]

andresdzphp tiene razon has olvidado el indice y por lo tanto haces referencia al array que contiene todos, el mismo que cuentas en el for.

De todas formas si lo que quieres es averiguar el nº correspondiente de habitaciones con respecto a un usuario, debes cruzar las tablas en la consulta, normalmente mediante un id (en este caso debe haber un id usuario en cada registro de habitaciones).

$consulta = "SELECT DISTINCT u.usuario, h." . $this->tipo_habita . " as total FROM usuarios u, habitaciones h WHERE u.usuario='" . $this->usuario . "' AND u.id_usuario = h.id_usuario";

Esto obtendra un campo "usuario" y uno "total" que tiene el valor del tipo de habitacion obtenido segun el post. Como ves se relaciona por el id_usuario, campo que he supuesto, deberas poner el que tu tengas en tu codigo. El distinct limpia el resultado de la consulta de elementos coincidentes.
Te recomiendo ejecutes las sentencias en el phpmyadmin y veas los distintos resultados.

O eso o no te he entendido bien...

[guardarmicorreo]

Iniciado por Sevillacode

andresdzphp tiene razon has olvidado el indice y por lo tanto haces referencia al array que contiene todos, el mismo que cuentas en el for.

De todas formas si lo que quieres es averiguar el nº correspondiente de habitaciones con respecto a un usuario, debes cruzar las tablas en la consulta, normalmente mediante un id (en este caso debe haber un id usuario en cada registro de habitaciones).

$consulta = "SELECT DISTINCT u.usuario, h." . $this->tipo_habita . " as total FROM usuarios u, habitaciones h WHERE u.usuario='" . $this->usuario . "' AND u.id_usuario = h.id_usuario";

Esto obtendra un campo "usuario" y uno "total" que tiene el valor del tipo de habitacion obtenido segun el post. Como ves se relaciona por el id_usuario, campo que he supuesto, deberas poner el que tu tengas en tu codigo. El distinct limpia el resultado de la consulta de elementos coincidentes.
Te recomiendo ejecutes las sentencias en el phpmyadmin y veas los distintos resultados.

O eso o no te he entendido bien...

tienes razón, soy un desastre lo siento, por querer correr no relacioné las tablas en la consulta, sí las tenía relacionadas en la propia base de datos.

la consulta ha quedado así

Código PHP:

$consulta = "SELECT usuario, " . $this->tipo_habita . " FROM usuarios, habitaciones WHERE (usuario='" . $this->usuario . "' AND usuarios.id=id_usuarios)";
 
//usuarios.id está en la tabla usuarios como campo id
//id_usuarios está en la tabla habitaciones como campo id_usuarios
//tanto clave primaria como foránea están ya relacionadas con el diseñador de //phpmyadmin
 

pero me sigue devolviendo el mismo error :S

perdón por todo este jaleo, pero llevo dos días estancado en esto y no logro resolverlo

[Sevillacode]

prueba a poner la consulta que te especifico. He adaptado los campos que inventé por lo que debe ser completamente valida

$consulta = "SELECT DISTINCT u.usuario, h." . $this->tipo_habita . " as total FROM usuarios u, habitaciones h WHERE u.usuario='" . $this->usuario . "' AND u.id = h.id_usuarios";

El problema que sigo viendo es que no especificas en que tabla esta cada campo, en mi consulta si te fijas estan todos especificados, solo que en lugar del nombre completo de la tabla (usuarios) le he asignado un alias por comodidad (u)
u.id es lo mismo que usuarios.id, h.id_usuarios es lo mismo que habitaciones.id_usuarios

si ejecutas la sentencia que te he puesto, y los campos coinciden con lo especificado, al montar un array asociativo debes obtener algo como esto
array{
['usuario'] = 'xxxxx'; (el usuario)
['total'] = xxxx; (la cantidad correspondiente a esa habitacion y ese usuario)
}

suerte

[guardarmicorreo]

Iniciado por Sevillacode

prueba a poner la consulta que te especifico. He adaptado los campos que inventé por lo que debe ser completamente valida

$consulta = "SELECT DISTINCT u.usuario, h." . $this->tipo_habita . " as total FROM usuarios u, habitaciones h WHERE u.usuario='" . $this->usuario . "' AND u.id = h.id_usuarios";

tu consulta sí me funciona, y el print_r me devolvía bien el array.

El problema que sigo viendo es que no especificas en que tabla esta cada campo, en mi consulta si te fijas estan todos especificados, solo que en lugar del nombre completo de la tabla (usuarios) le he asignado un alias por comodidad (u)
u.id es lo mismo que usuarios.id, h.id_usuarios es lo mismo que habitaciones.id_usuarios

no entiendo bien lo que dices.

cuando hago la consulta le digo que seleccione usuario y $this->tipo_habita de usuarios y habitaciones donde usuario es $this->usuario y relaciono las tablas usuarios.id=id_usuarios

la única diferencia con la consulta que me sugieres es que tú utilizas alias, nunca he utilizado alias y no se si es por eso mi problema

si ejecutas la sentencia que te he puesto, y los campos coinciden con lo especificado, al montar un array asociativo debes obtener algo como esto
array{
['usuario'] = 'xxxxx'; (el usuario)
['total'] = xxxx; (la cantidad correspondiente a esa habitacion y ese usuario)
}

suerte

efectivamente me ha devuelto el array correctamente. el problema estaba en otra parte. gracias a ti lo encontré.

en clases.php

Código PHP:

while ($fila = mysql_fetch_array($result_conexion))
        {
            $this->resultado_consulta[] = $fila; //aquí estaba el problema
        }
        return $this->resultado_consulta;
 
//estaba diciendole que solo igualara  resultado_consulta a fila[$this->tipo_habita]; claro, el error creo que al intentar recorrerlo como array me devolvía error ¿estoy en lo cierto?
 

en index.php

Código PHP:

for ($i = 0, $size = count($plazas); $i < $size; $i++)
        {
            echo $plazas[$i][$_POST['habitaciones']]; //aquí he recorrido el array con un id=$i y también he seleccionado del array el valor del tipo de habitación; me ha funcionado
        }
 

no se si así estará bien, porque aunque el código funcione no significa que esté bien escrito y estructurado.

estoy abierto a todo consejo y crítica, cuanto más pueda aprender mejor jeje

mil gracias por tu ayuda de verdad

[andresdzphp]

Yo te recomendaría que revises tu método de conexión, no está bien eso de conectarse por cada query que hagas y es mejor que uses mysqli o PDO ya que mysql quedará obsoleta desde PHP 5.5.0

[guardarmicorreo]

Iniciado por andresdzphp

Yo te recomendaría que revises tu método de conexión, no está bien eso de conectarse por cada query que hagas y es mejor que uses mysqli o PDO ya que mysql quedará obsoleta desde PHP 5.5.0

no sé a qué te refieres cuando dices que no está bien que conecte por cada query.

¿no tengo que conectar a la base de datos cada vez que haga una acción que lo requiera, es eso?

revisaré mysqli y PDO :P

si quieres te escribo por privado y continuamos esta conversación para no engordar el hilo. gracias de nuevo! :D

[andresdzphp]

Cada query que hagas estás conectando a mysql:

Código PHP:

class Conectar
{
 
    public static function conexion($TextoSQL)
    {
        $IDConexion = mysql_connect("localhost", "root", "");
 
        mysql_select_db("hoteles", $IDConexion);
 
        $IDConsulta = mysql_query($TextoSQL, $IDConexion) or die(mysql_error());
 
        if ($IDConsulta == false)
        {
            echo "ERROR: <br>\n";
            echo mysql_error($IDConexion);
        }
 
        return $IDConsulta;
    }
 
}
 

[Sevillacode]

andresdzphp te dice y coincido, que realices el constructor de conexion por un lado y una funcion query por otro, asi se crea una instancia de la conexion y se usa para todas las consultas.
Lo que tu tienes conecta siempre otra vez por cada consulta que haces.

y lo que no llegas a entender es precisamente lo que estas diciendome, en las consultas donde hay mas de una tabla, es necesario (si no conveniente, yo siempre lo hago) especificar a que tabla pertenece cada campo que vas a usar, creo que es la manera correcta.
Si mi consulta te funciona y la tuya no, tiene que ser por eso...

saludos

[guardarmicorreo]

si bien he entendido, lo que me sugerís es que realice un query genérico en forma de clase pasándole los valores al constructor y así no repetir querys no?

[Sevillacode]

exacto, una funcion constructora de la conexion y otra query que realiza la consulta