random nummer - sandsynlighed

Sandsynligheden er større end 0, så derfor skal du ikke gøre sådan.

Du kan gemme tidligere numre, så du vælger et nyt, hvis det allerede er brugt.

Men - øhm - hvorfor lige på den måde?  Når du kommer op i nærheden af 3000 ordrer, skal du prøve rigtig mange gange for at få et nyt - det er ikke sund programmering.

Jeg fik det ændret så den nu vælger mellem 1000 og 99999 - det gi'r lidt flere muligheder.

Den tjekker sådan her nu:

---------------------------------------------

$sql = "SELECT * FROM ordernum";
$result = mysql_query($sql);
if(mysql_num_rows($result) < 1) {
    mysql_query("INSERT INTO ordernum (ordernum) VALUES ('$ordernum')")or die(mysql_error());
} elseif(mysql_num_rows($result) > 0) {
    while($row = mysql_fetch_array($result)) {
        if($row["ordernum"] == $ordernum) {
            $ordernum = rand(1000, 99999);
        }
    }
}

--------------------------------------------------------

Er det en okay måde at lave det på?

Hvis du istedet for at genere et tilfældigt tal, lader mysql lave et tal som øges automatisk, så slipper du for at tjekke for tallet og du opnår en mere korrekt måde at behandle ordre på.

Hvordan lægger du ordre i din database ??

Nej. Det er enormt langsomt, og dur ikke mht samtidighed, hvad du gør. Ordrenummeret er vel en primær nøgle et eller andet sted. Så sæt ind og se hvor mange http://php.net/mysql_rows_affected - meget hurtigere.

Men det er ikke måden at gøre det på, helt generelt. Hvorfor ikke bare et fortløbende nummer, evt fra en auto_increment/identity  ?

"Det er enormt langsomt, og dur ikke mht samtidighed" ... og virker ikke - så 3 ting er galt! ;)

Ja okay, nu kan jeg godt se at det er en lidt uheldig måde at gøre det på.
Det skal nok bare laves med et stigende nummer så... Hvordan vil I foreslå det kunne laves?

Kunne ordrenummeret laves ved at den først tæller antallet af rækker og derefter lægger 1 til?

Noget med $ordernum = mysql_num_rows($result) + 1; ?

Eller næh... Nu er jeg selv lidt forvirret :)

Som begge de to andre har sagt, brug auto_increment i din tabel, det er det den er der for. Alle andre løsninger vil enten:

a) ikke virke,
b) være sårbare over for en race condition (to der vil indsætte samtidigt), eller
c) være langsommere.

Sagt på anden måde, du får ikke nogen god løsning ud af at gøre det selv. Benyt dig af det MySQL giver dig.

*går igen*

Hvordan ser dine tabeller ud ??

Er ved at ha' en idé om hvordan jeg skal lave det. Måske jeg lige skal nævne at det hele starter med en almindelig form der udfyldes og ved tryk på send, skal der sendes en mail til producenten og en kopi til brugeren - det virker top dollar. Men det er jo også her at ordrenummeret skal sendes afsted, så det skal jo genereres somehow.

Jeg kan vel i sagens natur, blot bruge det unikke ID som ordrenummer? Men hvorfor sendes dette så afsted i samme åndedrag?

Min tabel ser p.t. ud som dette:

orderID | ordernum

Struktur:

orderID int(11) not null auto_increment primary
ordernum int(11) not null

så skal der mere på, som navn, email osv. men det er uden betydning i den her sammenhæng.

Men hvorfor sendes ... skal være  Men hvordan sendes

eller ville problemet være noget helt andet :)

Det nemmeste ville at helt klart være at bruge din orderID istedet.
Når du laver en order, så ligger du vel hele ordren ind i databasen ?

Du kan med en mysql_insert_id() få det id som blev lavet ved sidste insert, se evt: http://dk.php.net/manual/da/function.mysql-insert-id.php

dkfire - din løsning ser ud til at virke upåklageligt! Smid et svar, så får du point!
Jeg takker!

lukker