[PHP] Random script geeft onjuist aantal

[TommyGun]

Beste mede Tweakers,

Ik heb hier een script dat wordt gebruikt om random tutorials van een Photoshop site te laten zien.

We hebben momenteel 36 tutorials wat dus een gelijk aantal is maar toch gaat het fout. Het probleem is namelijk dat er soms 4, soms 5, soms 3 en zelfs soms maar 2 tutorials worden getoond. Heel willekeurig. Soms dan geeft ie netjes een aantal keer achter elkaar 5 tutorials aan maar dan ineens 4, 3 en de volgende pagina 2.

Dit ondanks dat het aantal op 5 staat. Zie ook de regel $aantal = 5;.

De regel $image[] = "http://domein.ext/images/1.jpg"; staat er dus 36 keer in.

Script:

PHP:

$image = array();

$image[] = "http://domein.ext/images/1.jpg";

$gehad = array();

$aantal = 5;

while(sizeof($gehad) < min(sizeof($image), $aantal))

{

  $rand = rand(0, sizeof($image)-1);

  if(!in_array($rand, $gehad))

  {

    $gehad[] = $rand;

  }

}

$i = 0;

foreach($gehad as $v)

{

  $id = $v+1;

  echo "<a href=\"link.php?id=".$id."\">[img]\"".$image[$v]."\"[/img]</a>";

  $i++;

  if($i == 2) {

    echo "";

    $i = 0;

  }

}

Wat kan hier nu fout gaan? Ik heb ook al geprobeerd om 4 of 3 neer te zetten op de plek van $aantal = 5; maar dan nog blijft het probleem zich voort doen. Iemand enig idee hoe ik dit kan oplossen? Heel erg bedankt alvast.

[ Voor 48% gewijzigd door TommyGun op 29-04-2006 21:34 ]

[intoxicated]

Weet je zeker dat je die array goed opbouwt in de regels die je nu weg laat? Want met je code is zotezien niks mis.

offtopic:
http://nl2.php.net/manual/en/function.array-rand.php

[TommyGun]

intoxicated schreef op zaterdag 29 april 2006 @ 21:42:
Weet je zeker dat je die array goed opbouwt in de regels die je nu weg laat? Want met je code is zotezien niks mis.

offtopic:
http://nl2.php.net/manual/en/function.array-rand.php

Mij lijkt dat dit moet kloppen;

PHP:

$image[] = "images/tutorials/random/100_brush_text.png";

$image[] = "images/tutorials/random/100_bubble_text.png";

$image[] = "images/tutorials/random/100_button.png";

$image[] = "images/tutorials/random/100_button2.png";

$image[] = "images/tutorials/random/100_censoring.png";

$image[] = "images/tutorials/random/100_coloring.png";

$image[] = "images/tutorials/random/100_eye_ball.png";

$image[] = "images/tutorials/random/100_eye_color.png";

$image[] = "images/tutorials/random/100_fire_ball.png";

$image[] = "images/tutorials/random/100_gameboy_classic.png";

$image[] = "images/tutorials/random/100_hot_keys.png";

$image[] = "images/tutorials/random/100_image_text.png";

$image[] = "images/tutorials/random/100_ipod.png";

$image[] = "images/tutorials/random/100_mix_effect.png";

$image[] = "images/tutorials/random/100_nav_bar.png";

$image[] = "images/tutorials/random/100_neon_stick.png";

$image[] = "images/tutorials/random/100_nose_correction.png";

$image[] = "images/tutorials/random/100_orb.png";

$image[] = "images/tutorials/random/100_outcoming_text.png";

$image[] = "images/tutorials/random/100_pattren_diagonal.png";

$image[] = "images/tutorials/random/100_pattren_scanline.png";

$image[] = "images/tutorials/random/100_pattren_squares.png";

$image[] = "images/tutorials/random/100_photo_fuse.png";

$image[] = "images/tutorials/random/100_photo_touch_up.png";

$image[] = "images/tutorials/random/100_photo_poloroid.png";

$image[] = "images/tutorials/random/100_reflection.png";

$image[] = "images/tutorials/random/100_photo_screen.png";

$image[] = "images/tutorials/random/100_signature_border.png";

$image[] = "images/tutorials/random/100_skin_color.png";

$image[] = "images/tutorials/random/100_slice.png";

$image[] = "images/tutorials/random/100_style_flower.png";

$image[] = "images/tutorials/random/100_tattoo.png";

$image[] = "images/tutorials/random/100_teeth_whitner.png";

$image[] = "images/tutorials/random/100_tentacels.png";

$image[] = "images/tutorials/random/100_wires.png";

[ Voor 20% gewijzigd door TommyGun op 29-04-2006 21:45 ]

[BasieP]

offtopic:
aargh verkeerde knop

[ Voor 200% gewijzigd door BasieP op 29-04-2006 21:48 ]

[gvanh]

Is het niet makkelijker om deze functie te gebruiken:

http://php.nederland.net/array_rand

Dat scheelt je in ieder geval een hoop regels code. Wellicht dat het ook je probleem oplost.

[Michali]

Ik heb het hier even getest, en er lijkt niets mee mis te zijn. Weet je zeker dat er geen andere code in de weg zit? Anders lijkt me dit een hele vreemde bug. Probeer $gehad eens te dumpen nadat je hem hebt opgebouwd. Gewoon alles dumpen, dan weet je snel waar het mis gaat.

[TommyGun]

