Ir para conteúdo

POWERED BY:

Marcones Borges

Erro ao migrar código de php 5.6 para pdo

Recommended Posts

Olá, sou iniciante em php, estou tentando migrar um código de php 5.6 para pdo mais da erro, não roda....

Alguém pode me ajudar?


include '../Connections/functions.php';
         
function montaSelect()
        { 
$sql = "SELECT * FROM membros";
$result = $con->query($sql);

if ($result->num_rows > 0 ) 
{

  while($dados = $result->fetch_assoc())
  {
  $opt .= '<option value="'.$dados['id'].'">'.$dados['nome'].'</option>';
  }
} else {
          $opt = '<option value="0">Nenhum Membro cadastrado</option>';

    return $opt;
}
        }

 

Este era o código original:

 



// require 'topof.php';
// echo $usuario;
        /**
         * função que retorna o select
         */
        function montaSelect()
        {
                $sql = "SELECT id, nome FROM membros '";
                $query = mysql_query( $sql );

                if( mysql_num_rows( $query ) > 0 )
                {
                        while( $dados = mysql_fetch_assoc( $query ) )
                        {
                                $opt .= '<option value="'.$dados['id'].'">'.$dados['nome'].'</option>';
                        }
                }
                else
                        $opt = '<option value="0">Nenhum Membro cadastrado</option>';

                return $opt;
        }

Compartilhar este post


Link para o post
Compartilhar em outros sites

Olá @Marcones Borges.

 

7 horas atrás, Marcones Borges disse:

while($dados = $result->fetch_assoc())

 

Essa linha você precisa trocar para:

while( $dados = $result->fetch(PDO::FETCH_ASSOC) )

 

Já para a contagem de registros, geralmente iriam te recomendar usar fetchAll e contar os registros usando PHP com count() ou sizeof(), desse jeito:

$rows = $stmt->fetchAll();
$num_rows = count($rows);

Outra opção (talvez melhor) seria fazer uma consulta ao banco apenas para recuperar a quantidade de itens:

$numRows = $conn->query("SELECT COUNT(*) FROM tabela")->fetchColumn();
echo $numRows;

 

Mas eu lhe dou uma terceira opção, que considero melhor:

1 - Antes do loop registramos a quantidade de itens já processados, no caso 0:

$qtd = 0;

2 - Em cada iteração do loop, incrementamos 1 a essa variável:

// dentro do loop:
++$qtd;

Depois, verificamos se $qtd ainda é 0, pois isso significaria que o conjunto obtido pela consulta era vazio:

if ( !$qtd ) {
    echo 'Nenhum registro encontrado.';
}

 

Outro problema é que antes a conexão ao mysql era comumente utilizada de forma global como no seu código antigo, onde você chamava a função mysql_query.

Perceba que com o PDO você precisa da variável $con, que você criou e que contém a conexão com o banco.

Então precisamos passar a variável de conexão para dentro da função, uma vez que funções possuem um contexto fechado e desconhecem variáveis externas.

 

Um exemplo disso seria inserindo o parâmetro na declaração da função. Seria assim:

function montaSelect( $con ) {
   ...
}

// Agora chamamos:
montaSelect( $con );

Mas observe que sempre que quisermos chamar a função montaSelect agora devemos enviar a variável pra dentro da função. Meio chato, né?

Nesse caso podemos utilizar funções anônimas e explicitar o conhecimento global da conexão ao PDO. Ficaria assim:

$montaSelect = function () use ( $con ) {
   ...
}

// Agora chamamos:
$montaSelect();

 

Isso tudo desconsiderando orientação a objetos, que tornaria tudo mais profissional. Mas para um uso comum, rápido e prático deve servir :)

 

Reunindo tudo, ficaria assim:

$montaSelect = function () use ( $con ) {
    $sql = "SELECT * FROM membros";
    $result = $con->query($sql);
    
    $qtd = 0;
    $opt = '';
    
    while( $dados = $result->fetch( \PDO::FETCH_ASSOC ) ) {
        ++$qtd;
        $opt .= '<option value="'.$dados['id'].'">'.$dados['nome'].'</option>';
    }
    
    if ( !$qtd ) {
        $opt = '<option value="0">Nenhum Membro cadastrado</option>';
    }
    
    return $opt;
};

echo $montaSelect();

 

Compartilhar este post


Link para o post
Compartilhar em outros sites

Top, funcionou perfeitamente, Muito obrigado, tentei aplicar a mesma regra para segunda parte do código mais esta dando erro...

 

Código anterior


        function retorna( $igreja_nome=null, $id=null )
        {
                $id = (int)$id;
                $sql = "SELECT * FROM membros ";
                if( $id != null && $igreja_nome != null)
                        $sql .= "WHERE `id` = {$id} and `igreja` = '{$igreja_nome}';";
                $query = mysql_query( $sql );

                $json = '[';  
                if( mysql_num_rows( $query ) > 0 )
                {
                    $dados = mysql_fetch_assoc( $query );
                                        $json .= '{"celular":';
                                        $json .= '"';
                                        $json .= $dados["celular"];
                                        $json .= '"}';
                                }
                else
                    $json .= '{"celular": "não encontrado"}';

                $json .= ']';

                return $json;
        }

