PowerShell / PowerShell/PSScriptAnalyzer

PSAvoidAssignmentToAutomaticVariable Should detect auto-vars as function parameters

Offen
#1,026 5 Kommentare 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Area - Rules Issue - Enhancement
Vorherrschende Sprache
C#
Sterne
2.2k
Forks
414
Ø Merge
13 Std. 1 Min.
Gemergte PRs (30 T.)
2

Beschreibung

I did this recently and needed about 30 minutes to find my bug:

function Demo
{
    Param (
    [string] $input
    )
}

You can pass everything to this parameter, no warning, no error and $input is always $null !

See here, too: https://github.com/PowerShell/PSScriptAnalyzer/issues/712 "New Rule Suggestion: AvoidReuseOfAutomaticVariables"

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, die Regel PSAvoidAssignmentToAutomaticVariable zu finden und das zugehörige Issue #712 zu prüfen. Reproduziere das Funktionsparameter-Beispiel aus diesem Issue und ermittle anschließend, wie die Regel derzeit Zuweisungen an automatische Variablen erkennt. Als erledigt gilt die Aufgabe, wenn das Beispiel eine Warnung erzeugt, während gültige Parameter unbeeinträchtigt bleiben, und dieses Verhalten durch Tests abgedeckt ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
csharp, powershell
Bereich
tooling
Issue-Typ
Feature
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.