Michali schreef op zondag 30 april 2006 @ 15:51:
Ik heb het hier even getest, en er lijkt niets mee mis te zijn. Weet je zeker dat er geen andere code in de weg zit? Anders lijkt me dit een hele vreemde bug. Probeer $gehad eens te dumpen nadat je hem hebt opgebouwd. Gewoon alles dumpen, dan weet je snel waar het mis gaat.

Bedankt voor het testen. Ik zal hem morgen eens apart includen ipv gewoon in de index te gooien.

[BasieP]

lees dit ook eens:
http://php.nederland.net/manual/en/function.shuffle.php

[CH4OS]

Keep in mind, dat er ook tutorials bij (kunnen) komen. Als dat het geval is, dan zou je je script telkens weer aan moeten passen.

Kan je dan niet beter de image (filename) opslaan in de database?
Dan kan je middels SQL je array met gemak uitbreiden...

PHP:

$query = "SELECT COUNT(*) as number FROM `tutorials`";
$result = mysql_query($query);
if(!$result)
{
    echo mysql_error();
}
else
{
    while($row = mysql_fetch_assoc($result))
    {
        $number = $row['number'];
    }
}

$random = rand(1, $number);
$plaatje = "pad/naar/plaatje/". $random .".ext";

Of denk ik nu verkeerd?

[ Voor 11% gewijzigd door CH4OS op 30-04-2006 22:25 ]

[blizt]

Je idee is op zich goed GJ-tje, maar zitten wel wat dingetjes in die niet al te praktisch zijn
Als de query failed bijvoorbeeld, zal $number niet bestaan op line 15.
Een count op * kan in een grote db ook nog wel 'ns voor performanceproblemen zorgen...

Hoe dan ook lijkt het mij ook verstandiger om het gewoon in een database op te slaan, waarschijnlijk gebeurt dat nu toch ook met de URL e.d.?

[crisp]

Als je het al in een DB opslaat dan zou ik gewoon zoiets doen:

SQL:

SELECT id, image_url FROM tutorials ORDER BY RAND() LIMIT 5

[CH4OS]

blizt schreef op zondag 30 april 2006 @ 23:58:
Je idee is op zich goed GJ-tje, maar zitten wel wat dingetjes in die niet al te praktisch zijn
Als de query failed bijvoorbeeld, zal $number niet bestaan op line 15.
Een count op * kan in een grote db ook nog wel 'ns voor performanceproblemen zorgen...

Ik heb de code ook even snel geschreven...
Volgens mij word $number ook gemaakt, alleen heeft die geen waarde (null dus) en daar is een extra check natuurlijk zo op gemaakt...

[ Voor 7% gewijzigd door CH4OS op 01-05-2006 00:29 ]

[TommyGun]

De URL's naar de tutorials worden idd nog in de database gezet. Telkens het script aanpassen is niet zo'n probleem. Even met FTP progsel naar de host en het is zo gebeurd. Maar ik snap niet waarom het niet werkt. Zojuist even geinclude en gekeken of er andere scriptjes zouden draaien die zouden kunnen storen maar dat is niet het geval.

Ben zelf totaal geen held op PHP / MySQL gebied en heb dit script dan ook voor gedeeltes afgekeken en met een vriend gemaakt maar toch bedankt voor de hulp. Ik ga eens kijken voor een andere oplossing.

[Michali]

Geef de volledige code eens waarmee het op jouw server fout gaat. (Wel graag strippen tot een zo minimale vorm, maar die moet je zelf dan ook even testen of het nog steeds fout gaat) Dan kunnen we zien of daar is vreemds mee aan de hand is.

[TommyGun]

Michali schreef op maandag 01 mei 2006 @ 14:18:
Geef de volledige code eens waarmee het op jouw server fout gaat. (Wel graag strippen tot een zo minimale vorm, maar die moet je zelf dan ook even testen of het nog steeds fout gaat) Dan kunnen we zien of daar is vreemds mee aan de hand is.

Heb even de index in een zip gegooid; http://www.gamergun.com/files/1/index.zip

Bedankt

[Michali]

Sorry, maar daar heb ik niet zo veel aan. De betreffende code van hierboven staat daar ook niet eens in. Ik bedoel, maak een zo klein mogelijk scriptje waarin de fout nog steeds voorkomt, en post die even.

[TommyGun]

Michali schreef op maandag 01 mei 2006 @ 14:43:
Sorry, maar daar heb ik niet zo veel aan. De betreffende code van hierboven staat daar ook niet eens in. Ik bedoel, maak een zo klein mogelijk scriptje waarin de fout nog steeds voorkomt, en post die even.

Die staat geinclude ja als random.php. Maar strippen zal niet gaan gezien alles essenstieel is. Anders had ik het er net zo goed niet in kunnen coden he Maar goed heb de helft er uit gegooid behalve de includes en nog werkt het niet, zie;

http://www.gamergun.com/files/1/index2.zip

En de random.php dan ook maar; http://www.gamergun.com/files/1/random.zip

[Michali]

Als je random.php alleen draait, dan krijg je dus niet altijd 5 links? Hier wel namelijk.

[TommyGun]

Michali schreef op maandag 01 mei 2006 @ 14:55:
Als je random.php alleen draait, dan krijg je dus niet altijd 5 links? Hier wel namelijk.

Jawel dat werkt perfect. Het is echter de bedoeling dat het gewoon op de site werkt want anders heb ik er nog niet veel aan. Ik vind het maar vreemd. Hij zou alleen met de andere includes kunnen storen maar die kan ik niet weg halen.

[TommyGun]

Oke het werkt Er stonden wat links in die niet werktte maar gefixed en het werkt

Allemaal heel erg bedankt!