No funciona una consulta mysql dentro de una funcion php

[rutame]

Esta funcion no me funciona En realidad no sale nada de nada... alguien me puede aclarar el por qué? muchas gracias a todos.

Código PHP:

  function ofertas()
  {
    $sql = mysql_query ("SELECT * FROM productos, fotos 
                          WHERE 
                          estado ='oferta' 
                          and 
                          productos.referencia = fotos.referencia"); 
                          
     
      return $sql;
      
          while($res = mysql_fetch_array($sql))
            {        
                  echo "Hay esto";
               if($ofertas > 0) 
                {
                  echo "Hay ".$ofertas." ofertas";
                  echo "<img src=".$res['nombrefoto']."/>";
                  echo "Marca: ".$res['marca'];
                } 
                  else
                {
                  echo "No hay ofertas";
                }
             }
          return $res;   
  } 
   

[PIRRUMAN]

prueba si tu consulta se peude ejecutar en tu base de datos

[gldelossantos]

te falta comillas en esta linea

and
productos.referencia = 'fotos.referencia'");

y otra cosa amigo de donde estas tomando la variable oferta y fotos.referencia

[chwc]

ese return $sql; esta de mas, ademas, el return $res de el final no se para que te sirve, $res creo que te devolveria la ultima linea obtenida, creo que no sabes para que sirve return, no es para eliminar variables, las variables se eliminan solas despues de ejecutar una función, es para devolver una respuesta de la variable, por ejemplo

Código PHP:

function WeLoveFood(){
return " amo las papas fritas :D";
}
echo "yo".WeLoveFood();
 

saludos

[rutame]

Iniciado por PIRRUMAN

prueba si tu consulta se peude ejecutar en tu base de datos

Sí, se ejecuta desde mysql y devuelve resultados.

[rutame]

Iniciado por chwc

ese return $sql; esta de mas, ademas, el return $res de el final no se para que te sirve, $res creo que te devolveria la ultima linea obtenida, creo que no sabes para que sirve return, no es para eliminar variables, las variables se eliminan solas despues de ejecutar una función, es para devolver una respuesta de la variable, por ejemplo

Código PHP:

function WeLoveFood(){
return " amo las papas fritas :D";
}
echo "yo".WeLoveFood();
 

saludos

Muchas gracias. Es verdad, no se muy bien para que sirve return. El segundo: return $res, lo puse por desesperación a ver si salía algo.
Si no pongo el primer return, me da un error.

[rutame]

Y poniendo el ejemplo de una consulta mysql dentro de la función, como sería?? Ahora lo he dejado así:

Código PHP:

function ofertas()
  {
    $sql = mysql_query ("SELECT * FROM productos,fotos
                          WHERE 
                          productos.estado ='oferta' 
                     and productos.referencia = 'fotos.referencia'"); 
                          
     
      //return $sql;
      
          while($res = mysql_fetch_array($sql))
            {        
                  echo "Hay esto";
               if($ofertas > 0) 
                {
                  echo "Hay ".$ofertas." ofertas";
                  echo "<img src=".$res['nombrefoto']."/>";
                  echo "Marca: ".$res['marca'];
                } 
                  else
                {
                  echo "No hay ofertas";
                }
             }
             
  }
ofertas();
 

[rutame]

Hay una tabla con productos, que tiene un campo llamado referencia y hay otra tabla llamada fotos que tiene otro campo llamado referencia y son comunes. Así puedo hacer una consulta y que me extraiga datos de los dos campos de forma asociada.

[gldelossantos]

Iniciado por rutame

Hay una tabla con productos, que tiene un campo llamado referencia y hay otra tabla llamada fotos que tiene otro campo llamado referencia y son comunes. Así puedo hacer una consulta y que me extraiga datos de los dos campos de forma asociada.

Bno espero te sirva la solucion que te ofreci....

[gjx2]

Hola @rutame, te dejo aqui mi aporte para lo que buscas hacer.
hice uso de la funcion mysql_num_rows tienes que estudiarla un poco.

Código PHP:

function ofertas()
{ 
     # Aqui pon tu consulta, el or die quiere decir que si pasa un error durante la consulta entonces finalice el código y muestre el error
    $sql = mysql_query()or die(mysql_error()); 
     
     
     # Le cambie el nombre a la variable oferta para evitar confuciones futuras.
     # Con la funcion mysql_num_rows() de terminas la cantidad de ofertas que tienes en tu tabla                     
    $total_ofertas = mysql_num_rows($sql);
        
            # Verificas si tienes algunas ofertas 
          if ($total_ofertas > 0){
       
           
            while ($res = mysql_fetch_array($sql)){        
                
                # Aqui pones el codigo que quieras imprimir. 
                
            }
              
          } else {
               
                  echo "No hay ofertas";
            }    
  }
  
  
ofertas();
 

Saludos.

[rutame]

Muchas gracias GJX2. Ya está funcionando. La verdad es que cuando uno no está centrado le es bastante dificil darse cuenta de errores tonto y tenía varios :)

Un saludo y gracias a todo

[rutame]

La cosa quedó así:

Código PHP:

function ofertas()
  {
  global $ruta_mini;
    $sql = mysql_query ("SELECT * FROM fotos,productos WHERE productos.estado='oferta' AND productos.referencia = fotos.referencia LIMIT 3"); 
      $ofertas = mysql_num_rows($sql);
        if($ofertas > 0) 
                { 
            
            while($res = mysql_fetch_array($sql))
                {  
              
              ?>      
                  <div class="contenido">
                  <?php echo "<h5>".$res['marca']." ".$res['modelo']."</h5>";?>
                  <div class="oferta"><?php echo $res['pvp']."€";?></div>
               <img src="<?php echo $ruta_mini.$res['nombrefoto'];?>">
               </div>
                <?php  
             
                } 
                 
             }  else
                {
                  echo "No hay ofertas";
                }
  }