Ir para conteúdo

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 ILR master
      Fala galera, tudo bem?
       
      Tenho o seguinte codigo:
       
       class Data {
      public static function ExibirTempoDecorrido($date)
      {
          if(empty($date))
          {
              return "Informe a data";
          }
          $periodos = array("segundo", "minuto", "hora", "dia", "semana", "mês", "ano", "década");
          $duracao = array("60","60","24","7","4.35","12","10");
          $agora = time();
          $unix_data = strtotime($date);
          // check validity of date
          if(empty($unix_data))
          {  
              return "Bad date";
          }
          // is it future date or past date
          if($agora > $unix_data) 
          {  
              $diferenca     = $agora - $unix_data;
              $tempo         = "atrás";
          } 
          else 
          {
              $diferenca     = $unix_data - $agora;
              $tempo         = "agora";
          }
          for($j = 0; $diferenca >= $duracao[$j] && $j < count($duracao)-1; $j++) 
          {
              $diferenca /= $duracao[$j];
          }
          $diferenca = round($diferenca);
          if($diferenca != 1) 
          {
              $periodos[$j].= "s";
          }
          return "$diferenca $periodos[$j] {$tempo}";
      }
      }
       
      Funciona redondinho se o valor retornado for de algumas horas, mas...
      Quando passa de dois meses, ele retorna a palavra mess. Deve ser por conta dessa linha
      if($diferenca != 1) 
          {
              $periodos[$j].= "s";
          }
       
      Quero que modre:
       
      2 meses atrás
      e não
      2 mess atrás.
       
      Espero que tenham entendido.
       
      Valeu
    • Por Carlos Web Soluções Web
      Olá...
      Estou tentando fazer o seguinte !!
      Listando dados em tabela !!
      Gostaria que....se na listagem houver 4 linhas...indepedente de seu número de ID, faça a listagem em ID ser em ordem 1 2 3 4 !!
      Exemplo...se tiver uma listagem de dados que está em ID 1 3 3...faça ficar 1 2 3 !!

       
      echo "<table class='tabela_dados' border='1'> <tr> <td>ID</td> <td>Nome Empresa</td> <td>Responsável</td> <td>Telefone 1</td> <td>Telefone 2</td> <td>E-mail 1</td> <td>E-mail 2</td> <td>Endereço</td> <td>CEP</td> <td>Bairro</td> <td>AÇÃO 1</td> <td>AÇÃO 2</td> </tr> "; $sql = "SELECT ID FROM usuarios_dados WHERE Usuario='$usuario'"; $result = $conn->query($sql); $num_rows = $result->num_rows; $Novo_ID = 1; for ($i = 0; $i < $num_rows; $i++) { $registro = $result -> fetch_row(); $sql2 = "UPDATE usuarios_dados SET ID='$Novo_ID' WHERE ID='$Novo_ID'"; $result2 = $conn->query($sql2); $Novo_ID++; } $sql = "SELECT * FROM usuarios_dados"; $result = $conn->query($sql); if ($result->num_rows > 0) { // output data of each row while($row = $result->fetch_assoc()) { echo "<tr> <td>$row[ID]</td> <td>$row[Nome_Empresa]</td> <td>$row[Responsavel]</td> <td>$row[Telefone_1]</td> <td>$row[Telefone_2]</td> <td>$row[Email_1]</td> <td>$row[Email_2]</td> <td>$row[Endereço]</td> <td>$row[CEP]</td> <td>$row[Bairro]</td> <td> <form method='post' action='Editar_Dados.php'> <input type='hidden' name='usuario' value='$usuario'> <input type='hidden' name='senha' value='$senha'> <input type='hidden' name='ID' value='$row[ID]'> <input type='submit' style='padding: 10px;' value='EDITAR'> </form> </td> <td> <form method='post' action='Deletar_Dados.php'> <input type='hidden' name='usuario' value='$usuario'> <input type='hidden' name='senha' value='$senha'> <input type='hidden' name='ID' value='$row[ID]'> <input type='submit' style='padding: 10px;' value='DELETAR'> </form> </td> </tr> "; } } else { echo "0 results"; } $conn->close();  
    • Por ILR master
      Boa tarde pessoal, tudo bem ?
       
      Eu uso o tinymce para cadastro de textos no meu siite, porém, quero fazer um sistema para que os colunistas possam fazer o próprio post.
      O problema do tinymce, é que ele mantém a formatação do texto copiado, como tamanho de fonts, negritos, etc... Quero que o usuário cole o texto e a própria textarea limpe a formatação para que ele formate como quiser.
       
      A pergunta é:
       
      O tinymce tem uma opção para desabilitar a formatação quando um texto é colocado?
      Tem alguma função via java ou php para retirar a formatação assim que o texto é colado?
      Ou é melhor usar um outro editor?
       
      Agradeço deste já.
    • Por Giovanird
      Olá a todos!
      Tenho uma pagina que possui uma DIV onde coloquei uma pagina PHP.
      Uso a função setInterval para atualizar a pagina inclusa dentro da DIV.
      O problema é que ao acessar o site , a DIV só me mostra a pagina inclusa somente quando completo o primeiro minuto.
      Preciso que a pagina inclusa já inicie carregada
       
      Meu código JavaScript e a DIV com a pagina PHP
       
      <script> function atualiza(){ var url = 'direita.php'; $.get(url, function(dataReturn) { $('#direita').html(dataReturn); }); } setInterval("atualiza()",60000); </script> <div> <span id="direita"></span> </div>  
    • Por ILR master
      Fala pessoal.
       
      Seguinte:
       
      Quero selecionar duas tabelas e mostrar com resultados intercalados. Abaixo segue um código explicando para vcs terem uma ideia.
       
      $consulta = "SELECT A.*, B.* FROM tabela1 A, tabela2 B'";
      $resultado = mysqli_query($conexao, $consulta) or die ("erro");
      while($busca = mysqli_fetch_array($resultado)){
       
      print $busca['cod_evento']; --> traz o código da tabela1 
      print $busca['titulo_evento']; -->  traz o titulo da tabela1
      print $busca['cod_noticia']; --> traz o código da tabela2
      print $busca['titulo_noticia']; --> traz o tituloda tabela2
       
      }
       
      Espero que entendam. Grato
       
×

Informação importante

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