Qual desses dois códigos é mais eficiente?

[AnthraxisBR 56]

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 ?

 

[Williams Duarte 431]

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?

[AnthraxisBR 56]

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

 

.

 

 

[Williams Duarte 431]

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.

 

 

[Gabriel Heming 766]

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).

[AnthraxisBR 56]

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!

 

[Williams Duarte 431]

Recomendo que veja este vídeo!

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