在 Perl 中使用 foreach 代替 map 和 grep

Question

我正在尝试将map和grep代码完全转换为foreach循环，但是这样做时我遇到了麻烦。

这是获取事件的数组代码

my @events = [
  {
    Sponsor => ’Saash’,
    Time => ’05:00’,
    Date => ’01-03-2021’,
  },
  {
    Sponsor => ’Saash Tech’,
    Time => ’03:00’,
    Date => ’01-03-2021’,
  },
  {
    Sponsor => ’Saash Technology’,
    Time => ’04:00’,
    Date => ’01-03-2021’,
  },
  {
    Sponsor => ’Saash Techs’,
    Time => ’05:00’,
    Date => ’01-03-2021’,
  },
];

my $time = ’05:00’;

这是 grep 和 map 中使用的代码：

my $data = [
    map
    {
        my $hide;
        if ($_->{'sponsor'}) {
            $_->{'sponsor'} =~ ‘SaAsh’;
            $_->{‘sponsor'} = ‘Saash’ if $_->{’sponsor’} eq 'Saash Techs';
            if ($_->{‘time'} eq ’04:00’ && $_->{‘sponsor'} eq ‘SaAsh’) ) {
                $hide = 1;
            }
        }
        $hide ? () : ( $_ );
    }
    grep { $_->{‘time} =~ $time } @{events}
];

下面是我从 grep map 转换为foreach的代码

my $data;
foreach my $event (@events) {
    if ($event->{‘time’} =~ ’07:00’) {
        my $hide;
        if ($event->{‘sponsor’}) {
            $event->{‘sponsor'} =~ ‘SaAsh’;
            $event->{'sponsor'} = ‘Saash’ if $event->{’sponsor’} eq ‘Saash Techs';
            if ($event->{'time'} eq '04:00' && $event->{'sponsor'} eq ‘SaAsh’) ) {
                $hide = 1;
            }
        }
        $hide ? () : ( $event );
    }
}
$data = [$event];

与 grep 和 map 的代码相比，我在 foreach 代码中没有得到相同的 output。

有人可以帮助我哪里出错了吗？

Answer 1

foreach my $event (@events) {
   ...
   $hide ? () : ( $event )
}

$data = [$event];

应该

foreach my $event (@events) {
   ...
   push @$data, $hide ? () : ( $event )
}

---

my @a = map BLOCK LIST; 可以写成

my @a;
for (LIST) {
   push @a, do BLOCK;
}

my @a = grep BLOCK LIST; 可以写成

my @a;
for (LIST) {
   push @a, $_ if do BLOCK;
}

当然， do可以通过简化和重构来去掉，在你的情况下，这两个循环可以结合起来。 执行上述转换和一些清理后，我们得到以下信息：

for my $event (@events) {
   $event->{sponsor} = 'Saash'
      if $event->{sponsor}
      && $event->{sponsor} eq 'Saash Techs';
}

my $data = [];
for my $event (@events) {
   next if $event->{time} ne $time;
   next
      if ($event->{sponsor}
      && $event->{sponsor} eq 'SaAsh'
      && $event->{time} eq '04:00';

   push @$data, $event;
}

您只是在@events中将一些Saash Techs赞助商更改为Saash ，这就是我首先进行修复的原因。 像你一样偷偷改变@events是一种不好的做法。

Answer 2

这是答案代码吗？

foreach my $event (@events) {
   next if $_->{time} !~ /$time/;

   if ($event->{'sponsor'}) {
      $event->{'sponsor'} =~ ‘SaAsh’;
      $event->{'sponsor'} = 'Saash' if $event->{'sponsor'} eq 'Saash Techs';
      if ($event->{'time'} eq '04:00' && $event->{'sponsor'} eq ‘SaAsh’) ) {
          $hide = 1;
      }
   }

   push @$data, $hide ? () : ( $event );

}