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;}