Empty function return

Question

Could someone tell me why my insert2 function is returning null after insertion? she should be returning the string with the success message.

<?php
require_once 'conexao.php';
$con = new Conexao();

function insert($dados, $tabela, $campo_unico) : string{

    global $con;

    //Verifica se tem algum campo que tenha que ser único no BD
    if(is_null($campo_unico)) { 
        insert2($dados, $tabela);
    }else{ //Caso tenha verifica se já existe algum registro
        $sql = "SELECT * FROM ".$tabela." WHERE ".$campo_unico."=:".$campo_unico;
        $select = $con->conectar()->prepare($sql);

        if(is_numeric($dados[$campo_unico])){
            $select->bindValue(":".$campo_unico, $dados[$campo_unico], PDO::PARAM_INT);
        }else {
             $select->bindValue(":".$campo_unico, $dados[$campo_unico], PDO::PARAM_STR);
        }
        $select->execute();

        if( ($select->rowCount()) > 0) {
            return "O ".$campo_unico." informado já existe em nossos registros";
        }else {
            insert2($dados, $tabela);
        }

    }

}

function insert2($dados, $tabela) : string{
        global $con;

        $i = 0;
        $sql = "INSERT INTO " . $tabela . " (";
        foreach ($dados as $key => $valor) {
             if ($i == count($dados)-1) {
                $sql .= " ".$key." ";
            } else {
                $sql .= " ".$key.", ";
            }
            $i++;
        }
        $sql .= ")";
        $sql .= " values (";

        $i = 0;
        foreach ($dados as $key => $valor) {
             if ($i == count($dados)-1) {
                $sql .= " :".$key." ";
            } else {
                $sql .= " :".$key.", ";
            }
            $i++;
        }
        $sql .= ")";

        $insert = $con->conectar()->prepare($sql);
        foreach ($dados as $key => $valor) {
            if (is_numeric($valor)) {
                $insert->bindValue(":$key", $valor, PDO::PARAM_INT);
            } else {
                $insert->bindValue(":$key", $valor, PDO::PARAM_STR);
            }
        }
        if ($insert->execute()) {
            return "Registro inserido com sucesso";
        } else {
            return $insert->errorInfo();
        }
}

The mistake you’re making is this:

Fatal error: Uncaught Typeerror: Return value of Insert() must be of the type string, None returned in ...:34

Which is precisely why the return of the function is being null Stack trace:

Answer 1

If I understand the code, I believe that there is a lack of a:

if( ($select->rowCount()) > 0) {
    return "O ".$campo_unico." informado já existe em nossos registros";
}else {
    insert2($dados, $tabela);
}

Should be:

if( ($select->rowCount()) > 0) {
    return "O ".$campo_unico." informado já existe em nossos registros";
}else {
    return insert2($dados, $tabela); //Faltava o return
}

Answer 2

In typed languages it is quite common to specify the type of return of a method and it must obey this specification otherwise the compiler issues an error. In PHP7 works the same way or almost.

See that in insert() there is only one Return, there should be at least two. I left only the function skeleton to highlight the problem.

function insert($dados, $tabela, $campo_unico) : string{ //obriga a função retorna o tipo
    global $con;
    if(is_null($campo_unico)) {

    }else{ //Caso tenha verifica se já existe algum registro
        if(is_numeric($dados[$campo_unico])){

        }else {

        }
        $select->execute();

        if( ($select->rowCount()) > 0) {
            return "O ".$campo_unico." informado já existe em nossos registros";
        }else {
            insert2($dados, $tabela);
        }

    }
   //caso o código chegue aqui não existe nenhum return para satisfazer a assinatura do método.
}

By calling insert() in case of failure errorInfo(); returns an array or it will be necessary a treatment to convert it into string, not just give a return insert2($dados, $tabela); That also violates the signature.

insert2() suffers from the same problem see:

function insert2($dados, $tabela) : string{
 //código....

    if ($insert->execute()) {
        return "Registro inserido com sucesso";
    } else {
        return $insert->errorInfo(); //retorna um array o que quebra a assinatura
    }
 }

To solve the first step is to return the expected type, do not return errorInfo() but a

return implode(' - ', $insert->errorInfo());

Related:

PHP 7 has argument typing and return, but is optional. This is good or bad?

Typing the return in PHP 7. What are the advantages?