PowerShell / PowerShell/PSScriptAnalyzer

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

Aperta
#1,507 0 commenti 2 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Area - Rules Issue - New Rule Up-for-Grabs
Lingua principale
C#
Stelle
2.2k
Fork
414
Merge medio
13h 1m
PR unite (30g)
2

Descrizione

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.

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Non sono indicati file di implementazione, test o punti di ingresso. Inizia individuando le regole di analisi esistenti e i relativi test, quindi conferma se l’ambito previsto comprende sia break sia continue e come viene identificato il contesto di loop o trap. Il lavoro è completato quando la regola segnala l’uso improprio previsto e i test coprono il comportamento dell’avviso.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
csharp, powershell
Ambito
tooling
Tipo di issue
Funzionalità
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.