12

我在某些来源中看到了这行代码

( $self->{arg} ) = ( ( delete $self->{arg} ) =~ /(.*)/s ) if ${^TAINT}; 

我明白无染。我也知道删除

我的问题是,在什么情况下有必要或首选使用delete,使用更简单的还不够吗

( $self->{arg} ) = ( ( $self->{arg} ) =~ /(.*)/s ) if ${^TAINT};

例如

#!/usr/bin/env perl -T

use 5.014;
use warnings;

package Some {
    use Moose;
    has 'arg' => (is => 'rw', isa => 'Str');
    sub doit {
        my $self = shift;
        #( $self->{arg} ) = ( ( delete $self->{arg} ) =~ /(.*)/s ) if ${^TAINT};
        ( $self->{arg} ) = ( ( $self->{arg} ) =~ /(.*)/s ) if ${^TAINT};
    }
};

my $some = Some->new( arg => 'some text' );
$some->doit();
say $some->arg;
4

1 回答 1

3

使用普通哈希删除该值并重新插入将产生与修改它相同的结果。

提交没有提供任何关于他为什么删除它的信息,只是他从 Mason 1 复制了功能。但是如果您查看HTML::Mason::Lexer的源代码,您会发现以下评论:

我们需要清除组件,否则正则表达式将失败为 Perl 错误。删除很重要,因为我们需要创建一个全新的标量,而不仅仅是修改现有的标量。

($current->{comp_source}) = (delete $current->{comp_source}) =~ /(.*)/s if taint_is_on;

所以这样做的原因是有一个新的标量,尽管他没有为他正在去污的另一个地方这样做:Mason::Interp,所以我的猜测是在去污时是一个早期的 Perl 错误。

所以不同之处在于 withdelete会给你一个新的标量,尽管这很少有实际应用。(当然删除和插入也是一个较慢的操作。)

use strict;
my $hash->{test} = 'test';
print \($hash->{test}),"\n";
( $hash->{test} ) = ( ( $hash->{test} ) =~ /(.*)/s );
print \($hash->{test}),"\n";
( $hash->{test} ) = ( ( delete $hash->{test} ) =~ /(.*)/s );
print \($hash->{test}),"\n";

SCALAR(0x7f84d10047e8)
SCALAR(0x7f84d10047e8)
SCALAR(0x7f84d1029230)
于 2016-02-07T23:02:48.923 回答