Hjelp med W3 validatoren. Jeg skjønner ikke feilen

[Anders Moen]

Hei!

 

Har et lite problem her...nei, faktisk, et større problem:

Jeg prøver å validere kodinga mi hos W3 nå, og jeg har aldri hatt problemer med det før, men nå skjønner jeg rett og slett ikke hva som er feil.

 

Jeg får feilen der der jeg henter ut fra databasen, men det er ingen PHP/MySQL feil, bare 9 errors i validatoren.

 

http://validator.w3.org/check?uri=http%3A%...ype=Inline&ss=1

Der kan dere se selv, med kilden, bare ikke PHP koden selv, men den poster jeg her:

<?php
$con = mysql_connect("localhost","brukernavn","passord");
if (!$con)
 {
 die('Could not connect: ' . mysql_error());
 }

mysql_select_db("database_navn", $con);

$result = mysql_query("SELECT * FROM nyheter ORDER BY id DESC LIMIT 3");

while($row = mysql_fetch_array($result))
 {
echo "
<span style=\"font-size: 10pt; font-weight: bold; line-height: 12pt;\">
";
echo stripslashes(bbcode($row['tittel']));
echo "
</span>";
  echo "
<br />
<span style=\"font-size: 8pt; line-height: 8pt; color: gray;\">" . $row['dato'] . "</span>
<br />
<br />
";

echo "
<span style=\"font-size: 10pt; font-family: tahoma,arial;\">
";
echo stripslashes(bbcode($row['innhold1']));

echo "
</span>
<div style=\"text-align: center;\">
<a href=\"?page=view&id=" . $row['id'] . "\"><span style=\"font-size: 8pt;\">Read more</span></a><span style=\"font-size: 8pt;\"> / </span><a href=\"?page=view&id=" . $row['id'] . "\"><span 

style=\"font-size: 8pt;\">" . $row['replies'] . " comments<span></a>
</div>
<br />
<br />
<br />
<br />
";
 }

mysql_close($con);
?>

 

 

Håper noen kan hjelpe meg

[KillYou]

satt å så på validatoren nå de fleste feilene dine

<span style="font-size: 8pt;">0 comments<span></a>

 

ser du feilen? du har glemt å lukke span tagen?

 

og en liten ting vil tro koden din kan bli litt penere ved bruk av et eksternt css og ikke bruke style tagen.

[Anders Moen]

Å, takk

 

Bare tre errors igjen. Den jeg egentlig lurte på er igjen =/

 

Men ja, jeg vet at jeg burde bruke det i CSS fila i stedet for style="", men jeg orker ikke å gjøre om det akkurat nå...trøtt

 

 

Men veit du hvorfor jeg ikke får bruke div'en der da?

Endret 3. mars 2007 av Andy-Pandy

[KillYou]

når jeg ser på koden nå har jeg ikke peiling, er problemet der forsatt vist du tar vekk style="text-align: center;"?

[Anders Moen]

Ja...men jeg tok LIMIT 1 i stedet for 3, så det bare blir 1 error, i stedet for 3.

 

Her er error meldinga:

 

Error Line 46, column 4: document type does not allow element "div" here; missing one of "object", "applet", "map", "iframe", "button", "ins", "del" start-tag

[KillYou]

er vist valid nå hva gjorde du?

[Anders Moen]

Ja, den er det.

 

Nei, si det du

Har et slikt switch script, så får jeg ?page=etellerannet så inkluder den f. eks etellerannet.php der. Ja, du kan det sikkert.

 

Men i index.php har jeg satt på <p class="content">den switch koden her</p> så da ble det liksom en <p> som gikk rett igjenom div'en og slikt, hehe.

 

Jeg er så dum! hehe

 

 

 

Det skal jeg huske neste gang, ikke at jeg er dum (hihi), men at det er noe slikt noe hele tida.

 

Men takk for hjelpen. Hadde aldri funnet den spanen som ikke var lukket hadde det ikke vært for deg

[KillYou]

det er vel include script du mener:P

 

så du fikk en div inni en p eller?

[Anders Moen]

Hm, ja, et include script som bruker switch metoden.

 

<?php 
switch($_GET['page']) {
   case 'about':
       include "about.php";
       break;
   case 'portfolio';
       include "portfolio.php";
       break;
   case 'templates':
       include "templates.php";
       break;
   case 'tutorials':
       include "tutorials.php";
       break;
   case 'news':
       include "news.php";
       break;
   case 'contact':
       include "contact.php";
       break;
   case 'view':
       include "view.php";
       break;
   case '404':
       include "404.php";
       break;
   case '403':
       include "403.php";
       break;
   case '500':
       include "500.php";
       break;
   case 'admin':
       include "admin/index.php";
       break;
   case 'client':
       include "login.php";
       break;
   case 'guestbook':
       include "guestbook.php";
       break;
   default:
       include "main.php";
}
?>

Det er den jeg har. Så har et i mellom <title> og </title> også, slik at jeg får forskjellige titler på hver side, og en for menyen slik at det blir en annen farge på den linken som du er å besøker. Hvis du går til sida mi, så vil du se det hvis du klikker på en link.

[KillYou]

---

hvorfor gjøre det så komplisert?

 

les litt her:P vist du allerede ikke har gjort det http://www.hardware.no/guider/sikkerhet/si...ude-script/8764

 

og hvorfor ikke bare gjøre slik

<?PHP
echo "<title>Anders Moen :: $_GET

---

 </title>";
?>

[Anders Moen]

Fordi hvis noen da hadde skrevet ?page=n00b så hadde jo tittel blitt:

Anders Moen :: n00b

 

Det er ikke bra ^^

 

 

Edit: f***! Så på den sida du ga link til...d***enkelt egentlig =/

Endret 4. mars 2007 av Andy-Pandy

[KillYou]

---

hva med:P dette da:P

 

<?php
error_reporting(0);
$filnavn = $_GET['page'];
$tillatt = array("index", "articles", "links");
$filendelse = '.php';
if (in_array($filnavn, $tillatt)) {
 echo "<title>Anders Moen :: $_GET

---

</title>";
}
else {
echo "<title>Anders Moen</title>";
}
?>

 

så litt lengre der du vil inkludere siden :

<?PHP
if (in_array($filnavn, $tillatt)) {
 include $filnavn . $filendelse;
}
else {
 include '404.php';
}
?>