Thread Tomaten auf den augen (21 answers)
Opened by Alter at 2008-04-26 13:26

FoolAck
 2008-05-03 00:31
#109125 #109125
User since
2008-05-02
69 Artikel
BenutzerIn
[default_avatar]
Alter... :/
Das ist mal sowas von undurchschaubar... Wiese schließen indentierte (Klammer-)Konstrukte nicht mit ihren terminierenden Zeichen (= Klammern) ab? Wieso überhaupt die weiträumige extrem unübersichtliche Indentierung? Da verliert man doch total den Überblick wo jetzt was anfängt, was ein quote ist, ob alle Klammern "aufgehen", ob man sich in einem Array befindet, einem Konditional etc.
Also ich habs mal so wies hier steht durch perltidy jagen wollen, und perltidy told me:
perltidy.ERR
40: final indentation level: 7

Final nesting depth of '('s is 1
The most recent un-matched '(' is on line 7
7: $html->div(
^
40: To save a full .LOG file rerun with -g

Auf Deutsch: Räum deinen Code mal auf.
Indentierung ist einzig und allein dazu da die Lesbarkeit deines Codes zu erhöhen. Der Plan geht imho bei dir nicht auf.
Schau dir perltidy mal an.

Edit:
Ich hab mal am Anfang etwas entfernt, damit zumindest mal die while-Schleife aufgeräumt dargestellt werden konnte, danach kam dann schon wieder ein Fehler... (ein Problem mit einem nicht beendeten single-quote afaik *hust*...)
Code (perl): (dl )
1
2
3
4
5
6
7
8

while ( my $news = $news_dbh->fetchrow_array ) {
    print $html->start_table(
        $html->TR( $html->th("$$news{ueberschrift}"), ),
        $html->TR( $html->td("$$news{datum}"), ),
        $html->TR( $html->td("$$news{text}"), ),
      ),
      $html->end_table();
}

im Vergleich zum Original:
Code (perl): (dl )
1
2
3
4
5
6
7
8
9

 while (my $news = $news_dbh->fetchrow_array){                                                                   
 print                                                           $html->start_table(     
                                                                         $html->TR($html->th("$$news{ueberschrift}"),),
                                                                                 $html->TR(
                                                                                         $html->td("$$news{datum}"),),
                                                                                                 $html->TR(
                                                                                                         $html->td("$$news{text}"),),),
                                                                         $html->end_table();
 }

Da fallen einem glatt die Tomaten von den Augen ;)

View full thread Tomaten auf den augen