Ir para conteúdo

POWERED BY:

Arquivado

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

AnthraxisBR

Qual desses dois códigos é mais eficiente?

Recommended Posts

Tenho dois códigos que fazem a mesma coisa, mas de maneira diferente, um calcula de uma vez e coloca na array, outro, eu calculo um valor de cada vez.

 

O que calcula de uma vez:

<?php
public static function get_count_companies_by_tax_regime_by_financial_movement($tax_regime)
    {
        $sql = 'SELECT cod,internal_control_group,company_tax_regime FROM data_companies WHERE company_tax_regime = "' . $tax_regime . '"';
        $rs = MySQLComponents::get_rs_array_stmt_PDO($sql);
        $companies_cod = array_column($rs, 'cod');
        $SQL_companies_cod = implode("','", $companies_cod);
        $count = count($emps);
        $exercise = date('Y-m') . '-01';
        $sql_2 = 'SELECT * FROM fiscal_obligations_control WHERE relationship IN("' . $SQL_companies_cod . '") AND exercise = "' . $exercise . '"';
        $rs_2 = MySQLComponents::get_rs_array_stmt_PDO($sql_2);
        $CM = 0;
        $SM = 0;
        $outstanding= 0;
        foreach ($rs_2 as $row) {
            $apuracao = $row['status_apuracao'];
            if($apuracao == 0){
                $outstanding = $outstanding +1;
            }elseif($apuracao == 1){
                $SM = $SM + 1;
            }elseif($apuracao == 3 OR $apuracao == 4){
                $CM = $CM + 1;
            }else{
                return 0;
                break;
            }
        }
        $response = [];
        $response['CM'] = $CM;
        $response['SM'] = $SM;
        $response['outstanding'] = $outstanding;
        return $response;
    }

 

E fiz esse outro que calcula 1 valor de cada vez:

 

<?php
public static function get_count_companies_by_tax_regime_by_financial_movement($tax_regime, $financial_movement)
    {
        $sql = 'SELECT cod,internal_control_group,company_tax_regime FROM data_companies WHERE company_tax_regime = "' . $tax_regime . '"';
        $rs = MySQLComponents::get_rs_array_stmt_PDO($sql);
        $companies_cod = array_column($rs, 'cod');
        $SQL_companies_cod = implode("','", $companies_cod);
        $count = count($emps);
        $exercise = date('Y-m') . '-01';
        $sql_2 = 'SELECT * FROM fiscal_obligations_control WHERE relationship IN("' . $SQL_companies_cod . '") AND exercise = "' . $exercise . '"';
        $rs_2 = MySQLComponents::get_rs_array_stmt_PDO($sql_2);
        $value = 0;
        foreach ($rs_2 as $row) {
            $apuracao = $row['status_apuracao'];
            if ($apuracao == $financial_movement) {
                $value = $value + 1;
            } elseif ($apuracao == 1 OR $apuracao == 3 OR $apuracao == 4) {
                // Não soma
            } else {
                return 0;
                break;
            }
        }
        return $value;
    }

 

A primeira, eu chamaria uma vez, e espalharia os dados da array, mas ela é muito especifica.

 

A segunda precisa ser chamada toda vez que precisa de um dado, porém ele é mais 'abstrata'.

 

As duas são chamadas da mesma forma com uma requisição ajax quando o elemento com o data-attr "get_count_companies", com "primary-param", com "secondary-param", e param-numbs, e também com o className 'get_data_from_db'  é carregado, depois ele passa para o Controller, que identifica e chama essa função dentro do model, essa parte é ok, a dúvida é sobre o método, pois tenho vários métodos parecidos.

 

A dúvida é:

 

Citar

Entre esses dois métodos qual é o mais eficiente ?

 

Compartilhar este post


Link para o post
Compartilhar em outros sites

Teus códigos ao mesmo tempo que faz consulta no banco, faz calculo, estoura pipoca, ainda retorna um array.

 

Já ouviu falar em métodos privados, conceito de models, repositorios, services etc?

