Udtræk i alfabetisk rækkefølge

<?php
for ($i = "a"; $i <= "y"; $i++)
{
    echo "<a href=\"" . $_SERVER['PHP_SELF'] . "&bogstav=" . $i . "\">" . $i . "</a> ";
}

if (isset($_GET['bogstav']))
{
    echo "<hr />";
    $query = mysql_query("select `brugernavn` from `tabel` where `brugernavn` like '" . $_GET['bogstav'] . "%' order by `brugernavn` desc") or die (mysql_error());
    if (mysql_num_rows($query) >= 1)
    {
        while ($foo = mysql_fetch_object($query))
        {
            echo $foo->brugernavn . "<br />";
        }
    } else {
        echo "none";
    }
}
?>

Ja men på den måde du har lavet der kan man sende ekstra ting ind i databasen. og det er ikke smart, så gør det hellere på denne måde...

if( isset($_GET["bogstev"])
{
  if( ($_GET["bogstav"] == "a") || ($_GET["bogstav"] == "b") || ovs. )
  {
    $$query = mysql_query("select `brugernavn` from `tabel` where `brugernavn` like '" . $_GET['bogstav'] . "%' order by `brugernavn` desc") or die (mysql_error());
  }
  else
  {
    // vil blive vist hvis man ændre på url'en
    echo "FEJL: du kan ik søge sådan";
  }

  ovs. så videre i hans kode.

På denne måde sikrer man at folk ikke kan ændre på url'en og til f.eks. ?bogstav=ab og så den kun vil søge efter navne som starter med ab...

tænk over det... man må aldrig give brugere lov til at kunne ændre i udtrækket... for kan de det kan du også stoppe din sql sætning og meget andet... og det vil du vel ikke have...

bare en lille reminder.

Kender ikke lige LIKE men så'n her kan det også gøres

<?php
$valg = array();
for($i = "a"; $i <= "z"; $i++) {
    if(strlen($i) == 1) {
    $valg[] = $i;
    }
}

for($a = 0; $a < count($valg); $a++) {

echo "<a href='". $_SERVER["PHP_SELF"] ."?bogstav=". $valg[$a] ."'>". $valg[$a] ."</a><br />\r\n";
}

$sql = "SELECT * FROM `brugernavne`";

if(isset($_GET["bogstav"]) && in_array(strtolower($_GET["bogstav"]), $valg)) {
$sql .= " WHERE `brugernavn` LIKE '". $_GET["bogstav"] ."%'";
}

$sql .= " ORDER BY `brugernavn`";
echo $sql;
?>

Og husk at lave en index på det felt du bruger LIKE på, så stiger performance ;)

Nu har jeg udvidet gandso's eksempel til:

$stQrrelse = strlen($_GET['bogstav']);

if($stQrrelse > 1)
{
    echo "Du må ikke ændre i søgestrengen.. Fy skamme sig! Klik HER for at gå tilbage";
} else
{

    for ($i = "a"; $i <= "y"; $i++)
    {
        echo "<a href=\"" . $_SERVER['PHP_SELF'] . "?pageid=alle_brugere&bogstav=" . $i . "\">" . $i . "</a> ";
    }

    if (isset($_GET['bogstav']))
    {
        echo "<hr />";
        $query = mysql_query("SELECT `brugernavn` FROM `brugere` WHERE `brugernavn` LIKE '" . $_GET['bogstav'] . "%' order by `brugernavn` desc") or die (mysql_error());
        if (mysql_num_rows($query) >= 1)
        {
            while ($foo = mysql_fetch_object($query))
            {
                echo $foo->brugernavn . "<br />";
            }
        } else {
            echo "none";
        }
    }
}

Så er det problem med at man ændrer strengen til en større størrelse overstået. Udvider nok funktionen senere, men gider ikke lige nu :)

Har bare lige et sidste spørgsmål. Kan det udvides således, at 0-9 kommer med? Altså sådan at tal også er med i det?? :)

Gandso - du får point nu.. Håber du vil hjælpe alligevel :)

Takker til alle 4..

Forresten coderdk, hvad mener du? Forstår ikke :)

Hilsen Eiler...

Fyr denne af på din database:

ALTER TABLE `brugere` ADD INDEX (`brugernavn`);

Så vil din performance blive væsentligt bedre.

Kig på http://dev.mysql.com/doc/mysql/en/indexes.html
http://dev.mysql.com/doc/mysql/en/mysql-indexes.html
http://dev.mysql.com/doc/mysql/en/optimizing-database-structure.html

:)

Hehe fatter stadig ikke helt hvad du mener coderdk :) Er lidt dum til MySQL når det bliver lidt mere avanceret... Hvad gør den der du har sendt? Fatter ikke det som står på de sider du har skrevet :P

Forresten så lægger jeg gerne 25 mere i til dig, gandso, hvis du kan udvide din så den fungerer helt fra A-Å... Det ville være dejligt :) Kan nemlig ikke selv finde ud af det :P.. Eller giver 25 til anyone som kan udvide det til A-Å :)

Eiler :)

ALTER TABLE `brugere` ADD INDEX (`brugernavn`);

Gør at din query "koster" mindre på SQL-serveren - Derfor bliver den hurtigere. Hvis du søger igennem 1000 rækker bliver den ca. 100 gange hurtigere ;P Indexes er lidt avanceret ;P

Hehe okay... Meeen må ærligt indrømme at MySQL ikke lige er min stærke side når man skal andet end insert, select, delete og update... Og så lige lave strukturen (normal struktur, ikke noget hokus-pokus) :P...

Kan du hjælpe med mit lille problem mht. det kode eksempel jeg skrev? Sådan at det fatter alt fra A-Å?? så er du sød (og får 25 point) :D hehe..