switch($_SERVER['REQUEST_METHOD']){
        case 'GET': $the_request = $_GET;
        echo retorna();
        case 'POST': $the_request = $_POST;
        echo retorna($the_request['igreja'], $the_request['id']);
}

 

Alterei para este


$retorna = function () use ($conexao)
        {
                $id = (int)$id;
                $sql = "SELECT * FROM membros ";
                if( $id != null)
                        $sql .= "WHERE `id` = {$id};";
                $result = $conexao->query($sql);

                $json = '[';  
                if ( !$qtd )
                {
                    $dados = $result->fetch(PDO::FETCH_ASSOC);
                                        $json .= '{"celular":';
                                        $json .= '"';
                                        $json .= $dados["celular"];
                                        $json .= '"}';
                                }
                else
                    $json .= '{"celular": "não encontrado"}';

                $json .= ']';

                return $json;
        }

/* só se for enviado o parâmetro, que devolve o combo */
switch ($_SERVER["REQUEST_METHOD"]){
        case 'GET': $the_request = $_GET;
        echo $retorna();
        case 'POST': $the_request = $_POST;
        echo $retorna($the_request['igreja'], $the_request['id']);
}

 

Usei a mesma lógica mais esta dando erro.

Compartilhar este post


Link para o post
Compartilhar em outros sites

Então @Marcones Borges, o primeiro ponto é que você esqueceu dos parâmetros da função nessa nova versão:

$retorna = function ( $igreja_nome=null, $id=null ) use ( $conexao )

O segundo ponto é que nesse exemplo você teria a possibilidade de fazer um loop OU retornar um único registro se informar o $id na função, e por conta dessa flexibilidade o uso do conceito de $qtd fica um pouco mais complicado.

 

Diante do modelo de função que você postou, eu iria sugerir usar o fetchAll no lugar, como expliquei no meu post anterior. O fetchAll não lida bem quando você precisa recuperar MUITOS registros (pois ele armazena todos em memória ao invés de utilizar ponteiros para nevegar de um para o outro como nos outros casos), mas de todos os métodos ele é o mais prático.

 

E por fim, não há necessidade de você formatar a string para JSON na mão, pois o PHP faz isso para você.

 

Não pretendia fazer a função completa novamente, pois acredito que isso atrapalha o processo de aprendizado, mas são poucas linhas de código, então vamos lá:

$retorna = function ( $igreja_nome = NULL, $id = NULL ) use ( $conexao ) {
    $id = (int)$id;
    
    // Como é só o celular que você precisa, limitamos a consulta
    // Para exibir o "Não encontrado", fiz um pequeno IF no select
    // Esse IF poderia ser feito via PHP, mas aqui parece ser mais fácil
    $sql = 'SELECT IF( `celular`, `celular`, "Nao encontrado" ) `celular` FROM membros';
    
    // Não precisa do != null. $id é um inteiro e quando aplicado em um IF, 0=false, resto=true
    if ( $id )
        $sql .= " WHERE `id` = {$id};";
    
    $result = $conexao->query($sql);
    
    return json_encode( $result->fetchAll( PDO::FETCH_ASSOC ) );
    
}; // Não esqueça: Ao contrário de uma function normal,
   // nessa aqui você PRECISA colocar o ponto-e-vírgula
   // Isso pq essa está associada a uma variável,
   // e no final de todas as vars vem o ponto-e-vírgula.

 

Outro detalhe é que $igreja_nome não está sendo utilizada na função.

Alguns ajustes ainda podem ser feitos, mas a base da função seria essa aí.

Compartilhar este post


Link para o post
Compartilhar em outros sites

Crie uma conta ou entre para comentar

Você precisar ser um membro para fazer um comentário

Criar uma conta

Crie uma nova conta em nossa comunidade. É fácil!

Crie uma nova conta

Entrar

Já tem uma conta? Faça o login.

