Avatar billede blue-dragon Nybegynder
30. januar 2006 - 17:24 Der er 16 kommentarer og
1 løsning

Hvad fejler mit mysql script?

Hejsa!

jeg får denne meddelse: Parse error: parse error, unexpected $ in /web/www/frac/users/lgc/hitlist_show.php on line 54

Koden ser således ud:



<?
    print "<a href=hitlist_add.php?histlister=";
    print $user;
    print ">Tilføj en til hitlist</a>";
    print "<br><br><br>";
        if($bruger){

    $database = mysql_connect("host", "user", "pass");
    mysql_select_db("db", $database);
$foresp = mysql_query("SELECT dato, spillernavn, planetnavn, koordinater, grund, hitlisteren FROM hitlist ORDER BY dato DESC LIMIT 0,10");
   

    while($data = mysql_fetch_array($foresp)){
?>
<table>
<?
    print "<tr><td>";
    print date("j/n - Y", $data[0]);
    print "</td><td>";
    print $data[1];
    print "</td><td>";
    print $data[2];
    print "</td><td>";
    print $data[3];
    print "</td><td>";
    print $data[4];
    print "</td><td>";
    print $data[5];
    print "</td></tr>";
} ?>
</table>

</body>
</html>

Tak for hjælpen!
Avatar billede gizmo-gizmo Nybegynder
30. januar 2006 - 17:31 #1
Det står sådan set ret direkte, du har en syntax fejl på linje 54 i hitlist_show.php ... hvad indeholde denne linje?
Avatar billede gizmo-gizmo Nybegynder
30. januar 2006 - 17:32 #2
Iøvrigt kan jeg ikke spotte nogle "fejl" i det kode du har postet, andet end antallet af {'er ikke matcher antallet af }'er.
Avatar billede cronaldo Nybegynder
30. januar 2006 - 17:35 #3
if($bruger){ << den forstår jeg ikke? der står simpelthen: hvis $bruger.....
Jeg tror din browser fatter ligeså lidt ..

og sæt lige dette ind efter din </table>:

<? } ?>
Avatar billede blue-dragon Nybegynder
30. januar 2006 - 17:37 #4
Gizmo det er hele scriptet jeg har postet, eneste der mangler i det jeg har postet er noget CSS i starten af filen, men heller ikk andet...
Avatar billede gizmo-gizmo Nybegynder
30. januar 2006 - 17:39 #5
cronaldo: den tjekker bare om $bruger indeholder andet end 0/ingenting
Avatar billede cronaldo Nybegynder
30. januar 2006 - 17:40 #6
Okay. Ja, det ville jeg have gjort med if(isset($bruger)){}

så ved man hvad man gør ;)
-Men det er bare for at jeg skal have min opstilling i mine filer til at være overskuelig :)
Avatar billede gizmo-gizmo Nybegynder
30. januar 2006 - 17:42 #7
Cronaldo: du har ret, det beskriver ikke hvad sker særlig godt. if ($user_is_logged_in)  siger mere :-) 

Man ja, blue-dragon, prøv at smide det manglende } ind.
Avatar billede blue-dragon Nybegynder
30. januar 2006 - 17:48 #8
har jeg gjort, så får jeg den her: Parse error: parse error, unexpected '}' in /web/www/frac/users/lgc/hitlist_show.php on line 51
Avatar billede cronaldo Nybegynder
30. januar 2006 - 17:51 #9
<?
    print "<a href=hitlist_add.php?histlister=";
    print $user;
    print ">Tilføj en til hitlist</a>";
    print "<br><br><br>";
        if(isset($bruger)){

    $database = mysql_connect("host", "user", "pass");
    mysql_select_db("db", $database);
$foresp = mysql_query("SELECT dato, spillernavn, planetnavn, koordinater, grund, hitlisteren FROM hitlist ORDER BY dato DESC LIMIT 0,10");
 

    while($data = mysql_fetch_array($foresp)){
?>
<table>
<?
    print "<tr><td>";
    print date("j/n - Y", $data[0]);
    print "</td><td>";
    print $data[1];
    print "</td><td>";
    print $data[2];
    print "</td><td>";
    print $data[3];
    print "</td><td>";
    print $data[4];
    print "</td><td>";
    print $data[5];
    print "</td></tr>";
} ?>
</table>
<? } ?>

