8

有时我有非常大的函数,我觉得很难“压缩”或分离成更小的函数,因为这些函数不会在脚本的任何其他部分中使用。

所以,我想要一些建议:我应该创建不会在脚本的其他部分使用的函数,还是应该只在它们被多次使用时才创建它们?

谢谢!

概括:

  • 用目录中文件的信息填充数组。
  • 逐行处理 TXT,查看 TXT 中的 ID 是否与“已完成”文件数组匹配,并将其发布到外部产品中。如果没有,则检查其他数组以报告缺少的内容。
  • 保存在数组中找到的错误,然后将数组保存到 errors.txt
  • 文件。最后,返回报告。

目前我的功能是:

protected function processScanned()
  {
    try
    {
      // EJECUTAR BASH DE NAHUEL
      //
      //
      $PdfCPList = $this->model->getDirFilenames( $this->model->dirCartasPorte, 'pdf' );
      $PdfTBList = $this->model->getDirFilenames( $this->model->dirTicketsBalanza, 'pdf' );
      $PdfCompList = $this->model->getDirFilenames( $this->model->dirCompletos, 'pdf' );
      $PdfUnreconList = $this->model->getDirFilenames( $this->model->dirSinReconocer,'pdf' );
      // Adjuntar Novedades
      $newsToProcess = $this->model->getDirFilenames( $this->model->dirNovedades, 'txt', true);
      $this->appendNewsFiles($newsToProcess);
      $report = array();
      $report['info'] = array(
        'Documentos procesados correctamente'=>0,
        'Fecha de última actualización de datos'=>date('d/m/Y H:i:s',(int)file_get_contents($this->model->uriTxtInfo)),
      );
      if($file = fopen( $this->model->uriTxtProcesar, 'r' ) )
      {
$i = 0;
        $errors_file = fopen($this->model->uriTxtErrores,'w');
        while( $line = fgets( $file ) )
        {
          if( ! preg_match( '/^\s/', $line ) )
            continue;

          $lineData = array(
            'id'=> substr($line,3,9),
            'prefix'=>'1234-' . $i,
            'suffix'=>'1234-' . $i,
            'partner'=>'FAZON TIMBUES OMHSA',
            'date'=>time() - 222,
          );
$i++;
          $keywordsToPublish = array(
            'Nº de Operacion'=>$lineData['id'],
            'Prefijo'=>$lineData['prefix'],
            'Sufijo'=>$lineData['suffix'],
            'Socio'=>$lineData['partner'],
            'Fecha'=>date('Y/d/m',$lineData['date']),
          );

          if( $this->model->findInDocusearch( $lineData['id'] ) )
          {
            continue;
          }

          if( array_key_exists( $lineData['id'], $PdfCompList ) )
          {
            $lineData['docName'] = 'Carta de Porte - Ticket de Balanza';
            $lineData['docId'] = 'CP-TB';
            $lineData['path'] = $this->model->dirCompletos . '/' . $lineData['id'] . '.pdf';
            if( $id = $this->model->publishInDocusearch( $lineData, $keywordsToPublish ) ) {
              $report['info']['Documentos procesados correctamente']++;
              link( $this->model->dirDocusearchRepo . '/' . $id . '.pdf', 
                $this->model->dirBackupCliente . '/' . $lineData['partner'] . '_' . date('Ymd',$lineData['date']) . '_' . $lineData['id'] . '.pdf'
              );
            }
            unset( $PdfCompList[ $lineData['id'] ] );
          }
          else
          {
            fwrite($errors_file, $line); // Guarda la fila leida en el archivo de errores.
            // Valores por defecto
            $report[ 'errors' ][ $lineData['id'] ]['date'] = $lineData['date'];
            $report[ 'errors' ][ $lineData['id'] ]['id'] = $lineData['id'];
            $report[ 'errors' ][ $lineData['id'] ]['type'] = 'nn';
            $report[ 'errors' ][ $lineData['id'] ]['actions'] = array();
            // Valores por defecto

            if( array_key_exists( $lineData['id'], $PdfCPList ) )
            {
              $report[ 'errors' ][ $lineData['id'] ]['reportMsg'] = 'Falta Ticket de Balanza.';
              $report[ 'errors' ][ $lineData['id'] ]['type'] = 'cp';
              unset( $PdfCPList[ $lineData['id'] ] );
            }
            elseif( array_key_exists( $lineData['id'], $PdfTBList ) )
            {
              $report[ 'errors' ][ $lineData['id'] ]['reportMsg'] = 'Falta Carta de Porte.';
              $report[ 'errors' ][ $lineData['id'] ]['type'] = 'tb';
              unset( $PdfTBList[ $lineData['id'] ] );
            }
            else
            {
              $report[ 'errors' ][ $lineData['id'] ]['reportMsg'] = 'Ningún archivo digitalizado.';
            }
          }
        }

        fclose( $file );
        fclose( $errors_file );
        $this->fileRename( $this->model->uriTxtErrores, $this->model->uriTxtProcesar);
        foreach( $PdfCompList as $key=>$value )
        {
          $report[ 'errors' ][ $key ] = array( 
            'reportMsg'=>'Falta en archivo de datos del sistema externo.',
            'date'=>$value['date'],
            'id'=>$key,
            'type'=>'co',
            'actions'=>array('get','rename','delete'),
          );
        }
        foreach( $PdfCPList as $key=>$value )
        {
          $report[ 'errors' ][ $key ] = array(
            'reportMsg'=>'Carta de Porte no utilizada.',
            'date'=>$value['date'],
            'id'=>$key,
            'type'=>'cp',
            'actions'=>array('get','rename','delete'),
          );
        }
        foreach( $PdfTBList as $key=>$value )
        {
          $report[ 'errors' ][ $key ] = array(
            'reportMsg'=>'Ticket de Balanza no utilizado.',
            'date'=>$value['date'],
            'id'=>$key,
            'type'=>'tb',
            'actions'=>array('get','rename','delete'),
          );
        }
        foreach( $PdfUnreconList as $key=>$value )
        {
          $report[ 'errors' ][ $key ] = array(
            'reportMsg'=>'Documento no reconocido.',
            'date'=>$value['date'],
            'id'=>$key,
            'type'=>'un',
            'actions'=>array('get','rename','delete'),
          );
        }
        return $report;
      }
      else
      {
        throw new Exception('No se pudo abrir el archivo TXT');
      }
    }
    catch( Exception $e )
    {
      $this->mensaje = $e->getMessage();
      header('HTTP/1.1 500 ' . $this->mensaje);
    }
 }
