Keskustelu: Nettisivujen teko: Mikä php koodissa vikana kun email ja www ei näy

Mikä tässä osassa koodi on vikana? Kun netissä ei näy Email eikä www sivu kohdalla muutem mitään kuin vain sitten kun hiiri on päällä. Miten saan e-mail ja www kohdan näkymään?

echo " > <br><b>$ni</b></div>";echo "<p><table border=0 width=430 class=kortti style='margin-left: 0px;'>";if ($lo == true) {echo "<tr><td class='kortti_td' colspan='2'><a href='http://$ko' target='_blank'><img src='../logot/$lo' alt='$ni' border='0' style='margin-left:0px;margin-top:5px;'></a></td></tr>";}else {echo "";}echo "<tr><td class=kortti_td width=30%><b>Nimi</b>:</td><td width=70% class=kortti_td> $ni</td></tr><tr><td class=kortti_td><b>Puh</b>.</td><td class=kortti_td> $pu</td></tr>";if ($fa == true) {echo "<tr><td class=kortti_td><b>Fax</b>.</td><td class=kortti_td> $fa</td></tr>";}else {echo "";}echo "<tr><td class=kortti_td><b>Katuosoite</b>:</td><td class=kortti_td> $os</td></tr>";if ($pn == true) {echo "<tr><td class=kortti_td><b>Postinumero</b>:</td><td class=kortti_td>$pn</td></tr>";}else {echo "";}echo "<tr><td class=kortti_td><b>Paikkakunta</b>:</td><td class=kortti_td>$pk</td></tr>";echo "<tr><td class=kortti_td><b>Kuvaus</b>:</td><td class=kortti_td> $ku</td></tr>"; if ($sp == true) {echo "<tr><td class=kortti_td><b>Email</b>:</td><td class=kortti_td><a href='mailto:$sp' class='link_alat'>$sp</a></td></tr>";}else {echo "";}if ($ko == true) {echo "<tr><td class=kortti_td><b>www</b>:</td><td class=kortti_td><a href='http://$ko' target='_blank' class='link_alat'> http://$ko</a>";

Öh... Rivinvaihtoja ja kooditageja kehiin, kiitos. En kyllä edes yritä lukea tuota, ennen ku se on paremmin muotoiltu.

Jäsentäminen on ylimainostettua...

Suosittelen ainakin " -merkkien käyttöä tageissa (esim <td class="kortti_td">

Ja echo:n " -merkkien tilalle '

Mitään toiminnallisia takuita ei ole, ei ole testattu. Vain vähän sisennetty ja otettu pois ylimääräisiä kekkosia.

<?php
echo "<br>";
echo "<b>$ni</b>";

echo "<table border='0' width='430px' class='kortti' style='margin-left:0px;'>";

if ($lo == true)
{
echo "<tr>";
echo "<td class='kortti_td' colspan='2'>";
echo "<a href='http://' target='_blank'>
<img src='../logot/$lo'lt='$ni' border='0'
style='margin-left:0px; margin-top:5px;'></a>";
echo "</td>";
echo "</tr>";
}
else
{
echo '';
}

echo "<tr>";
echo "<td class='kortti_td' width='30%'>";
echo "<b>Nimi</b>:";
echo "</td>";
echo "<td width='70%' class='kortti_td'>$ni</td>";
echo "</tr>";

echo "<tr>";
echo "<td class='kortti_td'>";
echo "<b>Puh</b>.";
echo "</td>";

echo "<td class='kortti_td'>$pu</td>";
echo "</tr>";

if ($fa == true)
{
echo "<tr>";
echo "<td class='kortti_td'>";
echo "<b>Fax</b>.";
echo "</td>";

echo "<td class='kortti_td'>$fa</td>";
echo "</tr>";
}

else
{
echo '';
}
echo "<tr>";
echo "<td class='kortti_td'>";
echo "<b>Katuosoite</b>:";
echo "</td>";
echo "<td class='kortti_td'>$os</td>";
echo "</tr>";

if ($pn == true)
{
echo "<tr>";
echo "<td class='kortti_td'>";
echo "<b>Postinumero</b>:";
echo "</td>";
echo "<td class='kortti_td'>$pn</td>";
echo "</tr>";
}

else
{
echo '';
}

echo "<tr>";
echo "<td class='kortti_td'>";
echo "<b>Paikkakunta</b>:";
echho "</td>";
echo "<td class='kortti_td'>$pk</td>";
echo "</tr>";

echo "<tr>";
echo "<td class='kortti_td'>";
echo "<b>Kuvaus</b>:";
echo "</td>";

echo "<td class='kortti_td'>$ku</td>";
echo "</tr>";

if ($sp == true)
{
echo "<tr>";
echo "<td class='kortti_td'>";
echo "<b>Email</b>:";
echo "</td>";

echo "<td class='kortti_td'><a href='mailto:$sp' class='link_alat'>$sp</a>";
echo "</td>";
echo "</tr>";
}
else
{
echo '';
}
if ($ko == true)
{
echo "<tr>";
echo "<td class='kortti_td'>";
echo "<b>www</b>:";
echo "</td>";

echo "<td class='kortti_td'>
<a href='http://' target='_blank' class='link_alat'> http://</a>";
echo "</td>";
echo "</tr>";
?>

Hitusen verran parempi, tuota pystyy jo lukemaan, mutta ei jokaista html-tagia kannata erikseen tulostaa. Koodit ja toiminnallisuus sinne php-tagien sisään ja html niiden ulkopuolelle.

joo.. ei niitä kaikki erikseen kannata tulostaa, mutta semmosta kekkosta kun rupesi purkamaan näin sen henkilökohtaisesti itselleni helpommaksi urakaksi, kun pilkoin sen tarpeeksi pieneen kasaan. :)