Compartilhar este post


Link para o post
Compartilhar em outros sites
2 minutos atrás, Williams Duarte disse:

Teus códigos ao mesmo tempo que faz consulta no banco, faz calculo, estoura pipoca, ainda retorna um array.

 

Já ouviu falar em métodos privados, models, repositorios, services etc?

 

Sobre os métodos privado, models e repositórios é exatamente o que eu to tentando aprender, mas eu estou tendo muitos problemas pra entender a hora separar coisas de um método, eu to tentando dividir o máximo que eu consigo, tipo, uma função fazer apenas uma coisa, mas por exemplo.

 

Eu entendi que:

 

get_count_companies_by_tax_regime_by_financial_movement

 

Esse método de fazer a conta, era uma coisa só.

 

Mas agora que você disse, dei uma analisada melhor, parece que ele é várias coisas:

 

1 - Consulta

2 - Construção de Array

3 - Cálculo

 

Mas mesmo sabendo disso, não consigo imaginar como separar essas coisas em por exemplo um private function que faça a busca somente para essa classe, outro que monte a array, e um que calcule, até porque o calculo é o que vai preencher a array

 

.

 

 

Compartilhar este post


Link para o post
Compartilhar em outros sites

Primeiro passo é dar um nome mais conciso, ver o oque este método vai fazer ou retornar!

 

Este método faz oque?

 

Conta o total de empresas por regime tributário, e ao mesmo tempo por movimento financeiro?

 

Não está muito claro e bem confuso.

 

Métodos privados nada mais é do quebrar a aplicação em pequenos trechos de códigos, estes são auxiliares para os métodos protegidos ou públicos.

 

Seus exemplos acima, você pode quebrar em dois métodos privados, já que os métodos públicos nada mais é, do que sua interface, sendo interface, deixe as regras de negocio separados em métodos privados, ou use patterns.

 

 

Compartilhar este post


Link para o post
Compartilhar em outros sites

Apenas complementando, abra seu projeto em linux e use a biblioteca PHPMD, verá sua complexidade ciclomática no espaço (e eu nem precisei rodar para saber disso).

 

Existem um trilhão de regras, patterns e dicas para escrever um código mais eficiente, a maioria são boas, só que extensas. As que eu acho mais simples e diretas são Object Calisthenics ( do livro  The ThoughtWorks Anthology). Se colocar seu código conforme as 9 regras, terá um código bem eficiente:

http://williamdurand.fr/2013/06/03/object-calisthenics/

https://pt.slideshare.net/guilhermeblanco/php-para-adultos-clean-code-e-object-calisthenics

 

Entretanto, não menospreze as demais, existe muita coisa boa (conforme já foi mencionado no tópico).

Compartilhar este post


Link para o post
Compartilhar em outros sites

Bom dia!

 

@Williams Duarte Primeiro obrigado por responder quase todas as minhas perguntas kk

 

9 horas atrás, Williams Duarte disse:

Conta o total de empresas por regime tributário, e ao mesmo tempo por movimento financeiro?

 

É bem isso mesmo, eu preciso contar quantas empresas tiveram movimentação financeiro por regime (Simples, presumido e real), preciso disso pois tem um cálculo de eficiência dos funcionários com base nas obrigações que ele deveria fazer, e a movimentação financeira tem um valor grande nesse grau de eficiência, porque se tem movimentação, teve mais trabalho.

 

Eu tentei mudar o método, mas acabei deixando maior ainda ... Consegue apontar onde eu poderia fazer quebras pra eu dar uma estudada ?

 

