[PHP] maximaal aantal items met 'foreach'

[Verwijderd]

Ik gebruik een script wat een lijst met fotoalbums uit Flickr 'zuigt'. Nu is het alleen zo dat ik behoorlijk wat albums op Flickr heb staan en zodoende een meterslange lijst krijg.

In het script wat de connectie met Flickr staat worden de sets opgehaald via deze regel:

PHP:

foreach (array_keys($sets['photoset']) as $key)

Is het mogelijk om hier een maximaal aantal weer te geven items aan te hangen?

[ Voor 2% gewijzigd door Verwijderd op 24-04-2009 14:31 . Reden: Php code toegevoegd ]

[mathijs92]

ik zou dat met een for-loop doen. die zijn er voor gemaakt

[RAJH]

PHP:

$i = 1;
$max = 30;

foreach(array_keys($sets['photoset']) as $key)
{
    if ($i == $max)
    {
        break;
    }

    // Doe iets met de $key

    $i++
}

Je zou ook kunnen kijken naar een for-loop.

[Jaap-Jan]

Met matthijs92. Als je niet elk item wilt hebben, waarom gebruik je dan foreach?

PHP:

for ($set = array_keys($sets['photoset']); $i <= $max; $key = next($set), $i++) {

}

Hmm, de overzichtelijkheid is ver te zoeken op die manier

edit:
En hij 'vergeet' de eerste omdat hij het volgende element teruggeeft. Zinvol.

PHP:

for ($set = array_keys($sets['photoset']), $i = 0; $i <= $max; $key = $set[$i], $i++) {
        print $key;
}

Werkt wel omdat array_keys numerieke indexen gebruikt.

[ Voor 83% gewijzigd door Jaap-Jan op 24-04-2009 14:27 ]

[Voutloos]

Of gewoon wel foreach en zorgen dat je daarvoor al een gelimiteerd aantal albums ophaalt.

[Apen-nootjes]

met een for lus:

PHP:

<?php

$key = array_keys($sets['photoset']);

$start = 0; // Begin met ophalen bij record 0

/* 1 van de onderstaande regels gebruiken, niet beide ;) */
$end = count($key); // gebruik deze om alle keys op te halen
$end = 100; // Gebruik deze om er honderd op te halen

for($i=$start; $i<$end; $i++)
{
    print_r($key[$i]);
}

?>

Zoeits in ieder geval, ongetest en zo uit m'n hoofd opgeschreven

[BtM909]

Je vraagt iets over PHP, dus past je topic beter in Programming

Zie ook: Waar hoort mijn topic?

Ik verplaats daarom even je topic naar de juiste plek

[Verwijderd]

BtM909 schreef op vrijdag 24 april 2009 @ 14:09:
Je vraagt iets over PHP, dus past je topic beter in Programming

Zie ook: Waar hoort mijn topic?

Ik verplaats daarom even je topic naar de juiste plek

Excuus, ik dacht dat programming niet zozeer met internet/php te maken had. Prima wanneer hij op deze plek komt te staan. Ik ga kijken naar de bovenstaande suggesties. Ben geen php-held, hopelijk kom ik eruit.

[LuCarD]

