miałem taki kod.

[PHP] pobierz, plaintext 
<?
//screens
$screens = '';
$scr = explode("|||", $rdata[screenshots]);
$licz = count($scr);
if ($licz > 0)
{
	$scr_rand_array = array();
	if ($licz < 4)
	{
		$licz_temp = $licz;
		while ($licz_temp >= 0)
		{
			$scr_rand_array[] = rand(0, $licz);
			$licz_temp--;
		}
	}
	else
	{
		for ($licz_temp = 0; $licz_temp<4; $licz_temp++)
		{
			$scr_rand_array[] = rand(0, $licz);
		}
	}
	foreach ($scr_rand_array as $numerek)
	{
		if ($scr[$numerek] !== '')
		{
			$screens .= '<a href="reviews_screens.php?rid='.$_GET[rid].'&fid='.$numerek.'"><img src="thumbnail5.php?filename='.$scr[$numerek].'" border="0" alt=""></a><br>';
		}
	}
}
?>
[PHP] pobierz, plaintext 

i powiedzmy że wszystko działało jak trzeba mniej więcej. Natomiast po zmianie na ten kod, który miał rozwiązać parę rzeczy z powyższego:

[PHP] pobierz, plaintext 
<?
//screens
$screens = '';
$scr = explode("|||", $rdata[screenshots]);
 
$licz = count($scr)-1;
if ($licz > 0)
{
	$scr_rand_array = array();
	if ($licz < 4)
	{
		$licz_temp = $licz;
		while ($licz_temp >= 0)
		{
			while ($rand !== $licz_temp)
			{
				$rand = rand(0, $licz-1);
			}
			$scr_rand_array[] = $rand;
			$licz_temp--;
		}
	}
	else
	{
		for ($licz_temp = 0; $licz_temp<4; $licz_temp++)
		{
			while ($rand !== $scr_rand_array[$licz_temp])
			{
				$rand = rand(0, $licz-1);
			}
			$scr_rand_array[] = $rand;
		}
	}
	foreach ($scr_rand_array as $numerek)
	{
		if ($scr[$numerek] !== '')
		{
			$screens .= '<a href="reviews_screens.php?rid='.$_GET[rid].'&fid='.$numerek.'"><img src="thumbnail5.php?filename='.$scr[numerek].'" alt="" style="border: 4px solid #000000;"></a><br /><br />';
		}
	}
}
?>
[PHP] pobierz, plaintext 

przestało to działać w ogóle, tzn. nie wyswietlaja sie obrazki ani pod linkiem do nich nie ma zmiennej fid, tzn jest ale nie ma wartości i wyglada to http://.../reviews_screen.php?rid=276&fid=

moze mi ktos powiedziec co sie stało że sie z...?

podoba mi się ten kod:

[PHP] pobierz, plaintext 
<?php
while ($rand !== $licz_temp)
			{
				$rand = rand(0, $licz-1);
			}
?>
[PHP] pobierz, plaintext 

a wiesz ja można zapisać go krócej? o tak:

[PHP] pobierz, plaintext 
<?php
$rand = $licz_temp;
?>
[PHP] pobierz, plaintext 

A wiesz czemu? Bo ty w pętli krążysz tak długo, dopóki $rand jest różny od $licz_temp. Gdy się zrównają, to pętla się konczy.

a fid masz puste bo pewnie $numerek masz puste. A numerek masz puste, bo pewnie ci w tą petle nie wchodzi i do tablicy zapiujesz nic, bo ty nigdzie tego $rand nie inicjujesz na początku.

[PHP] pobierz, plaintext 
<?php
 
//screens
$screens = '';
$scr = explode("|||", $rdata[screenshots]);
$licz = count($scr)-1;
 
if ($licz > 0)
{
    $scr_rand_array = array();
    if ($licz < 4)
    {
        $licz_temp = $licz;
        while ($licz_temp >= 0)
        {
            while ($rand !== $licz_temp)
            {
                $rand = rand(0, $licz-1);
                
            }
            
            $scr_rand_array[] = $rand;
            $licz_temp--;
        }
    }
    
    else
    {
       
        for ($licz_temp = 0; $licz_temp<4; $licz_temp++)
        {
            
            while ($rand !== $scr_rand_array[$licz_temp]) 
            {
                $rand = rand(0, $licz-1);
                die(); // ten warunek nie jest spelniony, co za tym idzie $numerek 
                       // pozostaje NULL
            }
            
            $scr_rand_array[] = $rand;
        }
    }
    // var_export($scr_rand_array); 
    
    foreach ($scr_rand_array as $numerek)
    {
        // var_export($numerek); Numerek ma wartosc NULL dlatego jest pusto, 
        
        if ($scr[$numerek] !== '')
        {
            
            $screens .= '<a href="reviews_screens.php?rid='.$_GET[rid].'&fid='.$numerek.'"><img src="thumbnail5.php?filename='.$scr[$numerek].'" alt="" style="border: 4px solid #000000;"></a><br /><br />' . "n";
        }
    }
}
 
?>
[PHP] pobierz, plaintext 

Dodałem komentarze, teraz powinno Ci pójść łatwiej poprawanie...

Pozdrawiam!