4

3 回答 3

14

这完全取决于你。

但是,
将代码块分成不同的函数可以使代码更具可读性(当它没有做得太多时)。函数不仅用于重复使用代码,还旨在使代码更有组织性和更易于理解。如果您尝试通读一个并行执行大量任务的长函数,您可能会迷失方向,但是如果您采用此函数并将其某些部分分解为具有正确命名的较小函数,则该函数将更短更清晰将来维护或让下一个从事您项目的程序员了解您所做的工作。

此外,一个好的做法是创建将处理某些更具体任务的对象。这将允许(在许多其他好处中)通过扩展类来更新代码,而不必损害原始功能。

根据您的编辑,在您编写的“功能摘要”中可以找到确定是否应将功能拆分为多个部分的好方法。当您有超过 1-2 个任务时,最好将其分解为单独的功能。我建议为以下各项编写一个函数:

  • 用目录中的文件信息填充数组
  • 逐行处理 TXT,查看 TXT 中的 ID 是否与“已完成”文件数组匹配
  • 在外部产品中发布数组
  • 检查其他阵列以报告缺少的内容。
  • 保存在数组中找到的错误,然后将数组保存到 errors.txt
  • 当然,将所有内容包装在一起并在完成后返回报告的功能。
于 2013-07-05T14:20:04.890 回答
1

在 SOLID 中,您应该看看单一职责原则

在面向对象编程中,单一职责原则指出每个类都应该有单一职责,并且该职责应该完全由类封装。它的所有服务都应与该责任密切相关。

您也可以将其应用于单个功能 - 每个功能都应该是可读的并执行为其创建的主要任务。

您似乎也在谈论匿名函数-一次性。

PHP 手册

$greet = function($name)
{
    printf("Hello %s\r\n", $name);
};

你可以看到$greet一个函数如何返回它的值。

但是,如果您在代码中多次使用此函数,请将其设为真正的函数,您可以根据需要多次调用该函数。

于 2013-07-05T14:19:53.120 回答
0

命名函数很难。因此,每当您看到具有明显名称的代码块时,您都应该给它起这个名称。分离关注点是困难的。因此,每当您看到两个明显不同的关注点时,您应该将它们明确分开。重新排序操作很困难。因此,当您看到两个显然可以重新排序的操作时,您应该让它们易于重新排序。当然,编写只能使用一次的函数是有意义的。

于 2016-01-29T01:10:44.513 回答