waar die laatste <?php en ?> op slaan versta ik niet, maar zonder krijg je een parse error.
7 antwoorden
Gesponsorde links
Joel - 30/11/1999 00:00 (laatste wijziging 30/11/1999 00:00)
Eigenaar
Probeer dit eensrnrn[code]phprn$b = (int)$_GET['b'];rn$h = (int)$_GET['h'];rn$url = htmlspecialchars($_GET['url']);rnrnHTML rnHEAD rnTITLEFrametest op = $b; x = $h; TITLE rnframeset rows== $h; , rnframeset cols== $b; , rnframe name=boven src== $url; rnframe name=boven1 scrolling=auto rnframeset rnframe name=onder scrolling=auto rnnoframes rnbody rnpOp deze pagina worden frames gebruikt, maar uw browser ondersteunt geen rn frames.p rnbody rnnoframes rnframeset rnhtml[code]rnrnMaak gebruik van veilige variabelen ($_GET, $_POST, ...) !!!. Plaats nooit codes die geen nut hebben, in dit geval php ... Het kan zijn dat hierdoor een error niet wordt gezegd, maar er degelijk fouten in het script zijn..
Joeliewoeilie, waarom kopiëer je eerst die variabelenrn[code]$b = (int)$_GET['b'];rn$h = (int)$_GET['h'];rn$url = htmlspecialchars($_GET['url']);[code]rnDas toch niet nodig, je kan in je code direct die superglobals gebruiken... das pas onnodige code p
Joel - 30/11/1999 00:00 (laatste wijziging 30/11/1999 00:00)
Eigenaar
In dit geval wordt er meerdere keren gebruik gemaakt van de variabele $h en $b... Dus wat je zegt is niet waar, mijn code is sneller
we hebben de laatste tijd veel op elkaar staan roepen ivm. coding practices.rnAls je de PEAR manier wilt volgen (de meest verspreide, diegene die Henri verkondigt) dan mag je NOOIT zomaar een request variabele copiëren...rnVariabelen copiëren omdat er meerdere keren gebruik van wordt gemaakt is zinloos als je altijd dezelfde var gebruikt, gebruikt je script minder geheugen, en zal dus sneller lopen PUNT UIT.rnrnDit is dus letterlijk het script zoals het daarboven staatrn[code]HTML rnHEAD rnTITLEFrametest op =$_GET['b'] x $_GET['h']TITLE rnframeset rows=$_GET['h'], rnframeset cols=$_GET['b'], rnframe name=boven src=$_GET['url'] rnframe name=boven1 scrolling=auto rnframeset rnframe name=onder scrolling=auto rnnoframes rnbody rnpOp deze pagina worden frames gebruikt, maar uw browser ondersteunt geen rn frames.p rnbody rnnoframes rnframeset rnhtml[code]rnHet zou geen fouten mogen geven. Als het fouten geeft, zeg ons eens opw elke lijn het een fout zou geven en welke foutrnrn[b]Crew edit[b]rnrnHehe, men wordt eindelijk wakker!^^ ^^
Die code van je klopt ook niet wat betreft die framesets.rnDe framesets moeten buiten de headhead, dus dan zou je code zo wordenrn[code]rnHTML rnHEAD rnTITLEFrametest op =$_GET['b'] x $_GET['h']TITLE rnheadrnframeset rows=$_GET['h'], rnframeset cols=$_GET['b'], rnframe name=boven src=$_GET['url'] rnframe name=boven1 scrolling=auto rnframeset rnframe name=onder scrolling=auto rnnoframes rnbody rnpOp deze pagina worden frames gebruikt, maar uw browser ondersteunt geen rn frames.p rnbody rnnoframes rnframeset rnhtmlrnrn[code]