2013-03-19 68 views
1

我正在编写一个脚本,用于检查服务器中的空间并删除旧备份/的空间小于2 GB。脚本工作正常,但因为我使用严格;使用警告;在我的练习目的脚本中,这个错误被抛出。使用未初始化的值取消链接。 。 。

这里是

#!/usr/bin/perl 
use strict; 
use warnings; 

my @backups; 
my $now=time(); 
my $dayago=10; 

my (@space,@freesp); 
@space=grep /\/dev\/md0/,`df`; 

for(@space){ 
     chomp; 
     @freesp=split /\s+/ ,$_; 
     } 

chdir '/home/ftpusr/backup' or die "Can't cd to backup dir: $!\n"; 

while (($freesp[3]/1024/1024 < 2.0) && ($dayago > 0)){ 
       my ($sec,$min,$hour,$mday,$mon,$year,$wday,$yday,$isdst) = localtime($now-$dayago*60*60*24); 
       my $today_timestamp=sprintf("%4d%02d%02d",$year+1900,$mon+1,$mday); 
         @backups = glob("*NODE0000.CATN0000.$today_timestamp*"); 
         #print "@backups\n"; 
         $dayago -= 1; 
         #print "$dayago\n"; 
         unlink ($backups[0]); 
     } 

我已经初始化$ dayago参数为10,以便它检查从最近10天的备份,并配备近至当前的日期,所以据我了解的第一次迭代的脚本while循环没有找到特定的文件与时间空间,所以它无法链接,所以抛出了这个警告。有什么方法可以消除此警告?

这里是O/P当我运行该脚本

9 
    Use of uninitialized value in unlink at purge3.pl line 26. 

    8 
    Use of uninitialized value in unlink at purge3.pl line 26. 

    7 
    Use of uninitialized value in unlink at purge3.pl line 26. 

    6 
    Use of uninitialized value in unlink at purge3.pl line 26. 

    5 
    Use of uninitialized value in unlink at purge3.pl line 26. 

    4 
    Use of uninitialized value in unlink at purge3.pl line 26. 
    GSRTC.0.db2inst1.NODE0000.CATN0000.20130315102900.001 
    3 
    GSRTC.0.db2inst1.NODE0000.CATN0000.20130316150941.001 GSRTC.0.db2inst1.NODE0000.CATN0000.20130316171526.001 
    2 

    1 
    Use of uninitialized value in unlink at purge3.pl line 26. 
    GSRTC.0.db2inst1.NODE0000.CATN0000.20130318095532.001 
    0 

而在目录中的文件将被列出像,我需要删除它们

GSRTC.0.db2inst1.NODE0000.CATN0000.20130315102900.001 
GSRTC.0.db2inst1.NODE0000.CATN0000.20130316150941.001 
GSRTC.0.db2inst1.NODE0000.CATN0000.20130318095532.001 
AWDRT.0.db2inst1.NODE0000.CATN0000.20130319092156.001 
GSRTC.0.db2inst1.NODE0000.CATN0000.20130319095258.001 
+0

不要chdir外部。使用Perl'chdir'内置函数:http://perldoc.perl.org/functions/chdir.html – 2013-03-21 05:05:03

回答

1

如果$backups[0]undef,那么这意味着你的glob与任何文件都不匹配(大概是因为那天没有)。如果没有找到文件,则不需要unlink任何东西。

因此改变unlink线

unlink($backups[0]) if @backups; 

unlink($backups[0]) if $backups[0]; 
+0

谢谢你的工作 – mviswa 2013-03-20 05:48:08

1

如果你重写它有点不同,你可以得到的Perl做检测:

my @backups = glob "*NODE0000.CATN0000.$today_timestamp*"; 
unlink @backups; 

甚至

unlink glob "*NODE0000.CATN0000.$today_timestamp*"; # No conditionals! 

但我认为File::Find是解决您的问题的正确方法。

+0

将检查File :: Find ........ – mviswa 2013-03-19 12:28:14

+1

没有必要使用'unlink for @ backups'只需使用'unlink @ backups'。实际上,在某些系统上,您可能需要多次调用['unlink'](http://p3rl.org/unlink“perldoc unlink”)才能获取文件'1的所有版本,而unlink @ backups'。 – 2013-03-20 20:46:16

+0

@BradGilbert:没有意识到'取消链接'接受名单! – Zaid 2013-03-21 04:52:42