---

Jeg ved det bare er en dum fejl men jeg kan bare ikke få øje på den:

Script:

$string = "\n".'<table>'."\n";
for($i = 0; $i<sizeof($file_list); $i++)
{
$string .= '<tr>'."\n";
for($j = 0; $j<$pictures_pr_line; $j++)
{
if($i >= sizeof($file_list))
{
$string .= ' <td>&nbsp;</td>'."\n";
}
else
{
$string .= ' <td><img title="'.$file_list[$i].'"
alt="'.$file_list[$i].'" src="'.$path.$file_list[$i].'" /></td>'."\n";
}
$i++;
}
$string .= '</tr>'."\n";
}
$string .= '</table>';

/*********************/

$file_list indeholder f.eks. 13 elementer
$pictures_pr_line indeholder f.eks. 2

resultat->

Script udskriver en tabel med 9 billeder?!

sætter jeg $pictures_pr_line til 3 udskriver det 10 billeder?!

lige meget hvd jeg gør for jeg ikke 13 billeder ud

Hvad er det lige jeg har overset her????

Joe

---

Johannes Skov Frandsen skrev:

> Jeg ved det bare er en dum fejl men jeg kan bare ikke få øje på den:

Du opdaterer i to steder, og så har du en test der altid giver
falsk:

> $string = "\n".'<table>'."\n";
> for($i = 0; $i<sizeof($file_list); $i++)

Bemærk i++.
Bemærk at i er mindre end sizeof($file_list).

> {
> $string .= '<tr>'."\n";
> for($j = 0; $j<$pictures_pr_line; $j++)
> {
> if($i >= sizeof($file_list))

Det kan den aldrig blive, så det er hele tiden else-delen der er
aktiv.

> {
> $string .= ' <td>&nbsp;</td>'."\n";
> }
> else
> {
> $string .= ' <td><img title="'.$file_list[$i].'"
> alt="'.$file_list[$i].'" src="'.$path.$file_list[$i].'" /></td>'."\n";
> }
> $i++;

Bemærk i++.

> }
> $string .= '</tr>'."\n";
> }
> $string .= '</table>';

--
Bertel
http://bertel.lundhansen.dk/      http://fiduso.dk/

---

Johannes Skov Frandsen wrote:
> Jeg ved det bare er en dum fejl men jeg kan bare ikke få øje på den:
>
> Script:
>
> $string = "\n".'<table>'."\n";
> for($i = 0; $i<sizeof($file_list); $i++)

her itererer du $i ------------------^^^^

> {
> $string .= '<tr>'."\n";
> for($j = 0; $j<$pictures_pr_line; $j++)
> {
> if($i >= sizeof($file_list))

Du vil nok checke på $j i stedet for $i her..
....
> $i++;

og i bunden itererer du så $i igen..så prøv at fjerne den nederste
$i++; og rette $i til $j i din if..


--
Mvh. Nezar Nielsen
http://gorilla.dk

---

Nezar Nielsen wrote:

>> if($i >= sizeof($file_list))
>
>
> Du vil nok checke på $j i stedet for $i her..

Faktisk er det jo ikke fint, du vil i virkeligheden checke om $j >=
sizeof($file_list) % $pictures_pr_line tror jeg.

--
Mvh. Nezar Nielsen
http://gorilla.dk

---

Nezar Nielsen wrote:
> Johannes Skov Frandsen wrote:
>
>> Jeg ved det bare er en dum fejl men jeg kan bare ikke få øje på den:
>>
>> Script:
>>
>> $string = "\n".'<table>'."\n";
>> for($i = 0; $i<sizeof($file_list); $i++)
>
>
> her itererer du $i ------------------^^^^
>
>> {
>> $string .= '<tr>'."\n";
>> for($j = 0; $j<$pictures_pr_line; $j++)
>> {
>> if($i >= sizeof($file_list))
>
>
> Du vil nok checke på $j i stedet for $i her..
> ...
>
>> $i++;
>
>
> og i bunden itererer du så $i igen..så prøv at fjerne den nederste
> $i++; og rette $i til $j i din if..
>
>


Det var itereringen den var gal med:


$string = "\n".'<table>'."\n";
$i=0;
while($i<sizeof($file_list)) //while i stedet for for-løkke
{
$string .= '<tr>'."\n";
for($j = 0; $j<$pictures_pr_line; $j++)
{
if($i >= sizeof($file_list))
{
$string .= ' <td>&nbsp;</td>'."\n";
}
else
{
$string .= ' <td><img title="'.$file_list[$i].'"
alt="'.$file_list[$i].'" src="'.$path.$file_list[$i].'" /></td>'."\n";
}
$i++;
}
$string .= '</tr>'."\n";
}
$string .= '</table>';
return $string;

/*******************/

tak for kickstartet.... nogen gange er man bare blind

Joe