Avatar billede kack Nybegynder
07. april 2005 - 14:07 Der er 15 kommentarer

Parse error: parse error, unexpected $

Hej alle!

Er der en der kan fortælle mig hvad der er galt i dette stykke kode? Jeg er ikke så god til at få min kode til at se ordenlig ud, så den kan nok godt være lidt svær at finde rundt i :S

Men her er koden:

<?
$query = mysql_query("SELECT * FROM mm_brugere WHERE brugernavn = '$bruger' ORDER BY id");
while($row = mysql_fetch_array($query)) {
if ($row['admin'] == 1) {
        echo " - <a href='?side=vispost.php&slet=emne&id=".$_GET['id']."'>Slet emne</a>";
        }

      mysql_query("SELECT * FROM mm_post WHERE id = '".$_GET['id']."' LIMIT 1");
      if ($row['locked'] == 0) {
      if ($row['admin'] == 1) {
        echo " - <a href='?side=vispost.php&lock=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
        if ($row['admin'] == 2) {
        echo " - <a href='?side=vispost.php&lock=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
        }
        mysql_query("SELECT * FROM mm_post WHERE id = '".$_GET['id']."' LIMIT 1");
      if ($row['locked'] == 1) {
              if ($row['admin'] == 1) {
        echo " - <a href='?side=vispost.php&lockup=emne&id=".$_GET['id']."'>Lås op</a>";
        }
        if ($row['admin'] == 2) {
        echo " - <a href='?side=vispost.php&lockup=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
        }
if ($row['admin'] == 2) {
        echo " - <a href='?side=vispost.php&slet=emne&id=".$_GET['id']."'>Slet emne</a>";
        }

      mysql_query("SELECT * FROM mm_post WHERE id = '".$_GET['id']."' LIMIT 1");
      if ($row['locked'] == 0) {
      if ($row['admin'] == 1) {
        echo " - <a href='?side=vispost.php&lock=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
        if ($row['admin'] == 2) {
        echo " - <a href='?side=vispost.php&lock=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
        }
        mysql_query("SELECT * FROM mm_post WHERE id = '".$_GET['id']."' LIMIT 1");
      if ($row['locked'] == 1) {
              if ($row['admin'] == 1) {
        echo " - <a href='?side=vispost.php&lockup=emne&id=".$_GET['id']."'>Lås op</a>";
        }
        if ($row['admin'] == 2) {
        echo " - <a href='?side=vispost.php&lockup=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
        }
?>



Får denne fejl:


Parse error: parse error, unexpected $ in /customers/kack.dk/kack.dk/httpd.www/musicmania/vispost.php on line 255
Avatar billede bromer Nybegynder
07. april 2005 - 14:09 #1
hvor er linie 255?
Avatar billede kack Nybegynder
07. april 2005 - 14:10 #2
linie 255 er slet ikke noget af det der.

Min linie 255 er: ?>
Avatar billede mysli Nybegynder
07. april 2005 - 14:11 #3
Prøv at tjek for manglende '}' lukninger
Avatar billede bojo Nybegynder
07. april 2005 - 14:14 #4
mangler } til din while
Avatar billede bromer Nybegynder
07. april 2005 - 14:15 #5
kack: for at du kan finde din fejl må du tage udgangspunkt i den besked du får fra PHP parseren. Den er blevet forvirret ved linie 255, hvilket kan betyde flere ting. Jeg tror ikke det er en manglende "}" med mindre der ikke er flere linier i filen. Det kan være at du ikke har fået lukket en streng eller du mangler et semicolon.

Vi kan hjælpe dig bedre, hvis vi ser de - f.eks. 10 - linier får linie 255
Avatar billede bojo Nybegynder
07. april 2005 - 14:15 #6
dvs lige før ?>
Avatar billede kack Nybegynder
07. april 2005 - 14:17 #7
Tror bare ikke det er noget med det at gøre, for det var da jeg lige skulle lave om i denne kode den fejl kom.

Men jeg kan da godt poste de 10 sidste linier?
Avatar billede bromer Nybegynder
07. april 2005 - 14:18 #8
jeg tror bestemet at bojo har ret, så se lige på hvad han har skrevet. Min pointe var bare at det ikke behøver at være en manglede slut tuborg bare fordi man stopper PHP.
Avatar billede kack Nybegynder
07. april 2005 - 14:22 #9
Har fået det til at virke.

Men koden gør bare ikke helt det jeg vil ha den til. Vil ha' at hvis man er admin (1) eller moderator (2), kan man låse et emne, og låse det op igen.

Hvorfor kommer der ikke en "Lås op" knap, når emnet er låst? ET emne er låst når det er locked = 1.

Håber i forstår.

Ps. Det var fordi der manglede et }.
Avatar billede bojo Nybegynder
07. april 2005 - 14:50 #10
hvad laver alle de mysql_query der ikke bruges til noget udover at slå op i databasen - eller er det bare mig der ikke helt har fattet mysql?


btw - reformatteret kode her:

<?
$query = mysql_query("SELECT * FROM mm_brugere WHERE brugernavn = '$bruger' ORDER BY id");
while($row = mysql_fetch_array($query)) {
    if ($row['admin'] == 1) {
        echo " - <a href='?side=vispost.php&slet=emne&id=".$_GET['id']."'>Slet emne</a>";
    }

    mysql_query("SELECT * FROM mm_post WHERE id = '".$_GET['id']."' LIMIT 1");
    if ($row['locked'] == 0) {
        if ($row['admin'] == 1) {
            echo " - <a href='?side=vispost.php&lock=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
        if ($row['admin'] == 2) {
            echo " - <a href='?side=vispost.php&lock=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
    }
    mysql_query("SELECT * FROM mm_post WHERE id = '".$_GET['id']."' LIMIT 1");
    if ($row['locked'] == 1) {
        if ($row['admin'] == 1) {
            echo " - <a href='?side=vispost.php&lockup=emne&id=".$_GET['id']."'>Lås op</a>";
        }
        if ($row['admin'] == 2) {
            echo " - <a href='?side=vispost.php&lockup=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
    }
    if ($row['admin'] == 2) {
        echo " - <a href='?side=vispost.php&slet=emne&id=".$_GET['id']."'>Slet emne</a>";
    }

    mysql_query("SELECT * FROM mm_post WHERE id = '".$_GET['id']."' LIMIT 1");
    if ($row['locked'] == 0) {
        if ($row['admin'] == 1) {
            echo " - <a href='?side=vispost.php&lock=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
        if ($row['admin'] == 2) {
            echo " - <a href='?side=vispost.php&lock=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
    }
    mysql_query("SELECT * FROM mm_post WHERE id = '".$_GET['id']."' LIMIT 1");
    if ($row['locked'] == 1) {
        if ($row['admin'] == 1) {
            echo " - <a href='?side=vispost.php&lockup=emne&id=".$_GET['id']."'>Lås op</a>";
        }
        if ($row['admin'] == 2) {
            echo " - <a href='?side=vispost.php&lockup=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
    }
}
?>
Avatar billede kack Nybegynder
07. april 2005 - 14:55 #11
Tvivler på det er dig der ikke forstår det, - det er sq nok mig ;)

Men prøver det lige ;)
Avatar billede kack Nybegynder
07. april 2005 - 15:02 #12
Det virker desværre heller ikke helt endnu.

Dette er min kode:

<?
$query = mysql_query("SELECT * FROM mm_brugere WHERE brugernavn = '$bruger' ORDER BY id");
while($row = mysql_fetch_array($query)) {
    if ($row['admin'] == 1) {
        echo " - <a href='?side=vispost.php&slet=emne&id=".$_GET['id']."'>Slet emne</a>";
    }

    mysql_query("SELECT * FROM mm_post WHERE id = '".$_GET['id']."' LIMIT 1");
    if ($row['locked'] == 0) {
        if ($row['admin'] == 1) {
            echo " - <a href='?side=vispost.php&lock=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
        if ($row['admin'] == 2) {
            echo " - <a href='?side=vispost.php&lock=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
    }
    mysql_query("SELECT * FROM mm_post WHERE id = '".$_GET['id']."' LIMIT 1");
    if ($row['locked'] == 1) {
        if ($row['admin'] == 1) {
            echo " - <a href='?side=vispost.php&lockup=emne&id=".$_GET['id']."'>Lås op</a>";
        }
        if ($row['admin'] == 2) {
            echo " - <a href='?side=vispost.php&lockup=emne&id=".$_GET['id']."'>Lås op</a>";
        }
    }
    if ($row['admin'] == 2) {
        echo " - <a href='?side=vispost.php&slet=emne&id=".$_GET['id']."'>Slet emne</a>";
    }

    mysql_query("SELECT * FROM mm_post WHERE id = '".$_GET['id']."' LIMIT 1");
    if ($row['locked'] == 0) {
        if ($row['admin'] == 1) {
            echo " - <a href='?side=vispost.php&lock=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
        if ($row['admin'] == 2) {
            echo " - <a href='?side=vispost.php&lock=emne&id=".$_GET['id']."'>Lås emne</a>";
        }
    }
    mysql_query("SELECT * FROM mm_post WHERE id = '".$_GET['id']."' LIMIT 1");
    if ($row['locked'] == 1) {
        if ($row['admin'] == 1) {
            echo " - <a href='?side=vispost.php&lockup=emne&id=".$_GET['id']."'>Lås op</a>";
        }
        if ($row['admin'] == 2) {
            echo " - <a href='?side=vispost.php&lockup=emne&id=".$_GET['id']."'>Lås op</a>";
        }
    }
}
?>
Avatar billede bojo Nybegynder
07. april 2005 - 15:11 #13
jeg tror at du er helt ude på lars tynd...'s marker :-)

Så fremt programmet laver det jeg tror spørger jeg:

hvorfor alle de mysql_query - de laver intet ( eller er det bare mig der skal lære nyt igen? )

hvorfor alle de if - de laver det samme næsten

jeg formoder at $_GET['id'] kommer fra en form på forrige side, men skulle det ikke være $row['id'] ?

jeg ser lige om jeg kan overskue det og fjerne det overflødige ( efter min mening )
Avatar billede kack Nybegynder
07. april 2005 - 15:15 #14
Ville være dejligt hvis du kunne. Kan heller ikke helt overskue det selv :S
Avatar billede bojo Nybegynder
07. april 2005 - 15:20 #15
jeg siger ikke dette gør hvad du ønsker, men jeg tror det gør det samme som det du har


<?
$query = mysql_query("SELECT * FROM mm_brugere WHERE brugernavn = '$bruger' ORDER BY id");
while($row = mysql_fetch_array($query)) {

    if ($row['admin'] == 1 || $row['admin'] == 2) {
        echo " - <a href='?side=vispost.php&slet=emne&id=".$_GET['id']."'>Slet emne</a>";

        if ($row['locked'] == 0) {
            echo " - <a href='?side=vispost.php&lock=emne&id=".$_GET['id']."'>Lås emne</a>";
        } elseif ($row['locked'] == 1) {
            echo " - <a href='?side=vispost.php&lockup=emne&id=".$_GET['id']."'>Lås op</a>";
        }
    }
}
?>
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