While loop returned lege array waarde aan het einde

Door Snelle Jaap op 22-12-2015 10:58 gewijzigd op 22-12-2015 13:05

2.990 views

Thomas van den Heuvel

22-12-2015 14:22

Hoeveel resultaten heeft je query en uit hoeveel rijen bestaat $contentcr (let erop dat indexen van arrays beginnen bij 0)?

Grote kans dat dit door je while-constructie komt. De while-lust houdt op als fetch_array() false retourneert (en voor het begrip: de toegekende waarde is tevens de return-waarde van een toekenning zelf), maar dat resultaat stop je nog wel in $contentcr...

Daarnaast laat de naamgeving van je variabelen nogal te wensen over.
$content? dit is een querystring
$contentcon? dit is een resultset object

Daarna spring je je foreach in, waarom?
$content wordt op dit moment ook overschreven...

Snelle Jaap

22-12-2015 14:33

Er zijn 6 rijen in de array, maar de 6e is dus leeg. Dat komt inderdaad omdat die binnen de while loop false is. Hoe kan ik er voor zorgen dat hij die niet in $contentcr stopt? Of die false waarde niet uitleest?

$content is de query inderdaad en $contencon de variabele voor de connectie. Hoezo laat dat te wensen over?

En inspringen waar het kan is netjes toch?

- Roland -

22-12-2015 14:35

heb niet je hele tekst bestudeerd, maar dit is raar/onnodig:

"$contentcon = $conn->query($content);
$contentcr = array();
while ($contentcr[] = $contentcon->fetch_array());
"
maak daar eens gewoon:
$contentcon = $conn->query($content);
while ($content = $contentcon->fetch_array());

van !
fetch_array geeft ja al gewoon een array. (waarom zou je die weer als element in een array willen stoppen?)

Snelle Jaap

22-12-2015 14:38

Als ik die code aanpas staat er dat hij de foreach niet kan uitvoeren. Aan de gehele array moet je toch ook een variabele toekennen, die je daarna in je foreach kunt gebruiken?

Thomas van den Heuvel

22-12-2015 14:50

Wat Roland zegt klopt, je hebt hier een constructie met twee loops, terwijl een constructie met een enkele loop ook volstaat.

$content is de query inderdaad en $contencon de variabele voor de connectie. Hoezo laat dat te wensen over?

$conn bevat als het goed is een mysqli object (dit is dus je "variabele voor de connectie"), $contentcon bevat na aanroep van de query() methode ofwel een boolean ofwel een mysqli_result object.

En inspringen waar het kan is netjes toch?

Inspringen doe je niet waar het kan, maar waar het nodig is. In regel 8 van het eerste codefragment is dit niet nodig.

Snelle Jaap

22-12-2015 14:53

Ah op die manier. Bij mij in de editor staat die wel goed, ik zet ze altijd helemaal tegen de zijkant aan voordat ik hier copy paste, aangezien je anders moet scrollen om de code te kunnen zien.

Hoe kan ik voor elkaar krijgen dat hij door de hele array loopt met een while? (Of een foreach zoals ik nu al heb.

Toevoeging op 22/12/2015 14:57:37:

Laat maar, ik heb alles binnen de foreach in de while gezet, en de code aangepast zoals Roeland zei. Nu werkt het! Thx

Thomas van den Heuvel

22-12-2015 15:09 gewijzigd op 22-12-2015 15:13

Ik zou zoiets doen:

<?php
/* @var $res mysqli_result */
$res = $conn->query('SELECT * FROM `lb_content` WHERE catid = 13');
while ($row = $res->fetch_array()) {
    // Get image parameters of the article and split them.
    $pictures = json_decode($row['images']);
    ?><li class="job_listing">
        <a href="http://www.website.nl/_extern/website/content.php?alias=<?php echo $row['alias'] ?>">
            <div class="location job_img">
                <img name="images" src="/_extern/website/cms/<?php echo $pictures->{'image_intro'} ?>" alt="" class="company_logo" />
            </div>
            <div class="location boldfont"><?php echo $row['title'] ?></div>
            <div class="location specificaties"><?php echo $row['introtext'] ?></div>
            <div class="rating location">
                <div class="rating-stars">
                    <i class="fa fa-star"></i>
                    <i class="fa fa-star"></i>
                    <i class="fa fa-star"></i>
                    <i class="fa fa-star"></i>
                    <i class="fa fa-star"></i>
                </div>
            </div>
            <div class="location prijs"><?php
                echo $row['metakey'];
                ?><br />
                <div class="cursief">Vanaf 2 weken</div>
            </div>
            <div class="location">
                <button type="submit" class="btn btn-primary buttonblock" onClick="location.href='http://www.website.nl/_extern/website/content.php?page=<?php echo $row['alias'] ?>';">Nu huren</button>
            </div>
        </a>
    </li><?php
}
?>

Daarbij nog twee opmerkingen:
- de quotes van je button onclick klopt niet
- overal waar echo '...' staat zou ik deze output escapen met htmlspecialchars() tenzij het de bedoeling is dat je daar HTML kunt afdrukken

en overal waar $content stond $row gebruiker dus...

Snelle Jaap

22-12-2015 15:58 gewijzigd op 22-12-2015 15:59

Oke dat is niet zo'n drama van die button, aangezien je op het hele veld kunt klikken maar is wat netter als ik het even aanpas.

Ja ik wil graag dat er html opmaak mogelijk is (aangezien ik een tabel die is ingevoerd via een wysiwyg editor uitlees) dus dat is prima zo.

Wat bedoel je met dit laatste?

"en overal waar $content stond $row gebruiker dus..."

Reageren

Inloggen om te reageren