hay, er der en af jer som er super til php/mysql ?
- hader mysql som pesten!!!

Det jeg prøver på at lave er:

når jeg går ind på test.php?id=124, så kigger den i img finder id, og "udskriver" data fra.. kan bare ikke få det til at virke

Har lavet det her som viser alt info fra "img" databasen, men den skal jo kun vise data ud fra det id jeg skriver.



tænker på noget ala:

så hvis der sidder en af jer der ude der lige ved hvordan jeg fixer det, så må du gerne skrive! :D

super virker :)
- godt nok skulle jeg lige slette ; men det går :D

(11-11-2013, 18:45)ailicis Skrev: super virker :)
- godt nok skulle jeg lige slette ; men det går :D

Det lyder lidt underligt, du skulle slette ;. :)
PHP kode linier afsluttes jo med ;
Men ved ikke, om det er noget man kan sætte i php.ini eller noget.
Men bare det virker.

har jo en or die(mysql_error()); som afslutter det :P
- koden er nu

slettet slettet slettet slettet

super, tak for info.
- har ikke brugt mysql sammen med php før :)

Jeg vil også lige henvise til mysql_real_escape_string() funktionen, som sikrer at id-parametren ikke kan bruges til SQLi :)

Først og fremmest vil jeg fremhæve vores shcode tags endnu engang, prøv at bruge [shcod e=php] i stedet for [php ]. Så kommer der en fin syntax highlighting på der også er synlig på vores tema.

Ydermere vil jeg foreslå at du ikke bruger et MySQL library der er deprecated og bliver udfaset i næste version af PHP.
Brug enten MySQLi (MySQL improved) eller PDO.
Læs på http://dk1.php.net/manual/en/mysqli.overview.php i afsnittet http://dk1.php.net/manual/en/mysqli.overview.php.

Hvis jeg skulle revidere din kode ville det se nogenlunde sådan her ud:

<?php

    // Opret forbindelse til databasen

    $db = new mysqli("127.0.0.1", "bruger", "kode", "database");

    // Undgå SQL injection ved at sætte backslash før specialtegn i GET-værdien

    $id = $db->real_escape_string($_GET["id"]);

    // Send en anmodning og opbevar resultatet i $result

    $result = $db->query("SELECT * FROM `img` WHERE `id` = $id");



    // Vis GET-værdien for en sikkerheds skyld

    echo $id;

    echo "<br /><br />";

    // Fyld resultat fra forespørgslen ind i en tabel

    echo "<table>"; 

    while($row = $result->fetch_object()){ // Hent den næste række ind i $row

        echo "<tr>"; 

        echo "<th>id:</th><td>" . $row->id . "</td>";

        echo "<th>navn:</th><td>" . $row->name . " </td></tr>"; 

    } 

    // Afslut tabel

    echo "</table>"; 

    // Luk forbindelse til databasen

    $db->close();

?>

@BlimBlamBlar hvad er idéen i at typecaste til integer? Hvis ikke det var fordi PHP havde sin conversion matrix ville du bare få en parserfejl. Det giver overhovedet ingen mening.

EDIT: Som de andre siger, så sørg for at holde fejlene væk fra brugerne.
Hvis der sker en fatal fejl skal du give en exception og vise brugeren en besked á la "Beklager, men siden har nogle problemer i øjeblikket. Vi kigger på det hurtigst muligt.".
Når du debugger skal du enten bruge logfilen eller den indbyggede funktion der kan sende log-entries til din e-mail addresse. Hvis du har direkte adgang til serveren og den er linux-baseret kan jeg anbefale dig at bruge som vil vise dig fejlloggen og løbende opdaterer når der kommer noget nyt.

(11-11-2013, 23:34)Doctor Blue Skrev: Hvis jeg skulle revidere din kode ville det se nogenlunde sådan her ud:

<?php

    // Opret forbindelse til databasen

    $db = new mysqli("127.0.0.1", "bruger", "kode", "database");

    // Undgå SQL injection ved at sætte backslash før specialtegn i GET-værdien

    $id = $db->real_escape_string($_GET["id"]);

    // Send en anmodning og opbevar resultatet i $result

    $result = $db->query("SELECT * FROM `img` WHERE `id` = $id");



    // Vis GET-værdien for en sikkerheds skyld

    echo $id;

    echo "<br /><br />";

    // Fyld resultat fra forespørgslen ind i en tabel

    echo "<table>"; 

    while($row = $result->fetch_object()){ // Hent den næste række ind i $row

        echo "<tr>"; 

        echo "<th>id:</th><td>" . $row->id . "</td>";

        echo "<th>navn:</th><td>" . $row->name . " </td></tr>"; 

    } 

    // Afslut tabel

    echo "</table>"; 

?>

@BlimBlamBlar hvad er idéen i at typecaste til integer? Hvis ikke det var fordi PHP havde sin conversion matrix ville du bare få en parserfejl. Det giver overhovedet ingen mening.

Ville intval ikke være mere passende når værdien er int ? Tænker bare at man stadig ville kunne skrive noget tekst. Ikke at det hjælper, men så er man mere sikret mod fejl.

Og mht. BlimBlamBlar's int cast. Det er vel en form for at validere den parameter. Selvom jeg ikke ville bruge den løsning, virker det som det eneste logiske.
Den vil jo smide en fejl så snart du prøver på SQL injection.