</body>
</html>


<--- Brug den kode :)
Avatar billede jakobdo Ekspert
30. januar 2006 - 18:26 #10
Blue-Dragon, smid hele din kode herind.
Din kode i dit første indlæg har ikke linie 54, så derfor er fejlsøgning svært for os.
Avatar billede cronaldo Nybegynder
30. januar 2006 - 18:54 #11
jakobdo:

når den skriver det er en $-fejl er det vel højst sandsynligt antal { > antal }
Avatar billede blue-dragon Nybegynder
30. januar 2006 - 18:57 #12
cronaldo, det virkede!.... Lav et svar og få dine point :D
Avatar billede jakobdo Ekspert
30. januar 2006 - 19:01 #13
Eller manglende ; eller $
Avatar billede jakobdo Ekspert
30. januar 2006 - 19:11 #14
Ikke at cronaldo's script ikke er godt nok, men her er lidt ekstra fejlhåndtering:

<?
    echo "<a href=hitlist_add.php?histlister=" . $user . ">Tilføj en til hitlist</a>";
    echo "<br><br><br>";
   
    if(isset($bruger) && !empty($bruger))
    {
        $database = mysql_connect("host", "user", "pass");
        mysql_select_db("db", $database);
        $foresp = mysql_query("SELECT dato, spillernavn, planetnavn, koordinater, grund, hitlisteren FROM hitlist ORDER BY dato DESC LIMIT 0,10");
       
        if(mysql_num_rows($foresp)>0)
        {
            while($data = mysql_fetch_array($foresp))
            {
                echo "<table>";
                echo "<tr><td>";
                echo date("j/n - Y", $data[0]);
                echo "</td><td>";
                echo $data[1];
                echo "</td><td>";
                echo $data[2];
                echo "</td><td>";
                echo $data[3];
                echo "</td><td>";
                echo $data[4];
                echo "</td><td>";
                echo $data[5];
                echo "</td></tr>";
                echo "</table>";
            }
        }
        else
        {
            echo "Ingen brugere fundet";
        }
    }
    else
    {
        echo "Bruger ikke sat";
    }
?>
</body>
</html>
Avatar billede blue-dragon Nybegynder
30. januar 2006 - 19:19 #15
jakobdo, du må lige forklare hvad det ekstra fejlhåndtering går ud på, skulle jo os gerne lære lidt af det ;)
Avatar billede jakobdo Ekspert
30. januar 2006 - 20:02 #16
if(isset($bruger) && !empty($bruger))
er $bruger sat og indeholder den rent faktisk noget
$bruger = 0 eller NULL vil give true i isset.

if(mysql_num_rows($foresp)>0)
får vi rent faktisk nogle rækker retur. (tabellen kunne jo være tom)

else
        {
            echo "Ingen brugere fundet";
        }
    }
    else
    {
        echo "Bruger ikke sat";
    }
Der fortæller hvis tabellen var tom eller $bruger ikke sat.

Ligeledes bruges echo, den er hurtigere en print.
print returnere antal tegn den har printet!
Avatar billede cronaldo Nybegynder
30. januar 2006 - 21:50 #17
jeg ved altså ikke hvis der er rigtigt..
Her er et svar, så kan du acceptere hvis mit var fint nok :)
Avatar billede Ny bruger Nybegynder

Din løsning...

Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] Web- og emailadresser omdannes automatisk til links. Der sættes "nofollow" på alle links.

Loading billede Opret Preview
Kategori
Vi tilbyder markedets bedste kurser inden for webudvikling

Log ind eller opret profil

Hov!

For at kunne deltage på Computerworld Eksperten skal du være logget ind.

Det er heldigvis nemt at oprette en bruger: Det tager to minutter og du kan vælge at bruge enten e-mail, Facebook eller Google som login.

Du kan også logge ind via nedenstående tjenester