問題タブ [perl-critic]

Perlのベストプラクティスとopendir同じポリシーがない理由はありますか？open

私は少なくともこれらの2つのポリシーについて考えています：

• Perl :: Critic :: Policy :: InputOutput :: ProhibitBarewordFileHandles
• Perl :: Critic :: Policy :: InputOutput :: RequireBriefOpen

最近、自分のコードでPerl::Criticをもっと頻繁に使い始めることにしました。Perl でのプログラミングを 7 年近く行った後、私は長い間、ほとんどの Perl のベスト プラクティスに慣れてきましたが、常に改善の余地があることを知っています。しかし、私を悩ませていることの 1 つは、Perl::Criticがサブルーチン用に @_ をアンパックする方法を気に入らないという事実です。例として：

これは私が常に行ってきた方法であり、PerlMonks と Stack Overflow の両方で議論されているように、必ずしも悪いことでもありません。

上記のコード スニペットを次のように変更すると...

...も機能しますが、読みにくいと思います。また、Damian Conway の著書Perl Best Practicesも読んだことがありますが、 Perl::Critic@_が暗示しているように、直接使用を避けるという彼の提案に、私の推奨する展開方法がどのように該当するのか、よくわかりません。私は常に、コンウェイが次のような意地悪について話しているという印象を受けてきました。

上記の例は下手で読みにくいので、私はこれを製品コードに書くことを考えたことはありません。

要するに、なぜPerl::Criticは私の好みのやり方が悪いと考えるのですか? シフトを使って開梱する私は本当に凶悪な犯罪を犯していますか?

これは、私以外の人がPerl::Criticのメンテナーと共に育てるべきだと考えるものでしょうか?

PHP用のPerlCriticに相当するものを探しています。PerlCritcは静的なソースコードアナライザーであり、コードをqritiquesし、未使用の変数から、データを処理するための安全でない方法まで、ほとんどすべてについて警告します。

ソースコード分析を自動化できるように、（できれば）IDEの外部で実行できるPHPのようなものはありますか？

からの引用Perl::Critic::Policy::InputOutput::ProhibitExplicitStdin

Perlには、コマンドラインをチェックし、引数がある場合はそれらを開いてファイルとして読み取る*ARGVと呼ばれる便利な魔法のファイルハンドルがあります。引数がない場合、*ARGVは代わりに*STDINのように動作します。STDINから読み取るプログラムを作成する場合、この動作はほとんどの場合必要なものです。これは多くの場合、次の2つの同等の形式のいずれかで記述されます。

• それは「単なる慣習」なのか、それとも使用しない確かな理由があるの<STDIN>でしょうか。

または<STDIN>を使用するよりも、コードの意図が明確になると思います。<><ARGV>

私のコードの流れはこんな感じです

perlcriticcroakは、die次のコード、完全に正常に動作するボイラープレート DBI のものは、

これはすべて、ダイは私にとってはうまくいくようです。

Perl のサムライ戦士にとって、鳴き声は、物事がうまくいかないときに実際に死ぬよりも名誉に値すると思います。冗談はさておき

croak代わりになぜ私が必要なのdieですか？

perlcritic のアドバイスに耳を傾けないことの結果は何ですか?

次のperlコードは、PerlCriticで（Activestateによる）警告を生成します。

生成される警告は次のとおりです。

リスト関数の$_を変更しないでください

その警告についての詳細はこちら

$ _を変更しているとは思わないので、警告がわかりません。ただし、変更する必要があると思います。誰かが私にそれを説明してもらえますか？

Perl::Critic出力に問題のあるポリシー名を表示することは可能であると言われています。しかし、これをオンにするために私がしなければならないことを思い出せません。どうすればこれを有効にできperlcritic.rcますか？

私はPerlcriticの使用を開始しようとしていますが、同意しない、または適切とは思わないすべてのルールを除外して、独自の構成を改良することに取り組んでいます。

.perlcrirticrcファイルを使用してこれを実行できること、およびそのファイルでexcludeディレクティブを使用して、同意しないポリシーを除外できることを知っていますが、そのためには、次のようなポリシーの名前を見つける必要があります。抑制したいエラーを引き起こしています。

CPANのドキュメントでポリシーのリストがどこにも見つかりません。また、エラーメッセージに生成されているポリシーの名前が表示されないため、どのように生成するのかまったくわかりません。リストを除外します。

誰かがエラーをポリシーにマッピングする方法を教えてくれたり、コアのポリシーのリストを教えてくれたら、私は最も感謝しています！

私はスクリプトを持っており、perlcritic を使用して悪い慣行を排除しようとしています。

私が持っている1行は次のとおりです。

これにより、次のエラーが発生します:行 xxx、列 x で使用される 2 つの引数「オープン」。PBP の 207 ページを参照してください。(重大度: 5)

それを修正する方法についてのアイデアはありますか？

フォローコードは評論家によって受け入れられません、重大度4：

このエラーが発生します：

ただし、このコードはコメントなしで受け入れられます。

一時変数を使用して戻り値を記述する方が本当に良いComma used to separate statementsのでしょうか、それとも、ハッシュ参照を作成して返すだけの時間を検出する際に批評家が間違っているのでしょうか。