Forbedre ett script?

[Sono Juventino]

Har skrevet ett script som henter ut ting som siste innlegg, bruker, tid og svar i ett forum (phpBB), fra en database og lurer på om scriptet kan forbedres. Garanterer nesten at det kan forbedres siden jeg har skrevet det selv og ikke er så god i koding. Desuten virker bare linkene hvis scriptet ligger i selve forum-mappa, noe jeg synes den ikke burde gjøre, siden det ligger på en hovedside og derfor burde hete index.php.

 

Noen som kan fikse litt på dette?

 

<?php 
// Koble til databasen 
$adresse = "localhost"; 
$brukernavn = "db_username"; 
$passord = "Password"; 
$database = "db_name"; 
$conncetion = mysql_connect($adresse,$brukernavn,$passord) or die ("Oppkobling mot databasen mislyktes. Prøv igjen senere"); 
echo mysql_error(); 
$db = mysql_select_db($database,$conncetion) or die("Fant ikke valgt database"); 

// Ditt bildeikon 
$ikon = 'folder.gif'; 

$result = mysql_query("SELECT phpbb_topics.topic_title as topic_title, 
phpbb_topics.topic_time as topic_time, 
phpbb_topics.topic_views as topic_views, 
phpbb_topics.topic_replies as topic_replies, 
phpbb_topics.topic_id as topic_id, 
phpbb_topics.forum_id as forum_id, 
phpbb_topics.topic_poster as topic_poster, 
phpbb_users.username as username 
FROM phpbb_topics, phpbb_users 
WHERE phpbb_topics.topic_poster = phpbb_users.user_id 
ORDER BY topic_last_post_id DESC LIMIT 3"); 

echo "<table width=\"100%\" bgcolor=\"#FFFFFF\" cellspacing=\"1\"> 
<tr> 
<td class=\"style17 style33 style34\"> #</td> 
<td class=\"style17 style33 style34\" nowrap=\"nowrap\"> Tittel</td> 
<td class=\"style17 style33 style34\"> Skrevet av</td> 
<td class=\"style17 style33 style34\"> Tid<td>
<td class=\"style17 style33 style34\" nowrap=\"nowrap\"> Svar</td>
</tr>"; 

while ($row = mysql_fetch_object($result)) {
$row->topic_time = date("j/m/Y G:i", $row->topic_time);

echo "<tr> 
<td class=\"style17 style33 style34\"> <img src=\"$ikon\" alt=\"[*]\" title=\"[*]\" /></td> 
<td class=\"style17 style33 style34\" nowrap=\"nowrap\"> <a href=\"viewtopic.php?topic=$row->topic_id&forum=$row->forum_id\">$row->topic_title</a></td> 
<td class=\"style17 style33 style34\"> <a href=\"viewtopic.php?topic=$row->topic_id&forum=$row->forum_id\">$row->username</a></td>
<td class=\"style17 style33 style34\"> <a href=\"viewtopic.php?topic=$row->topic_id&forum=$row->forum_id\">$row->topic_time</a></td>
<td class=\"style17 style33 style34\"> <a href=\"viewtopic.php?topic=$row->topic_id&forum=$row->forum_id\">$row->topic_replies</a></td> 
</tr>";
} 

echo "</table>"; 

?>

 

Takker for seriøse svar!

[LateFot]

$adresse = "localhost"; 
$brukernavn = "db_username"; 
$passord = "Password"; 
$database = "db_name"; 
$conncetion = mysql_connect($adresse,$brukernavn,$passord) or die ("Oppkobling mot databasen mislyktes. Prøv igjen senere"); 
echo mysql_error(); 
$db = mysql_select_db($database,$conncetion) or die("Fant ikke valgt database");

 

denne delen ville jeg ha lagt i en egen fil og kalt den f.eks. connection.php, så inkludert den i scriptet. når du har den infoen i samme fil som bruker infoen er det ikke nødvendig å sette opp variabler.

[Sono Juventino]

Kommer det til å løse problemet til scriptet, med at det må ligge i samme mappe som forumet? Hvis det ikke gjør det kan noen forklare meg hvordan?

[kakkle]

nei...

da må du endre på linkene...

Hvis f.eks forumet ligger i www.domene.com/forum/, og du vil ha dette inn på www.domene.com/index.php, så må det legges til "forum/" foran hver viewtopic i hrefen...

 

det burde ihvertfall funke...