Модераторы: korob2001, ginnie
  

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Refactoring, что это и для чего это нужно 
:(
    Опции темы
korob2001
Дата 30.5.2005, 02:58 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
****


Профиль
Группа: Комодератор
Сообщений: 2871
Регистрация: 29.12.2002

Репутация: 31
Всего: 61



Возможно кому то будет интересно пообщаться на эту тему. Начнем с того, что синтаксис 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>

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


--------------------
"Время проходит", - привыкли говорить вы по неверному пониманию. 
"Время стоит - проходите вы".
PM MAIL WWW ICQ MSN   Вверх
  
Ответ в темуСоздание новой темы Создание опроса
Правила форума "Perl"
korob2001
sharq
  • В этом разделе обсуждаются общие вопросы по языку Perl
  • Если ваш вопрос относится к системному программированию, задавайте его здесь
  • Если ваш вопрос относится к CGI программированию, задавайте его здесь
  • Интерпретатор Perl можно скачать здесь ActiveState, O'REILLY, The source for Perl
  • Справочное руководство "Установка perl-модулей", можно скачать здесь


Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, korob2001, sharq.

 
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | Perl: Общие вопросы | Следующая тема »


 




[ Время генерации скрипта: 0.0756 ]   [ Использовано запросов: 23 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.