PHP/MYSQL: if / else in while lus

[Verwijderd]

Ik heb een while lus die adressen uit een mysql database haalt. Nu wil ik dat er af en toe bij een bepaald zoekwoord, (bijvoorbeeld: ' schilder'). Daarna geeft hij dus een lijst adressen tot 10 per pagina. In deze while lus zit de functie "reclamebanner" deze functie heeft de volgende code:

code:

<?php

    function reclamebanner ($zoek) {

        if ($zoek == "schilder"){ //het zoekwoord
        
            $bannergeweest = 0;
            
            while($bannergeweest == 0 ){
            
                //if($bannergeweest == 1){
                //  echo "<br>";
                //  echo "de banner is al geweest";
                //  echo "<br>";
                //}
            
                if($bannergeweest == 0){
                    $welofniet = rand(1,3);
                    if ($welofniet == 1) {
                    echo ?> [img]"banners/banner1.gif"[/img]<br> <?
                    echo "Hier is een reclamebanner";
                    echo "<br>";
                    $bannergeweest = $bannergeweest + 1;
                    }
                }
            
            }

        }
    }
?>

Nu werkt het niet helemaal zoals ik zou willen. De code werkt wel maar het is mijn bedoeling dat er maar maximaal 1 keer per pagina een reclamebanner komt te staan. Met de random functie probeer ik ervoor te zorgen dat hij niet altijd bovenaan staat maar ook soms tussen plek 1 en 2 of 3 en 4 etc etc..

Hoop dat iemand het begrijpt.

[Verwijderd]

Bij elke aanroep verander je $bannergeweest in 0...

Doe het dus zo:

PHP:

$bannergeweest = 0;

function reclamebanner ($zoek) {
  global $bannergeweest;

  // de rest gewoon hetzelfde
}

Nu wil ik dat er af en toe bij een bepaald zoekwoord, (bijvoorbeeld: ' schilder').

Hier moet vermoedelijk nog iets achter van een banner tonen

Dit kan beter met een ads management systeem als phpadsnew, maar goed.

echo ?> [img]"banners/banner1.gif"[/img]<br> <?

Dit slaat ook nergens op...

[ Voor 52% gewijzigd door Verwijderd op 09-02-2006 14:11 ]

[Verwijderd]

Super! thanx het werkt nu. maar nu ben ik wel benieuwd wat global doet. want dat wist ik nog niet..

[Verwijderd]

Verwijderd schreef op donderdag 09 februari 2006 @ 14:11:
Super! thanx het werkt nu. maar nu ben ik wel benieuwd wat global doet. want dat wist ik nog niet..

Op jouw manier definieer je bij elke aanroep die variabele opnieuw. Als je hem globaal (global) maakt, wordt hij alleen op 0 gezet bij het aanroepen van het php-script. Bij het veranderen van die waarde in de functie 'onthoudt' hij dat bij de daaropvolgende aanroep

[Verwijderd]

oh de banner moet je nog niet op letten hoor. want dat ga ik nu verder uitwerken. ik moest eerst het systeem hebben voor een random banner bij een bepaald zoekwoord.

[Verwijderd]

dat phpadsnew systeem waar je het over hebt is dat open source??

[Verwijderd]

Verwijderd schreef op donderdag 09 februari 2006 @ 14:20:
dat phpadsnew systeem waar je het over hebt is dat open source??

Is idd gratis te downloaden

[ZroBioNe]

Het is niet netjes om HTML in je functie te echo'en, en zeker niet door 't php block af te sluiten (dmv ?>).
Met de functie kan je beter wat returnen, en dan in je normale script de echo doen, of het php block sluiten.

En ook vreemd, je doet:

PHP:

$bannergeweest  = 0;
while ($bannergeweest == 0) {
/* code */
}

Is dit niet oneindig?
Want opzich is $bannergeweest == 0 ALTIJD true

[Verwijderd]

ziet er inderdaad goed uit maar kun jij het ook aanpassen aan je eigen systeem. Dus ik ik bijvoorbeeld de functie reclamebanner combineer met het systeem. Want ik wil namelijk reclamebanners van schilderbedrijven alleen als men in deze branche zoekt zodat deze schilder daar profeit van heeft.

Dus dat je het naar elkaar laat luisteren of iets dergelijks.

[apNia]

Wat is er in het algemeen lelijk aan HTML binnen PHP code tonen door te exitten via ?>

[.oisyn]

ZroBioNe: Nee, want de code past $bannergeweest aan. Een do..while was hier idd beter op z'n plek geweest, alhoewel de code in z'n huidige vorm al helemaal geen loop nodig heeft omdat altijd exact 1 keer geïtereerd wordt, en $bannergeweest daarna 1 is.

[ Voor 4% gewijzigd door .oisyn op 09-02-2006 14:29 ]

[Verwijderd]

ZroBioNe schreef op donderdag 09 februari 2006 @ 14:23:
Het is niet netjes om HTML in je functie te echo'en, en zeker niet door 't php block af te sluiten (dmv ?>).
Met de functie kan je beter wat returnen, en dan in je normale script de echo doen, of het php block sluiten.

En ook vreemd, je doet:

