Funcionamiento de funciones

[Dude--]

Código php:

function comprobarlogin($email, $pass)
{
    //conexión a la bd
    //comprobacion de los datos
    if ()//si los datos coinciden crea las variables de sesion
    { 
        $_SESSION["usuario_valido"] = "si";
        return "loginOK";
    }
    else //devuelve error
    {
        return "Tus datos no son correctos";
    }
}
    
if ( $accion == "comprobaridentificacion")
{
    $comprobarlogin = comprobarlogin($email, $pass);
    if ($comprobarlogin != "loginOK")
    {   $errores_identificacion[] = $comprobarlogin;}
    
    foreach ($errores_identificacion as $indice => $valor)
    { echo '<div id="mensages">Error nº'.($indice + 1).': '.$valor.'</div>';}
}
 

No se si es correcta la forma de resolver el problema de que datos devuelve la funcion, si un "ok" o un "error"...

También había pensado esta otra opción para la parte final:

Código php:

$comprobarlogin = comprobarlogin($email, $pass);
if ( !$_SESSION["usuario_valido"] ) 
//No hay variable de sesion que indique que los datos de identificacion son correctos.
{$errores_identificacion[] = $comprobarlogin;}
 

[pacmanaman]

En la linea 7 del primer codigo pone ...

$_SESSION["usuario_valido"] = TRUE;

De la linea 15 a la 23 no va a llegar nunca por que ya con el primer if capturas lo que pasa y siempre va retornar o "LoginOK" o en caso de error "Tus datos no son correctos"

Y en la función tenes la variable $accion, en que momento tiene data, por que no veo que se la pases como argumento a la función.

SaluDOS!

[Dude--]

Perdona, olvide mencionar que la llamada a "comprobaridentificacion" se hace desde un formulario, mediante el cual también se envian $email y $pass:
<form id="identificacion" action="index.php?accion=comprobaridentificacion". ......

De todas formas la consulta es relativa a que la función cuando detecta un error de identificación devuelve mediante return el error, pero cuando los datos son correctos devuelve loginOK en el primer caso o $_SESSION["usuario_valido"] = "si" en el segundo. La cuestión es si esta es la forma más optima de hacerlo.

PD: if ( $accion == "comprobaridentificacion") NO FORMA PARTE DE LA FUNCION, he editado la tabulación del código para que se pueda leer mejor.