<?php
private static function make_imploded_list_by_field($field_values,$separator,$field)
    {
        // Retira o campo correspondente ao relacionamento da array
        $field_to_implode = array_column($field_values, $field);
        // IMPLODE e dispôe em linha separador por '","'
        $imploded_field = implode($separator, $field_to_implode);
        return $imploded_field;
    }
    
    public static function get_count_companies_by_tax_regime_by_financial_movement($tax_regime, $responsible, $exercise)
    {
        // Verifica se o o responsável foi definido, se não, define como o usuário da sessão
        if ($responsible == '') {
            $responsible = $_SESSION['relationship'];
        }
        // Verifica se o exercicio foi definido, se nao, define o mês atual ---> .'-01' no final é apenas padrão do banco de dados.
        if ($exercise == '') {
            $exercise = date('Y-m') . '-01';
        } else {
            //Se o exercicio foi definido, executa função componente de formatação de data.
            $exercise = Components::exec_format_date($exercise, 'm/Y', 'Y-m') . '-01';
        }
        // Método para agilizar a interpretação que busca na tabela cache se o usuário POSSUI a algúm grupo como responsável.
        $user_group = geralModel::verif_if_collaborator_has_group($responsible);
        // Se o usuário possuir grupo busca os usuários relacionados a este grupo
        if ($user_group != '') {
            // Efetua a busca
            $relacionated_collaborators = geralModel::list_collaborator_dependents($responsible);
            //Função para criar lista de busca separada por '","'
            $SQL_collaborators_cod = self::make_imploded_list_by_field($relacionated_collaborators,'","','user_relationship');
            // Complemento para a query de consulta
            $sql_final_comp = '  exercise_responsible IN ("' . $SQL_collaborators_cod . '")';
        } else {
            //Se o usuário não possuir grupo, usar o complemento simples para a query
            $sql_final_comp = ' exercise_responsible = "' . $responsible . '"';
        }
        //Query para buscar as empresas para o regime tributário definido
        $sql = 'SELECT cod,internal_control_group,company_tax_regime FROM data_companies WHERE company_tax_regime = "' . $tax_regime . '"';
        //Função componente para eecução da query
        $rs = MySQLComponents::get_rs_array_stmt_PDO($sql);
        //
        $SQL_companies_cod = self::make_imploded_list_by_field($rs,'","','cod');
       //Query que busca dentro da tabema do controle de obrigações as empresas para o exercicio/responsavel/regime definidos
        $sql_2 = 'SELECT * FROM fiscal_obligations_control WHERE relationship IN("' . $SQL_companies_cod . '") AND exercise = "' . $exercise . '" AND ' . $sql_final_comp . '';
        //Função componente para execução de query
        $rs_2 = MySQLComponents::get_rs_array_stmt_PDO($sql_2);
        //Define como 0 as váriaveis 
        //$CM = Com Movimento
        //$SM = Sem Movimento
        //$outstandig = Pendentes
        $CM = 0;
        $SM = 0;
        $outstanding = 0;
        //Abreo loop pela $rs_2 
        foreach ($rs_2 as $row) {
            //Define a variavel apuração
            $apuracao = $row['status_apuracao'];
            //Comparativos de soma
            if ($apuracao == 0) {
                $outstanding = $outstanding + 1;
            } elseif ($apuracao == 1) {
                $SM = $SM + 1;
            } elseif ($apuracao == 2 or $apuracao == 3 or $apuracao == 4) {
                $CM = $CM + 1;
            } else {
                return 0;
                break;
            }
        }
        //Inicia e grava os dados na array
        $response = [];
        $response['CM'] = $CM;
        $response['SM'] = $SM;
        $response['outstanding'] = $outstanding;
        //Devovle a array resposta
        return $response;
    }

 

@Gabriel Heming Muito bom esse texto sobre Object Calisthenics, é de fácil compreensão, me sacou uma dúvida sobre divisão de arquivos/classes que eu tava a muito tempo, também vou estudar esse slide, mas o melhor é ir aprendendo nomes de coisas pra estudar, como por exemplo eu não conhecia o termo ' complexidade ciclomática' kk

 

Obrigado de novo!

 

Compartilhar este post


Link para o post
Compartilhar em outros sites

Recomendo que veja este vídeo!

Assiste, vai te ajudar entender um pouco mais sobre S.O.L.I.D. já mencionado no slide.

 

 

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.