PHP:

$bannergeweest  = 0;
while ($bannergeweest == 0) {
/* code */
}

Is dit niet oneindig?
Want opzich is $bannergeweest == 0 ALTIJD true

Nee, want er komt daarna een if op $bannergeweest ==0 -> daar wordt hij verhoogd

/me mept .oisyn

De code slaat idd nergens op...

ziet er inderdaad goed uit maar kun jij het ook aanpassen aan je eigen systeem. Dus ik ik bijvoorbeeld de functie reclamebanner combineer met het systeem. Want ik wil namelijk reclamebanners van schilderbedrijven alleen als men in deze branche zoekt zodat deze schilder daar profeit van heeft.

Dus dat je het naar elkaar laat luisteren of iets dergelijks.

Lees de tutorial van phpAdsNew even... Dit is inderdaad wel mogelijk.

[ Voor 30% gewijzigd door Verwijderd op 09-02-2006 14:31 ]

[NMe]

ZroBioNe schreef op donderdag 09 februari 2006 @ 14:23:
En ook vreemd, je doet:

PHP:

//snip

Is dit niet oneindig?
Want opzich is $bannergeweest == 0 ALTIJD true

Kijk eens goed naar regel 23?

Maar die if op regel 17 is dan weer wel heel overbodig. Zolang je in die while zit kun je er wel zeker van zijn dat $bannergeweest 0 is, anders zat je daar niet in. Trouwens, met deze code hoeft de banner ook niet per se te verschijnen...ik zou zoiets doen:

PHP:

define('MAXPOSITIE', 4)

$positie = rand(1, MAXPOSITIE) - 1;
$bannerGeweest = false;
for ($i = 0; !$bannerGeweest; $i++) {
  if ($i == $positie) {
    $bannerGeweest = true;
    echo 'hier komt je banner';
  }
  //vergeet niet hier andere dingen te echoën, want anders doet deze functie vrij weinig
}

Verder, bsdesign, ik zie hier al een heel aantal kleine vraagjes die je makkelijk zelf op had kunnen lossen door Google te gebruiken. Als je hier een term tegenkomt die je niet kent, probeer die dan eerst zelf op te zoeken voordat je ernaar vraagt.

[ Voor 8% gewijzigd door NMe op 09-02-2006 14:34 ]

[Verwijderd]

Ik heb het nu zo:

PHP:

<?php

    $bannergeweest = 0; 

    function reclamebanner ($zoek) {
        
        global $bannergeweest;

        if ($zoek == "schilder"){ //het zoekwoord
                                
                 if($bannergeweest == 1){
                    echo "<br>";
                    echo "de banner is al geweest";
                    echo "<br>";
                }
            
                if($bannergeweest == 0){
                    $welofniet = rand(1,3);
                    if ($welofniet == 1) {
                    echo ?> [img]"banners/banner1.gif"[/img]<br> <?
                    echo "Hier is een reclamebanner";
                    echo "<br>";
                    $bannergeweest = $bannergeweest + 1;
                    }
                }
            
        }
    }
?>

[Verwijderd]

Die echo uit regel 20 is overbodig... Tevens hoeft de banner idd niet getoond te worden nu.

[ Voor 43% gewijzigd door Verwijderd op 09-02-2006 14:37 ]

[Verwijderd]

even voor alle duidelijkeheid nog, Ik maag website's in adobe golive en als ik php dan niet afsluit dmv. ?> <? krijg ik de banner niet als plaatje te zien. Dit wil ik wel graag want in deze pagina staan nog andere functies en zo scroll ik makkelijk naar de juiste functie dmv visiueel beeld anders geeft golive alleen maar een klein paars php blokje weer.

[Verwijderd]

Verwijderd schreef op donderdag 09 februari 2006 @ 14:40:
even voor alle duidelijkeheid nog, Ik maag website's in adobe golive en als ik php dan niet afsluit dmv. ?> <? krijg ik de banner niet als plaatje te zien. Dit wil ik wel graag want in deze pagina staan nog andere functies en zo scroll ik makkelijk naar de juiste functie dmv visiueel beeld anders geeft golive alleen maar een klein paars php blokje weer.

Dat is ook geen probleem, maar de afsluittags echo-en slaat nergens op Als je die echo weghaalt, werkt het ook namelijk...

[ZroBioNe]

Ok ok, daar had ik dus overheen gelezen

[apNia]

apNia schreef op donderdag 09 februari 2006 @ 14:28:
Wat is er in het algemeen lelijk aan HTML binnen PHP code tonen door te exitten via ?>

iemand?

[ZroBioNe]

apNia schreef op vrijdag 10 februari 2006 @ 10:36:
[...]

iemand?

In het algemeen is het niet lelijk, maar in een functie staat het gewoon niet netjes

[JHS]

apNia: Dat is an sich niet lelijk, het gaat meer om de scheiding van presentatie- en andere logica. Het zou dus mooier zijn als je een .php bestand hebt waarin je variabele genereerd waarin staat óf er een banner getoond moet worden en zoja waar die vandaan komt, en dan in een template achtig .php bestand dat eruitpoept. Waar je best kan exitten met behulp van ?> [html] <? .