[SOLUCIONADO] Obteniendo un valor de una funcion en una llamada

[Hyemin]

Buentas tardes, estoy trabajando con una aplicacion que genera un listado cuyo contenido muestra el resultado de una consulta MySQL sobre una base de datos, uno de los campos es la fecha de vencimiento de un certificado.

Mi intencion era que cuando el certificado estuvioera por vencer el color del texto fuera rojo y si no verde.

Para eso estoy usando este codigo que funciona bien:

Código PHP:

  while ($result_ar = mysqli_fetch_assoc($resultado)) {

date_default_timezone_set('America/Mexico_City');   
$hoy = new datetime();       
$vencimiento = new DateTime($result_ar['vencimiento']);
$total = $hoy->diff($vencimiento)->format('%a');

if ($total<90) {
    $color = red;
} else {
    $color = green;

}
        
   
         echo "<td> <a  style='color:$color' href='detalles.php?id=".$result_ar['id']."'>".$result_ar['vencimiento']."</a></td>";
         echo "</td></tr>\n";
     }
      echo "</table>";
    
     $conexion->close(); 
   

El codigo precedido me genera correctamente la tabla con el color, la variable $color toma el que coresponde.

Ahora, quiero hacer el código mas simple y estoy tratando de separar todo lo posible el diseño de la prograacion asi que desearía dejar la menor cantidad de código en la pagina utilizando entonces funciones.

Para ello hice un fichero llamado funciones,php y lo puse en "require_nce" del header

Código PHP:

  function comparar (){
global $color;            
date_default_timezone_set('America/Mexico_City');   
$hoy = new datetime();       
$vencimiento = new DateTime($result_ar['vencimiento']);
$total = $hoy->diff($vencimiento)->format('%a');

if ($total<90) {
    $color = red;
} else {
    $color = green;

}
return  $color;
} 
   

Luego modifico el bucle while en la pagina para que quede solo la llamada a la funcion

Código PHP:

  
while ($result_ar = mysqli_fetch_assoc($resultado)) {
call_user_func($comparar);
        
         
 echo "<td> <a  style='color:$color' href='detalles.php?id=".$result_ar['id']."'>".$result_ar['vencimiento']."</a></td>";
echo "</td></tr>\n";
     }
      echo "</table>";
    
     $conexion->close();
     
        ?> 
  

El tema es que la funcion no esta dando resultados, o sea no logro obtener de regreso el valor $color

[carlos_belisario]

usar variables globales no es lo más correcto es mejor que retornes el valor que necesitas y lo asignes a la variale

Código PHP:

function comparar()
{
    // toda tu logica 
    if ($total<90) {
        $color = red;
    } else {
        $color = green;
    }
    return $color;
}
$color = comparar();
 

[Hyemin]

HIce estas dos cosas, una funciona y la otra no, no entiendo la diferencia

prueba.php (Funciona perfecto y mestra el contenido de $result_ar en rojo)

Código PHP:

  <?php
include_once 'funciones.php';

$color = comparar();
$result_ar = texto;

echo "<td> <a  style='color:$color' href='detalles.php?id=".$result_ar['id']."'>".$result_ar."</a></td>";   

index.php (no funciona e incluso tranca la pagina sin error pero sin mostrar ada, queda en blanco la tabla)

Código PHP:

  while ($result_ar = mysqli_fetch_assoc($resultado)) {
       
         $color = comparar();
         echo "<tr class=\$class\">";
         echo "<td>".$result_ar['nombre']."</td>";
         echo "<td> <a  style='color:$color' href='detalles.php?id=".$result_ar['id']."'>".$result_ar['vencimiento']."</a></td>";
         echo "</td></tr>\n";
     }
      echo "</table>";
    
     $conexion->close();
     
        ?> 
  

[carlos_belisario]

no tienes el error_reporting activado, activalo en el php.ini o has lo siguiente al principio del archivo

Código PHP:

error_reporting(E_ALL);
ini_set('display_errors', 1);
 

lo más seguro es que te de un error de undefined index por $result_ar no esta dentro del scope de la función, creo que tu función debería de esta así

Código PHP:

function comparar ($vence) 
{
    date_default_timezone_set('America/Mexico_City');   
    $hoy = new datetime();       
    $vencimiento = new DateTime($vence);
    $total = $hoy->diff($vencimiento)->format('%a');
    if ($total<90) {
        $color = red;
    } else {
        $color = green;
   }
}
 

y lo usas

Código PHP:

$color = comparar($result_ar['vencimiento']);
 

prueba y nos comentas

[Hyemin]

Lo probe y casi funciona :)

Funciones.php si lo hago de esta forma por cada registro devuelto me da el siguente error

Código:

Notice: Use of undefined constant green - assumed 'green' in /var/www/funciones.php on line 12

Código PHP:

  <?php

    function comparar ($vence)
    {
        date_default_timezone_set('America/Mexico_City');  
        $hoy = new datetime();      
        $vencimiento = new DateTime($vence);
        $total = $hoy->diff($vencimiento)->format('%a');
        if ($total<90) {
            $color = red;
        } else {
            $color = green;
       }
    }   

Si cambio $color=green por $color="green" o 'green', no da el error pero tampoco funciona (al menos en ambos casos las tablas se muestran)

[Alexis88]

Tienes que asignar el nombre del color entre comillas pues es una cadena y no una constante, además, te falta retornar el valor de $color al final de la función (después de la condición).

Código PHP:

return $color;
 

Saludos

[Hyemin]

Iniciado por Alexis88

Tienes que asignar el nombre del color entre comillas pues es una cadena y no una constante, además, te falta retornar el valor de $color al final de la función (después de la condición).

Código PHP:

return $color;
 

Saludos

Excelente!!! muchas gracias a ambos por el aporte, en este thread aprendi muchisimo mas que en algunos capitulos de los varios libros de php para principiantes que he estado leyendo.

Gracias por el tiempo que le dedicaron

[pateketrueke]

Hyemin, te sugiero leer el siguiente manual: http://www.php.net/manual/es/functions.user-defined.php