Entrar Agora

  • Conteúdo Similar

    • Por Carcleo
      Tenho uma abela de usuarios e uma tabela de administradores e clientes.
      Gostaria de uma ajuda para implementar um cadastro
       
      users -> name, login, passord (pronta) admins -> user_id, registratiom, etc.. client -> user_id, registratiom, etc...
      Queria ajuda para extender de user as classes Admin e Client
      Olhem como estáAdmin
      <?php namespace App\Models; use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Model; class Admin extends User {     use HasFactory;            protected $fillable = [         'name',         'email',         'password',         'registration'     ];      private string $registration;     public function create(         string $name,          string $email,          string $password,         string $registration     )     {         //parent::create(['name'=>$name, 'email'=>$email, 'password'=>$password]);         parent::$name = $name;         parent::$email = $email;         parent::$password = $password;         $this->registration = $registration;     } } User
      <?php namespace App\Models; // use Illuminate\Contracts\Auth\MustVerifyEmail; use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Foundation\Auth\User as Authenticatable; use Illuminate\Notifications\Notifiable; use Illuminate\Database\Eloquent\Relations\BelongsToMany; class User extends Authenticatable {     /** @use HasFactory<\Database\Factories\UserFactory> */     use HasFactory, Notifiable;     static string $name;     static string $email;     static string $password;     /**      * The attributes that are mass assignable.      *      * @var list<string>      */     protected $fillable = [         'name',         'email',         'password',     ];          /**      * The attributes that should be hidden for serialization.      *      * @var list<string>      */     protected $hidden = [         'remember_token',     ];     /**      * Get the attributes that should be cast.      *      * @return array<string, string>      */     protected function casts(): array     {         return [             'email_verified_at' => 'datetime',             'password' => 'hashed',         ];     }          public function roles() : BelongsToMany {         return $this->belongsToMany(Role::class);     }       public function hasHole(Array $roleName): bool     {                 foreach ($this->roles as $role) {             if ($role->name === $roleName) {                 return true;             }         }         return false;     }         public function hasHoles(Array $rolesName): bool     {                 foreach ($this->roles as $role) {             foreach ($rolesName as $rolee) {             if ($role->name === $rolee) {                 return true;             }          }         }         return false;     }         public function hasAbility(string $ability): bool     {         foreach ($this->roles as $role) {             if ($role->abilities->contains('name', $ability)) {                 return true;             }         }         return false;     }     } Como gravar um Admin na tabela admins sendo que ele é um User por extensão?
      Tentei assim mas é claro que está errado...
      public function store(Request $request, Admin $adminModel) {         $dados = $request->validate([             "name" => "required",             "email" => "required|email",             "password" => "required",             "registration" => "required"         ]);         $dados["password"] =  Hash::make($dados["password"]);                  $admin = Admin::where("registration",  $dados["registration"])->first();                  if ($admin)              return                    redirect()->route("admin.new")                             ->withErrors([                                 'fail' => 'Administrador já cadastrados<br>, favor verificar!'                   ]);                            $newAdmin = $adminModel->create(                                    $dados['name'],                                    $dados['email'],                                    $dados['password'],                                    $dados['registration']                                 );         dd($newAdmin);         $adminModel->save();         //$adminModel::create($admin);                  return redirect()->route("admin.new")->with("success",'Cadastrado com sucesso');     }  
    • Por violin101
      Caros amigos, saudações.
       
      Gostaria de tirar uma dúvida com os amigos, referente a PDV.
       
      Estou escrevendo um Sistema com Ponto de Vendas, a minha dúvida é o seguinte, referente ao procedimento mais correto.

      Conforme o caixa vai efetuando a venda, o Sistema de PDV já realiza:
      a baixa direto dos produtos no estoque
      ou
      somente após concretizar a venda o sistema baixa os produtos do estoque ?
       
      Grato,
       
      Cesar
       
    • Por violin101
      Caros amigos do grupo, saudações e um feliz 2025.
       
      Estou com uma pequena dúvida referente a Teclas de Atalho.

      Quando o Caps Lock está ativado o Comando da Tecla de Atalho não funciona.
      ou seja:
      se estiver para letra minúscula ====> funciona
      se estiver para letra maiúscula ====> não funciona
       
      Como consigo evitar essa falha, tanto para Letra Maiúscula quanto Minúscula ?

      o Código está assim:
      document.addEventListener( 'keydown', evt => { if (!evt.ctrlKey || evt.key !== 'r' ) return;// Não é Ctrl+r, portanto interrompemos o script evt.preventDefault(); });  
      Grato,
       
      Cesar
    • Por ILR master
      Fala galera, tudo certo?
       
      Seguinte: No servidor A estou tentando fazer uma consulta com o servidor B, mas está dando erro.
      Estou usando o mesmo código de conexão do servidor B que funciona perfeitamente, mas no servidor A, dá erro.
      Segue código:
       
      $host = 'servidor B';
      $user = 'user';
      $pass = '********';
      $db   = 'banco';
       
      // conexão e seleção do banco de dados
      $conexao = mysqlI_connect($host, $user, $pass, $db);
      mysqlI_set_charset($conexao,"utf8");
      //print "Conexão rodando e OK!"; 
      //mysqlI_close($conexao);
       
      Alguém pode me ajudar?
    • Por violin101
      Caros amigos, saudações.
       
      Por favor, poderiam me ajudar.

      Estou com a seguinte dúvida:
      --> como faço para para implementar o input código do produto, para quando o usuário digitar o ID o sistema espera de 1s a 2s, sem ter que pressionar a tecla ENTER.

      exemplo:
      código   ----   descrição
           1       -----   produto_A
       
      Grato,
       
      Cesar
×

Informação importante

Ao usar o fórum, você concorda com nossos Termos e condições.