Perl 循环读取文件卡住？

Question

结束这个问题。 会喝红牛。 睡觉。 编码并返回带有单元测试用例的全新问题。

更新：新文件位于此处< /a>

配置文件也在这里

我再次重构了代码：

sub getColumns {
    open my $input, '<', $ETLSplitter::configFile
        or die "Error opening '$ETLSpliter::configFile': $!";

    my $cols;
    while( my $conline = <$input> ) {
        chomp $conline;
        my @values = split (/=>/, $conline);
        if ($ETLSplitter::name =~ $values[0] ) {
            $cols = $values[1];
            last;
        }
    }

    if($cols) {
        @ETLSplitter::columns = split (':', $cols);
    }
    else {
        die("$ETLSplitter::name is not specified in the config file");
    }
}

此代码总是死在这里 die("配置文件中未指定 $ETLSplitter::name");.

另一个线索是，如果我将 split (':', $cols); 更改为 split (/:/, $cols); 我会收到此错误。

 perl -wle "
 use modules::ETLSplitter;
 \$test = ETLSplitter->new('cpr_operator_metric_actual_d2', 'frame/');
 \$test->prepareCSV();"
 syntax error at modules/ETLSplitter.pm line 154, near "}continue"
 Compilation failed in require at -e line 2.
 BEGIN failed--compilation aborted at -e line 2.

Answer 1

此问题的最终帖子：根据您的最新更新，我相信以下代码说明了使用 /:/ 作为 的第一个参数如何没有问题拆分。 它还指出，当人们使用函数参数而不是依赖全局变量时，阅读代码会更容易：

#!/usr/bin/perl

use strict;
use warnings;

use Data::Dumper;

for my $varname ( qw( adntopr.cpr.smtref.actv cpr_operator_detail )) {
    print $varname, "\n";
    print Dumper get_columns(\*DATA, $varname);
}

sub get_columns {
    my ($input_fh, $varname) = @_;

    while ( my $line = <$input_fh> ) {
        chomp $line;
        my @values = split /=>/, $line;
        next unless $varname eq $values[0];
        return [ split /:/, $values[1] ];
    }
    return;
}

__DATA__
adntopr.cpr.smtref.actv=>3:8:18:29:34:38:46:51:53:149
adntopr.smtsale2=>3:8:16:22:27:37:39:47:52:57:62:82:102:120:138:234:239:244:249:250:259:262:277:282:287:289:304:319:327:331:335:339:340:341:342:353:364:375:386:397:408
cpr_operator_detail=>3:11:18:28:124:220:228:324
cpr_operator_org_unit_map=>7:12
cpr_operator_metric_actual=>8:15:25:33:38:40:51

C:\Temp> tjm
adntopr.cpr.smtref.actv
$VAR1 = [
          '3',
          '8',
          '18',
          '29',
          '34',
          '38',
          '46',
          '51',
          '53',
          '149'
        ];
cpr_operator_detail
$VAR1 = [
          '3',
          '11',
          '18',
          '28',
          '124',
          '220',
          '228',
          '324'
        ];

该代码中有很多缺陷。 这是我对您想要执行的操作的解释：

更新：鉴于您最近关于模式中的正则表达式特殊字符的评论，如果您打算在模式中使用它们来分割，请确保引用它们。 $ETLSpliter::name 也有可能包含其他特殊字符。 我修改了代码来处理这种可能性。

sub getColumns {
    open my $input, '<', $ETLSpliter::configFile
          or die "Error opening '$ETLSpliter::configFile': $!");
      my @columns;
      while( my $conline = <$input> ) {
          my @values = split /=>/, $conline;
          print "not at: ".$conline;
          push @columns, $values[1] if $values[0] =~ /\Q$ETLSpliter::name/;
      }
      return @columns;
  }

另一个更新：

因此，根据您在下面的评论，该模式确实是 /=>/ 。 然后：

my $conline = q{cpr_operator_detail=>3:11:18:28:124:220:228:324};
my @values = split /=>/, $conline;

use Data::Dumper;
print Dumper \@values;
__END__

C:\Temp> tml
$VAR1 = [
          'cpr_operator_detail',
          '3:11:18:28:124:220:228:324'
        ];

没有错误 ... 没有警告 因此，还有其他事情正在发生，但您坚持不向我们展示。

其他备注：

1. 使用词法文件句柄，让 perl 告诉你它可能遇到什么错误，而不是推测。

2. 在最小适用范围内声明变量。

3. 当您可以在 while 语句中执行此操作时，无需在循环体中将 $_ 分配给 $conline。< /p>

4. 在原始代码中，您没有在 @columns 中放入任何内容，也没有对 $colData 执行任何有用的操作。

5. 缓和言辞。 计算机按照 GIGO 原理工作。

6. 查看您发布的链接处的代码，看起来您并没有意识到您可以做：

   use File::Spec::Functions qw( catfile ); 
     ... 
     catfile($ETLSpliter::filepath_results, $ETLSpliter::actual_name); 
     

此外，看起来你正在使用一个散列可以完成这项工作的包：

$ETLSpliter{filepath}

最后，你确实意识到 Spliter 是不正确的。 ITYM：分离器。

Answer 2

你确定它被卡住了吗？ 您永远不会在 @columns 中存储任何数据，因此您的代码将始终返回空列表。

其他注意事项：

• 您的 die 调用应包含 $! （操作系统错误）。 除了文件不存在之外，还有其他原因导致 open 失败，$! 会告诉您真正的问题是什么。
• 您可能应该执行 chomp $conline 来删除换行符。
• 您可以执行 while (my $conline =) 而不是从 $_ 复制值。
• 双参数 open（特别是隐式 < 模式）的形式很差。 首选使用三参数形式（最好使用词法文件句柄）：open(my $fh, '<', $filename) or die...

Answer 3

$ETLSpliter::name 中的内容 - 任何 / 字符都应该被转义。

代码片段中的许多其他问题已经得到解决，所以我不会去那里。

Answer 4

终于弄清楚了！！！！ 哇，睡眠是一种神奇的力量。

反正。 问题出在我的死亡消息中的 $ETLSplitter::configFile 中。

die ('Error opening '.$ETLSpliter::configFile.': '.$!);

其中有 winblows 路径分隔符“/”。 因为我用双引号输出，perl 将路径中的“/”解释为模式。 从这里

die "Error opening some/path/to/ ...

到

...  /=>/, 

哪个弄乱了子例程中的整个程序流程。 通过这样做解决了这个问题。

die ('Error opening '.$ETLSpliter::configFile.': '.$!);