PowerShell / PowerShell/PSScriptAnalyzer

New rule to warn against break/continue outside of loops or traps

Offen
#1,507 0 Kommentare 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

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

Beschreibung

From https://github.com/PowerShell/PowerShell/issues/12742.

Today break will take control out of a function and up to the first loop in the callstack, possibly exiting the whole script.

Some existing code depends on this, but it's not a good behaviour.

Ideally a rule should warn about this, since using it break outside a loop or trap is usually a mistake that could seriously affect the program.

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

Es werden keine Implementierungsdateien, Tests oder Einstiegspunkte genannt. Beginne damit, vorhandene Analyzer-Regeln und deren Tests zu finden, und bestätige anschließend, ob der vorgesehene Umfang sowohl break als auch continue abdeckt und wie der Schleifen- oder Trap-Kontext identifiziert wird. Als erledigt gilt die Aufgabe, wenn die Regel vor dem gezielten Fehlgebrauch warnt und die Tests das Warnverhalten abdecken.

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.