Der Martin: Parse error: syntax error, unexpected T_STRING, expecting ',' or

Beitrag lesen

Hallo,

»» echo $before."<a href=\"" . add_trailing_slash($scriptPath) . "viewtopic.php?f={$row['forum_id']}&t={$row['topic_id']}&p={$row['post_id']}#p{$row['post_id']}\" title=\"{$row['topic_title']}\">{$row['topic_title']}</a><br/> " . date($gtm, $row['post_time']) <a href=\"" . add_trailing_slash($scriptPath) . "memberlist.php?mode=viewprofile&u=" . $row['poster_id'] . "\">" . $row['username'] ."</a>" .$after;

Abgesehen von der Syntaxhervorhebung: Schriebest du nicht solche Monsterdinger sondern setztest sie in kleinen Teilen und übersichtlich notiert zusammen, fiele dir auch vielleicht die Stelle besser auf, an der du Punkt und Anführungszeichen vergessen hast zu setzen.

das "in kleinen Teilen" ist nicht einmal nötig - wenn man nur denselben Ausdruck etwas übersichtlicher formatiert (eine Zeile pro Denkschritt), wird es gleich schon viel besser:

»» ~~~php

echo $before

»»    . "<a href=""
»»    . add_trailing_slash($scriptPath)
»»    . "viewtopic.php?f={$row['forum_id']}&t={$row['topic_id']}&p={$row['post_id']}#p{$row['post_id']}" title="{$row['topic_title']}">{$row['topic_title']}</a><br/> "
»»    . date($gtm, $row['post_time'])
»»    <a href=""
»»    . add_trailing_slash($scriptPath)
»»    . "memberlist.php?mode=viewprofile&u="
»»    . $row['poster_id']
»»    . "">" . $row['username']
»»    . "</a>"
»»    . $after;

  
So sieht man auf den ersten Blick, in welchem Teilterm "der Wurm drin" ist.  
  

> Auch Funktionen wie sprintf() oder in deinem Fall printf() können zur Erhöhung der Übersichtlichkeit und Wartbarkeit beitragen.  
  
Ja, dann würde sich der eine noch verbliebene Monsterstring auch noch etwas einfacher darstellen.  
  
Schönen Abend noch,  
 Martin  

-- 
Die letzten Worte des Neandertalers:  
Möchte doch zu gern wissen, was in der Höhle ist ...