Best option to be used in error handling?

Question

I’m creating a library in PHP to make the connection to the database and manipulate data from it, but do not know what is the best option to handle the errors, if it is better to use try/catch, echo/return or die.

I am currently using several if/else to go through (validate) the data, in case of error I display a echo with the mistake and I give a return false (not to execute the remaining code), if there is no error, just continue my checks and at the end return true or array (depending on function). See an example of my simplest function (the deletar):

function deletar($tabela, $where = NULL)
{
    if(!function_exists("conectar"))
    {   //falta include de conexao.php
        echo "Não há uma conexão ativa com o seu banco de dados!\n<br><i>Inclua a página ../conexao.php<br>";
        return false;
    }
    else
    {
        //conexao feita
        if($tabela)
            $tabela = "DELETE FROM ".$tabela." ";
        else
        {
            echo "<br>Não foi indicada nenhum tabela.<br>";
            return false;
        }

        $where = minwhere($where);

        echo $sql = $tabela.$where;

        if($conn = conectar())
        {
            if($result = $conn->query($sql))
            {
                $stmt = $conn->prepare( $sql );
                if($result = $stmt->execute())
                        echo "<br>Deletado!<br>";
                else
                    echo "<br>Query inválida!<br>";

                $conn = null;
                return true;
            }
            else
            {
                echo "<br>Query inválida!<br>";
                return false;
            }  
        }
        else
        {
            echo "<br>Não foi possível conectar-se ao banco de dados!<br>\n<i>Verifique as variáveis do arquivo ../conexao.php</i>";
            return false;
        }
    }
};

Follows the rest of the library (if someone wants to analyze other functions).

What can improve error handling? The mode I’m doing is a valid alternative or bad code?

Answer 1

Basically right, it could only be more organized. I could organize even more than I did below, but I would run away from the style being done.

That pile of echo It’s always a shallow way to deal with the mistake. In serious applications, the treatment would be done in a totally different way, never play a text anyway on the page. But for that you would have to restructure the entire application, not just this section. I even understand that most people do so, but it is the "pig" way of doing.

Honestly, because it’s a library, I thought it was a horrible code. It sounds like code from someone who’s starting programming. I wouldn’t follow her to learn.

function deletar($tabela, $where = NULL) {
    if(!function_exists("conectar")) {
        echo "Não há uma conexão ativa com o seu banco de dados!\n<br><i>Inclua a página ../conexao.php<br>";
        return false;
    }
    if(!$tabela) {
        echo "<br>Não foi indicada nenhum tabela.<br>";
        return false;
    }
    $tabela = "DELETE FROM ".$tabela." ";
    $where = minwhere($where);
    echo $sql = $tabela . $where;
    if(!($conn = conectar())) {
        echo "<br>Não foi possível conectar-se ao banco de dados!<br>\n<i>Verifique as variáveis do arquivo ../conexao.php</i>";
        return false;
    }
    if($result = $conn->query($sql)) {
        $stmt = $conn->prepare( $sql );
        echo ($result = $stmt->execute()) ? "<br>Deletado!<br>" : "<br>Query inválida!<br>";
        $conn = null; //isto provavelmente é um erro
        return true;
    }
    echo "<br>Query inválida!<br>";
    return false;
}

I put in the Github for future reference.

I deal with the subject in another question.

Answer 2

Solution using try catch

try {

    if(!function_exists("conectar")) {
        throw new Exception( "Não há uma conexão ativa com o seu banco de dados!\n<br><i>Inclua a página ../conexao.php<br>" );
    }
    if(!$tabela) {
        throw new Exception( "<br>Não foi indicada nenhum tabela.<br>" );
    }

    $tabela = "DELETE FROM ".$tabela." ";
    $where = minwhere($where);
    echo $sql = $tabela . $where

    if(!($conn = conectar())) {
        throw new Exception( "<br>Não foi possível conectar-se ao banco de dados!<br>\n<i>Verifique as variáveis do arquivo ../conexao.php</i>" );
    }

    $result = @$conn->query($sql);

    if(!$result) {
        throw new Exception( "<br>Query inválida!<br>" );
    }

    $stmt = $conn->prepare( $sql );
    echo ($result = $stmt->execute()) ? "<br>Deletado!<br>" : "<br>Query inválida!<br>";
    $conn = null; //isto provavelmente é um erro
    return true;

} catch( Exception $e ) {

    echo $e->getMessage();
    return false;
}