Misschien dat een array_slice makkelijker is ( http://nl3.php.net/manual/en/function.array-slice.php )

[Quadro!]

Waar wordt $sets gedefinieerd? Je zegt dat ze daar opgehaald worden, maar dat kan ik mij bijna niet voorstellen, aangezien je daar over een array in een variabele looped. Die moet dus eerder al gedefinieerd zijn. Nu heb ik geen ervaring met Flickr en PHP, dus correct me if I'm wrong, maar is het niet mogelijk direct in de Flickr API call miner photosets op te halen?

Oh ja, je kunt ook gewoon de keys krijgen door het volgende te doen:

PHP:

foreach ($sets['photoset'] as $key => $value) { /* */ }

[ Voor 10% gewijzigd door Quadro! op 24-04-2009 14:20 ]

[Verwijderd]

Misschien is het duidelijker wanneer ik de hele code van het ophalen van de setlijsten (albums) laat zien:

PHP:

        function DisplaySetList($username, $title, $intro)
        {
                // Find the NSID of the username
                $nsid = $this->f->people_findByUsername($username);
                if (!$nsid) {
                        echo "<p>" . $this->Lang('flickrerrorusername', $username) . "</p>\n";
                        return;
                }

                // Get list of photo sets
                $sets = $this->f->photosets_getList($nsid);

                // Slightly alternative way of doing the foreach - this is so that we can obtain
                // a reference to $set and not a copy of $set.
                foreach (array_keys($sets['photoset']) as $key)
                {
                        $set = &$sets['photoset'][$key];
                        $set['url'] = "http://farm4.static.flickr.com/" . $set['server'] . "/" . $set['primary'] . "_" . $set['secret'] . "_s.jpg";
                        $set['middle'] = "http://farm4.static.flickr.com/" . $set['server'] . "/" . $set['primary'] . "_" . $set['secret'] . "_m.jpg";
                        $set['link'] = $this->GetURL($set['id']);
                        $set['description'] = html_entity_decode($set['description']);
                        $set['title'] = html_entity_decode($set['title']);
                }

                // Assign variables to Smarty and process template
                $this->smarty->assign_by_ref('setlist', $sets['photoset']);
                $this->smarty->assign('title', $title);
                $this->smarty->assign('intro', $intro);
 
                echo $this->ProcessTemplate($this->setlist_template);
        }

[ Voor 0% gewijzigd door een moderator op 24-04-2009 14:27 . Reden: Doe het zelf wel ]

[.oisyn]

Misschien is het duidelijker als je je code in [code=php]...[/code] tags zet

[Verwijderd]

.oisyn schreef op vrijdag 24 april 2009 @ 14:23:
Misschien is het duidelijker als je je code in [code=php]...[/code] tags zet

Handig dat daar geen knop voor is in de opmaakbalk

[RobIII]

Verwijderd schreef op vrijdag 24 april 2009 @ 14:25:
[...]


Handig dat daar geen knop voor is in de opmaakbalk

We hebben een shitload aan UBB codes; je wil echt niet voor iedere scheet een knop op die balk
code tags

Het was overigens minder moeite geweest even die "=php" er in te knallen voor je dan die zinloze reply te schrijven.

[ Voor 12% gewijzigd door RobIII op 24-04-2009 14:27 ]

[Verwijderd]

RobIII schreef op vrijdag 24 april 2009 @ 14:26:
[...]

We hebben een shitload aan UBB codes; je wil echt niet voor iedere scheet een knop op die balk
code tags

Het was overigens minder moeite geweest even die "=php" er in te knallen voor je dan die zinloze reply te schrijven.

Dat dat voor jou/jullie vanzelfsprekend is hoeft nog niet te betekenen dat dit voor anderen ook zo is. Bedankt voor het aangeven in ieder geval. Weer iets geleerd

[Quadro!]

In dit geval is het inderdaad het handigst om zoals eerder aangegeven door RAJH iets te doen als:

PHP:

$i = 0;
$i_max = 10; // laat max 10 photosets zien
foreach (/* je foreach statement */) {
    $i++;
    if ($i > $i_max) break;

    /* doe je ding */
}

[Guillome]

Met forEach en een $max is naar mijn inzicht de beste oplossing. Als je minder items hebt dan max, werkt dit prima. Met een for($x=0;$x<$max) ding moet je ook nog controleren of er nog items zijn.

[Verwijderd]

Dank allen, ik ga ermee aan de slag. De code van Quadro! ga ik proberen te implementeren.

[Bosmonster]

nm, volgorde omgedraaid

[ Voor 92% gewijzigd door Bosmonster op 24-04-2009 14:47 ]

[Verwijderd]

Helaas, het werkt niet. De foreach die ik hierboven aangaf is alleen bedoeld voor de thumbnails die geladen moeten worden. Met de php toevoeging erbij laadt 'ie alleen de eerste 10 thumbnails, de andere info (teksten, album title etc.) van de overige albums blijven wél staan.

De flickr module werkt met templates, maar die zijn niet in php opgebouwd maar in html. Het stukje code wat de lijst genereert met flickr-albums is alsvolgt:

HTML:

<ul class="flickr_setlist">
{foreach from=$setlist item=set}
    <li>
        <a href="{$set.link}"><img alt="{$set.title}" src="{$set.url}" /></a>
        <h2>{$set.title}</h2><div class="klein">({$set.photos} foto's)</div>
    </li>
{/foreach}
</ul>

De php-code met $ = 1; etc. hierin plakken geeft errors omdat het geen php-document is (denk ik). Iemand een suggestie hoe ik het nu aan kan pakken? Het mooiste zou zijn wanneer ik hier, in de template, kan aangeven hoeveel albums ik wil laden. Dan kan ik m'n flickr-module onaangetast laten.

[LuCarD]

Je wilt het dus op template niveau oplossen...

Dat kan toevallig met smarty wel, maar dan moet je section gebruiken in plaats van foreach.

http://smarty.net/manual/en/language.function.section.php

HTML:

<ul class="flickr_setlist">
{section max=10 item=$setlist name=set}
    <li>
        <a href="{$setlist[set].link}"><img alt="{$setlist[set].title}" src="{$setlist[set].url}" /></a>
        <h2>{$setlist[set].title}</h2><div class="klein">({$setlist[set].photos} foto's)</div>
    </li>
{/section}
</ul>

[ Voor 82% gewijzigd door LuCarD op 24-04-2009 15:31 ]

[Verwijderd]

LuCarD schreef op vrijdag 24 april 2009 @ 15:27:
Je wilt het dus op template niveau oplossen...

Dat kan toevallig met smarty wel, maar dan moet je section gebruiken in plaats van foreach.

http://smarty.net/manual/en/language.function.section.php

HTML:

<ul class="flickr_setlist">
{section max=10 item=$setlist name=set}
    <li>
        <a href="{$setlist[set].link}"><img alt="{$setlist[set].title}" src="{$setlist[set].url}" /></a>
        <h2>{$setlist[set].title}</h2><div class="klein">({$setlist[set].photos} foto's)</div>
    </li>
{/section}
</ul>

Hmm, in dat geval krijg ik deze foutmelding:

string(150) "Smarty error: [in module_file_tpl:Flickr;setlist.tpl line 18]: syntax error: unknown section attribute - 'item' (Smarty_Compiler.class.php, line 1096)"

En daaronder wordt alleen het laatst toegevoegde album weergegeven. We zijn wel op de goede weg denk ik...

[iH8]

.iteration

iteration contains the current loop iteration and always starts at one, unlike index. It is incremented by one on each iteration.

http://smarty.net/manual/en/language.function.foreach.php


edit: ik kan alleen zo geen manier vinden om die iteratie te stopppen als ie 10 bereikt. section is voor meerdere arrays.

[ Voor 63% gewijzigd door iH8 op 24-04-2009 15:54 ]

[LuCarD]

Verwijderd schreef op vrijdag 24 april 2009 @ 15:38:
[...]


Hmm, in dat geval krijg ik deze foutmelding:

string(150) "Smarty error: [in module_file_tpl:Flickr;setlist.tpl line 18]: syntax error: unknown section attribute - 'item' (Smarty_Compiler.class.php, line 1096)"

En daaronder wordt alleen het laatst toegevoegde album weergegeven. We zijn wel op de goede weg denk ik...

errrm sorry item moet loop zijn....


Maar dat had je ook zelf kunnen zien op de link die ik had gegeven

Je kan er namelijk nog meer instellen dus misschien is het handig om even te kijken....

[Verwijderd]

errrm sorry item moet loop zijn....


Maar dat had je ook zelf kunnen zien op de link die ik had gegeven

Je kan er namelijk nog meer instellen dus misschien is het handig om even te kijken....

Jaaaa!!! :-) Wow, dit werk! En nogmaals, sommige dingen zijn voor jullie misschien vanzelfsprekend, maar voor mij als php/smarty-leek niet echt. Top dat het nu op template-niveau op te lossen is. Op deze manier kan ik ook redelijk eenvoudig een archief gaan aanmaken.

Bedankt!