Ir para conteúdo

POWERED BY:

Arquivado

Este tópico foi arquivado e está fechado para novas respostas.

reinaldorti

origem

Recommended Posts

Boa tarde, criei esse script  onde eu salvo no banco ou atualizo a origem do usuário dentro do meu site.. mais quando eu navego no site ele cadastra  a minha url também..

Alguém pode me dar alguma dica para eu melhorar esse script? valeu...
O código está funcionando normal só queria dar uma melhorada nele. gostaria de algumas dicas...
 

$origem = filter_input(INPUT_SERVER, 'HTTP_REFERER', FILTER_DEFAULT);
    $read->ExeRead(DB_ORIGEM,"WHERE url = :u", "u={$origem}");   
    if ($read->getResult()):
        $url = ['views' => $read->getResult()[0]['views'] + 1, 'updated' => date('Y-m-d H:i:s')];
        $update->ExeUpdate(DB_ORIGEM, $url, "WHERE id = :id", "id={$read->getResult()[0]['id']}");
    else:
        $create = new Create;
        $url = ['url' => strtolower($origem), 'views' => 1, 'created' => date('Y-m-d H:i:s'), 'status' => 1];
        $create->ExeCreate(DB_ORIGEM, $url);
    endif;    

 

Compartilhar este post


Link para o post
Compartilhar em outros sites

Que tipo de melhoria/dicas você gostaria? Leitura, manutenção, execução, velocidade, complexidade ciclomática, qual biblioteca utilizar?

Compartilhar este post


Link para o post
Compartilhar em outros sites
4 minutos atrás, Gabriel Heming disse:

Que tipo de melhoria/dicas você gostaria? Leitura, manutenção, execução, velocidade, complexidade ciclomática, qual biblioteca utilizar?

Olá, tudo bem?  estou utilizando pdo, mais a melhoria pode ser uma sugestão mesmo...Qualquer coisa que possa ajudar...uma coisa que estou mexendo é, ele esta gravando as url próprias do site e uma coisa que não poderia fazer é as url interna e sim somente as externa...

Compartilhar este post


Link para o post
Compartilhar em outros sites

HTTP_REFERER vai retornar a URL de onde o usuário veio. Ou seja, a última página que ele esteve.

 

O que você pode fazer para melhorar o seu código em questões de leitura e manutenção é utilizar o PHPMD e criar significados para cada trecho do código.

 

Tais quais:

O que esse trecho de código faz?

$read->getResult()[0]['views'] + 1

Por que está pegando o primeiro registro? Por que está somando + 1? etc..

 

No lugar dele, crie uma função que deixe explícito o que ele realmente faz. Isso para cada trecho do código aonde não é explícito o suficiente para quem não escreveu o código.

Compartilhar este post


Link para o post
Compartilhar em outros sites

  • Conteúdo Similar

    • Por violin101
      Caros amigos, saudações.
       
      Por favor, me permita tirar uma dúvida com os amigos.

      Tenho um Formulário onde o Usuário digita todos os Dados necessários.

      Minha dúvida:
      --> como faço após o usuário digitar os dados e salvar, o Sistema chamar uma Modal ou mensagem perguntando se deseja imprimir agora ?

      Grato,
       
      Cesar
    • 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 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.