perl - Perl::Critic in Brutal Mode

Question

所以我最近开始使用Perl::Critic来检查我编写的代码的质量。我正在以残酷的模式运行它，并提出了一个我不理解为问题的建议。输出是：

Return value of flagged function ignored - print at line 197, column 13. See pages 208,278 of PBP. (Severity: 1)

这基本上是对 print 函数的调用，并带有一条输出到控制台的短消息。那我为什么要捕获几乎肯定总是 1 的返回值，因为我想不出任何不是 1 的用例。

野蛮模式“太野蛮”了吗？还是我错过了什么？我应该补充一点，我确实阅读了 PBP 的第 208 页和第 278 页，但我并不清楚答案。

score 6 · Accepted Answer

我同意大多数时候print不会失败。但是，您可以通过创建一个.perlcriticrc文件并将这些行添加到其中来禁用此功能（就像我一样）：

# Check all builtins except "print"
[InputOutput::RequireCheckedSyscalls]
functions = :builtins
exclude_functions = print

此外，如果您不同意 Brutal 设置的所有政策，您可以使用其他 4 个不那么残酷的设置之一。该工具是高度可配置的。

这是一个print可能失败的小例子（打印到关闭的文件句柄）：

open my $fh, '>', 'out';
print $fh "555\n";
close $fh;
print $fh "888\n" or die "print failed: $!";

# we shouldn't get here
print "777\n";

在如此短的代码中，很明显您只是关闭了文件句柄，然后您将永远不会尝试打印到它。但是，如果你有很多（设计不佳的）代码，它可能会发生。

还有其他原因print可能会失败，例如另一个进程出现并删除了目录或禁用了对打开文件的写入权限。

我为自己创建了一个脚本来运行perlcritic，它可以轻松访问给定策略的 POD：排序和汇总 perlcritic 输出

score 0 · Accepted Answer

失败的一个用例print "something";是当 STDOUT 已打开文件并且文件系统已满时。但在我的项目中，我也不检查print.

score -1 · Accepted Answer

实施修复

print "$updated_service_name\n";

利用

my $printed = (print "$updated_service_name\n");
if (!$printed) {
  die "Unable to write to stdout\n";
}

3 回答 3