Função com PHP - syntax error, unexpected T_STRING in

[lucas martins 0]

Estou iniciando a utilização do php, js, etc, etc para a construção do meu TG, e estou tentando construir uma função no meu arquivo .php(function getCategoria($codigo)), fazendo uma select e atribuindo o resultado à variavel $codigo e no momento de passar para o meu INSERT, está retornando o erro: "syntax error, unexpected T_STRING in".

 

Se puderem me ajudar, por gentileza. Agradeço.

$categoria          = $_POST["categoria"]; guarda o código da categoria


function getCategoria($codigo){
    $sql = "SELECT CAT_DESCRICAO FROM categoria WHERE CAT_ID = ".$categoria."";            
    $qry = mysqli_query($conexao, $sql) or die(mysqli_error($conexao));    
    $codigo = $categoria;
    return $codigo;
}

switch($processo){
     case 'incluir':

        $sql = "INSERT INTO anuncio
        (ANU_DTANUNCIO,
        ANU_TITULO, 
        ANU_DESCRICAO, 
        CAT_ID, 
        CAT_DESCRICAO,
        ANU_VALOR, 
        ANU_CEP, 
        ANU_HORARIO,
        ANU_QTDIMG) 
        VALUES ('".date("Y-m-d")."', '".$titulo."', '".$descricao."', '".$categoria."', '"getCategoria(.$codigo.)"', '".$valor."', '".$cep."', '".$horaFuncionamento."', '".$i."')";                        
        $resultado = mysqli_query($conexao, $sql) or die(mysqli_error());        
        if(!$resultado){
            $resposta["mensagem"] = "Anúncio não enviado com sucesso!";    
        }else{
            $resposta["mensagem"] = "Anúncio enviado com sucesso!";
        }
    
        print json_encode($resposta);                        
    break;

 

[Diego Brocanelli 0]

Olá, Lucas Martins.

 

Com base no código que você enviou, o erro informado encontra-se na seguinte linha:

 

$categoria          = $_POST["categoria"]; guarda o código da categoria

 

Pois há um comentário escrito, porém sem a declaração de comentário de linha, veja resolução abaixo:

$categoria          = $_POST["categoria"]; //guarda o código da categoria

 

Abro espaço para algumas recomendações:

• Utilize a PDO ao invés de mysqli_*.
• Não utilize consultas SQL somadas a sua regra de negócio, isole as responsabilidades.
  • Isso lhe trará um código mais legível, manutenivel e reutilizável.

Espero ter lhe auxiliado.

 

Att,

Diego Brocanelli

[Rodrigo Vieira E da Silva 0]

Olá Diego, o por que da não utilização da Mysqli? Se ela é um drive também de conexão com o banco de dados qual o problema de não utilizá-la?

 

Me dê esse feedback

[tetsuo 0]

Eu gosto muito de mysqli
Mas uso a forma OO
Mas se não não viu, dê uma olhada aqui pra aumentar seu repertório:
https://imasters.com.br/back-end/diferencas-mysqli-versus-pdo-versus-mysql-benchmark-para-comparacao-de-desempenho-seguranca-e-conversor-que-funciona-em-2016-e-2017

[Diego Brocanelli 0]

Olá, Rodrigo Vireia.

 

O uso da PDO é incentivada, pois é uma camada de abstração que fortalece a manipulação da camada do DB. mysqli_+ não é o problema, tendo em vista que tem seu uso no interno da PDO, porém com o uso do mysqli você sai da OO, seu código fica com mescla de estruturado e OO, a camada de abstração com o banco de dados fica enfraquecida, vide a string SQL que foi passada para a manipulação e não ocorreu nenhuma sanitização de dados. Claro que somente a PDO não deixa sua aplicação invulnerável.

 

Lembrando que a aplicação tem que ser manutenivel, escalável e segura. Em minha opinião seria mais sensato o uso de ORMs para isso, porém descartando o uso deles minha escolha sem dúvida seria a PDO pela sua API muito mais amigável, flexível, de fácil compreensão, facilidade na troca do DB e segura.

 

Espero ter lhe auxiliado.

 

Att,

Diego Brocanelli