Onderwerp: Php newbie vraagje

In nl.internet.www.server-side Toni Van Remortel <tonivr@chopout.pandora.be> wrote:
> <?php
> if ( !isset($_GET["cmdOk"]) )
> {
[knip]
> <?php
> }
> else
> {
> header('Location: Toertochten_'.$_GET["cmbMaand"].'.htm');
> }
> ?>
>
> Dat is dus veel korter, en redelijk dynamisch. Have fun.

Maar ook fout (Location dient absoluut te zijn) en nogal "bad practice"
door enig gebrek aan controle op de invoer.

--

Daniel Tryba

Jeps, bedankt, 't is misschien wel niet correct maar't werkt wel

bedankt allemaal

"Toni Van Remortel" <tonivr@chopout.pandora.be> wrote in message
newsan.2003.10.06.19.19.20.579128@chopout.pandora.be.. .
> On Mon, 06 Oct 2003 19:16:52 +0000, Toni Van Remortel wrote:
>
> > <form action="toertochten.php" method="get">
>
> kan je action="<?php echo $_SERVER["PHP_SELF"];?>" van maken.
> --
> Groeten, Toni
> Nokelu - NoorderKempense Linux users
> http://www.nokelu.be
>

Sven Daems wrote:
> Hieronder nog eens de code zoals ik ze nu heb. Het moet voor mij niet
> op deze manier hoor, als er maar een werkende oplossing uit de bus komt
> ben ik heel tevreden.
>
> <?php
>
> $_GET[cmbMaand]
>
> if ($cmbMaand=="September") {
> $URL=="http://www.heidestoempers.be/Toertochten_sept.htm";

Dit zal in elk geval
$URL="http://www.heidestoempers.be/Toertochten_sept.htm";
moeten zijn:
x=y is een toekenning (hierna heeft x de waarde van y)
x==y is een test (geeft waar terug als x gelijk is aan y)

De instructie $URL=="..." doet dus eigenlijk niets: het geeft gewoon onwaar
terug.

Robrecht

> Grappig, ik vind die wel. Dus was de code die je poste wel daadwerkelijk
> de door jou gebruikte code?

Ik had deze code gebruikt

<?php

$_GET[cmbMaand]

if ($cmbMaand=="September") {
$URL=="http://www.heidestoempers.be/Toertochten_sept.htm";
header ("Location: $URL");
} elseif ($cmbMaand=="Oktober") {
$URL=="http://www.heidestoempers.be/Toertochten_okt.htm";
header ("Location: $URL");
} else {
$URL=="http://www.heidestoempers.be/Toertochten_nov.htm";
header ("Location: $URL");
}

?>

Wat is er daar fout aan bij de <?php of ?>

Sven Daems vertrouwde ons op Mon, 6 Oct 2003 21:39:46 +0200 in
<3f81c505$0$24163$ba620e4c@reader0.news.skynet.b e> het volgende toe:

>$_GET[cmbMaand]

Moet daar geen ; achter?


--
Door mij verstuurde briefpost wordt immer voldoende gefrankeerd.

In nl.internet.www.server-side Sven Daems <cyberbike9@hotmail.com> wrote:
>
>> Grappig, ik vind die wel. Dus was de code die je poste wel daadwerkelijk
>> de door jou gebruikte code?
>
> Ik had deze code gebruikt
>
> <?php
> $_GET[cmbMaand]
[knip]
> ?>
>
> Wat is er daar fout aan bij de <?php of ?>

Niets, maar daar gaat
http://nl.php.net/manual/en/language...separation.php
ook helemaal niet over. Ik zal 1 van de 2 regels op deze pagina even
quoten en het sleutel woord onderstrepen en je code inkorten tot de
regel waar op het fout gaat:
Instructions are separated the same as in C or Perl - terminate each
statement with a semicolon.
^^^^^^^^^

MAW ik raad je nogmaals aan een handleiding te gaan lezen voor je verder
gaat, er wordt wel vaker in deze groep gevraagt naar goede
nederlandstalige php beginners boeken, doorzoek dus even de archiven.

--

Daniel Tryba

In article <3f81b997$0$24164$ba620e4c@reader0.news.skynet.be> ,
Sven Daems <cyberbike9@hotmail.com> wrote:
>$_GET[cmbMaand]

Zo heet de variabele waarin de maand staat die je in het
formulier hebt ingevuld. Er is dus geen variabele $cmbMaand.

>if ($cmbMaand=="September") {

Je moet hier dus
if ($_GET[cmdMaand]=="September") {
van maken.

> $URL=="http://www.heidestoempers.be/Toertochten_sept.htm";

De == gebruik je om te vergelijken. Gebruik = om iets toe te
kennen:
$URL="http://www.heidestoempers.be/Toertochten_sept.htm";

Arnoud

--
Arnoud Engelfriet, Dutch patent attorney - Speaking only for myself
Patents, copyright and IPR explained for techies: http://www.iusmentis.com/

A long time ago in a galaxy far, far away, Sven Daems wrote:

> Hieronder nog eens de code zoals ik ze nu heb. Het moet voor mij
> niet op deze manier hoor, als er maar een werkende oplossing uit
> de bus komt ben ik heel tevreden.
>
>
> <?php
>
> $_GET[cmbMaand]

moet zijn:

$cmbMaand=$_GET['cmbMaand'];



> if ($cmbMaand=="September") {
> $URL=="http://www.heidestoempers.be/Toertochten_sept.htm";
> header ("Location: $URL");
> } elseif ($cmbMaand=="Oktober") {
> $URL=="http://www.heidestoempers.be/Toertochten_okt.htm";
> header ("Location: $URL");
> } else {
> $URL=="http://www.heidestoempers.be/Toertochten_nov.htm";
> header ("Location: $URL");
> }
>
> ?>




--
Joris "flotspe" Janssens
http://www.xhtml.be/inferno/

Sven Daems wrote:

> <?php
>
> $_GET[cmbMaand]
>
> if ($cmbMaand=="September") {
> $URL=="http://www.heidestoempers.be/Toertochten_sept.htm";
> header ("Location: $URL");
> } elseif ($cmbMaand=="Oktober") {
> $URL=="http://www.heidestoempers.be/Toertochten_okt.htm";
> header ("Location: $URL");
> } else {
> $URL=="http://www.heidestoempers.be/Toertochten_nov.htm";
> header ("Location: $URL");
> }
>
> ?>

Los van de andere opmerkingen, ik zou hier een switch-statement
gebruiken. Zie ook
<http://nl.php.net/manual/en/control-...res.switch.php>.

--
Cheers, Maarten Wierda
cp: The Stooges - We Will Fall

Q: What do you call someone who hangs around with musicians?
A: A drummer.

Op Mon, 06 Oct 2003 19:31:07 +0000, schreef Daniel Tryba :

> In nl.internet.www.server-side Toni Van Remortel
> <tonivr@chopout.pandora.be> wrote:
>> <?php
>> if ( !isset($_GET["cmdOk"]) )
>> {
> [knip]
>> <?php
>> }
>> else
>> {
>> header('Location: Toertochten_'.$_GET["cmbMaand"].'.htm');
>> }
>> ?>
>>
>> Dat is dus veel korter, en redelijk dynamisch. Have fun.
>
> Maar ook fout (Location dient absoluut te zijn) en nogal "bad practice"
> door enig gebrek aan controle op de invoer.

Dan maak je de location absoluut, het was maar als voorbeeld (en location
MOET niet absoluut zijn, daar het relatief tov de webroot is als er geen
protocol gespecifieerd is).
En de controle op invoer is ook snel verder uitgebreid. Trouwens, een
'foute' invoer rechtstreeks in de URL brengt je toch bij een niet
bestaande pagina.

Again: het was een voorbeeld.

--
Groeten, Toni Van Remortel
Wetenschappelijk medewerker, D-science lab, Hogeschool Antwerpen
Real time Linux for embedded systems: http://linemb.d-sciencelab.com