Perl::Critic, was mache ich hier "falsch" (Allgemeines zu Perl)

[thread]13984[/thread]

Perl::Critic, was mache ich hier "falsch"

Readers: 26

Articles: hide open all | hide show old branches

+4 replies
roli

2009-09-05 16:13
User since
2004-12-31
424 articles
BenutzerIn
Hi,

ich habe damit angefangen meinen Code von Perl::Critic mal checken zu lassen. Im Moment hänge ich an folgendem:
Quote
Loop iterator is not lexical at line 1354, column 3. See page 108 of PBP. (Severity: 5)

Hier Zeile 1354:
Code: (dl )

foreach $s_key ( sort keys %h_key_all){
ich habe doch "$s_key" verwendet anstelle von "$_", wo liegt also der Fehler?

Danke
Roland
--
"Steh vorn, während du fragst;
sitzen soll, wer antwortet."
Aus "Die Edda des Snorri Sturluson" "Gylfis Täuschung" Strophe 2
- Dubu
  
  2009-09-05 16:20
  User since
  2003-08-04
  2145 articles
  ModeratorIn + EditorIn
  Du hast $s_key weder in der Schleife noch vorher als lexikalische Variable deklariert. Es sollte so heißen:
  
  Code: (dl )
  
  foreach my $s_key ( sort keys %h_key_all){
- +2 replies
- Linuxer
  
  2009-09-05 16:23
  User since
  2006-01-27
  3891 articles
  HausmeisterIn
  $s_key scheint in einem größeren Scope definiert zu sein (ich gehe mal davon aus, dass Du "strict" benutzt, oder?).
  
  Wenn nur eine Laufvariable gebraucht wird, dann würde ich sie immer mit "my" deklarieren:
  
  Code: (dl )
  
  1 2
  
  foreach my $s_key ( sort keys %h_key_all ) { }
  
  meine Beiträge: I.d.R. alle Angaben ohne Gewähr und auf Linux abgestimmt!
  Die Sprache heisst Perl, nicht PERL. - Bitte Crossposts als solche kenntlich machen!
  - roli
    
    2009-09-05 19:12
    
    User since
    2004-12-31
    424 articles
    BenutzerIn
    
    2009-09-05T14:23:05 Linuxer
    $s_key scheint in einem größeren Scope definiert zu sein (ich gehe mal davon aus, dass Du "strict" benutzt, oder?).
    Jepp, so ist es
    
    2009-09-05T14:23:05 Linuxer
    Wenn nur eine Laufvariable gebraucht wird, dann würde ich sie immer mit "my" deklarieren
    
    Gerade gemacht, und schon ist Critic zufrieden mit mir. Jetzt will ich mal den nächsten Level von Critic angehen ...
    
    Danke
    Roland
    --
    "Steh vorn, während du fragst;
    sitzen soll, wer antwortet."
    Aus "Die Edda des Snorri Sturluson" "Gylfis Täuschung" Strophe 2

View all threads created 2009-09-05 16:13.