Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > Perl: Общие вопросы > Refactoring, что это и для чего это нужно

Автор: korob2001 30.5.2005, 02:58

Возможно кому то будет интересно пообщаться на эту тему. Начнем с того, что синтаксис perl позволяет писать весьма лаконичный и хорошочитаемый код. Рефакторинг кода это по сути приведение кода программы к лаконичности и удобочитаемости, избавлению от лишних переменных, убиранию глобальных пременных, включению обработки исключительных ситуаций и т.д. и т.п.
Например, есть такой скрипт, который принимает два параметра и распечатывает файл, заданый вторым параметром, с нумерацией строк:

Код


#!/usr/bin/perl -w
@arr = @ARGV;
$i=1;
if (@arr and $arr[0] eq 'l') {
  open (FH, $arr[1]);
  while ($str = <FH>) {
    chomp($str);
    print $i.$str."\n";
    $i++;
  }
  close FH;
} else {
  print "Using:\n";
  print "printfile.pl l <filename>\n";
}

Для того что бы предотвратить использование глобальных переменных и для большей "секьюрности", добавим use strict; в начало кода. Наш код примет такой вид:

Код


#!/usr/bin/perl -w
use strict;
my @arr = @ARGV;
my $i=1;
if (@arr and $arr[0] eq 'l') {
    open (FH, $arr[1]);
    my $str;
    while ($str = <FH>) {
        chomp($str);
        print $i.$str."\n";
        $i++;
    }
    close FH;
} else {
    print "Using:\n";
    print "printfile.pl l <filename>\n";
}

Теперь вспомним о "переменной по умолчанию" и вобще критически просмотрим код на наличие лишних переменных:

Код


#!/usr/bin/perl -w
use strict;
my $i=1;
if (@ARGV and $ARGV[0] eq 'l') {
    open (FH, $ARGV[1]);
    while (<FH>) {
        chomp;
        print $i.$_."\n";
        $i++;
    }
    close FH;
} else {
    print "Using:\n";
    print "printfile.pl l <filename>\n";
}

Есть хорошее правило, более короткий код в конструкции if {} else стоит ставить первым, плюс добавим обработку исключительных ситуаций:

Код


#!/usr/bin/perl -w
use strict;
my $i=1;
if (@ARGV < 2 or $ARGV[0] ne 'l' or not (-f $ARGV[1])) {
    print "Using:\nprintfile.pl l <filename>\n";
} else {
    open (FH, $ARGV[1]) or die "Can't open file $ARGV[1], $!\n";
    while (<FH>) {
        chomp;
        print $i.$_."\n";
        $i++;
    }
    close FH;
}

Теперь попробуем еще свернуть код:

Код


#!/usr/bin/perl -w
use strict;
if (@ARGV < 2 or $ARGV[0] ne 'l' or not (-f $ARGV[1])) {
    print "Using:\nprintfile.pl l <filename>\n";
} else {
    my $i=1;
    open (FH, $ARGV[1]) or die "Can't open file $ARGV[1], $!\n";
    chomp,print $i++."$_\n" while (<FH>);
}

Если же для нас некритична платформонезависимость кода, то можно его еще "ужать":

Код


#!/usr/bin/perl -w
use strict;
my $i=1;
(@ARGV < 2 or $ARGV[0] ne 'l' or not (-f $ARGV[1])) ?  print "Using:\nprintfile.pl l <filename>\n" :  
            print $i++."$_\n" for split /\n/, `cat $ARGV[1]`;

Свернуть все это можно к "однострочнику":

Код


perl -ne 'print ++$i."$_"' <filename>

Главное не слишком увлекаться, что бы красота кода не повлияла на его стабильность и работу
Надеюсь это кому то пригодится.